diff mbox series

parisc: Remove pte_inserted define

Message ID 20181020003329.GA22259@mx3210.localdomain (mailing list archive)
State Accepted, archived
Headers show
Series parisc: Remove pte_inserted define | expand

Commit Message

John David Anglin Oct. 20, 2018, 12:33 a.m. UTC
The attached change removes the pte_inserted from pgtable.h.  As a result,
we always flush the TLB entry when the associated page table entry is changed.

This change doesn't impact performance signifcantly and it may catch some
cases where the TLB needs flushing but wasn't.


Signed-off-by: John David Anglin <dave.anglin@bell.net>
diff mbox series

Patch

diff --git a/arch/parisc/include/asm/pgtable.h b/arch/parisc/include/asm/pgtable.h
index 94c0ef7a9e03..b941ac7d4e70 100644
--- a/arch/parisc/include/asm/pgtable.h
+++ b/arch/parisc/include/asm/pgtable.h
@@ -55,10 +55,6 @@  static inline void purge_tlb_entries(struct mm_struct *mm, unsigned long addr)
                 *(pteptr) = (pteval);                           \
         } while(0)
 
-#define pte_inserted(x)						\
-	((pte_val(x) & (_PAGE_PRESENT|_PAGE_ACCESSED))		\
-	 == (_PAGE_PRESENT|_PAGE_ACCESSED))
-
 #define set_pte_at(mm, addr, ptep, pteval)			\
 	do {							\
 		pte_t old_pte;					\
@@ -66,8 +62,7 @@  static inline void purge_tlb_entries(struct mm_struct *mm, unsigned long addr)
 		spin_lock_irqsave(&pa_tlb_lock, flags);		\
 		old_pte = *ptep;				\
 		set_pte(ptep, pteval);				\
-		if (pte_inserted(old_pte))			\
-			purge_tlb_entries(mm, addr);		\
+		purge_tlb_entries(mm, addr);			\
 		spin_unlock_irqrestore(&pa_tlb_lock, flags);	\
 	} while (0)
 
@@ -493,8 +488,7 @@  static inline pte_t ptep_get_and_clear(struct mm_struct *mm, unsigned long addr,
 	spin_lock_irqsave(&pa_tlb_lock, flags);
 	old_pte = *ptep;
 	set_pte(ptep, __pte(0));
-	if (pte_inserted(old_pte))
-		purge_tlb_entries(mm, addr);
+	purge_tlb_entries(mm, addr);
 	spin_unlock_irqrestore(&pa_tlb_lock, flags);
 
 	return old_pte;