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

[Top] [All Lists]
<prev [Date] next>
[Advanced]
 <prev [Thread] next>

[Xen-devel] Re: [PATCH, v2]: xl: Check domain existance when doing domai

from [Ian Jackson] [Permanent Link][Original]
To: Gianni Tedesco <gianni.tedesco@xxxxxxxxxx>
Subject: [Xen-devel] Re: [PATCH, v2]: xl: Check domain existance when doing domain identifier lookups
From: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
Date: Tue, 25 Jan 2011 18:28:47 +0000
Cc: Christoph Egger <Christoph.Egger@xxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>, Stefano Stabellini <Stefano.Stabellini@xxxxxxxxxxxxx>
Delivery-date: Tue, 25 Jan 2011 10:29:23 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <1295976920.25215.5.camel@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/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: <A24AE1FFE7AEC5489F83450EE98351BF2BF2EC4C9D@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <1295888759.28333.14.camel@xxxxxxxxxxxxxxxxxxxxxx> <201101241827.46797.Christoph.Egger@xxxxxxx> <1295891031.28333.25.camel@xxxxxxxxxxxxxxxxxxxxxx> <1295893129.28333.31.camel@xxxxxxxxxxxxxxxxxxxxxx> <19775.2104.742709.639002@xxxxxxxxxxxxxxxxxxxxxxxx> <1295976920.25215.5.camel@xxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx 
Gianni Tedesco writes ("Re: [PATCH, v2]: xl: Check domain existance when doing 
domain identifier lookups"):
> Yes I think you are right, except for that we can make the same argument
> against more than just destroy. In other words, let's change the logic
> such that only non-existant domain causes this to fail and other errors
> are ignored.

Hrm, yes.  Thanks.  I have applied your patch.

I corrected some minor code style problems (whitespace placement) in
your patch.

Regards,
Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
[More with this subject...]
<Prev in Thread] Current Thread [Next in Thread>
Previous by Date:  [Xen-devel] Re: [PATCH]: xl: Check a domain exists before destroying it, Ian Jackson
Next by Date:  Re: [Xen-devel] linux kernel 2.6.37, Konrad Rzeszutek Wilk
Previous by Thread:  [Xen-devel] Re: [PATCH, v2]: xl: Check domain existance when doing domain identifier lookups, Gianni Tedesco
Next by Thread:  [Xen-devel] Re: [PATCH]: xl: Check a domain exists before destroying it, Ian Jackson
Indexes:  [Date] [Thread] [Top] [All Lists]
 
   

Copyright , Citrix Systems Inc. All rights reserved. Legal and Privacy