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] Shouldn't XendDomainInfo.py: _releaseDevices call destro

To: "Petersson, Mats" <Mats.Petersson@xxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: Re: [Xen-devel] Shouldn't XendDomainInfo.py: _releaseDevices call destroyDevice?
From: Keir Fraser <keir@xxxxxxxxxxxxx>
Date: Wed, 16 May 2007 13:33:38 +0100
Delivery-date: Wed, 16 May 2007 05:32:09 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <907625E08839C4409CE5768403633E0B018E1D26@xxxxxxxxxxxxxxxxx>
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>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AceXskvMVkMzYNlrRCmrPOw8aBeWLgABCGsW
Thread-topic: [Xen-devel] Shouldn't XendDomainInfo.py: _releaseDevices call destroyDevice?
User-agent: Microsoft-Entourage/11.3.3.061214
This is how it looks to me too! Unless Ewan or Tom knows better, this is
probably worth trying out. It would be a neater solution for the console
backend cleanup than your hacky patch, if it works.

 -- Keir

On 16/5/07 13:04, "Petersson, Mats" <Mats.Petersson@xxxxxxx> wrote:

> It seems like something is missing in the _releaseDevices, and I think
> the correct way to fix it would be to call destroyDevice - is there any
> reason I shouldn't use that function, rather than the current method of
> removing (some of) the device info from XenStore in _releaseDevices?
> 
> --
> Mats
> 
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-devel


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