Home » Mailing list archives » linux-kernel » 2008 » June » 19

Re: [PATCH 1 of 4] mm: add a ptep_modify_prot transaction abstraction

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
[view in full thread]
From: Jeremy Fitzhardinge <jeremy@...>
To: <benh@...>
Cc: xen-devel <xen-devel@...>, Peter Zijlstra <a.p.zijlstra@...>, kvm-devel <kvm-devel@...>, <x86@...>, LKML <linux-kernel@...>, Virtualization Mailing List <virtualization@...>, Hugh Dickins <hugh@...>, Ingo Molnar <mingo@...>, Linus Torvalds <torvalds@...>, Thomas Gleixner <tglx@...>
Subject: Re: [PATCH 1 of 4] mm: add a ptep_modify_prot transaction abstraction
Date: Wednesday, June 18, 2008 - 8:15 pm

Jeremy Fitzhardinge wrote:
Along the lines of:

--- a/include/asm-x86/pgtable.h	Wed Jun 18 13:54:49 2008 -0700
+++ b/include/asm-x86/pgtable.h	Wed Jun 18 17:03:39 2008 -0700
@@ -491,7 +491,11 @@
 static inline void ptep_set_wrprotect(struct mm_struct *mm,
 				      unsigned long addr, pte_t *ptep)
 {
-	clear_bit(_PAGE_BIT_RW, (unsigned long *)&ptep->pte);
+	pte_t pte = ptep_modify_prot_start(mm, addr, ptep);
+	
+	pte = pte_wrprotect(pte);
+
+	ptep_modify_prot_commit(mm, addr, ptep, pte);
 	pte_update(mm, addr, ptep);
 }
 

	J

--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
Messages in current thread:
[PATCH 0 of 4] mm+paravirt+xen: add pte read-modify-write ab..., Jeremy Fitzhardinge, (Mon Jun 16, 7:29 am)
[PATCH 2 of 4] paravirt: add hooks for ptep_modify_prot_star..., Jeremy Fitzhardinge, (Mon Jun 16, 7:30 am)
[PATCH 4 of 4] xen: add mechanism to extend existing multica..., Jeremy Fitzhardinge, (Mon Jun 16, 7:30 am)
[PATCH 3 of 4] xen: implement ptep_modify_prot_start/commit, Jeremy Fitzhardinge, (Mon Jun 16, 7:30 am)
[PATCH 1 of 4] mm: add a ptep_modify_prot transaction abstra..., Jeremy Fitzhardinge, (Mon Jun 16, 7:30 am)
Re: [PATCH 1 of 4] mm: add a ptep_modify_prot transaction ab..., Benjamin Herrenschmidt, (Wed Jun 18, 7:23 pm)
Re: [PATCH 1 of 4] mm: add a ptep_modify_prot transaction ab..., Jeremy Fitzhardinge, (Wed Jun 18, 7:59 pm)
Re: [PATCH 1 of 4] mm: add a ptep_modify_prot transaction ab..., Jeremy Fitzhardinge, (Wed Jun 18, 8:15 pm)
Re: [PATCH 1 of 4] mm: add a ptep_modify_prot transaction ab..., Linus Torvalds, (Wed Jun 18, 8:24 pm)
Re: [PATCH 1 of 4] mm: add a ptep_modify_prot transaction ab..., Benjamin Herrenschmidt, (Wed Jun 18, 8:39 pm)
Re: [PATCH 1 of 4] mm: add a ptep_modify_prot transaction ab..., Jeremy Fitzhardinge, (Thu Jun 19, 1:03 am)
Re: [PATCH 1 of 4] mm: add a ptep_modify_prot transaction ab..., Benjamin Herrenschmidt, (Thu Jun 19, 3:20 am)
Re: [PATCH 1 of 4] mm: add a ptep_modify_prot transaction ab..., Jeremy Fitzhardinge, (Thu Jun 19, 1:57 pm)
Re: [PATCH 1 of 4] mm: add a ptep_modify_prot transaction ab..., Jeremy Fitzhardinge, (Wed Jun 18, 8:37 pm)
Re: [PATCH 1 of 4] mm: add a ptep_modify_prot transaction ab..., Linus Torvalds, (Wed Jun 18, 8:49 pm)
Re: [PATCH 1 of 4] mm: add a ptep_modify_prot transaction ab..., Linus Torvalds, (Thu Jun 19, 12:03 am)
Re: [PATCH 1 of 4] mm: add a ptep_modify_prot transaction ab..., Ingo Molnar, (Thu Jun 19, 7:58 am)
Re: [PATCH 1 of 4] mm: add a ptep_modify_prot transaction ab..., Linus Torvalds, (Thu Jun 19, 12:30 pm)
Re: [PATCH 1 of 4] mm: add a ptep_modify_prot transaction ab..., Ingo Molnar, (Thu Jun 19, 12:47 pm)
Re: [PATCH 1 of 4] mm: add a ptep_modify_prot transaction ab..., Ingo Molnar, (Fri Jun 20, 6:10 am)
Re: [PATCH 1 of 4] mm: add a ptep_modify_prot transaction ab..., Jeremy Fitzhardinge, (Fri Jun 20, 3:06 pm)
Re: [PATCH 1 of 4] mm: add a ptep_modify_prot transaction ab..., Linus Torvalds, (Fri Jun 20, 3:15 pm)
Re: [PATCH 1 of 4] mm: add a ptep_modify_prot transaction ab..., Ingo Molnar, (Fri Jun 20, 3:56 pm)
Re: [PATCH 1 of 4] mm: add a ptep_modify_prot transaction ab..., Jeremy Fitzhardinge, (Fri Jun 20, 4:05 pm)
Re: [PATCH 1 of 4] mm: add a ptep_modify_prot transaction ab..., Linus Torvalds, (Fri Jun 20, 4:03 pm)
Re: [PATCH 1 of 4] mm: add a ptep_modify_prot transaction ab..., Jeremy Fitzhardinge, (Fri Jun 20, 4:16 pm)
Re: [PATCH 1 of 4] mm: add a ptep_modify_prot transaction ab..., Jeremy Fitzhardinge, (Fri Jun 20, 4:22 pm)
Re: [PATCH 1 of 4] mm: add a ptep_modify_prot transaction ab..., Ingo Molnar, (Sat Jun 21, 2:06 am)
Re: [PATCH 1 of 4] mm: add a ptep_modify_prot transaction ab..., Akinobu Mita, (Thu Jun 19, 8:20 am)
Re: [PATCH 1 of 4] mm: add a ptep_modify_prot transaction ab..., Ingo Molnar, (Thu Jun 19, 8:03 am)
Re: [PATCH 1 of 4] mm: add a ptep_modify_prot transaction ab..., Linus Torvalds, (Mon Jun 16, 1:29 pm)
Re: [PATCH 1 of 4] mm: add a ptep_modify_prot transaction ab..., Hugh Dickins, (Mon Jun 16, 2:13 pm)
Re: [PATCH 1 of 4] mm: add a ptep_modify_prot transaction ab..., Ingo Molnar, (Mon Jun 16, 2:49 pm)