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

["Jan Beulich" <JBeulich@xxxxxxxxxx>]

>>> On 05.01.11 at 09:55, "Wei, Gang" <gang.wei@xxxxxxxxx> wrote:
> Jan Beulich wrote on 2011-01-05:
>>>>> On 05.01.11 at 09:22, Keir Fraser <keir@xxxxxxx> wrote:
>>> 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).
> 
> Ok, I accept such kind of usages. So how about simply check in my patch and 
> do remove these restrictions in your following patches?

Yes, that's what we all seem to agree to now (and I just sent out
that other patch).

Jan


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