[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v6 12/22] block: Use bdrv_open_image() in bdrv_open(
From: |
Max Reitz |
Subject: |
[Qemu-devel] [PATCH v6 12/22] block: Use bdrv_open_image() in bdrv_open() |
Date: |
Thu, 19 Dec 2013 20:47:13 +0100 |
Using bdrv_open_image() instead of bdrv_file_open() directly in
bdrv_open() is easier.
Signed-off-by: Max Reitz <address@hidden>
---
block.c | 18 +++++-------------
1 file changed, 5 insertions(+), 13 deletions(-)
diff --git a/block.c b/block.c
index 76b6c25..9e4e85f 100644
--- a/block.c
+++ b/block.c
@@ -1128,8 +1128,6 @@ int bdrv_open(BlockDriverState *bs, const char *filename,
QDict *options,
/* TODO: extra byte is a hack to ensure MAX_PATH space on Windows. */
char tmp_filename[PATH_MAX + 1];
BlockDriverState *file = NULL;
- QDict *file_options = NULL;
- const char *file_reference;
const char *drvname;
Error *local_err = NULL;
@@ -1215,17 +1213,11 @@ int bdrv_open(BlockDriverState *bs, const char
*filename, QDict *options,
flags |= BDRV_O_ALLOW_RDWR;
}
- qdict_extract_subqdict(options, &file_options, "file.");
- file_reference = qdict_get_try_str(options, "file");
-
- if (filename || file_reference || qdict_size(file_options)) {
- ret = bdrv_file_open(&file, filename, file_reference, file_options,
- bdrv_open_flags(bs, flags | BDRV_O_UNMAP),
- &local_err);
- qdict_del(options, "file");
- if (ret < 0) {
- goto fail;
- }
+ ret = bdrv_open_image(&file, filename, options, "file",
+ bdrv_open_flags(bs, flags | BDRV_O_UNMAP), true,
true,
+ &local_err);
+ if (ret < 0) {
+ goto fail;
}
/* Find the right image format driver */
--
1.8.5.1
- [Qemu-devel] [PATCH v6 02/22] blkdebug: Don't require sophisticated filename, (continued)
- [Qemu-devel] [PATCH v6 02/22] blkdebug: Don't require sophisticated filename, Max Reitz, 2013/12/19
- [Qemu-devel] [PATCH v6 03/22] qdict: Add qdict_array_split(), Max Reitz, 2013/12/19
- [Qemu-devel] [PATCH v6 04/22] qapi: extend qdict_flatten() for QLists, Max Reitz, 2013/12/19
- [Qemu-devel] [PATCH v6 05/22] qemu-option: Add qemu_config_parse_qdict(), Max Reitz, 2013/12/19
- [Qemu-devel] [PATCH v6 06/22] blkdebug: Always call read_config(), Max Reitz, 2013/12/19
- [Qemu-devel] [PATCH v6 07/22] blkdebug: Use command-line in read_config(), Max Reitz, 2013/12/19
- [Qemu-devel] [PATCH v6 08/22] block: Allow reference for bdrv_file_open(), Max Reitz, 2013/12/19
- [Qemu-devel] [PATCH v6 09/22] block: Pass reference to bdrv_file_open(), Max Reitz, 2013/12/19
- [Qemu-devel] [PATCH v6 10/22] block: Allow block devices without files, Max Reitz, 2013/12/19
- [Qemu-devel] [PATCH v6 11/22] block: Add bdrv_open_image(), Max Reitz, 2013/12/19
- [Qemu-devel] [PATCH v6 12/22] block: Use bdrv_open_image() in bdrv_open(),
Max Reitz <=
- [Qemu-devel] [PATCH v6 13/22] block: Allow recursive "file"s, Max Reitz, 2013/12/19
- [Qemu-devel] [PATCH v6 14/22] blockdev: Move "file" to legacy_opts, Max Reitz, 2013/12/19
- [Qemu-devel] [PATCH v6 15/22] blkdebug: Allow command-line file configuration, Max Reitz, 2013/12/19
- [Qemu-devel] [PATCH v6 16/22] blkverify: Allow command-line configuration, Max Reitz, 2013/12/19
- [Qemu-devel] [PATCH v6 18/22] qapi: Add "errno" to the list of polluted words, Max Reitz, 2013/12/19
- [Qemu-devel] [PATCH v6 17/22] blkverify: Don't require protocol filename, Max Reitz, 2013/12/19
- [Qemu-devel] [PATCH v6 19/22] qapi: QMP interface for blkdebug and blkverify, Max Reitz, 2013/12/19
- [Qemu-devel] [PATCH v6 20/22] qemu-io: Make filename optional, Max Reitz, 2013/12/19
- [Qemu-devel] [PATCH v6 21/22] iotests: Test new blkdebug/blkverify interface, Max Reitz, 2013/12/19
- [Qemu-devel] [PATCH v6 22/22] iotests: Test file format nesting, Max Reitz, 2013/12/19