From: Ian Campbell <ian.campbell@citrix.com>
To: Christoffer Dall <christoffer.dall@linaro.org>
Cc: Hangaohuai <hangaohuai@huawei.com>,
Stefano Stabellini <stefano.stabellini@eu.citrix.com>,
"Huangpeng (Peter)" <peter.huangpeng@huawei.com>,
xen-devel <xen-devel@lists.xen.org>,
Julien Grall <julien.grall@citrix.com>,
Stefano Stabellini <stefano.stabellini@citrix.com>,
Shannon Zhao <shannon.zhao@linaro.org>,
Jan Beulich <JBeulich@suse.com>,
Shannon Zhao <zhaoshenglong@huawei.com>,
ParthDixit <parth.dixit@linaro.org>
Subject: Re: Design doc of adding ACPI support for arm64 on Xen - version 2
Date: Thu, 13 Aug 2015 11:34:54 +0100 [thread overview]
Message-ID: <1439462094.23981.74.camel@citrix.com> (raw)
In-Reply-To: <20150813102929.GB19874@cbox>
On Thu, 2015-08-13 at 12:29 +0200, Christoffer Dall wrote:
> On Thu, Aug 13, 2015 at 11:22:19AM +0100, Ian Campbell wrote:
> > On Thu, 2015-08-13 at 11:13 +0100, Stefano Stabellini wrote:
> > >
> > > > > For example it is only natural for the kernel to try to use the
> > > > > GIC
> > > > > hyp
> > > > > functionalities if they are described, while actually they are
> > > > > not
> > > > > emulated by Xen at all.
> > > >
> > > > See Ian's earlier reply: It can also be considered natural for it
> > > > to
> > > > be aware that when run in EL2 to not use EL1 functionality.
> >
> > NB EL2 == Hyp and EL1 == Kernel, so it's the other way round, FWIW.
> >
> > > It is not just about the GIC Hyp functionalities.
> >
> > What else is there which is not subject to this logic? Timers are too,
> > it
> > even applies to IOMMU's which have both stage1 and stage2 bits.
> >
> > BTW, I think kernels _already_ need to deal with a lot of this because
> > in
> > reality nobody modifies the DTB when they use a firmware which launches
> > the
> > kernel in EL1. IOW I think the kernel is already aware of which
> > resources
> > can be used by which privilege level.
> >
> Yes, for resources specific to EL2 I believe that is indeed the case
> (the GIC driver doesn't look at the hypervisor control register address,
> and KVM does not even get that far if you're not booted in EL2, and the
> timer only uses the virtual timer if not booted in EL2 - we never
> attempt to use the hyp timer until Marc's VHE patches land, but they
> also depend on being booted in hyp mode).
Right, and I think that's almost always going to be the case by virtue of
the architecture. You can't use these resources from EL1 (however you got
there) so you need checks.
> However, what about for other resources? Having code somewhere that
> says "hide this random piece of hardware if you're Xen dom0" sounds
> awful to me. I know it's only the serial port right now, but still.
Right, that's the only bit I'm aware of that I'm not sure about, which is
why I was curious how x86 managed it (I've seen but not digested Jan's
answer).
At least this has the virtue of being an extra table, and tagging on a new
one of those has more limited impact on the tables I think.
Ian.
next prev parent reply other threads:[~2015-08-13 10:34 UTC|newest]
Thread overview: 82+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-08-07 2:11 Design doc of adding ACPI support for arm64 on Xen - version 2 Shannon Zhao
2015-08-07 9:45 ` Stefano Stabellini
2015-08-07 10:33 ` Julien Grall
2015-08-07 10:37 ` Christoffer Dall
2015-08-07 10:42 ` Stefano Stabellini
2015-08-07 10:44 ` Julien Grall
2015-08-11 2:09 ` Shannon Zhao
2015-08-11 9:46 ` Julien Grall
2015-08-11 10:27 ` Shannon Zhao
2015-08-11 14:12 ` Ian Campbell
2015-08-11 14:51 ` David Vrabel
2015-08-11 14:59 ` Ian Campbell
2015-08-11 15:02 ` David Vrabel
2015-08-11 15:11 ` Julien Grall
2015-08-11 15:19 ` Ian Campbell
2015-08-11 15:25 ` David Vrabel
2015-08-11 16:01 ` Julien Grall
2015-08-12 2:42 ` Shannon Zhao
2015-08-12 8:46 ` Ian Campbell
2015-08-12 8:46 ` Ian Campbell
2015-08-12 9:02 ` Julien Grall
2015-08-12 15:48 ` Jan Beulich
2015-08-11 14:19 ` Ian Campbell
2015-08-11 14:21 ` Ian Campbell
2015-08-11 15:29 ` Boris Ostrovsky
2015-08-11 15:35 ` Ian Campbell
2015-08-11 15:52 ` Boris Ostrovsky
2015-08-12 2:47 ` Shannon Zhao
2015-08-12 8:47 ` Ian Campbell
2015-08-12 9:00 ` Shannon Zhao
2015-08-17 10:36 ` Roger Pau Monné
2015-08-18 1:44 ` Shannon Zhao
2015-08-11 16:19 ` Julien Grall
2015-08-12 3:04 ` Shannon Zhao
2015-08-12 8:52 ` Ian Campbell
2015-08-12 9:21 ` Julien Grall
2015-08-12 10:36 ` Andrew Turner
2015-08-12 10:48 ` Stefano Stabellini
2015-08-12 11:23 ` Ian Campbell
2015-08-12 12:11 ` Julien Grall
2015-09-02 11:27 ` Ian Campbell
2015-08-12 12:20 ` Julien Grall
2015-09-02 11:30 ` Ian Campbell
2015-09-02 11:39 ` Julien Grall
2015-09-02 12:05 ` Ian Campbell
2015-08-12 10:17 ` Stefano Stabellini
2015-08-12 7:22 ` Shannon Zhao
2015-08-12 9:11 ` Julien Grall
2015-08-14 14:05 ` Shannon Zhao
2015-08-14 14:17 ` xen/arm: Crash when allocating memory for ACPI table (Was Re: Design doc of adding ACPI support for arm64 on Xen - version 2) Julien Grall
2015-08-14 14:35 ` Shannon Zhao
2015-08-14 14:41 ` Julien Grall
2015-08-14 14:49 ` Shannon Zhao
2015-08-14 14:53 ` Julien Grall
2015-08-14 14:55 ` Shannon Zhao
2015-08-12 15:45 ` Design doc of adding ACPI support for arm64 on Xen - version 2 Jan Beulich
2015-08-12 15:51 ` Christoffer Dall
2015-08-12 15:58 ` Jan Beulich
2015-08-12 16:18 ` Julien Grall
2015-08-13 6:41 ` Jan Beulich
2015-08-13 8:01 ` Christoffer Dall
2015-08-13 8:11 ` Jan Beulich
2015-08-13 8:18 ` Jan Beulich
2015-08-13 9:05 ` Ian Campbell
2015-08-13 9:20 ` Jan Beulich
2015-08-13 10:40 ` Julien Grall
2015-08-13 9:43 ` Stefano Stabellini
2015-08-13 10:03 ` Jan Beulich
2015-08-13 10:13 ` Stefano Stabellini
2015-08-13 10:22 ` Ian Campbell
2015-08-13 10:29 ` Christoffer Dall
2015-08-13 10:32 ` Stefano Stabellini
2015-08-13 10:34 ` Ian Campbell [this message]
2015-08-13 10:48 ` Shannon Zhao
2015-08-13 10:54 ` Jan Beulich
2015-08-13 11:00 ` Julien Grall
2015-08-13 11:07 ` Stefano Stabellini
2015-08-13 11:55 ` Jan Beulich
2015-08-13 12:08 ` Julien Grall
2015-08-13 14:49 ` Ian Campbell
2015-08-14 5:34 ` Shannon Zhao
2015-08-14 9:54 ` Stefano Stabellini
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=1439462094.23981.74.camel@citrix.com \
--to=ian.campbell@citrix.com \
--cc=JBeulich@suse.com \
--cc=christoffer.dall@linaro.org \
--cc=hangaohuai@huawei.com \
--cc=julien.grall@citrix.com \
--cc=parth.dixit@linaro.org \
--cc=peter.huangpeng@huawei.com \
--cc=shannon.zhao@linaro.org \
--cc=stefano.stabellini@citrix.com \
--cc=stefano.stabellini@eu.citrix.com \
--cc=xen-devel@lists.xen.org \
--cc=zhaoshenglong@huawei.com \
/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.