From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jeremy Fitzhardinge Subject: Re: [PATCH][pvops_dom0][0/4] parse ACPI Cx/Px info for Xen Date: Mon, 20 Jul 2009 12:34:27 -0700 Message-ID: <4A64C6C3.90904@goop.org> References: <4D05DB80B95B23498C72C700BD6C2E0B2FBF1987@pdsmsx502.ccr.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <4D05DB80B95B23498C72C700BD6C2E0B2FBF1987@pdsmsx502.ccr.corp.intel.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: "Yu, Ke" Cc: "Tian, Kevin" , "xen-devel@lists.xensource.com" List-Id: xen-devel@lists.xenproject.org On 07/18/09 23:45, Yu, Ke wrote: > Note that the 04 patch xen-vcpu-pcpu-mapping.patch is a bit hacky. Its purpose is fixing the gap between dom0 vcpu and physical acpi info. but due to that this is a the architecture issue, no clean way is found with current architecture. There is thread talking about moving the ACPI parser from dom0 to hypervisor. this issue will gone under such architecture. But by far, we have to use this kind of hacky patch. > > Also this series of patches are rebased version of the previous post. Most of the comments Jeremy provided last time is applied, with only one exception: use xen specific cpufreq driver for the Px info. I tried this approach, and finally find this approach need many hacks in cpufreq path to fix the gap of vcpu and physical cpu. so I still keep the Px logic in external control logic instead of cpufreq driver. > Thats unfortunate. Could you go into a bit more detail about the problem with the pcpu/vcpu gap? I wonder if this could be dealt with in some other way? For example, since it never(?) makes sense for cpufreq to operate in terms of vcpus, could the interface be defined to always operate in terms of pcpus, this erasing the gap? (I can see lots of problems with this suggestion, but I'm just throwing it up as a discussion point.) J