WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] shadow2 corrupting PV guest state

To: Doi.Tsunehisa@xxxxxxxxxxxxxx
Subject: Re: [Xen-devel] shadow2 corrupting PV guest state
From: Tim Deegan <Tim.Deegan@xxxxxxxxxxxxx>
Date: Fri, 20 Oct 2006 14:57:18 +0100
Cc: Jeremy Fitzhardinge <jeremy@xxxxxxxx>, Xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>, Tim Deegan <Tim.Deegan@xxxxxxxxxxxxx>, Chris Wright <chrisw@xxxxxxxxxxxx>, Michael A Fetterman <Michael.Fetterman@xxxxxxxxxxxx>
Delivery-date: Fri, 20 Oct 2006 12:46:45 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <200610201342.k9KDgdF01725@xxxxxxxxxxxxxxxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <453020EE.4080603@xxxxxxxx> <200610201342.k9KDgdF01725@xxxxxxxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.5.13 (2006-08-11)
Hi,

At 22:42 +0900 on 20 Oct (1161384159), Doi.Tsunehisa@xxxxxxxxxxxxxx wrote:
>   So, I found that:
> 
>   * Before shadow2 age, x86 and ia64 use same logic for domain
>     destruction.
>     - at first, release gnttab references
>     - destruct page table for VCPU
>     - destruct P2M table for domain
>     - relinquish memory for domain
> 
>   * After shadow2 age, x86 introduces delayed P2M table destruction.
>     - release gnttab references
>     - destruct page table for VCPU
>     - relinquish memory for domain
>     - destruct P2M table for domain in domain_destroy()
>     *** I don't have confidence in my investigation. 
>     *** Am I right ?

Yep.  The P2M table can't be destroyed in domain_relinquish_resources,
as it is needed when pulling down grant references, and foreign domains
may have outstanding grant references to the dying domain's memory even
after domain_relinquish_resources.

>   If my speculation is correct, shadow2 may occur a problem of memory
> corruption.

I don't follow quite why this would lead to memory corruption.  Can you
explain?

Tim.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel