[PATCH] tweaks for page_convert_anon

Hugh Dickins (hugh@veritas.com)
Fri, 4 Apr 2003 17:41:42 +0100 (BST)


A couple of small additions on top of Dave's objfix-2.5.66-mm3-3:

To maintain an accurate nr_mapped, page_remove_rmap must check
page_mapped with pte_chain_lock held, both at end and at start:
particularly now it's being called speculatively (in ignorance
of whether this pte is already listed or not).

Coincidentally, Pete Zaitcev's "gcc 3.2 breaks rmap on s390x" problem
reported yesterday would also be corrected by this, though it doesn't
fix the root of the problem (no barrier in pte_chain_lock on s390x).

Also, page_convert_anon remember pte_unmap after successful find_pte.

Hugh

--- 2.5.66-mm3-3/mm/rmap.c Fri Apr 4 12:20:40 2003
+++ linux/mm/rmap.c Fri Apr 4 16:13:42 2003
@@ -398,10 +398,10 @@
BUG();
if (!pfn_valid(page_to_pfn(page)) || PageReserved(page))
return;
- if (!page_mapped(page))
- return; /* remap_page_range() from a driver? */

pte_chain_lock(page);
+ if (!page_mapped(page))
+ goto outer;

/*
* If this is an object-based page, just uncount it. We can
@@ -461,10 +461,10 @@
}
}
out:
- pte_chain_unlock(page);
if (!page_mapped(page))
dec_page_state(nr_mapped);
- return;
+outer:
+ pte_chain_unlock(page);
}

/**
@@ -831,6 +831,7 @@
/* Make sure this isn't a duplicate */
page_remove_rmap(page, pte);
pte_chain = page_add_rmap(page, pte, pte_chain);
+ pte_unmap(pte);
}
spin_unlock(&vma->vm_mm->page_table_lock);
}
@@ -850,6 +851,7 @@
/* Make sure this isn't a duplicate */
page_remove_rmap(page, pte);
pte_chain = page_add_rmap(page, pte, pte_chain);
+ pte_unmap(pte);
}
spin_unlock(&vma->vm_mm->page_table_lock);
}

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/