diff mbox series

[4/4] mm/hwpoison: fix some obsolete comments

Message ID 20210814105131.48814-5-linmiaohe@huawei.com (mailing list archive)
State New
Headers show
Series Cleanups and fixup for memcontrol | expand

Commit Message

Miaohe Lin Aug. 14, 2021, 10:51 a.m. UTC 
Since commit cb731d6c62bb ("vmscan: per memory cgroup slab shrinkers"),
shrink_node_slabs is renamed to drop_slab_node. And doit argument is
changed to forcekill since commit 6751ed65dc66 ("x86/mce: Fix
siginfo_t->si_addr value for non-recoverable memory faults").

Signed-off-by: Miaohe Lin <linmiaohe@huawei.com>
---
 mm/memory-failure.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

Comments

HORIGUCHI NAOYA(堀口 直也) Aug. 17, 2021, 7:28 a.m. UTC | #1 
On Sat, Aug 14, 2021 at 06:51:31PM +0800, Miaohe Lin wrote:
> Since commit cb731d6c62bb ("vmscan: per memory cgroup slab shrinkers"),
> shrink_node_slabs is renamed to drop_slab_node. And doit argument is
> changed to forcekill since commit 6751ed65dc66 ("x86/mce: Fix
> siginfo_t->si_addr value for non-recoverable memory faults").
> 
> Signed-off-by: Miaohe Lin <linmiaohe@huawei.com>

Acked-by: Naoya Horiguchi <naoya.horiguchi@nec.com>
diff mbox series

Patch

diff --git a/mm/memory-failure.c b/mm/memory-failure.c
index a4e585f812c1..a6f2384d9933 100644
--- a/mm/memory-failure.c
+++ b/mm/memory-failure.c
@@ -296,7 +296,7 @@  void shake_page(struct page *p, int access)
 	}
 
 	/*
-	 * Only call shrink_node_slabs here (which would also shrink
+	 * Only call drop_slab_node here (which would also shrink
 	 * other caches) if access is not potentially fatal.
 	 */
 	if (access)
@@ -391,8 +391,8 @@  static void add_to_kill(struct task_struct *tsk, struct page *p,
 /*
  * Kill the processes that have been collected earlier.
  *
- * Only do anything when DOIT is set, otherwise just free the list
- * (this is used for clean pages which do not need killing)
+ * Only do anything when FORCEKILL is set, otherwise just free the
+ * list (this is used for clean pages which do not need killing)
  * Also when FAIL is set do a force kill because something went
  * wrong earlier.
  */