* [PATCH] perf bench messaging: Fix the formatting issue when printing.
@ 2025-09-22 9:17 liujing
2026-09-15 22:11 ` Ian Rogers
0 siblings, 1 reply; 4+ messages in thread
From: liujing @ 2025-09-22 9:17 UTC (permalink / raw)
To: peterz
Cc: mingo, acme, namhyung, mark.rutland, alexander.shishkin, jolsa,
irogers, adrian.hunter, kan.liang, linux-perf-users, linux-kernel,
liujing
The unsigned int type should use the %u format specifier when
printing with printf or related functions.
Signed-off-by: liujing <liujing@cmss.chinamobile.com>
---
tools/perf/bench/sched-messaging.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tools/perf/bench/sched-messaging.c b/tools/perf/bench/sched-messaging.c
index 93dcd9dba3d0..0a447f279d68 100644
--- a/tools/perf/bench/sched-messaging.c
+++ b/tools/perf/bench/sched-messaging.c
@@ -338,9 +338,9 @@ int bench_sched_messaging(int argc, const char **argv)
switch (bench_format) {
case BENCH_FORMAT_DEFAULT:
- printf("# %d sender and receiver %s per group\n",
+ printf("# %u sender and receiver %s per group\n",
num_fds, thread_mode ? "threads" : "processes");
- printf("# %d groups == %d %s run\n\n",
+ printf("# %u groups == %u %s run\n\n",
num_groups, num_groups * 2 * num_fds,
thread_mode ? "threads" : "processes");
printf(" %14s: %lu.%03lu [sec]\n", "Total time",
--
2.27.0
^ permalink raw reply related [flat|nested] 4+ messages in thread* [PATCH] perf bench messaging: Fix the formatting issue when printing.
2025-09-22 9:17 [PATCH] perf bench messaging: Fix the formatting issue when printing liujing
@ 2026-09-15 22:11 ` Ian Rogers
2026-09-15 22:16 ` sashiko-bot
2026-09-16 13:55 ` Arnaldo Carvalho de Melo
0 siblings, 2 replies; 4+ messages in thread
From: Ian Rogers @ 2026-09-15 22:11 UTC (permalink / raw)
To: peterz
Cc: mingo, acme, namhyung, mark.rutland, alexander.shishkin, jolsa,
irogers, adrian.hunter, kan.liang, linux-perf-users, linux-kernel,
liujing
From: liujing <liujing@cmss.chinamobile.com>
The unsigned int type should use the %u format specifier when
printing with printf or related functions.
Signed-off-by: liujing <liujing@cmss.chinamobile.com>
Reviewed-by: Ian Rogeres <irogers@google.com>
---
tools/perf/bench/sched-messaging.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tools/perf/bench/sched-messaging.c b/tools/perf/bench/sched-messaging.c
index 4fb6657fc826..c5c9a7a1d7db 100644
--- a/tools/perf/bench/sched-messaging.c
+++ b/tools/perf/bench/sched-messaging.c
@@ -338,9 +338,9 @@ int bench_sched_messaging(int argc, const char **argv)
switch (bench_format) {
case BENCH_FORMAT_DEFAULT:
- printf("# %d sender and receiver %s per group\n",
+ printf("# %u sender and receiver %s per group\n",
num_fds, thread_mode ? "threads" : "processes");
- printf("# %d groups == %d %s run\n\n",
+ printf("# %u groups == %u %s run\n\n",
num_groups, num_groups * 2 * num_fds,
thread_mode ? "threads" : "processes");
printf(" %14s: %lu.%03lu [sec]\n", "Total time",
--
2.55.0.1032.g73a4cd73de-goog
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH] perf bench messaging: Fix the formatting issue when printing.
2026-09-15 22:11 ` Ian Rogers
@ 2026-09-15 22:16 ` sashiko-bot
2026-09-16 13:55 ` Arnaldo Carvalho de Melo
1 sibling, 0 replies; 4+ messages in thread
From: sashiko-bot @ 2026-09-15 22:16 UTC (permalink / raw)
To: Ian Rogers; +Cc: linux-perf-users
> From: liujing <liujing@cmss.chinamobile.com>
>
> The unsigned int type should use the %u format specifier when
> printing with printf or related functions.
>
> Signed-off-by: liujing <liujing@cmss.chinamobile.com>
Sashiko has reviewed this patch and found no issues. It looks great!
--
Sashiko AI review · https://sashiko.dev/#/patchset/20260915221122.2174928-1-irogers@google.com?part=1
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] perf bench messaging: Fix the formatting issue when printing.
2026-09-15 22:11 ` Ian Rogers
2026-09-15 22:16 ` sashiko-bot
@ 2026-09-16 13:55 ` Arnaldo Carvalho de Melo
1 sibling, 0 replies; 4+ messages in thread
From: Arnaldo Carvalho de Melo @ 2026-09-16 13:55 UTC (permalink / raw)
To: Ian Rogers
Cc: peterz, mingo, namhyung, mark.rutland, alexander.shishkin, jolsa,
adrian.hunter, kan.liang, linux-perf-users, linux-kernel, liujing
On Tue, Sep 15, 2026 at 03:11:22PM -0700, Ian Rogers wrote:
> From: liujing <liujing@cmss.chinamobile.com>
>
> The unsigned int type should use the %u format specifier when
> printing with printf or related functions.
>
> Signed-off-by: liujing <liujing@cmss.chinamobile.com>
> Reviewed-by: Ian Rogeres <irogers@google.com>
s/Rogeres/Rogers/g
Applied,
- Arnaldo
> ---
> tools/perf/bench/sched-messaging.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tools/perf/bench/sched-messaging.c b/tools/perf/bench/sched-messaging.c
> index 4fb6657fc826..c5c9a7a1d7db 100644
> --- a/tools/perf/bench/sched-messaging.c
> +++ b/tools/perf/bench/sched-messaging.c
> @@ -338,9 +338,9 @@ int bench_sched_messaging(int argc, const char **argv)
>
> switch (bench_format) {
> case BENCH_FORMAT_DEFAULT:
> - printf("# %d sender and receiver %s per group\n",
> + printf("# %u sender and receiver %s per group\n",
> num_fds, thread_mode ? "threads" : "processes");
> - printf("# %d groups == %d %s run\n\n",
> + printf("# %u groups == %u %s run\n\n",
> num_groups, num_groups * 2 * num_fds,
> thread_mode ? "threads" : "processes");
> printf(" %14s: %lu.%03lu [sec]\n", "Total time",
> --
> 2.55.0.1032.g73a4cd73de-goog
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2026-09-16 13:55 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-09-22 9:17 [PATCH] perf bench messaging: Fix the formatting issue when printing liujing
2026-09-15 22:11 ` Ian Rogers
2026-09-15 22:16 ` sashiko-bot
2026-09-16 13:55 ` Arnaldo Carvalho de Melo
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.