2009-05-04 18:38:09 +02:00
|
|
|
From cb982fd919a52ff86f01025d0f92225bc7b2a956 Mon Sep 17 00:00:00 2001
|
|
|
|
From: Kevin Wolf <kwolf@suse.de>
|
|
|
|
Date: Tue, 10 Mar 2009 16:44:31 +0100
|
|
|
|
Subject: [PATCH 5/6] ioemu: Fail on too small blktap disks
|
|
|
|
|
|
|
|
The blktap infrastructure doesn't seems to be able to cope with images
|
|
|
|
that are smaller than a sector, it produced hangs for me. Such an
|
|
|
|
image isn't really useful anyway, so just fail gracefully.
|
|
|
|
|
|
|
|
Signed-off-by: Kevin Wolf <kwolf@suse.de>
|
|
|
|
---
|
|
|
|
hw/xen_blktap.c | 6 ++++++
|
|
|
|
1 files changed, 6 insertions(+), 0 deletions(-)
|
|
|
|
|
2010-09-28 00:07:21 +02:00
|
|
|
Index: xen-4.0.1-testing/tools/ioemu-qemu-xen/hw/xen_blktap.c
|
2008-11-08 20:32:12 +01:00
|
|
|
===================================================================
|
2010-09-28 00:07:21 +02:00
|
|
|
--- xen-4.0.1-testing.orig/tools/ioemu-qemu-xen/hw/xen_blktap.c
|
|
|
|
+++ xen-4.0.1-testing/tools/ioemu-qemu-xen/hw/xen_blktap.c
|
2010-03-02 01:46:56 +01:00
|
|
|
@@ -258,6 +258,12 @@ static int open_disk(struct td_state *s,
|
2008-11-08 20:32:12 +01:00
|
|
|
s->size = bs->total_sectors;
|
|
|
|
s->sector_size = 512;
|
|
|
|
|
|
|
|
+ if (s->size == 0) {
|
|
|
|
+ fprintf(stderr, "Error: Disk image %s is too small\n",
|
|
|
|
+ path);
|
|
|
|
+ return -ENOMEM;
|
|
|
|
+ }
|
|
|
|
+
|
|
|
|
s->info = ((s->flags & TD_RDONLY) ? VDISK_READONLY : 0);
|
|
|
|
|
|
|
|
#ifndef QEMU_TOOL
|
2010-09-28 00:07:21 +02:00
|
|
|
Index: xen-4.0.1-testing/tools/python/xen/xend/server/DevController.py
|
2008-11-08 20:32:12 +01:00
|
|
|
===================================================================
|
2010-09-28 00:07:21 +02:00
|
|
|
--- xen-4.0.1-testing.orig/tools/python/xen/xend/server/DevController.py
|
|
|
|
+++ xen-4.0.1-testing/tools/python/xen/xend/server/DevController.py
|
2009-05-04 18:38:09 +02:00
|
|
|
@@ -155,7 +155,7 @@ class DevController:
|
2008-11-08 20:32:12 +01:00
|
|
|
(devid, self.deviceClass))
|
|
|
|
|
|
|
|
elif status == Error:
|
|
|
|
- self.destroyDevice(devid, False)
|
|
|
|
+ self.destroyDevice(devid, True)
|
|
|
|
if err is None:
|
|
|
|
raise VmError("Device %s (%s) could not be connected. "
|
|
|
|
"Backend device not found." %
|