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] libxl: remove console 0 backend directory from x

To: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: [Xen-devel] Re: [PATCH] libxl: remove console 0 backend directory from xenstore on devices destroy
From: Ian Campbell <Ian.Campbell@xxxxxxxxxxxxx>
Date: Mon, 4 Oct 2010 14:37:31 +0100
Delivery-date: Mon, 04 Oct 2010 06:38:20 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <d59be7a8e1e3bc439bb0.1286198519@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/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>
Organization: Citrix Systems, Inc.
References: <d59be7a8e1e3bc439bb0.1286198519@xxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
On Mon, 2010-10-04 at 14:21 +0100, Ian Campbell wrote:
> # HG changeset patch
> # User Ian Campbell <ian.campbell@xxxxxxxxxx>
> # Date 1286198484 -3600
> # Node ID d59be7a8e1e3bc439bb01a0f89d43411210fedcb
> # Parent  1cae52330eebbe448586c0a173945bc2bf8d631f
> libxl: remove console 0 backend directory from xenstore on devices destroy
> 
> The is (deliberately) no frontend directory for console 0 in
> xenstore. Therefore to avoid leaking
> /local/domain/0/backend/console/$domid/0 check explicitly for this
> backend directory and remove.

Thinking about this some more it might be preferable to
write /local/domain/domid/console/{backend,backend-id} giving us a
concrete pointer to the backend we wish to destroy, this would cope
better with console backends in dom!=0 too.

I'm assuming its not a mistake that the backend exists in the first
place?

Ian.


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