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] Linux-specific blkif.py change

To: Ewan Mellor <ewan@xxxxxxxxxxxxx>
Subject: Re: [Xen-devel] Linux-specific blkif.py change
From: John Levon <levon@xxxxxxxxxxxxxxxxx>
Date: Thu, 2 Nov 2006 23:38:04 +0000
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Thu, 02 Nov 2006 21:32:23 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <20061102232117.GA30953@xxxxxxxxxxxxxxxxxxxxxx>
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: <20061102014250.GA1575@xxxxxxxxxxxxxxxxxxxx> <20061102232117.GA30953@xxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.5.9i
On Thu, Nov 02, 2006 at 11:21:17PM +0000, Ewan Mellor wrote:

> > 1) util/blkif.py logs to xend-debug.log if the stat() fails. This is
> > needlessly chatty, and indicates there's some kind of error, when there
> > is not.
> > 
> > 2) util/blkif.py has a load of Linux gook for getting the device
> > numbers. Luckily Solaris has a completely different naming scheme, but
> > wouldn't this go horribly wrong if a domU just happened to use the same
> > name, different device number?
> > 
> > It's not clear to us why Linux even needs to do this?
> 
> I think that the correct fix would be for the tools to pass the untranslated
> device name into the guest, rather than translating it to a device number

Sounds sensible to me.

> that it's not an old Linux guest.  The change was intended to improve the
> error message that you receive in this case, so at the least, the failure
> ought to be logged (unless you can come up with some way to detect old Linux
> guests, and only complain in that case).

Is there some other way to indicate the failure later? We'd like
xend-debug.log to be essentially silent during normal operation for a
non-debug xend...

> Would you like to put together a patch along these lines?

I can do a patch for xend, but I'm not familiar enough to update the
Linux side of things.

I see that the 'is none' hack has been committed along with removing the
message in blkif.py, so that solves the immediate issue for us.

thanks,
john

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