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] earlier remove the backend of tapdisk device in xenstore

To: Jan Beulich <JBeulich@xxxxxxxxxx>, James Song <JSong@xxxxxxxxxx>
Subject: Re: [Xen-devel] earlier remove the backend of tapdisk device in xenstore to release the resource allocated in backend driver lies in dom0'kernel
From: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
Date: Fri, 23 Apr 2010 09:54:43 +0100
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Fri, 23 Apr 2010 01:55:24 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <4BD172C9020000780003B9E4@xxxxxxxxxxxxxxxxxx>
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>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcrivPeQ/QMg9uSzRlmGnzmdbTTSkQABaYVW
Thread-topic: [Xen-devel] earlier remove the backend of tapdisk device in xenstore to release the resource allocated in backend driver lies in dom0'kernel
User-agent: Microsoft-Entourage/12.24.0.100205
On 23/04/2010 09:13, "Jan Beulich" <JBeulich@xxxxxxxxxx> wrote:

>> Moving the device destruction a little ahead of  killing qemu-dm would
>> triger blktapctl thread send CTRMSG_CLOSE to "qemu-dm" before it exit.
>> And then, qemu-dm would notify backend driver to release resouce by
>> calling release() of driver through closing the opened device file of
>> "/dev/xen/blktapN"
> 
> Even though Keir already applied your patch, I continue to disagree:

I agreed with you and Jim, and subsequently reverted it.

 K.



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