From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
To: Attilio Rao <attilio.rao@citrix.com>
Cc: Stefano Stabellini <Stefano.Stabellini@eu.citrix.com>,
xen-devel@lists.xen.org
Subject: Re: [PATCH 0/2] Document pagetable_reserve PVOP and enforce a better semantic
Date: Mon, 13 Aug 2012 16:42:18 -0400 [thread overview]
Message-ID: <20120813204218.GB12550@phenom.dumpdata.com> (raw)
In-Reply-To: <1344608227-30910-1-git-send-email-attilio.rao@citrix.com>
On Fri, Aug 10, 2012 at 03:17:05PM +0100, Attilio Rao wrote:
> When looking for documenting the pagetable_reserve PVOP, 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 PVOP.
> 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.
> A preliminary version of this patch has been already reviewed by
> Stefano Stabellini.
>
> Attilio Rao (2):
> XEN, X86: Improve semantic support for pagetable_reserve PVOP
> Document the semantic of the pagetable_reserve PVOP
The titles need a prefix, like 'x86' or 'xen/x86'.
s/PVOP/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
prev parent reply other threads:[~2012-08-13 20:42 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-08-10 14:17 [PATCH 0/2] Document pagetable_reserve PVOP and enforce a better semantic Attilio Rao
2012-08-10 14:17 ` [PATCH 1/2] XEN, X86: Improve semantic support for pagetable_reserve PVOP Attilio Rao
2012-08-13 20:43 ` Konrad Rzeszutek Wilk
2012-08-10 14:17 ` [PATCH 2/2] Document the semantic of the " Attilio Rao
2012-08-13 20:42 ` Konrad Rzeszutek Wilk [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20120813204218.GB12550@phenom.dumpdata.com \
--to=konrad.wilk@oracle.com \
--cc=Stefano.Stabellini@eu.citrix.com \
--cc=attilio.rao@citrix.com \
--cc=xen-devel@lists.xen.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.