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/
Home Products Support Community News


Re: [Xen-devel] [PATCH] Fix stubdom-dm using "grep" improperly [and 1 mo

To: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] Fix stubdom-dm using "grep" improperly [and 1 more messages]
From: Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>
Date: Tue, 11 Jan 2011 16:59:02 +0000
Cc: Keir Fraser <keir@xxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>, John Weekes <lists.xen@xxxxxxxxxxxxxxxxxx>, Stefano Stabellini <Stefano.Stabellini@xxxxxxxxxxxxx>
Delivery-date: Tue, 11 Jan 2011 09:00:39 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <19756.34953.50436.947990@xxxxxxxxxxxxxxxxxxxxxxxx>
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/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <4D2A94A0.6030505@xxxxxxxxxxxxxxxxxx> <4D2AB68E.6050804@xxxxxxxxxxxxxxxxxx> <19756.34953.50436.947990@xxxxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Alpine 2.00 (DEB 1167 2008-08-23)
On Tue, 11 Jan 2011, Ian Jackson wrote:
> John Weekes writes ("[Xen-devel] [PATCH] Fix stubdom-dm using "grep" 
> improperly"):
> > stubdom-dm uses "grep" on "xm list" output to determine whether it is 
> > already running. The existing behavior is to use "grep $domname-dm" but 
> > this will result in a false-positive in the case of another domU running 
> > whose name ends with the full new name; for instance, if "abctest-dm" is 
> > running, a new "test-dm" will spin forever, waiting for it the end.
> Urgh.  Thanks for this patch.  This code is pretty horrid stuff, I
> have to say.  I have applied your fix to xen-unstable.
> Keir, can you please also commit this to 4.0-testing ?  It applies
> cleanly.
> Stefano: does xl in stubdom mode still use stubdom-dm ?  If so can it
> please not ?  Certainly "xm list" isn't going to work!

It doesn't.

Xen-devel mailing list