mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-10-03 07:38:10 +00:00
perf bench numa: Address compiler error on s390
[ Upstream commitf8ac1c4784
] The compilation on s390 results in this error: # make DEBUG=y bench/numa.o ... bench/numa.c: In function ‘__bench_numa’: bench/numa.c:1749:81: error: ‘%d’ directive output may be truncated writing between 1 and 11 bytes into a region of size between 10 and 20 [-Werror=format-truncation=] 1749 | snprintf(tname, sizeof(tname), "process%d:thread%d", p, t); ^~ ... bench/numa.c:1749:64: note: directive argument in the range [-2147483647, 2147483646] ... # The maximum length of the %d replacement is 11 characters because of the negative sign. Therefore extend the array by two more characters. Output after: # make DEBUG=y bench/numa.o > /dev/null 2>&1; ll bench/numa.o -rw-r--r-- 1 root root 418320 May 19 09:11 bench/numa.o # Fixes:3aff8ba0a4
("perf bench numa: Avoid possible truncation when using snprintf()") Suggested-by: Namhyung Kim <namhyung@gmail.com> Signed-off-by: Thomas Richter <tmricht@linux.ibm.com> Cc: Heiko Carstens <hca@linux.ibm.com> Cc: Sumanth Korikkar <sumanthk@linux.ibm.com> Cc: Sven Schnelle <svens@linux.ibm.com> Cc: Vasily Gorbik <gor@linux.ibm.com> Link: https://lore.kernel.org/r/20220520081158.2990006-1-tmricht@linux.ibm.com Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com> Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
parent
e80fcc3f04
commit
2332db753f
1 changed files with 1 additions and 1 deletions
|
@ -1626,7 +1626,7 @@ static int __bench_numa(const char *name)
|
||||||
"GB/sec,", "total-speed", "GB/sec total speed");
|
"GB/sec,", "total-speed", "GB/sec total speed");
|
||||||
|
|
||||||
if (g->p.show_details >= 2) {
|
if (g->p.show_details >= 2) {
|
||||||
char tname[14 + 2 * 10 + 1];
|
char tname[14 + 2 * 11 + 1];
|
||||||
struct thread_data *td;
|
struct thread_data *td;
|
||||||
for (p = 0; p < g->p.nr_proc; p++) {
|
for (p = 0; p < g->p.nr_proc; p++) {
|
||||||
for (t = 0; t < g->p.nr_threads; t++) {
|
for (t = 0; t < g->p.nr_threads; t++) {
|
||||||
|
|
Loading…
Reference in a new issue