The patch titled
mm: strictly require elevated page refcount in isolate_lru_page()
has been removed from the -mm tree. Its filename was
mm-strictly-require-elevated-page-refcount-in-isolate_lru_page.patch
This patch was dropped because it was merged into mainline or a subsystem tree
The current -mm tree may be found at http://userweb.kernel.org/~akpm/mmotm/
------------------------------------------------------
Subject: mm: strictly require elevated page refcount in isolate_lru_page()
From: Konstantin Khlebnikov <khlebnikov@xxxxxxxxxx>
isolate_lru_page() must be called only with stable reference to the page,
this is what is written in the comment above it, this is reasonable.
current isolate_lru_page() users and its page extra reference sources:
mm/huge_memory.c
__collapse_huge_page_isolate() - reference from pte
mm/memcontrol.c
mem_cgroup_move_parent() - get_page_unless_zero()
mem_cgroup_move_charge_pte_range() - reference from pte
mm/memory-failure.c
soft_offline_page() - fixed, reference from get_any_page()
delete_from_lru_cache() - reference from caller or get_page_unless_zero()
[seems like there bug, because __memory_failure() can call page_action() for
hpages tail, but it is ok for isolate_lru_page(), tail getted and not in lru]
mm/memory_hotplug.c
do_migrate_range() - fixed, get_page_unless_zero()
mm/mempolicy.c
migrate_page_add() - reference from pte
mm/migrate.c
do_move_page_to_node_array() - reference from follow_page()
mlock.c - various external references
mm/vmscan.c
putback_lru_page() - reference from isolate_lru_page()
It seems that all isolate_lru_page() users are ready now for this restriction.
So, let's replace redundant get_page_unless_zero() with get_page() and
add page initial reference count check with VM_BUG_ON()
Signed-off-by: Konstantin Khlebnikov <khlebnikov@xxxxxxxxxx>
Cc: Andi Kleen <andi@xxxxxxxxxxxxxx>
Cc: KAMEZAWA Hiroyuki <kamezawa.hiroyu@xxxxxxxxxxxxxx>
Cc: KOSAKI Motohiro <kosaki.motohiro@xxxxxxxxxxxxxx>
Cc: Mel Gorman <mel@xxxxxxxxx>
Cc: Lee Schermerhorn <lee.schermerhorn@xxxxxx>
Cc: Rik van Riel <riel@xxxxxxxxxx>
Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
---
mm/vmscan.c | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff -puN mm/vmscan.c~mm-strictly-require-elevated-page-refcount-in-isolate_lru_page mm/vmscan.c
--- a/mm/vmscan.c~mm-strictly-require-elevated-page-refcount-in-isolate_lru_page
+++ a/mm/vmscan.c
@@ -1207,13 +1207,16 @@ int isolate_lru_page(struct page *page)
{
int ret = -EBUSY;
+ VM_BUG_ON(!page_count(page));
+
if (PageLRU(page)) {
struct zone *zone = page_zone(page);
spin_lock_irq(&zone->lru_lock);
- if (PageLRU(page) && get_page_unless_zero(page)) {
+ if (PageLRU(page)) {
int lru = page_lru(page);
ret = 0;
+ get_page(page);
ClearPageLRU(page);
del_page_from_lru_list(zone, page, lru);
_
Patches currently in -mm which might be from khlebnikov@xxxxxxxxxx are
origin.patch
--
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
