Re: [PATCH bpf-next v3] tools/bpf: Remove legacy BPF tools

[bot+bpf-ci]

> tools/bpf: Remove legacy BPF tools

TL;DR: Commit message only: the changelog doesn't say why bpf_asm, bpf_dbg and
bpf_jit_disasm are removed or what replaces them. Not a bug.

>     tools/bpf: Remove legacy BPF tools
>
>     Remove bpf_asm, bpf_dbg and bpf_jit_disasm, along with the assembler's
>     lexer and parser, as suggested by Daniel.
>
>     Drop their build and install rules, feature checks and ignore entries.
>     Remove the tool-specific documentation and update the PTP classifier and
>     packet socket test comments. Keep the classic BPF instruction descriptions
>     and assembly examples.
>
>     Keep the bpftool and resolve_btfids build and clean targets, and the
>     bpftool install target.
>
>     Suggested-by: Daniel Borkmann <daniel@xxxxxxxxxxxxx>
>     Link: https://lore.kernel.org/bpf/61f7315e-c835-425e-a99b-d2470ba9a875@xxxxxxxxxxxxx/
>     Signed-off-by: Xia Yan <yanxia@xxxxxxxxxxxxxxxxxxxx>

This isn't a bug, but could the changelog say briefly why these tools are
being removed and what users should use instead?

The body lists what is removed (build and install rules, feature checks,
ignore entries, documentation, comments), but the only stated reason is "as
suggested by Daniel" plus the Link: tag.

The documentation hunks also drop the in-tree explanation of how to produce
xt_bpf/cls_bpf bytecode and how to disassemble a JIT image dump from the
kernel log.  A short pointer to the replacements, for example tcpdump -ddd
for the bytecode or bpftool for JIT dumps, would let the commit stand on its
own without relying on the Link:.


---
AI reviewed your patch. Please fix the bug or email reply why it's not a bug.
See: https://github.com/kernel-patches/vmtest/blob/master/ci/claude/README.md

CI run summary: https://github.com/kernel-patches/bpf/actions/runs/39405560147