From: Christoffer Dall <christoffer.dall@linaro.org>
To: Ian Campbell <ian.campbell@citrix.com>
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 12:29:29 +0200 [thread overview]
Message-ID: <20150813102929.GB19874@cbox> (raw)
In-Reply-To: <1439461339.23981.64.camel@citrix.com>
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).
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.
-Christoffer
next prev parent reply other threads:[~2015-08-13 10:29 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 [this message]
2015-08-13 10:32 ` Stefano Stabellini
2015-08-13 10:34 ` Ian Campbell
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=20150813102929.GB19874@cbox \
--to=christoffer.dall@linaro.org \
--cc=JBeulich@suse.com \
--cc=hangaohuai@huawei.com \
--cc=ian.campbell@citrix.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).