WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] [PATCH] Allow dom0 to write MSR IA32_ENERGY_PERF_BIAS

To: Jan Beulich <JBeulich@xxxxxxxxxx>, Gang Wei <gang.wei@xxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] Allow dom0 to write MSR IA32_ENERGY_PERF_BIAS
From: Keir Fraser <keir@xxxxxxx>
Date: Wed, 05 Jan 2011 08:22:46 +0000
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Wed, 05 Jan 2011 00:23:31 -0800
Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:sender:user-agent:date :subject:from:to:cc:message-id:thread-topic:thread-index:in-reply-to :mime-version:content-type:content-transfer-encoding; bh=KueEUmGUzLlkCxEr7mbgvIW6U5PdqH8FekjLBNhAs7k=; b=GkoMF79tto/e8mgWNTckJNu+GACaUFFRP7K88wRYkmOkfXIUE55levSHanb+IGsDu3 GcD/aE68Lwy3HTmABJYe/Bw+F+NOVk4O6oWViMEKhzKXeiUf0ILM/R+Nb/gsYt0xgXc0 KRjqV/vVXjP/EIUH+7KdYV5MxXfSupEoWm+co=
Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=sender:user-agent:date:subject:from:to:cc:message-id:thread-topic :thread-index:in-reply-to:mime-version:content-type :content-transfer-encoding; b=wktKh78SlxDWAdUm9mQsrFlt/X/3zxwsh1Oc0lZyKqu3EUx6kNcfr4DHPXiuszLFUd cJ5UjsdjkbrUWC4vs4YbyNOnPqXYCs60S/hk9fvndNXFIibKZ4lKXx1Zgkj3Di06WNS9 drEAVR5W99ZrzU4JzTHi7T18qqmAuzxzqXfn8=
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <4D24371B020000780002A62B@xxxxxxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcussbsyZDzRWLjCoEGAAIyOBhzTuQ==
Thread-topic: [Xen-devel] [PATCH] Allow dom0 to write MSR IA32_ENERGY_PERF_BIAS
User-agent: Microsoft-Entourage/12.28.0.101117
On 05/01/2011 08:17, "Jan Beulich" <JBeulich@xxxxxxxxxx> wrote:

>>>> On 05.01.11 at 09:13, Keir Fraser <keir.fraser@xxxxxxxxxxxxx> wrote:
>> On 05/01/2011 07:59, "Jan Beulich" <JBeulich@xxxxxxxxxx> wrote:
>> 
>>>>> Why would you allow this only if Dom0 has its vcpus pinned?
>>>> 
>>>> It is meaningless if dom0 can't control all pcpus exactly. Only in case of
>>>> dom0 vcpus pinned, it makes sense.
>>> 
>>> Disagree. The user mode tool could set its own affinity (virtual and
>>> physical) and then issue the MSR write. Please don't enforce
>>> restrictions where not really needed (I actually suppose that the
>>> restriction should be removed for MSR_IA32_THERM_CONTROL too).
>> 
>> If so, it deserves a separate patch to strip out *all* the is_pinned checks
>> at the same time.
> 
> You certainly don't mean *all*, but yes, I'm intending to submit such
> a patch.

The ones in x86/traps.c (WRMSR emulation) and x86/domain.c
(VCPUOP_get_physid) are both unnecessary, at least.

 -- Keir

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



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