From: Shannon Zhao <zhaoshenglong@huawei.com>
To: Christoffer Dall <christoffer.dall@linaro.org>,
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>,
ParthDixit <parth.dixit@linaro.org>
Subject: Re: Design doc of adding ACPI support for arm64 on Xen - version 2
Date: Thu, 13 Aug 2015 18:48:40 +0800 [thread overview]
Message-ID: <55CC7608.2000903@huawei.com> (raw)
In-Reply-To: <20150813102929.GB19874@cbox>
On 2015/8/13 18:29, 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).
>
> 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.
>
It needs to modify MADT table according to dom0_max_vcpus and modify EFI
MMAP table according to dom0_mem. And modify FADT table to set
hypervisor_id as "XenVMM" to tell Dom0 that it runs on Xen hypervisor.
--
Shannon
next prev parent reply other threads:[~2015-08-13 10:48 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
2015-08-13 10:48 ` Shannon Zhao [this message]
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=55CC7608.2000903@huawei.com \
--to=zhaoshenglong@huawei.com \
--cc=JBeulich@suse.com \
--cc=christoffer.dall@linaro.org \
--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 \
/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).