Commit 9144785b authored by Matthew Wilcox (Oracle)'s avatar Matthew Wilcox (Oracle)
Browse files

filemap: Remove PageHWPoison check from next_uptodate_page() 



Pages are individually marked as suffering from hardware poisoning.
Checking that the head page is not hardware poisoned doesn't make
sense; we might be after a subpage.  We check each page individually
before we use it, so this was an optimisation gone wrong.

Signed-off-by: default avatarMatthew Wilcox (Oracle) <willy@infradead.org>
Reviewed-by: default avatarChristoph Hellwig <hch@lst.de>
Reviewed-by: default avatarWilliam Kucharski <william.kucharski@oracle.com>
parent 2a987e65

mm/filemap.c

+0 −2
Original line number Diff line number Diff line
@@ -3253,8 +3253,6 @@ static struct page *next_uptodate_page(struct page *page, 
			goto skip;

		if (!PageUptodate(page) || PageReadahead(page))

			goto skip;

		if (PageHWPoison(page))

			goto skip;

		if (!trylock_page(page))

			goto skip;

		if (page->mapping != mapping)