From: Ian Campbell <ian.campbell@citrix.com>
To: Gareth Stockwell <Gareth.Stockwell@arm.com>
Cc: "stefano.stabellini@citrix.com" <stefano.stabellini@citrix.com>,
"tklengyel@sec.in.tum.de" <tklengyel@sec.in.tum.de>,
"xen-devel (xen-devel@lists.xen.org)" <xen-devel@lists.xen.org>
Subject: Re: memaccess: skipping mem_access_send_req
Date: Wed, 15 Apr 2015 10:26:52 +0100 [thread overview]
Message-ID: <1429090012.15516.155.camel@citrix.com> (raw)
In-Reply-To: <DA845EDCE27355428C520DC5B8DC05CE763FB38550@GEORGE.Emea.Arm.com>
On Fri, 2015-04-10 at 16:04 +0100, Gareth Stockwell wrote:
> We would like to use memaccess to perform (1) - but rather than
> pausing the VCPU in (2), instead simply directly inject the exception
> into the VCPU.
That is, into the VCPUs whose permissions have been modified behind its
back and not into some controlling domain?
Is the guest expected to be aware of this, i.e. to be somewhat
paravirtualised? I suppose it must have to be in order to accept
seemingly spurious page faults.
Which leads me to wonder whether an extra shared ring between the
hypervisor and target VCPU would be desirable, i.e. to allow more fine
grained semantics than just "computer says no". Specifically if you need
to care about the reason for the fault being the actions of an external
arbiter rather than some other guest-internal thing.
If your application is just to allow the guest OS to kill a process
which has tried to touch memory in a way which the external controller
has disallowed then a page fault seems like a simple and effective way
though.
> b) Define new xenmem_access_t values which cause the exception handler
> to reinject rather than adding a message to the ring buffer.
I'm no xenaccess guru but that's the option I'd be inclined to take.
My only concern would be limitations on the number of xenaccess types.
On ARM we couldn't fit the type into the spare P2M PTE bits so we will
already have a separate lookup and therefore not much limitation, but I
don't know about on x86.
Ian.
next prev parent reply other threads:[~2015-04-15 9:26 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-04-10 15:04 memaccess: skipping mem_access_send_req Gareth Stockwell
2015-04-10 15:11 ` Razvan Cojocaru
2015-04-10 15:23 ` Julien Grall
2015-04-15 9:18 ` Ian Campbell
2015-04-10 15:28 ` Tamas K Lengyel
2015-04-10 16:03 ` Andrew Cooper
2015-04-17 9:35 ` Gareth Stockwell
2015-04-15 9:26 ` Ian Campbell [this message]
2015-04-17 9:35 ` Gareth Stockwell
2015-04-17 10:31 ` Ian Campbell
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=1429090012.15516.155.camel@citrix.com \
--to=ian.campbell@citrix.com \
--cc=Gareth.Stockwell@arm.com \
--cc=stefano.stabellini@citrix.com \
--cc=tklengyel@sec.in.tum.de \
--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.