[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Xen-devel] RE: [PATCH] Enable Px/Cx related CPUID/MSR bits for dom0


  • To: "Keir Fraser" <keir.fraser@xxxxxxxxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxx>
  • From: "Wei, Gang" <gang.wei@xxxxxxxxx>
  • Date: Mon, 5 May 2008 23:48:49 +0800
  • Delivery-date: Mon, 05 May 2008 08:50:12 -0700
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>
  • Thread-index: Aciucp7xXvPGFQVHQc62vzQNeMPItQAH5bLaAAyz9wA=
  • Thread-topic: [PATCH] Enable Px/Cx related CPUID/MSR bits for dom0

On Monday, May 05, 2008 5:27 PM, Keir Fraser wrote:
> Applied your patches, but why would you expose MWAIT as a usable
feature?

I can't exactly tell why. What I am sure is that X86_FEATURE_EST should
be exposed, otherwise ACPI _CST method invoking in dom0 will always
failed. MWAIT feature & MSR_IA32_MISC_ENABLE_MONITOR_ENABLE bit are
exposed just in case of possible needs. If I could make sure MWAIT
feature & MONITOR_ENABLE are not needed, I will send patch to remove
them.

> Maybe worth a comment explaining that, as otherwise it's not unlikely
to
> be cleaned up as a 'mistake' sometime in the future.

Ok, I will add comment after I can explain clearly.

Jimmy

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.