Re: [RFC PATCH bpf-next v4 3/3] selftests/bpf: build each test runner instance in its own sub-make

Correct - the message was stale (the derivation was dropped in v4 per
review); wording fixed in v5, no code change needed:
https://lore.kernel.org/bpf/20260804170156.1709916-1-nickolay.lysenko@gmail=
.com/

On Thu, Jul 23, 2026 at 9:35=E2=80=AFPM <[email protected]> wrote:
>
> Thank you for your contribution! Sashiko AI review found 1 potential issu=
e(s) to consider:
> - [Medium] The commit message claims that light-skeleton demand lists are=
 derived dynamically, but the implementation retains hardcoded lists.
> --
>
> --- Patch [3]: [RFC PATCH bpf-next v4 3/3] selftests/bpf: build each test=
 runner instance in its own sub-make ---
> commit a1ebeec9064cd1ef457a1d6c67e818e26521e31b
> Author: Mykola Lysenko <[email protected]>
>
> selftests/bpf: build each test runner instance in its own sub-make
>
> This commit replaces the double-expansion test runner logic with a
> Makefile.runner script invoked per test runner instance. It aims to simpl=
ify
> the build system and fixes a latent parallel-build race condition.
>
> >     standalone binaries and the kselftest lib.mk contract - and delegat=
es to
> >     the runner through explicit per-instance rules. Light-skeleton dema=
nd
> >     lists are derived from the tests' own '#include "*.lskel.h"' lines;=
 the
> >     bench object list is derived from $(wildcard benchs/bench_*.c).
>
> [Severity: Medium]
> The commit message mentions that the light-skeleton demand lists are deri=
ved
> dynamically, but the implementation in Makefile.runner retains the hardco=
ded
> lists.
>
> Was the dynamic derivation logic dropped from this patch, or does the com=
mit
> message need to be updated to match the code?
>
> For reference, Makefile.runner still hardcodes the lists here:
>
> > diff --git a/tools/testing/selftests/bpf/Makefile.runner b/tools/testin=
g/selftests/bpf/Makefile.runner
> > new file mode 100644
> > index 0000000000000..ced44aad7eded
> > --- /dev/null
> > +++ b/tools/testing/selftests/bpf/Makefile.runner
> [ ... ]
> > +LSKELS :=3D fexit_sleep.c trace_printk.c trace_vprintk.c map_ptr_kern.=
c        \
> > +     core_kern.c core_kern_overflow.c test_ringbuf.c                 \
> > +     test_ringbuf_n.c test_ringbuf_map_key.c test_ringbuf_write.c    \
> > +     test_ringbuf_overwrite.c
> > +
> > +LSKELS_SIGNED :=3D fentry_test.c fexit_test.c atomics.c
> > +
> > +# Generate both light skeleton and libbpf skeleton for these
> > +LSKELS_EXTRA :=3D test_ksyms_module.c test_ksyms_weak.c kfunc_call_tes=
t.c \
> > +     kfunc_call_test_subprog.c
>
> --
> Sashiko AI review =C2=B7 https://sashiko.dev/#/patchset/20260724042600.17=
[email protected]?part=3D3