* [PATCH v2] tools/bpf:Fix the wrong format specifier
@ 2024-07-24 8:52 Zhu Jun
2024-07-24 9:05 ` Quentin Monnet
0 siblings, 1 reply; 2+ messages in thread
From: Zhu Jun @ 2024-07-24 8:52 UTC (permalink / raw)
To: qmo
Cc: ast, daniel, martin.lau, john.fastabend, kpsingh, sdf, haoluo,
bpf, linux-kernel, zhujun2
The format specifier of "unsigned int" in printf() should be "%u", not
"%d".
Signed-off-by: Zhu Jun <zhujun2@cmss.chinamobile.com>
---
Changes
v2:modify commit info
tools/bpf/bpftool/xlated_dumper.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tools/bpf/bpftool/xlated_dumper.c b/tools/bpf/bpftool/xlated_dumper.c
index 567f56dfd9f1..3efa639434be 100644
--- a/tools/bpf/bpftool/xlated_dumper.c
+++ b/tools/bpf/bpftool/xlated_dumper.c
@@ -349,7 +349,7 @@ void dump_xlated_plain(struct dump_data *dd, void *buf, unsigned int len,
double_insn = insn[i].code == (BPF_LD | BPF_IMM | BPF_DW);
- printf("% 4d: ", i);
+ printf("% 4u: ", i);
print_bpf_insn(&cbs, insn + i, true);
if (opcodes) {
@@ -415,7 +415,7 @@ void dump_xlated_for_graph(struct dump_data *dd, void *buf_start, void *buf_end,
}
}
- printf("%d: ", insn_off);
+ printf("%u: ", insn_off);
print_bpf_insn(&cbs, cur, true);
if (opcodes) {
--
2.17.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH v2] tools/bpf:Fix the wrong format specifier
2024-07-24 8:52 [PATCH v2] tools/bpf:Fix the wrong format specifier Zhu Jun
@ 2024-07-24 9:05 ` Quentin Monnet
0 siblings, 0 replies; 2+ messages in thread
From: Quentin Monnet @ 2024-07-24 9:05 UTC (permalink / raw)
To: Zhu Jun
Cc: ast, daniel, martin.lau, john.fastabend, kpsingh, sdf, haoluo,
bpf, linux-kernel
2024-07-24 01:52 UTC-0700 ~ Zhu Jun <zhujun2@cmss.chinamobile.com>
> The format specifier of "unsigned int" in printf() should be "%u", not
> "%d".
>
> Signed-off-by: Zhu Jun <zhujun2@cmss.chinamobile.com>
> ---
> Changes
> v2:modify commit info
>
> tools/bpf/bpftool/xlated_dumper.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tools/bpf/bpftool/xlated_dumper.c b/tools/bpf/bpftool/xlated_dumper.c
> index 567f56dfd9f1..3efa639434be 100644
> --- a/tools/bpf/bpftool/xlated_dumper.c
> +++ b/tools/bpf/bpftool/xlated_dumper.c
> @@ -349,7 +349,7 @@ void dump_xlated_plain(struct dump_data *dd, void *buf, unsigned int len,
>
> double_insn = insn[i].code == (BPF_LD | BPF_IMM | BPF_DW);
>
> - printf("% 4d: ", i);
> + printf("% 4u: ", i);
Please see feedback on v1 and address the warning.
pw-bot: changes-requested
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2024-07-24 9:05 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-07-24 8:52 [PATCH v2] tools/bpf:Fix the wrong format specifier Zhu Jun
2024-07-24 9:05 ` Quentin Monnet
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox