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

Re: [Xen-devel] [PATCH] 32on64 vCPU cleanup fix


  • To: Jan Beulich <jbeulich@xxxxxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxx>
  • From: Keir Fraser <keir@xxxxxxxxxxxxx>
  • Date: Thu, 26 Apr 2007 18:36:50 +0100
  • Delivery-date: Thu, 26 Apr 2007 10:35:44 -0700
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>
  • Thread-index: AceIKXh/ttz6kfQcEduB4QAX8io7RQ==
  • Thread-topic: [Xen-devel] [PATCH] 32on64 vCPU cleanup fix

On 26/4/07 16:55, "Jan Beulich" <jbeulich@xxxxxxxxxx> wrote:

> Since vcpu_destroy_pagetables() clears v->arch.guest_table for hvm vCPUs,
> it must be avoided to free the page with MFN zero.

Up to yesterday there was never a case that we'd have an HVM vcpu with
IS_COMPAT()==1 (although there was some code that tried to handle this
case). Now that we support this there's a general need for a cleanup so that
code that is compat-mode-dependent explicitly checks for PV vs. HVM if that
matters to it.

I've applied an initial general fix as c/s 14961:f18e1ca6. I think
everything else is okay, but I'm not 100% sure about shadow code (although
that was one place that was explicitly checking for PV 32bit guests where it
cared, so I hope it's okay).

Please take a look and make sure I'm not very confused about what's going
on. :-)

 -- Keir


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