[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v12 22/31] block: Add BlockDriver.bdrv_gather_child_
From: |
Max Reitz |
Subject: |
[Qemu-devel] [PATCH v12 22/31] block: Add BlockDriver.bdrv_gather_child_options |
Date: |
Mon, 17 Dec 2018 23:43:39 +0100 |
Some follow-up patches will rework the way bs->full_open_options is
refreshed in bdrv_refresh_filename(). The new implementation will remove
the need for the block drivers' bdrv_refresh_filename() implementations
to set bs->full_open_options; instead, it will be generic and use static
information from each block driver.
However, by implementing bdrv_gather_child_options(), block drivers will
still be able to override the way the full_open_options of their
children are incorporated into their own.
We need to implement this function for VMDK because we have to prevent
the generic implementation from gathering the options of all children:
It is not possible to specify options for the extents through the
runtime options.
For quorum, the child names that would be used by the generic
implementation and the ones that we actually (currently) want to use
differ. See quorum_gather_child_options() for more information.
Note that both of these are cases which are not ideal: In case of VMDK
it would probably be nice to be able to specify options for all extents.
In case of quorum, the current runtime option structure is simply broken
and needs to be fixed (but that is left for another patch).
Signed-off-by: Max Reitz <address@hidden>
Reviewed-by: Alberto Garcia <address@hidden>
---
include/block/block_int.h | 24 +++++++++++++++++++++++
block/quorum.c | 40 +++++++++++++++++++++++++++++++++++++++
block/vmdk.c | 19 +++++++++++++++++++
3 files changed, 83 insertions(+)
diff --git a/include/block/block_int.h b/include/block/block_int.h
index 40f00aa44e..8df23ab79c 100644
--- a/include/block/block_int.h
+++ b/include/block/block_int.h
@@ -141,6 +141,30 @@ struct BlockDriver {
void (*bdrv_refresh_filename)(BlockDriverState *bs, QDict *options);
+ /*
+ * Gathers the open options for all children into @target.
+ * A simple format driver (without backing file support) might
+ * implement this function like this:
+ *
+ * QINCREF(bs->file->bs->full_open_options);
+ * qdict_put(target, "file", bs->file->bs->full_open_options);
+ *
+ * If not specified, the generic implementation will simply put
+ * all children's options under their respective name.
+ *
+ * @backing_overridden is true when bs->backing seems not to be
+ * the child that would result from opening bs->backing_file.
+ * Therefore, if it is true, the backing child's options should be
+ * gathered; otherwise, there is no need since the backing child
+ * is the one implied by the image header.
+ *
+ * Note that ideally this function would not be needed. Every
+ * block driver which implements it is probably doing something
+ * shady regarding its runtime option structure.
+ */
+ void (*bdrv_gather_child_options)(BlockDriverState *bs, QDict *target,
+ bool backing_overridden);
+
/*
* Returns an allocated string which is the directory name of this BDS: It
* will be used to make relative filenames absolute by prepending this
diff --git a/block/quorum.c b/block/quorum.c
index 1af6458dc4..3984f0aa4f 100644
--- a/block/quorum.c
+++ b/block/quorum.c
@@ -1094,6 +1094,45 @@ static void quorum_refresh_filename(BlockDriverState
*bs, QDict *options)
bs->full_open_options = opts;
}
+static void quorum_gather_child_options(BlockDriverState *bs, QDict *target,
+ bool backing_overridden)
+{
+ BDRVQuorumState *s = bs->opaque;
+ QList *children_list;
+ int i;
+
+ /*
+ * The generic implementation for gathering child options in
+ * bdrv_refresh_filename() would use the names of the children
+ * as specified for bdrv_open_child() or bdrv_attach_child(),
+ * which is "children.%u" with %u being a value
+ * (s->next_child_index) that is incremented each time a new child
+ * is added (and never decremented). Since children can be
+ * deleted at runtime, there may be gaps in that enumeration.
+ * When creating a new quorum BDS and specifying the children for
+ * it through runtime options, the enumeration used there may not
+ * have any gaps, though.
+ *
+ * Therefore, we have to create a new gap-less enumeration here
+ * (which we can achieve by simply putting all of the children's
+ * full_open_options into a QList).
+ *
+ * XXX: Note that there are issues with the current child option
+ * structure quorum uses (such as the fact that children do
+ * not really have unique permanent names). Therefore, this
+ * is going to have to change in the future and ideally we
+ * want quorum to be covered by the generic implementation.
+ */
+
+ children_list = qlist_new();
+ qdict_put(target, "children", children_list);
+
+ for (i = 0; i < s->num_children; i++) {
+ qlist_append(children_list,
+ qobject_ref(s->children[i]->bs->full_open_options));
+ }
+}
+
static char *quorum_dirname(BlockDriverState *bs, Error **errp)
{
/* In general, there are multiple BDSs with different dirnames below this
@@ -1121,6 +1160,7 @@ static BlockDriver bdrv_quorum = {
.bdrv_open = quorum_open,
.bdrv_close = quorum_close,
.bdrv_refresh_filename = quorum_refresh_filename,
+ .bdrv_gather_child_options = quorum_gather_child_options,
.bdrv_dirname = quorum_dirname,
.bdrv_co_flush_to_disk = quorum_co_flush,
diff --git a/block/vmdk.c b/block/vmdk.c
index b3fc3b9e12..0b3a19804d 100644
--- a/block/vmdk.c
+++ b/block/vmdk.c
@@ -27,6 +27,7 @@
#include "qapi/error.h"
#include "block/block_int.h"
#include "sysemu/block-backend.h"
+#include "qapi/qmp/qdict.h"
#include "qapi/qmp/qerror.h"
#include "qemu/error-report.h"
#include "qemu/module.h"
@@ -2377,6 +2378,23 @@ static int vmdk_get_info(BlockDriverState *bs,
BlockDriverInfo *bdi)
return 0;
}
+static void vmdk_gather_child_options(BlockDriverState *bs, QDict *target,
+ bool backing_overridden)
+{
+ /* No children but file and backing can be explicitly specified (TODO) */
+ qdict_put(target, "file",
+ qobject_ref(bs->file->bs->full_open_options));
+
+ if (backing_overridden) {
+ if (bs->backing) {
+ qdict_put(target, "backing",
+ qobject_ref(bs->backing->bs->full_open_options));
+ } else {
+ qdict_put_null(target, "backing");
+ }
+ }
+}
+
static QemuOptsList vmdk_create_opts = {
.name = "vmdk-create-opts",
.head = QTAILQ_HEAD_INITIALIZER(vmdk_create_opts.head),
@@ -2447,6 +2465,7 @@ static BlockDriver bdrv_vmdk = {
.bdrv_get_specific_info = vmdk_get_specific_info,
.bdrv_refresh_limits = vmdk_refresh_limits,
.bdrv_get_info = vmdk_get_info,
+ .bdrv_gather_child_options = vmdk_gather_child_options,
.supports_backing = true,
.create_opts = &vmdk_create_opts,
--
2.19.2
- [Qemu-devel] [PATCH v12 10/31] block: bdrv_get_full_backing_filename_from_...'s ret. val., (continued)
- [Qemu-devel] [PATCH v12 10/31] block: bdrv_get_full_backing_filename_from_...'s ret. val., Max Reitz, 2018/12/17
- [Qemu-devel] [PATCH v12 11/31] block: bdrv_get_full_backing_filename's ret. val., Max Reitz, 2018/12/17
- [Qemu-devel] [PATCH v12 13/31] block: Fix bdrv_find_backing_image(), Max Reitz, 2018/12/17
- [Qemu-devel] [PATCH v12 14/31] block: Add bdrv_dirname(), Max Reitz, 2018/12/17
- [Qemu-devel] [PATCH v12 15/31] blkverify: Make bdrv_dirname() return NULL, Max Reitz, 2018/12/17
- [Qemu-devel] [PATCH v12 16/31] quorum: Make bdrv_dirname() return NULL, Max Reitz, 2018/12/17
- [Qemu-devel] [PATCH v12 17/31] block/nbd: Make bdrv_dirname() return NULL, Max Reitz, 2018/12/17
- [Qemu-devel] [PATCH v12 18/31] block/nfs: Implement bdrv_dirname(), Max Reitz, 2018/12/17
- [Qemu-devel] [PATCH v12 19/31] block: Use bdrv_dirname() for relative filenames, Max Reitz, 2018/12/17
- [Qemu-devel] [PATCH v12 20/31] iotests: Add quorum case to test 110, Max Reitz, 2018/12/17
- [Qemu-devel] [PATCH v12 22/31] block: Add BlockDriver.bdrv_gather_child_options,
Max Reitz <=
- [Qemu-devel] [PATCH v12 21/31] block: Add strong_runtime_opts to BlockDriver, Max Reitz, 2018/12/17
- [Qemu-devel] [PATCH v12 23/31] block: Generically refresh runtime options, Max Reitz, 2018/12/17
- [Qemu-devel] [PATCH v12 25/31] block: Do not copy exact_filename from format file, Max Reitz, 2018/12/17
- [Qemu-devel] [PATCH v12 26/31] block/nvme: Fix bdrv_refresh_filename(), Max Reitz, 2018/12/17
- [Qemu-devel] [PATCH v12 24/31] block: Purify .bdrv_refresh_filename(), Max Reitz, 2018/12/17
- [Qemu-devel] [PATCH v12 27/31] block/curl: Harmonize option defaults, Max Reitz, 2018/12/17
- [Qemu-devel] [PATCH v12 28/31] block/curl: Implement bdrv_refresh_filename(), Max Reitz, 2018/12/17
- [Qemu-devel] [PATCH v12 30/31] block: BDS options may lack the "driver" option, Max Reitz, 2018/12/17
- [Qemu-devel] [PATCH v12 29/31] block/null: Generate filename even with latency-ns, Max Reitz, 2018/12/17
- [Qemu-devel] [PATCH v12 31/31] iotests: Test json:{} filenames of internal BDSs, Max Reitz, 2018/12/17