On Tue, May 26, 2020 at 01:51:13PM -0600, Jens Axboe wrote: > The read-ahead shouldn't block, so allow it to be done even if > IOCB_NOWAIT is set in the kiocb. > > Signed-off-by: Jens Axboe <ax...@kernel.dk>
Acked-by: Johannes Weiner <han...@cmpxchg.org> Looks reasonable. Especially after patch 1 - although it seems that even before that, IOCB_NOWAIT could have occasionally ended up in page_cache_async_readahead(), which isn't too different from the sync variant except for the range calculations, and may have blocked on IO submission in the past.