On Mon, Mar 23, 2020 at 06:40:32AM -0700, Matthew Wilcox wrote: > Oh, I see that now. It uses readahead_gfp_mask(), and I was grepping for > GFP_NORETRY so I didn't spot it. It falls back to block_read_full_page() > which we can't do. That will allocate smaller BIOs, so there's an argument > that we should do the same. How about this: That looks silly to me. This just means we'll keep iterating over small bios for readahead.. Either we just ignore the different gfp mask, or we need to go all the way and handle errors, although that doesn't really look nice.
