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] That xenstored console leak...

To: Keir Fraser <Keir.Fraser@xxxxxxxxxxxx>
Subject: Re: [Xen-devel] That xenstored console leak...
From: John Levon <levon@xxxxxxxxxxxxxxxxx>
Date: Tue, 15 Jan 2008 14:31:54 +0000
Cc: Jim Fehlig <jfehlig@xxxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Tue, 15 Jan 2008 06:33:05 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <C3B26CD9.1AB80%Keir.Fraser@xxxxxxxxxxxx>
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: <C3B26C45.1AB7D%Keir.Fraser@xxxxxxxxxxxx> <C3B26CD9.1AB80%Keir.Fraser@xxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.5.9i
On Tue, Jan 15, 2008 at 01:59:53PM +0000, Keir Fraser wrote:

> >> I don't think so - domain destruction already removes the /vm path (I'd
> >> confused myself here). I just verified this with our 3.1 bits.
> > 
> > We could officially make /vm/ have the same lifetime as /local/domain, in
> > which case it should die even on domain localhost migration, or we could
> > take the patch that Jim suggests, and revert the other two. I don't have a
> > particularly strong opinion either way.
> 
> Actually if we attach/detach lots of virtual devices to a single domain,
> will that end up leaking xenstore nodes with Jim's patch?

I've become confused about exactly what changes Jim's patch entails,
since he's backing stuff out too, but it does seem like it.

Theoretically, if we made "xm create" domains have an XML file too, we
wouldn't ever need /vm right?

Anyway, seems like we need a simple working fix for 3.1 series, even if
there's larger surgery done.

regards
john

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