The patch titled
Subject: mm: memcontrol: clarify the uncharge_list() loop
has been removed from the -mm tree. Its filename was
mm-memcontrol-clarify-the-uncharge_list-loop.patch
This patch was dropped because it was merged into mainline or a subsystem tree
------------------------------------------------------
From: Johannes Weiner <hannes@xxxxxxxxxxx>
Subject: mm: memcontrol: clarify the uncharge_list() loop
uncharge_list() does an unusual list walk because the function can take
regular lists with dedicated list_heads as well as singleton lists where a
single page is passed via the page->lru list node.
This can sometimes lead to confusion as well as suggestions to replace the
loop with a list_for_each_entry(), which wouldn't work.
Signed-off-by: Johannes Weiner <hannes@xxxxxxxxxxx>
Acked-by: Michal Hocko <mhocko@xxxxxxxx>
Cc: Vladimir Davydov <vdavydov@xxxxxxxxxxxxx>
Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
---
mm/memcontrol.c | 4 ++++
1 file changed, 4 insertions(+)
diff -puN mm/memcontrol.c~mm-memcontrol-clarify-the-uncharge_list-loop mm/memcontrol.c
--- a/mm/memcontrol.c~mm-memcontrol-clarify-the-uncharge_list-loop
+++ a/mm/memcontrol.c
@@ -5420,6 +5420,10 @@ static void uncharge_list(struct list_he
struct list_head *next;
struct page *page;
+ /*
+ * Note that the list can be a single page->lru; hence the
+ * do-while loop instead of a simple list_for_each_entry().
+ */
next = page_list->next;
do {
unsigned int nr_pages = 1;
_
Patches currently in -mm which might be from hannes@xxxxxxxxxxx are
--
To unsubscribe from this list: send the line "unsubscribe mm-commits" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
