Xen 
 
Home About Xen.org Xen Xen Summit Wiki Mailing List Bug Tracker Xen Downloads
 
   
 

xen-devel

Re: [Xen-devel] [RFC][PATCH] Use ioemu block drivers through blktap

To: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [RFC][PATCH] Use ioemu block drivers through blktap
From: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
Date: Mon, 31 Mar 2008 14:37:33 +0100
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx, Kevin Wolf <kwolf@xxxxxxx>
Delivery-date: Mon, 31 Mar 2008 06:38:43 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <C416A653.1EA20%keir.fraser@xxxxxxxxxxxxx>
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: <18416.58810.250091.262660@xxxxxxxxxxxxxxxxxxxxxxxx> <C416A653.1EA20%keir.fraser@xxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Keir Fraser writes ("Re: [Xen-devel] [RFC][PATCH] Use ioemu block drivers 
through blktap"):
> The SIGHUP was only introduced in the first place to allow cleanup when
> using stub domains. In that case the SIGHUP goes to a qemu-dm 'proxy script'
> which then kills the actual stub domain. That's another case where the
> mechanism for tearing down such pervasive state really belongs in xend, in
> my opinion. I would be very much up for seeing us going back to SIGKILL of
> qemu-dm processes.

Using SIGKILL is indeed much better because it guarantees that any
necessary cleanup happens even if qemu-dm (or some associated process)
is terminated abruptly - because it makes that be the usual case.

Ian.

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