From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
To: "Tian, Kevin" <kevin.tian@intel.com>
Cc: liang tang <liang.tang@oracle.com>,
Konrad Rzeszutek Wilk <konrad@darnok.org>,
"jeremy@goop.org" <jeremy@goop.org>,
"xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>,
"Ian.Campbell@citrix.com" <Ian.Campbell@citrix.com>,
"mike.mcclurg@citrix.com" <mike.mcclurg@citrix.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"stefan.bader@canonical.com" <stefan.bader@canonical.com>,
"rjw@sisk.pl" <rjw@sisk.pl>,
"linux-acpi@vger.kernel.org" <linux-acpi@vger.kernel.org>,
"Yu, Ke" <ke.yu@intel.com>,
"konrad@kernel.org" <konrad@kernel.org>,
"lenb@kernel.org" <lenb@kernel.org>,
"Nakajima, Jun" <jun.nakajima@intel.com>
Subject: Re: [Xen-devel] [PATCH 3/8] ACPI: processor: add __acpi_processor_[un]register_driver helpers.
Date: Tue, 17 Jan 2012 12:13:14 -0500 [thread overview]
Message-ID: <20120117171314.GB5494@phenom.dumpdata.com> (raw)
In-Reply-To: <AADFC41AFE54684AB9EE6CBC0274A5D10FCD37A9@SHSMSX102.ccr.corp.intel.com>
> > I was trying to figure out how difficult it would be to just bring Pxx states to
> > the Xen hypervisor using the existing ACPI interfaces. And while it did not pass
> > all the _Pxx states (seems that all the _PCT, _PSS, _PSD, _PPC flags need to
> > be enabled in the hypercall to make this work), it demonstrates what I had in
> > mind.
.. snip..
> > /* TODO: Under Xen, the C-states information is not present.
> > * Figure out why. */
>
> it's possible related to this long thread:
>
> http://lists.xen.org/archives/html/xen-devel/2011-08/msg00511.html
>
> IOW, Xen doesn't export mwait capability to dom0, which impacts _PDC setting.
> Final solution is to have a para-virtualized PDC call for that.
Aaah. Let me play with that a bit. Thanks for the pointer.
.. snip..
> the prerequisites for this module to work correctly, is that dom0 has the right
> configurations to have all necessary Cx/Px information ready before this
> module is loaded. That may mean enabling full CONFIG_CPU_IDLE and CONFIG_CPUFREQ,
Right.
> which in current form may add some negative impact, e.g. dom0 will try to control
> Px/Cx to conflict with Xen. So some tweaks may be required in that part.
Yup. Hadn't even looked at the cpufreq tries to do yet.
>
> given our purpose now, is to come up a cleaner approach which tolerate some
> assumptions (e.g. #VCPU of dom0 == #PCPU), there's another option following this
> trend (perhaps compensate your idea). We can register a Xen-cpuidle and
> xen-cpufreq driver to current Linux cpuidle and cpufreq framework, which plays
> mainly two roles:
> - a dummy driver to prevent dom0 touching actual Px/Cx
> - parse ACPI Cx/Px information to Xen, in a similar way you did above
Yeah, I like where you are heading.
>
> there may have some other trickiness, but the majority code will be self-contained.
<nods>
>
> Thanks
> Kevin
> --
> To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2012-01-17 17:16 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-11-30 17:20 [RFC PATCH] Exporting ACPI Pxx/Cxx states to other kernel subsystems (v1) Konrad Rzeszutek Wilk
2011-11-30 17:20 ` [PATCH 1/8] ACPI: processor: export necessary interfaces Konrad Rzeszutek Wilk
2011-12-16 21:33 ` Konrad Rzeszutek Wilk
2011-12-19 5:43 ` Tian, Kevin
2011-12-19 14:17 ` [Xen-devel] " Konrad Rzeszutek Wilk
2011-11-30 17:20 ` [PATCH 2/8] ACPI: processor: cache acpi_power_register in cx structure Konrad Rzeszutek Wilk
2011-11-30 17:20 ` [PATCH 3/8] ACPI: processor: add __acpi_processor_[un]register_driver helpers Konrad Rzeszutek Wilk
2011-12-16 22:03 ` Konrad Rzeszutek Wilk
2011-12-19 5:48 ` Tian, Kevin
2011-12-19 14:26 ` [Xen-devel] " Konrad Rzeszutek Wilk
2011-12-20 2:29 ` Tian, Kevin
2011-12-20 15:31 ` Konrad Rzeszutek Wilk
2011-12-21 0:35 ` Tian, Kevin
2011-12-23 3:01 ` Konrad Rzeszutek Wilk
2011-12-26 1:31 ` Tian, Kevin
2012-01-03 20:59 ` Konrad Rzeszutek Wilk
2012-01-06 1:07 ` Tian, Kevin
2012-01-13 22:24 ` Konrad Rzeszutek Wilk
2012-01-17 3:03 ` Tian, Kevin
2012-01-17 17:13 ` Konrad Rzeszutek Wilk [this message]
2012-01-17 18:19 ` Konrad Rzeszutek Wilk
2012-01-23 16:53 ` Konrad Rzeszutek Wilk
2011-11-30 17:21 ` [PATCH 4/8] ACPI: processor: Don't setup cpu idle driver and handler when we do not want them Konrad Rzeszutek Wilk
2011-12-16 21:36 ` Konrad Rzeszutek Wilk
2011-11-30 17:21 ` [PATCH 5/8] ACPI: add processor driver for Xen virtual CPUs Konrad Rzeszutek Wilk
2011-12-01 9:24 ` [Xen-devel] " Jan Beulich
2011-12-12 17:29 ` Konrad Rzeszutek Wilk
2011-12-13 7:45 ` Jan Beulich
2011-12-13 9:26 ` liang tang
2011-12-16 22:21 ` Konrad Rzeszutek Wilk
2012-02-10 17:18 ` Konrad Rzeszutek Wilk
2011-11-30 17:21 ` [PATCH 6/8] ACPI: processor: override the interface of register acpi processor handler for Xen vcpu Konrad Rzeszutek Wilk
2011-11-30 17:21 ` [PATCH 7/8] ACPI: xen processor: add PM notification interfaces Konrad Rzeszutek Wilk
2011-11-30 17:21 ` [PATCH 8/8] ACPI: xen processor: set ignore_ppc to handle PPC event for Xen vcpu Konrad Rzeszutek Wilk
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=20120117171314.GB5494@phenom.dumpdata.com \
--to=konrad.wilk@oracle.com \
--cc=Ian.Campbell@citrix.com \
--cc=jeremy@goop.org \
--cc=jun.nakajima@intel.com \
--cc=ke.yu@intel.com \
--cc=kevin.tian@intel.com \
--cc=konrad@darnok.org \
--cc=konrad@kernel.org \
--cc=lenb@kernel.org \
--cc=liang.tang@oracle.com \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mike.mcclurg@citrix.com \
--cc=rjw@sisk.pl \
--cc=stefan.bader@canonical.com \
--cc=xen-devel@lists.xensource.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).