Re: FAILED: patch "[PATCH] bpf: Fix toctou on read-only map's constant scalar tracking" failed to apply to 5.4-stable tree

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Lee, [ also Cc'ing Rafael, ]

On 2/21/22 11:44 AM, Lee Jones wrote:
On Mon, 21 Feb 2022, Greg KH wrote:
On Mon, Feb 21, 2022 at 09:52:34AM +0000, Lee Jones wrote:
On Thu, 17 Feb 2022, Lee Jones wrote:
On Thu, 17 Feb 2022, Greg KH wrote:
On Thu, Feb 17, 2022 at 03:57:23PM +0000, Lee Jones wrote:
On Mon, 22 Nov 2021, gregkh@xxxxxxxxxxxxxxxxxxx wrote:

The patch below does not apply to the 5.4-stable tree.
If someone wants it applied there, or to any other stable or longterm
tree, then please email the backport, including the original git commit
id to <stable@xxxxxxxxxxxxxxx>.

We are in receipt of a bug report which cites this patch as the fix.

Does the bug report really say that this issue is present in the 5.4
kernel tree?  Anything older?

Not specifically, but the commit referenced in the Fixes tag landed in
v5.5. and was successfully back-ported to v5.4.144.

Another potential avenue might to be revert the back-ported commit
which caused the issue from v5.4.y.

Unless that was fixing a different issue?  I have no idea at this point
what commit you are talking about, sorry :(

The bad-commit mentioned in "the Fixes tag":

Fixes: a23740ec43ba ("bpf: Track contents of read-only maps as scalars")

Which as you say, could well have been fixing another issue.

In fact, yes it was:

https://lore.kernel.org/stable/20210821203108.215937-2-rafaeldtinoco@xxxxxxxxx/

Daniel, what do you suggest please?

Hm, okay, so a23740ec43ba ("bpf: Track contents of read-only maps as scalars") was
backported to 5.4.144 given Rafael needed it to fix a failing regression test [0].

Normally, I would have said that we should just revert a23740ec43ba given it was
not a 'fix' in the first place, but then we are getting into a situation where it
would break Rafael's now functioning test case again on 5.4.144+ released kernels.

Rafael, given you need this, do you have some cycles to help out Lee on this backport
for 5.4 stable?

Thanks guys,
Daniel

 [0] https://lore.kernel.org/stable/20210821203108.215937-1-rafaeldtinoco@xxxxxxxxx/



[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux