diff options
author | Naoya Horiguchi <naoya.horiguchi@nec.com> | 2022-03-22 22:44:03 +0100 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2022-03-22 23:57:06 +0100 |
commit | ae483c20062695324202d19e5283819b11b83eaa (patch) | |
tree | 7e21467929de0f725ae2d7ae40492bd140b114ad /mm/memory-failure.c | |
parent | mm/page_alloc: check high-order pages for corruption during PCP operations (diff) | |
download | linux-ae483c20062695324202d19e5283819b11b83eaa.tar.xz linux-ae483c20062695324202d19e5283819b11b83eaa.zip |
mm/memory-failure.c: remove obsolete comment
With the introduction of mf_mutex, most of memory error handling process
is mutually exclusive, so the in-line comment about subtlety about
double-checking PageHWPoison is no more correct. So remove it.
Link: https://lkml.kernel.org/r/20220125025601.3054511-1-naoya.horiguchi@linux.dev
Signed-off-by: Naoya Horiguchi <naoya.horiguchi@nec.com>
Suggested-by: Mike Kravetz <mike.kravetz@oracle.com>
Reviewed-by: Miaohe Lin <linmiaohe@huawei.com>
Reviewed-by: Anshuman Khandual <anshuman.khandual@arm.com>
Reviewed-by: Oscar Salvador <osalvador@suse.de>
Reviewed-by: Yang Shi <shy828301@gmail.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to '')
-rw-r--r-- | mm/memory-failure.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/mm/memory-failure.c b/mm/memory-failure.c index 97a9ed8f87a9..0f6413a2f301 100644 --- a/mm/memory-failure.c +++ b/mm/memory-failure.c @@ -2150,12 +2150,6 @@ static int __soft_offline_page(struct page *page) .gfp_mask = GFP_USER | __GFP_MOVABLE | __GFP_RETRY_MAYFAIL, }; - /* - * Check PageHWPoison again inside page lock because PageHWPoison - * is set by memory_failure() outside page lock. Note that - * memory_failure() also double-checks PageHWPoison inside page lock, - * so there's no race between soft_offline_page() and memory_failure(). - */ lock_page(page); if (!PageHuge(page)) wait_on_page_writeback(page); |