On Mon, Sep 07, 2020 at 09:21:02AM -0600, Jens Axboe wrote: > Pipe only looks at O_NONBLOCK for non-blocking operation, which means that > io_uring can't easily poll for it or attempt non-blocking issues. Check for > IOCB_NOWAIT in locking the pipe for reads and writes, and ditto when we > decide on whether or not to block or return -EAGAIN. > > Signed-off-by: Jens Axboe <axboe@xxxxxxxxx> > > --- > > If this is acceptable, then I can add S_ISFIFO to the whitelist on file > descriptors we can IOCB_NOWAIT try for, then poll if we get -EAGAIN > instead of using thread offload. Will check. In the meanwhile, blacklist eventpoll again. Because your attempts at "nonblocking" there had been both ugly as hell *AND* fail to prevent blocking. And frankly, I'm very tempted to rip that crap out entirely. Seriously, *look* at the code you've modified in do_epoll_ctl(). And tell me why the hell is grabbing ->mtx in that function needs to be infested with trylocks, while exact same mutex taken in loop_check_proc() called under those is fine with mutex_lock(). Ditto for calls of vfs_poll() inside ep_insert(), GFP_KERNEL allocations in ep_ptable_queue_proc(), synchronize_rcu() callable from ep_modify() (from the same function), et sodding cetera. No, this is _not_ an invitation to spread the same crap over even more places in there; I just want to understand where had that kind of voodoo approach comes from. And that's directly relevant for this patch, because it looks like the same kind of thing. What is your semantics for IOCB_NOWAIT? What should and what should _not_ be waited for?
