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

[Xen-devel] Re: [PATCH] Error message for device not found at blkif.py

To: Glauber de Oliveira Costa <gcosta@xxxxxxxxxx>
Subject: [Xen-devel] Re: [PATCH] Error message for device not found at blkif.py
From: Anthony Liguori <aliguori@xxxxxxxxxx>
Date: Wed, 18 Oct 2006 16:37:38 -0500
Cc: xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Wed, 18 Oct 2006 14:38:19 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <20061018212846.GC16266@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: <20061018200845.GB16266@xxxxxxxxxx> <453695B2.9080307@xxxxxxxxxx> <20061018212846.GC16266@xxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Thunderbird 1.5.0.7 (X11/20060918)
Glauber de Oliveira Costa wrote:
On Wed, Oct 18, 2006 at 03:59:30PM -0500, Anthony Liguori wrote:
Instead of throwing a VmError, could you subclass VmError with a more specific error and throw that?
Fore sure I can. But what does this condition have so differently from
others that justifies that for it only?

Good question. Previously, we only threw opaque errors back over the wire. A few months ago, we changed that so that we could pass exceptions over the wire with specific Fault ids. The new Xend API should be even better for this.

So, the answer we know have the ability to do useful things with this info so it's now a best practice for new code :-)

I'm not suggesting you go change everything, just in your patch.

Regards,

Anthony Liguori

Although I agree with you that such a specificiness is good, VmError is being thrown everywhere, meaning that your proposal would require touching a great amount of code change.
Is there any plans/opposals for that?


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