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

Re: [Xen-devel] [PATCH] clean up vmx_world_restore and its comments.


  • To: "Li, Xin B" <xin.b.li@xxxxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxx>
  • From: Keir Fraser <keir@xxxxxxxxxxxxx>
  • Date: Mon, 13 Aug 2007 17:21:46 +0100
  • Delivery-date: Mon, 13 Aug 2007 09:22:26 -0700
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>
  • Thread-index: AcfduaFp8Gbanf1SS1m6FgcICbDg0QADGmH7
  • Thread-topic: [Xen-devel] [PATCH] clean up vmx_world_restore and its comments.

+    /* guest setup page table in real mode, and enables PE, then PG. */
+    if ( !hvm_paging_enabled(v) && v->arch.hvm_vmx.vmxassist_enabled )
+        v->arch.hvm_vcpu.guest_cr[3] = c->cr3;

Any reason to gate this on vmxassist_enabled? Must you avoid modifying
guest_cr[3] when switching into vmxassist mode?

I'd really expect this code to just look a lot like vmx_vmcs_restore(). That
is, update all of CR0-CR4, then unconditionally get a reference on cr3 (if
paging enabled) and drop old cr3 reference.

 -- Keir

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

> Clean up vmx_world_restore and its comments.
> Also some other small cleanups.
> 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®.