[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v6 16/25] block: Add 'base-directory' BDS option
From: |
Max Reitz |
Subject: |
[Qemu-devel] [PATCH v6 16/25] block: Add 'base-directory' BDS option |
Date: |
Fri, 29 Sep 2017 18:53:38 +0200 |
Using this option, one can directly override what bdrv_dirname() will
return. This is useful if one uses e.g. qcow2 on top of quorum (with
only protocol BDSs under the quorum BDS) and wants to be able to use
relative backing filenames.
Signed-off-by: Max Reitz <address@hidden>
---
qapi/block-core.json | 9 +++++++++
include/block/block_int.h | 2 ++
block.c | 14 ++++++++++++++
3 files changed, 25 insertions(+)
diff --git a/qapi/block-core.json b/qapi/block-core.json
index 750bb0c77c..c57741f4c6 100644
--- a/qapi/block-core.json
+++ b/qapi/block-core.json
@@ -3137,6 +3137,14 @@
# (default: off)
# @force-share: force share all permission on added nodes.
# Requires read-only=true. (Since 2.10)
+# @base-directory May be specified for any node. Normally, whenever a filename
+# is specified which is supposed to be relative to this node
+# (such as relative backing filenames), the base directory to
be
+# used is the directory the image file of this node is in,
which
+# is simply prepended to the relative filename. Using this
+# option, the string which is prepended (i.e. the base
+# directory) can be overridden.
+# (Since 2.10)
#
# Remaining options are determined by the block driver.
#
@@ -3145,6 +3153,7 @@
{ 'union': 'BlockdevOptions',
'base': { 'driver': 'BlockdevDriver',
'*node-name': 'str',
+ '*base-directory': 'str',
'*discard': 'BlockdevDiscardOptions',
'*cache': 'BlockdevCacheOptions',
'*read-only': 'bool',
diff --git a/include/block/block_int.h b/include/block/block_int.h
index 3a71be3734..3c53a373e1 100644
--- a/include/block/block_int.h
+++ b/include/block/block_int.h
@@ -615,6 +615,8 @@ struct BlockDriverState {
QDict *full_open_options;
char exact_filename[PATH_MAX];
+ char *dirname;
+
BdrvChild *backing;
BdrvChild *file;
diff --git a/block.c b/block.c
index fd67b667b7..c055bcce5a 100644
--- a/block.c
+++ b/block.c
@@ -1250,6 +1250,12 @@ QemuOptsList bdrv_runtime_opts = {
.type = QEMU_OPT_BOOL,
.help = "always accept other writers (default: off)",
},
+ {
+ .name = "base-directory",
+ .type = QEMU_OPT_STRING,
+ .help = "String to prepend to filenames relative to this BDS for "
+ "making them absolute",
+ },
{ /* end of list */ }
},
};
@@ -1322,6 +1328,8 @@ static int bdrv_open_common(BlockDriverState *bs,
BlockBackend *file,
trace_bdrv_open_common(bs, filename ?: "", bs->open_flags,
drv->format_name);
+ bs->dirname = g_strdup(qemu_opt_get(opts, "base-directory"));
+
bs->read_only = !(bs->open_flags & BDRV_O_RDWR);
if (use_bdrv_whitelist && !bdrv_is_whitelisted(drv, bs->read_only)) {
@@ -3383,6 +3391,8 @@ static void bdrv_delete(BlockDriverState *bs)
bdrv_close(bs);
+ g_free(bs->dirname);
+
/* remove from list, if necessary */
if (bs->node_name[0] != '\0') {
QTAILQ_REMOVE(&graph_bdrv_states, bs, node_list);
@@ -5032,6 +5042,10 @@ char *bdrv_dirname(BlockDriverState *bs, Error **errp)
{
BlockDriver *drv = bs->drv;
+ if (bs->dirname) {
+ return g_strdup(bs->dirname);
+ }
+
if (!drv) {
error_setg(errp, "Node '%s' is ejected", bs->node_name);
return NULL;
--
2.13.6
- [Qemu-devel] [PATCH v6 06/25] block: bdrv_get_full_backing_filename_from_...'s ret. val., (continued)
- [Qemu-devel] [PATCH v6 06/25] block: bdrv_get_full_backing_filename_from_...'s ret. val., Max Reitz, 2017/09/29
- [Qemu-devel] [PATCH v6 07/25] block: bdrv_get_full_backing_filename's ret. val., Max Reitz, 2017/09/29
- [Qemu-devel] [PATCH v6 08/25] block: Add bdrv_make_absolute_filename(), Max Reitz, 2017/09/29
- [Qemu-devel] [PATCH v6 09/25] block: Fix bdrv_find_backing_image(), Max Reitz, 2017/09/29
- [Qemu-devel] [PATCH v6 10/25] block: Add bdrv_dirname(), Max Reitz, 2017/09/29
- [Qemu-devel] [PATCH v6 11/25] blkverify: Make bdrv_dirname() return NULL, Max Reitz, 2017/09/29
- [Qemu-devel] [PATCH v6 12/25] quorum: Make bdrv_dirname() return NULL, Max Reitz, 2017/09/29
- [Qemu-devel] [PATCH v6 13/25] block/nbd: Make bdrv_dirname() return NULL, Max Reitz, 2017/09/29
- [Qemu-devel] [PATCH v6 14/25] block/nfs: Implement bdrv_dirname(), Max Reitz, 2017/09/29
- [Qemu-devel] [PATCH v6 15/25] block: Use bdrv_dirname() for relative filenames, Max Reitz, 2017/09/29
- [Qemu-devel] [PATCH v6 16/25] block: Add 'base-directory' BDS option,
Max Reitz <=
- [Qemu-devel] [PATCH v6 17/25] iotests: Add quorum case to test 110, Max Reitz, 2017/09/29
- [Qemu-devel] [PATCH v6 18/25] block: Add sgfnt_runtime_opts to BlockDriver, Max Reitz, 2017/09/29
- [Qemu-devel] [PATCH v6 19/25] block: Add BlockDriver.bdrv_gather_child_options, Max Reitz, 2017/09/29
- [Qemu-devel] [PATCH v6 20/25] block: Generically refresh runtime options, Max Reitz, 2017/09/29
- [Qemu-devel] [PATCH v6 22/25] block: Do not copy exact_filename from format file, Max Reitz, 2017/09/29
- [Qemu-devel] [PATCH v6 21/25] block: Purify .bdrv_refresh_filename(), Max Reitz, 2017/09/29
- [Qemu-devel] [PATCH v6 23/25] block: Fix FIXME from "Add BDS.backing_overridden", Max Reitz, 2017/09/29
- [Qemu-devel] [PATCH v6 24/25] block/curl: Implement bdrv_refresh_filename(), Max Reitz, 2017/09/29
- [Qemu-devel] [PATCH v6 25/25] block/null: Generate filename even with latency-ns, Max Reitz, 2017/09/29