Commit 69e1e688 authored Sep 22, 2005 by Paolo 'Blaisorblade' Giarrusso Committed by Linus Torvalds Sep 22, 2005

[PATCH] uml: don't redundantly mark pte as newpage in pte_modify



pte_modify marks a page as needing flush, which is redundant because the
resulting PTE is still set with set_pte, which already handles that.

Signed-off-by: Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>
Cc: Jeff Dike <jdike@addtoit.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>

parent a8bfb94c

include/asm-um/pgtable.h

+0 −1

Original line number	Diff line number	Diff line
		@@ -346,7 +346,6 @@ static inline void set_pte(pte_t *pteptr, pte_t pteval)
		static inline pte_t pte_modify(pte_t pte, pgprot_t newprot)
		{
		pte_set_val(pte, (pte_val(pte) & _PAGE_CHG_MASK), newprot);
		if(pte_present(pte)) pte = pte_mknewpage(pte_mknewprot(pte));
		return pte;
		}