Re: [PATCH bpf-next v12 2/6] bpf: Add bpf_prog_run_array_sleepable()

Kumar Kartikeya Dwivedi
2026-04-22T18:02:44Z

On Wed, 22 Apr 2026 at 19:57, Kumar Kartikeya Dwivedi <memxor@gmail.com> wrote:
>
> On Wed, 22 Apr 2026 at 19:00, Alexei Starovoitov
> <alexei.starovoitov@gmail.com> wrote:
> >
> > On Wed, Apr 22, 2026 at 9:36 AM Mykyta Yatsenko
> > <mykyta.yatsenko5@gmail.com> wrote:
> > >
> > > >> +
> > > >> + if (prog->sleepable) {
> > > >> + guard(rcu)();
> > > >> + ret &= run_prog(prog, ctx);
> > > >> + } else {
> > > >> + ret &= run_prog(prog, ctx);
> > > >> + }
> > > >
> > >
> > > Yep, this one should be inverted.
> >
> > Ohh and CI was green. Looks like there is a gap in test coverage.
> > I thought you added a test that does something like bpf_copy_from_user.
> > We should have seen the splat with config_debug_atomic_sleep.
> > What happened?
>
> CI will likely be green even with splat, since it may not be
> registered as test failure?

I can see the warning here:
https://github.com/kernel-patches/bpf/actions/runs/24787752184/job/72539372270#step:7:8176
We likely need to add some post processing step on dmesg that errors
out on new BUG/WARNING lines in dmesg.