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

Re: [Xen-devel] [PATCH] No need to readjust cpu based control when vTPR is not available.


  • To: "Li, Xin B" <xin.b.li@xxxxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxx>
  • From: Keir Fraser <keir@xxxxxxxxxxxxx>
  • Date: Thu, 09 Aug 2007 13:35:30 +0100
  • Delivery-date: Thu, 09 Aug 2007 05:33:22 -0700
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>
  • Thread-index: AcfabgHg27pvyYD5RFuUjNpVgGfjAwAAOgZ3AAQnuxAAAI8drg==
  • Thread-topic: [Xen-devel] [PATCH] No need to readjust cpu based control when vTPR is not available.

Now that bit is known to be set-to-enable, we know there will no processor
that does not support TPR-shadow that requires that bit forced high. So
unconditionally *clearing* that feature is fine.

 -- Keir

On 9/8/07 13:20, "Li, Xin B" <xin.b.li@xxxxxxxxx> wrote:

> Then we need this, right?
> 
> -Xin 
> 
>> -----Original Message-----
>> From: Keir Fraser [mailto:keir@xxxxxxxxxxxxx]
>> Sent: Thursday, August 09, 2007 6:21 PM
>> To: Li, Xin B; xen-devel@xxxxxxxxxxxxxxxxxxx
>> Subject: Re: [Xen-devel] [PATCH] No need to readjust cpu based
>> control when vTPR is not available.
>> 
>> It's nice to be consistent about laundering all control flags through
>> adjust_vmx_controls().
>> 
>> -- Keir
>> 
>> 
>> On 9/8/07 11:14, "Li, Xin B" <xin.b.li@xxxxxxxxx> wrote:
>> 
>>> No need to readjust cpu based control when vTPR is not available.
>>> Signed-off-by: Xin Li <xin.b.li@xxxxxxxxx>
>>> _______________________________________________
>>> 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


_______________________________________________
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®.