summary refs log tree commit diff stats
diff options
context:
space:
mode:
authorAnthony PERARD <anthony.perard@citrix.com>2021-04-30 17:34:32 +0100
committerAnthony PERARD <anthony.perard@citrix.com>2021-05-10 13:43:58 +0100
commit1898293990702c5601e225dac9afd2402fc46e2d (patch)
tree75e93fd1403280161afd3e0f6d16be0cf4278961
parentf1e43b6026500690fc402828fa7cc735175b93b6 (diff)
downloadfocaccia-qemu-1898293990702c5601e225dac9afd2402fc46e2d.tar.gz
focaccia-qemu-1898293990702c5601e225dac9afd2402fc46e2d.zip
xen-block: Use specific blockdev driver
... when a xen-block backend instance is created via xenstore.

Following 8d17adf34f50 ("block: remove support for using "file" driver
with block/char devices"), using the "file" blockdev driver for
everything doesn't work anymore, we need to use the "host_device"
driver when the disk image is a block device and "file" driver when it
is a regular file.

Signed-off-by: Anthony PERARD <anthony.perard@citrix.com>
Acked-by: Paul Durrant <paul@xen.org>
Message-Id: <20210430163432.468894-1-anthony.perard@citrix.com>
Signed-off-by: Anthony PERARD <anthony.perard@citrix.com>
-rw-r--r--hw/block/xen-block.c14
1 files changed, 13 insertions, 1 deletions
diff --git a/hw/block/xen-block.c b/hw/block/xen-block.c
index 83754a4344..674953f1ad 100644
--- a/hw/block/xen-block.c
+++ b/hw/block/xen-block.c
@@ -728,6 +728,8 @@ static XenBlockDrive *xen_block_drive_create(const char *id,
     XenBlockDrive *drive = NULL;
     QDict *file_layer;
     QDict *driver_layer;
+    struct stat st;
+    int rc;
 
     if (params) {
         char **v = g_strsplit(params, ":", 2);
@@ -761,7 +763,17 @@ static XenBlockDrive *xen_block_drive_create(const char *id,
     file_layer = qdict_new();
     driver_layer = qdict_new();
 
-    qdict_put_str(file_layer, "driver", "file");
+    rc = stat(filename, &st);
+    if (rc) {
+        error_setg_errno(errp, errno, "Could not stat file '%s'", filename);
+        goto done;
+    }
+    if (S_ISBLK(st.st_mode)) {
+        qdict_put_str(file_layer, "driver", "host_device");
+    } else {
+        qdict_put_str(file_layer, "driver", "file");
+    }
+
     qdict_put_str(file_layer, "filename", filename);
     g_free(filename);