* [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; 3+ 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 [flat|nested] 3+ 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-16 13:55 ` Arnaldo Carvalho de Melo
0 siblings, 1 reply; 3+ 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 [flat|nested] 3+ messages in thread
* Re: [PATCH] perf bench messaging: Fix the formatting issue when printing.
2026-09-15 22:11 ` Ian Rogers
@ 2026-09-16 13:55 ` Arnaldo Carvalho de Melo
0 siblings, 0 replies; 3+ 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] 3+ messages in thread
end of thread, other threads:[~2026-09-16 13:55 UTC | newest]
Thread overview: 3+ 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-16 13:55 ` Arnaldo Carvalho de Melo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®