From mboxrd@z Thu Jan 1 00:00:00 1970 From: Attilio Rao Subject: [PATCH v2 0/2] XEN/X86: Document pagetable_reserve PVOPS and enforce a better semantic Date: Tue, 14 Aug 2012 13:24:20 +0100 Message-ID: <1344947062-4796-1-git-send-email-attilio.rao@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: xen-devel@lists.xen.org, Konrad Rzeszutek Wilk , Stefano Stabellini Cc: Attilio Rao List-Id: xen-devel@lists.xenproject.org When looking for documenting the pagetable_reserve PVOPS, I realized that it assumes start == pgt_buf_start. I think this is not semantically right (even if with the current code this should not be a problem in practice) and what we really want is to extend the logic in order to do the RO -> RW convertion also for the range [pgt_buf_start, start). This patch then implements this missing conversion, adding some smaller cleanups and finally provides documentation for the PVOPS. Please look at 2/2 for more details on how the comment is structured. If we get this right we will have a reference to be used later on for others PVOPS. The difference with v1 is that sh_start local variable in xen_mapping_pagetable_reserve() is renamed as begin. Also, the commit messages have been tweaked. Attilio Rao (2): XEN, X86: Improve semantic support for pagetable_reserve PVOPS Xen: Document the semantic of the pagetable_reserve PVOPS arch/x86/include/asm/x86_init.h | 19 +++++++++++++++++-- arch/x86/mm/init.c | 4 ++++ arch/x86/xen/mmu.c | 22 ++++++++++++++++++++-- 3 files changed, 41 insertions(+), 4 deletions(-) -- 1.7.2.5