xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Julien Grall <julien.grall@citrix.com>
To: Jan Beulich <JBeulich@suse.com>, Shannon Zhao <zhaoshenglong@huawei.com>
Cc: Hangaohuai <hangaohuai@huawei.com>,
	Ian Campbell <ian.campbell@citrix.com>,
	Stefano Stabellini <stefano.stabellini@eu.citrix.com>,
	"Huangpeng (Peter)" <peter.huangpeng@huawei.com>,
	xen-devel <xen-devel@lists.xen.org>,
	StefanoStabellini <stefano.stabellini@citrix.com>,
	Shannon Zhao <shannon.zhao@linaro.org>,
	ParthDixit <parth.dixit@linaro.org>,
	Christoffer Dall <christoffer.dall@linaro.org>
Subject: Re: Design doc of adding ACPI support for arm64 on Xen - version 2
Date: Thu, 13 Aug 2015 12:00:18 +0100	[thread overview]
Message-ID: <55CC78C2.2060005@citrix.com> (raw)
In-Reply-To: <55CC938D020000780009AA72@prv-mh.provo.novell.com>

On 13/08/15 11:54, Jan Beulich wrote:
>>>> On 13.08.15 at 12:48, <zhaoshenglong@huawei.com> wrote:
>> On 2015/8/13 18:29, Christoffer Dall wrote:
>>> 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.
> 
> None of this should be required:
> 
> Unaltered MADT may (or should I say will) be needed to drive P- or
> C-states.

We have to alter MADT for different reasons:
	- restricting the number of vCPUs
	- update the CPU bring up method
	- changing the CPUID

Maybe we should pass a backup but we don't support cpufreq driver right
now and it would need quite a look of work to do it (see [1]).

> UEFI memmap shouldn't be exposed to Dom0 at all.

We have to craft a UEFI memmap in order to notify Dom0 where are the RAM
regions.

> Detecting running on Xen should (hopefully) be possible by other
> means for Dom0.

How there is no cpuid like on ARM. So no possibility to know whether you
are running on Xen or another hypervisor...

Modifying the FADT is the only viable solution we have today in order to
tell that it's running on Xen.

Regards,

[1]http://lists.xen.org/archives/html/xen-devel/2014-10/msg02883.html

-- 
Julien Grall

  reply	other threads:[~2015-08-13 11:00 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
2015-08-13 10:54                       ` Jan Beulich
2015-08-13 11:00                         ` Julien Grall [this message]
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=55CC78C2.2060005@citrix.com \
    --to=julien.grall@citrix.com \
    --cc=JBeulich@suse.com \
    --cc=christoffer.dall@linaro.org \
    --cc=hangaohuai@huawei.com \
    --cc=ian.campbell@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).