On Wed, Jun 14, 2023 at 11:23:52AM -0700, Robert Kolchmeyer wrote:
Hi all,
I believe 5.10.168 and 5.15.93 introduced a regression that impacts the Cilium project. Some information on the nature of the regression is available at https://github.com/cilium/cilium/issues/25500. The primary symptom seems to be the error `BPF program is too large.`
My colleague has found that reverting the following two commits:
8de8c4a "bpf: Support <8-byte scalar spill and refill" 9ff2beb "bpf: Fix incorrect state pruning for <8B spill/fill"
resolves the regression.
If we revert these in the stable tree, there may be a few changes that depend on those that also need to be reverted, but I'm not sure yet.
Would it make sense to revert these changes (and any dependent ones) in the 5.10 and 5.15 trees? If anyone has other ideas, I can help test possible solutions.
Can you actually test if those reverts work properly for you and if there are other dependencies involved?
And is this issue also in 6.1.y and Linus's tree? If not, why not, are we just missing a commit? We can't revert something from a stable release if you are going to hit the same issue when moving to a new release, right?
thanks,
greg k-h