diff options
| author | Matthew Wilcox (Oracle) <[email protected]> | 2021-12-10 14:46:18 -0800 |
|---|---|---|
| committer | Linus Torvalds <[email protected]> | 2021-12-10 17:10:55 -0800 |
| commit | 0c941cf30b913d4a684d3f8d9eee60e0daffdc79 (patch) | |
| tree | e8e947137bfffc6d7bdca780cf78da0e81b27b3e | |
| parent | d020d9e63d5396fbcc3a2c01cee38e28c7d20a3d (diff) | |
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. It will
cause us to fall back to the slow path when there was no need to do
that
Link: https://lkml.kernel.org/r/[email protected]
Signed-off-by: Matthew Wilcox (Oracle) <[email protected]>
Reviewed-by: Naoya Horiguchi <[email protected]>
Cc: Yang Shi <[email protected]>
Cc: "Kirill A . Shutemov" <[email protected]>
Cc: Hugh Dickins <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
| -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 daa0e23a6ee6..39c4c46c6133 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) |