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

Re: [Xen-devel] [PATCH][VMX] a small clean to vmx_world_restore


  • To: "Li, Xin B" <xin.b.li@xxxxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxx>
  • From: Keir Fraser <Keir.Fraser@xxxxxxxxxxxx>
  • Date: Sat, 13 Oct 2007 11:57:54 +0100
  • Delivery-date: Sat, 13 Oct 2007 03:53:22 -0700
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>
  • Thread-index: AcgNfqFRSPZ6n/dbRgWjVrQ0bynVxQACBJIHAAAUspAAADhYYQ==
  • Thread-topic: [Xen-devel] [PATCH][VMX] a small clean to vmx_world_restore

The current code is correct in that case, and more generally, and also one
line shorter than the proposed patch.

 -- Keir

On 13/10/07 11:49, "Li, Xin B" <xin.b.li@xxxxxxxxx> wrote:

> If guest exits real mode, in vmx_world_restore, c->cr0 is equal to
> guest_cr0.
> -Xin 
> 
>> I think it's correct as is and changing it in this way could break it.
>> Whether a reference is acquired depends on new CR0.PG. Whether
>> a reference
>> is dropped depends on old CR0.PG. Which is how the code
>> currently works.
>> 
>> -- Keir
>> 
>> On 13/10/07 10:51, "Li, Xin B" <xin.b.li@xxxxxxxxx> wrote:
>> 
>>> a small clean to vmx_world_restore.
>>> 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


 


Rackspace

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