aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNaoya Horiguchi <[email protected]>2022-03-22 14:44:03 -0700
committerLinus Torvalds <[email protected]>2022-03-22 15:57:06 -0700
commitae483c20062695324202d19e5283819b11b83eaa (patch)
tree7e21467929de0f725ae2d7ae40492bd140b114ad
parent77fe7f136a7312954b1b8b7eeb4bc91fc3c14a3f (diff)
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/[email protected] Signed-off-by: Naoya Horiguchi <[email protected]> Suggested-by: Mike Kravetz <[email protected]> Reviewed-by: Miaohe Lin <[email protected]> Reviewed-by: Anshuman Khandual <[email protected]> Reviewed-by: Oscar Salvador <[email protected]> Reviewed-by: Yang Shi <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
-rw-r--r--mm/memory-failure.c6
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);