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: [Xen-changelog] Deal with domain info going away (handle

To: xen-devel@xxxxxxxxxxxxxxxxxxx
Subject: [Xen-devel] Re: [Xen-changelog] Deal with domain info going away (handle new error path introduced by
From: Daniel Stekloff <dsteklof@xxxxxxxxxx>
Date: Thu, 15 Sep 2005 07:58:48 -0700
Cc: steven@xxxxxxxxxxxxx
Delivery-date: Thu, 15 Sep 2005 14:56:43 +0000
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <E1EFl8U-0005GC-RK@xxxxxxxxxxxxxxxxxxxxx>
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: <E1EFl8U-0005GC-RK@xxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
On Thu, 2005-09-15 at 04:14 +0000, Xen patchbot -unstable wrote:
> # HG changeset patch
> # User shand@xxxxxxxxxxxxxxxxxxxxxxxxxxx
> # Node ID bd477fcd32aba8a3196873b702ae6ea0d8a7e43b
> # Parent  36f8c5900e8646b0465d92d602bd07f3d3b5a966
> Deal with domain info going away (handle new error path introduced by 
> cset 6828). Believe Dan Smith is working up a more comprehensive patch.
> 
> Signed-off-by: Steven Hand <steven@xxxxxxxxxxxxx>
> 
> diff -r 36f8c5900e86 -r bd477fcd32ab tools/python/xen/xend/XendDomainInfo.py
> --- a/tools/python/xen/xend/XendDomainInfo.py Thu Sep 15 01:46:40 2005
> +++ b/tools/python/xen/xend/XendDomainInfo.py Thu Sep 15 04:12:54 2005
> @@ -353,7 +353,12 @@
>      def update(self, info=None):
>          """Update with  info from xc.domain_getinfo().
>          """
> -        self.info = info or dom_get(self.domid)
> +        if info:
> +            self.info = info
> +        else:
> +            di = dom_get(self.domid)
> +            if not di:
> +                return 



Um.... shouldn't there be a "self.info = di" somewhere?




>          self.memory = self.info['mem_kb'] / 1024
>          self.ssidref = self.info['ssidref']



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