poll_wait: add mb() to fix theoretical race between waitqueue_active() and .poll()
Status from this message: Applied to the vfs-6.14.poll branch of the vfs/vfs.git tree.
Get series with b4
b4 am [email protected]Versions newest first
Patches in v1 Applied
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/5 | poll_wait: add mb() to fix theoretical race between waitqueue_active() and .poll()include/ | +9 | 0 replies | |
| 2/5 | poll_wait: kill the obsolete wait_address checkinclude/ | +1 | 0 replies | |
| 3/5 | io_uring_poll: kill the no longer necessary barrier after poll_wait()io_uring/ | +4 | 0 replies | |
| 4/5 | sock_poll_wait: kill the no longer necessary barrier after poll_wait()include/ | +7 | 0 replies | |
| 5/5 | poll: kill poll_does_not_wait()include/ | +3 | 0 replies |
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.
Cover letter [PATCH 0/5] poll_wait: add mb() to fix theoretical race between waitqueue_active() and .poll()
Linus,
I misread fs/eventpoll.c, it has the same problem. And more __pollwait()-like functions, for example p9_pollwait(). So 1/5 adds mb() into poll_wait(), not into __pollwait().
WangYuli, after 1/5 we can reconsider your patch.
Oleg.
---
include/linux/poll.h | 26 ++++++++++++-------------- include/net/sock.h | 17 +++++++---------- io_uring/io_uring.c | 9 ++++----- 3 files changed, 23 insertions(+), 29 deletions(-)