On Sat, Oct 31, 2020 at 09:59:58AM +0100, Christoph Hellwig wrote: > Move the call to filemap_make_page_uptodate for a newly allocated page > into filemap_new_page, which turns the new vs lookup decision into a > plain if / else statement, rename two identifier to be more obvious > and the function itself to filemap_read_pages which describes > it a little better while being much shorter. We don't need to do this -- filemap_read_page waits for the page to become unlocked, so we already know that it's uptodate (or an error!) and we know it isn't going to have the readahead bit set.