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

[Xen-devel] RE: [PATCH][RESEND] only BSP can really do clear_all_shadow_status


  • To: "Keir Fraser" <Keir.Fraser@xxxxxxxxxxxx>
  • From: "Li, Xin B" <xin.b.li@xxxxxxxxx>
  • Date: Sun, 16 Apr 2006 03:05:07 +0800
  • Cc: xen-devel Devel <xen-devel@xxxxxxxxxxxxxxxxxxx>
  • Delivery-date: Sat, 15 Apr 2006 12:05:34 -0700
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>
  • Thread-index: AcZgp7YcJFDEJqkHQJCC85BNK/f39gAFS33A
  • Thread-topic: [PATCH][RESEND] only BSP can really do clear_all_shadow_status

>On 15 Apr 2006, at 15:54, Li, Xin B wrote:
>
>> only BSP can really do clear_all_shadow_status.
>> This fixes SMP IA32 VMX guest booting on IA32 xen.
>
>Why can only VCPU0 do this? Is the argument to 
>clear_all_shadow_status() always current domain? If so that should 
>probably be asserted, or the argument removed.

Both Jun and I think clear_all_shadow_status is overkilled,
update_pagetables should have done the cleanup things, so we thought
about removing it, but the test shows that removing it breaks windows on
PAE xen, and I'm looking at this issue.

Actually, this patch should be a right direction, and changeset 9626 has
alrealdy changed shadow.c like what this patch does to shadow32.c.

For long term, maybe we will move to per vcpu shadow.

-Xin


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