Re: [PATCH bpf-next v3 1/5] bpf: do not print a newline after disassembly in bpf_verbose_insn()
Quentin Monnet <[email protected]> Mon, 3 Aug 2026 01:22:54 +0100
| Newsgroups | org.kernel.vger.bpf |
|---|---|
| Message-ID | <[email protected]> |
On 02/08/2026 21:24, Eduard Zingerman wrote: > At the moment there are more callsites that want bpf_verbose_insn() to > not print a newline after the instruction, than callsites that want a > newline. Drop '\n' from disasm.c. Non-functional change. > > The changes in bpftool are verified by writing a bpf program using a > variety of instructions and comparing `prog dump xlated` output in the > following modes: plain, opcodes, visual, visual opcodes. > The output before and after the changes is identical. > > Cc: Quentin Monnet <[email protected]> > Signed-off-by: Eduard Zingerman <[email protected]> Looks good, and looking at the diff, the change makes sense. Thanks! Reviewed-by: Quentin Monnet <[email protected]>