[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 2/6] blkdebug: Don't require sophisticated filename
From: |
Max Reitz |
Subject: |
[Qemu-devel] [PATCH 2/6] blkdebug: Don't require sophisticated filename |
Date: |
Thu, 21 Nov 2013 20:05:50 +0100 |
If the filename is not prefixed by "blkdebug:" in
blkdebug_parse_filename(), the blkdebug driver was not selected through
that protocol prefix, but by an explicit command line option
(file.driver=blkdebug or something similar). Contrary to the current
reaction, this is not a problem at all; we just need to store the
filename (in the x-image option) and can go on; the user just has to
manually specify the config option.
Signed-off-by: Max Reitz <address@hidden>
---
block/blkdebug.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/block/blkdebug.c b/block/blkdebug.c
index 76f80c0..59d33a8 100644
--- a/block/blkdebug.c
+++ b/block/blkdebug.c
@@ -315,7 +315,9 @@ static void blkdebug_parse_filename(const char *filename,
QDict *options,
/* Parse the blkdebug: prefix */
if (!strstart(filename, "blkdebug:", &filename)) {
- error_setg(errp, "File name string must start with 'blkdebug:'");
+ /* There was no prefix; therefore, all options have to be already
+ present in the QDict (except for the filename) */
+ qdict_put(options, "x-image", qstring_from_str(filename));
return;
}
--
1.8.4.2
- [Qemu-devel] [PATCH 0/6] blkdebug/blkverify: Allow command-line configuration, Max Reitz, 2013/11/21
- [Qemu-devel] [PATCH 1/6] blkdebug: Use errp for read_config(), Max Reitz, 2013/11/21
- [Qemu-devel] [PATCH 2/6] blkdebug: Don't require sophisticated filename,
Max Reitz <=
- [Qemu-devel] [PATCH 4/6] blkdebug: Always call read_config(), Max Reitz, 2013/11/21
- [Qemu-devel] [PATCH 5/6] blkdebug: Use command-line in read_config(), Max Reitz, 2013/11/21
- [Qemu-devel] [PATCH 3/6] qemu-option: Add qemu_config_parse_qdict(), Max Reitz, 2013/11/21
- [Qemu-devel] [PATCH 6/6] blkverify: Don't require protocol filename, Max Reitz, 2013/11/21