Björn Töpel wrote: > John Fastabend <john.fastabend@xxxxxxxxx> writes: > > >> Took the series for a run, and it does solve crash, but I'm getting > >> additional failures: > > > > Thanks! I'm guessing those tests were failing even without the patch > > though right? > > Correct. Thanks Bjorn! > > test_sockmap did however pass the full suite in 6.12. So, something > changed with the addition of [1]. Agree with finding this. I also have a compliance test failing in one of our nginx/apache/bpf test suites so might be related. I'll dig into it. > > I guess: > > Tested-by: Björn Töpel <bjorn@xxxxxxxxxx> Because when I find the bug that gets here I would have stacked this on top of any fix as well I think lets apply this now. Reviewed-by: John Fastabend <john.fastabend@xxxxxxxxx> > > can be added to this series, and not crashing is nice, but it would be > interesting to know how it got there. > > > Björn > > > [1] https://lore.kernel.org/all/20241106222520.527076-1-zijianzhang@xxxxxxxxxxxxx/ >
