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] [PATCH][VT] Patch to allow VMX domainsto be destroyedor

To: Khoa Huynh <khoa@xxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH][VT] Patch to allow VMX domainsto be destroyedor shut down cleanly
From: Keir Fraser <Keir.Fraser@xxxxxxxxxxxx>
Date: Mon, 19 Sep 2005 18:39:33 +0100
Cc: "Jiang, Yunhong" <yunhong.jiang@xxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxx, xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Delivery-date: Mon, 19 Sep 2005 17:32:27 +0000
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <OFF1F7D12C.4E717F75-ON85257081.005F7B46-86257081.005FCF84@xxxxxxxxxx>
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: <OFF1F7D12C.4E717F75-ON85257081.005F7B46-86257081.005FCF84@xxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx

On 19 Sep 2005, at 18:26, Khoa Huynh wrote:

If the shadow page reference counts are handled correctly now (I look
forward to receiving a complete patch!), do we really need
shadow_tainted_refcnts flag in the domain structure ?  What's
the real purpose of this flag ?  Is it used to indicate that
shadow mode was enabled but now disabled ?

Although the general ref counts are okay, I think the type counts can be wrong. Certainly the only use of tainted_refcnts check is to avoid an assertion in common/page_alloc.c (it asserts that if the general count is zero, teh type count must also be zero).

Do a debug build of Xen, remove the tainted_refcnts check from page_alloc.c, and see if destroying a vmx domain kills Xen. :-)

If not, maybe we can remove that flag.

 -- Keir


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