Re: [PATCH bpf-next 04/13] bpf: move bpf_get_btf_vmlinux() to btf.c

[bot+bpf-ci]

> bpf: move bpf_get_btf_vmlinux() to btf.c

TL;DR: Commit message only: the author's Signed-off-by is behind a
TODO-NEEDS-IHOR placeholder, so it won't parse as a trailer; most other
patches in the series have the same placeholder.

> bpf: move bpf_get_btf_vmlinux() to btf.c
>
> Move bpf_get_btf_vmlinux(), bpf_peek_btf_vmlinux() and btf_vmlinux_lock
> from verifier.c to btf.c, next to btf_parse_vmlinux().
>
> Parsing the vmlinux BTF on first use (later in this series) makes the
> accessor apply deferred kfunc, dtor kfunc and struct_ops registrations
> and kept module BTF under btf_vmlinux_lock.  That state lives in btf.c;
> with the accessor and its lock there, it can stay static.
>
> No functional change.
>
> TODO-NEEDS-IHOR: Signed-off-by: Ihor Solodrai <ihor.solodrai@xxxxxxxxx>
> Signed-off-by: Jay Wang <wanjay@xxxxxxxxxx>

The author's sign-off is behind a placeholder prefix, so git
interpret-trailers will read that line as a TODO-NEEDS-IHOR trailer and not
as a Signed-off-by.  That leaves only the Signed-off-by from Jay Wang in the
trailer block.

Documentation/process/submitting-patches.rst asks for the author's
Signed-off-by ahead of the sender's.  Should the placeholder line be
replaced with a real "Signed-off-by: Ihor Solodrai
<ihor.solodrai@xxxxxxxxx>" once the author provides it?

The same placeholder shows up in most of the other commits in this series
(2a58754b9ead, 32ad8adbb1a8, cc9b43c0e23c, 699a8e9536e8, c97dea7ddbb0,
38e2751cdb40, 66c4a6a9293d, e46b5971bc91, cae96a9f7395), so those would need
the same fix.


---
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/41205212136