From: Jeremy Fitzhardinge <jeremy@goop.org>
To: "Yu, Ke" <ke.yu@intel.com>
Cc: "Brown, Len" <len.brown@intel.com>,
"linux-acpi@vger.kernel.org" <linux-acpi@vger.kernel.org>,
"Tian, Kevin" <kevin.tian@intel.com>
Subject: Re: [Xen-devel] [PATCH][pvops_dom0][2/4] Introduce the external control operation interface for domain0 ACPI parser
Date: Wed, 29 Jul 2009 09:50:09 -0700 [thread overview]
Message-ID: <4A707DC1.9070708@goop.org> (raw)
In-Reply-To: <4D05DB80B95B23498C72C700BD6C2E0B315D65B6@pdsmsx502.ccr.corp.intel.com>
On 07/28/09 23:20, Yu, Ke wrote:
> Your question is reasonable. there is also debate here before. People discuss if it is possible to add acpica stuff to xen hypervisor and let xen control the acpi completely. Unfortunately, this will lead dilemma here, i.e. some devices are controlled by dom0 and also need acpi info, e.g. battery, thermal, etc. and pci hotplug in dom0 is another example. Tian Kevin has detail description on this issue in the attached mail.
>
What would happen if we special-cased dom0 VCPUs to be bound 1:1 to
PCPUs. Would that simplify this stuff? Wouldn't that make CPU control
equivalent to battery, fan, etc?
> On the other hand, the dom0 acpica approach has other benefit, i.e. current linux acpica stuff is pretty mature and has numerous bug fix. Leveraging acpica in linux kernel is more practical.
>
My understanding is that all that code is supposed to be
kernel-independent. We could lift all that code as-is, write some new
kernel interface shims and shove it all into Xen. But I don't know if
that solves any more problems than it causes.
>> s/extcntl/xen/ to make it clear why this code exists --
>> or is there an expected "external control" other than Xen?
>>
>> -Len
>>
>
> Ok, I can try this. BTW, besides this naming change, do you have other comment on the code? So that I can make it more clean.
>
I don't think that's a good idea. If we need to do that, then there's a
deeper design problem we need to address. (And, if nothing else, people
have become hypersensitive to naked Xen-specific code references in
non-Xen files, and I'm tired of having those arguments.)
J
next prev parent reply other threads:[~2009-07-29 16:50 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-07-29 2:55 FW: [Xen-devel] [PATCH][pvops_dom0][2/4] Introduce the external control operation interface for domain0 ACPI parser Yu, Ke
2009-07-29 4:14 ` Brown, Len
2009-07-29 6:20 ` Yu, Ke
2009-07-29 16:50 ` Jeremy Fitzhardinge [this message]
2009-07-30 9:18 ` Yu, Ke
2009-07-30 16:00 ` Len Brown
2009-07-30 20:36 ` Jeremy Fitzhardinge
2009-07-30 17:23 ` Jeremy Fitzhardinge
2009-07-30 15:37 ` Len Brown
2009-07-30 20:52 ` Jeremy Fitzhardinge
2009-07-29 14:47 ` Yu, Ke
2009-07-30 16:29 ` Len Brown
2009-07-30 22:04 ` Jeremy Fitzhardinge
2009-07-31 8:05 ` Yu, Ke
2009-07-29 16:43 ` Jeremy Fitzhardinge
2009-07-30 8:59 ` Yu, Ke
2009-07-30 15:03 ` Brown, Len
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=4A707DC1.9070708@goop.org \
--to=jeremy@goop.org \
--cc=ke.yu@intel.com \
--cc=kevin.tian@intel.com \
--cc=len.brown@intel.com \
--cc=linux-acpi@vger.kernel.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