Re: [Xen-devel] [PATCH] blkfront: don't change to closing if we're busy
From: Andrew Jones <hidden>
Date: 2012-02-17 12:50:11
Also in:
xen-devel
----- Original Message -----
On Thu, Feb 16, 2012 at 01:17:09PM +0100, Andrew Jones wrote:quoted
We just reported to xenbus that we can't close yet, because blkfront is still in use. So we shouldn't then immediately state that we are closing.What happens if the user uses --force to unplug the device? Will that still work?
With RHEL5 tooling I get the same results. That is, the device is forcibly detached, and then any task in the guest that still attempts to use (or even just unmount) the device hangs. I don't know anything about xl, but afaict block-detach doesn't take a 'force' option? Drew
quoted
Signed-off-by: Andrew Jones <redacted> --- drivers/block/xen-blkfront.c | 1 - 1 files changed, 0 insertions(+), 1 deletions(-)diff --git a/drivers/block/xen-blkfront.cb/drivers/block/xen-blkfront.c index 5d45688..b53cae4 100644--- a/drivers/block/xen-blkfront.c +++ b/drivers/block/xen-blkfront.c@@ -1134,7 +1134,6 @@ blkfront_closing(struct blkfront_info *info) if (bdev->bd_openers) { xenbus_dev_error(xbdev, -EBUSY, "Device in use; refusing to close"); - xenbus_switch_state(xbdev, XenbusStateClosing); } else { xlvbd_release_gendisk(info); xenbus_frontend_closed(xbdev); --1.7.7.5 _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel