On Sat, 2023-12-02 at 18:09 -0500, Andrei Matei wrote: > [...] > > > --- a/tools/testing/selftests/bpf/progs/verifier_raw_stack.c > > +++ b/tools/testing/selftests/bpf/progs/verifier_raw_stack.c > > @@ -5,9 +5,10 @@ > > #include <bpf/bpf_helpers.h> > > #include "bpf_misc.h" > > > > -SEC("tc") > > +SEC("socket") > > __description("raw_stack: no skb_load_bytes") > > -__failure __msg("invalid read from stack R6 off=-8 size=8") > > +__success > > +__failure_unpriv __msg_unpriv("invalid read from stack R6 off=-8 size=8") > > __naked void stack_no_skb_load_bytes(void) > > { > > Please confirm that changing this program type is OK. I wasn't sure here. > > [...] lgtm, does not seem matter in this case.