Patch "fs/pipe: Read pipe->{head,tail} atomically outside pipe->mutex" has been added to the 5.10-stable tree
A message that is not in the archive (sent off-list or not yet received): [email protected]
Note that this was a real fix, but the fix only matters if commit aaec5a95d596 ("pipe_read: don't wake up the writer if the pipe is still full") is in the tree.
Now, the bug was pre-existing, and *maybe* it could be hit without that commit aaec5a95d596, but nobody has ever reported it, so it's very very unlikely.
Also, this fix then had some fall-out, and while I think you've queued all the fallout fixes too, I think it might be a good idea to wait for more reports from the development tree before considering these for stable.
Put another way: this fix caused some pain. It might not be worth back-porting to stable at all, and if it is, it might be worth waiting to see that there's no other fallout.
Linus
Quoted 4 lines · Sasha Levin
This is a note to let you know that I've just added the patch titled
fs/pipe: Read pipe->{head,tail} atomically outside pipe->mutex
Quoted 8 lines · Linus Torvalds
Note that this was a real fix, but the fix only matters if commit
aaec5a95d596("pipe_read: don't wake up the writer if the pipe is still full") is in the tree.Now, the bug was pre-existing, and *maybe* it could be hit without that commit
aaec5a95d596, but nobody has ever reported it, so it's very very unlikely.
Quoted 8 lines
Also, this fix then had some fall-out, and while I think you've queued all the fallout fixes too, I think it might be a good idea to wait for more reports from the development tree before considering these for stable.
Put another way: this fix caused some pain. It might not be worth back-porting to stable at all, and if it is, it might be worth waiting to see that there's no other fallout.
Thanks for letting us know, I've dropped these and the other "fixups" for the pipe patch from all stable queues now.
greg k-h