On 9/8/21 8:57 PM, Jens Axboe wrote: > On 9/8/21 1:49 PM, Pavel Begunkov wrote: >> In case of !SQPOLL, io_cqring_ev_posted_iopoll() doesn't provide a >> memory barrier required by waitqueue_active(&ctx->poll_wait). There is >> a wq_has_sleeper(), which does smb_mb() inside, but it's called only for >> SQPOLL. > > We can probably get rid of the need to even do so by having the slow > path (eg someone waiting on cq_wait or poll_wait) a bit more expensive, > but this should do for now. You have probably seen smp_mb__after_spin_unlock() trick [1], easy way to get rid of it for !IOPOLL. Haven't figured it out for IOPOLL, though [1] https://github.com/isilence/linux/commit/bb391b10d0555ba2d55aa8ee0a08dff8701a6a57 -- Pavel Begunkov
