diff options
| author | Matthew Wilcox (Oracle) <willy@infradead.org> | 2021-05-25 10:11:25 -0400 |
|---|---|---|
| committer | Matthew Wilcox (Oracle) <willy@infradead.org> | 2022-01-02 20:28:35 -0500 |
| commit | 9144785b02765409f63b2fe16948bc746248cf1f (patch) | |
| tree | 7a0c490bcead29729e8a624c1091e034e22bab13 /mm/filemap.c | |
| parent | 2a987e65025e2b79c6d453b78cb5985ac6e5eb26 (diff) | |
| download | linux-9144785b02765409f63b2fe16948bc746248cf1f.tar.gz | |
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: Matthew Wilcox (Oracle) <willy@infradead.org>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: William Kucharski <william.kucharski@oracle.com>
Diffstat (limited to 'mm/filemap.c')
| -rw-r--r-- | mm/filemap.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/mm/filemap.c b/mm/filemap.c index daa0e23a6ee666..39c4c46c61337e 100644 --- a/mm/filemap.c +++ b/mm/filemap.c @@ -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) |