[src/trunk]: src/sys/external/bsd/drm2/dist/drm/amd/powerplay amdgpu: amdgpu_...

[riastradh <riastradh%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/148330aaf98d
branches:  trunk
changeset: 1028868:148330aaf98d
user:      riastradh <riastradh%NetBSD.org@localhost>
date:      Sun Dec 19 12:21:10 2021 +0000

description:
amdgpu: amdgpu_renoir_ppt.c

This abuses snprintf with SIZE_MAX as a substitute for sprintf, which
would be super-sketchy, except this code is all dead because it's
only reached via sysfs stuff that isn't wired up in NetBSD.

diffstat:

 sys/external/bsd/drm2/dist/drm/amd/powerplay/amdgpu_renoir_ppt.c |  14 +++++-----
 1 files changed, 7 insertions(+), 7 deletions(-)

diffs (52 lines):

diff -r d16a00bef6a8 -r 148330aaf98d sys/external/bsd/drm2/dist/drm/amd/powerplay/amdgpu_renoir_ppt.c
--- a/sys/external/bsd/drm2/dist/drm/amd/powerplay/amdgpu_renoir_ppt.c  Sun Dec 19 12:21:02 2021 +0000
+++ b/sys/external/bsd/drm2/dist/drm/amd/powerplay/amdgpu_renoir_ppt.c  Sun Dec 19 12:21:10 2021 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: amdgpu_renoir_ppt.c,v 1.2 2021/12/18 23:45:26 riastradh Exp $  */
+/*     $NetBSD: amdgpu_renoir_ppt.c,v 1.3 2021/12/19 12:21:10 riastradh Exp $  */
 
 /*
  * Copyright 2019 Advanced Micro Devices, Inc.
@@ -24,7 +24,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: amdgpu_renoir_ppt.c,v 1.2 2021/12/18 23:45:26 riastradh Exp $");
+__KERNEL_RCSID(0, "$NetBSD: amdgpu_renoir_ppt.c,v 1.3 2021/12/19 12:21:10 riastradh Exp $");
 
 #include "amdgpu.h"
 #include "amdgpu_smu.h"
@@ -270,12 +270,12 @@
                        else
                                i = 1;
 
-                       size += sprintf(buf + size, "0: %uMhz %s\n", min,
+                       size += snprintf(buf + size, SIZE_MAX/*XXX*/, "0: %uMhz %s\n", min,
                                        i == 0 ? "*" : "");
-                       size += sprintf(buf + size, "1: %uMhz %s\n",
+                       size += snprintf(buf + size, SIZE_MAX/*XXX*/, "1: %uMhz %s\n",
                                        i == 1 ? cur_value : RENOIR_UMD_PSTATE_GFXCLK,
                                        i == 1 ? "*" : "");
-                       size += sprintf(buf + size, "2: %uMhz %s\n", max,
+                       size += snprintf(buf + size, SIZE_MAX/*XXX*/, "2: %uMhz %s\n", max,
                                        i == 2 ? "*" : "");
                }
                return size;
@@ -301,7 +301,7 @@
 
        for (i = 0; i < count; i++) {
                GET_DPM_CUR_FREQ(clk_table, clk_type, i, value);
-               size += sprintf(buf + size, "%d: %uMhz %s\n", i, value,
+               size += snprintf(buf + size, SIZE_MAX/*XXX*/, "%d: %uMhz %s\n", i, value,
                                cur_value == value ? "*" : "");
        }
 
@@ -851,7 +851,7 @@
                if (workload_type < 0)
                        continue;
 
-               size += sprintf(buf + size, "%2d %14s%s\n",
+               size += snprintf(buf + size, SIZE_MAX/*XXX*/, "%2d %14s%s\n",
                        i, profile_name[i], (i == smu->power_profile_mode) ? "*" : " ");
        }