[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [Qemu-devel] [PATCH v3 2/4] block: auto-generated node-
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-block] [Qemu-devel] [PATCH v3 2/4] block: auto-generated node-names |
Date: |
Tue, 13 Oct 2015 09:39:57 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Jeff Cody <address@hidden> writes:
> If a node-name is not specified, automatically generate the node-name.
>
> Generated node-names will use the "block" sub-system identifier.
>
> Reviewed-by: Eric Blake <address@hidden>
> Reviewed-by: John Snow <address@hidden>
> Signed-off-by: Jeff Cody <address@hidden>
> ---
> block.c | 19 ++++++++++++-------
> 1 file changed, 12 insertions(+), 7 deletions(-)
>
> diff --git a/block.c b/block.c
> index 1f90b47..5947704 100644
> --- a/block.c
> +++ b/block.c
> @@ -763,12 +763,15 @@ static void bdrv_assign_node_name(BlockDriverState *bs,
> const char *node_name,
> Error **errp)
> {
> + char *gen_node_name = NULL;
> +
> if (!node_name) {
> - return;
> - }
> -
> - /* Check for empty string or invalid characters */
> - if (!id_wellformed(node_name)) {
> + node_name = gen_node_name = id_generate(ID_BLOCK);
> + } else if (!id_wellformed(node_name)) {
> + /*
> + * Check for empty string or invalid characters, but not if it is
> + * generated (generated names use characters not available to the
> user)
> + */
> error_setg(errp, "Invalid node name");
> return;
> }
I'd drop the comment, as the code is obvious enough now. Could be done
on commit.
> @@ -777,18 +780,20 @@ static void bdrv_assign_node_name(BlockDriverState *bs,
> if (blk_by_name(node_name)) {
> error_setg(errp, "node-name=%s is conflicting with a device id",
> node_name);
> - return;
> + goto out;
> }
>
> /* takes care of avoiding duplicates node names */
> if (bdrv_find_node(node_name)) {
> error_setg(errp, "Duplicate node name");
> - return;
> + goto out;
> }
>
> /* copy node name into the bs and insert it into the graph list */
> pstrcpy(bs->node_name, sizeof(bs->node_name), node_name);
> QTAILQ_INSERT_TAIL(&graph_bdrv_states, bs, node_list);
> +out:
> + g_free(gen_node_name);
> }
>
> static QemuOptsList bdrv_runtime_opts = {
Reviewed-by: Markus Armbruster <address@hidden>
- Re: [Qemu-block] [Qemu-devel] [PATCH v3 1/4] util - add automated ID generation utility, (continued)
[Qemu-block] [PATCH v3 3/4] block: add filter for generated node-names, Jeff Cody, 2015/10/12
[Qemu-block] [PATCH v3 4/4] qemu-iotests: update tests for generated node-names, Jeff Cody, 2015/10/12
[Qemu-block] [PATCH v3 2/4] block: auto-generated node-names, Jeff Cody, 2015/10/15
- Re: [Qemu-block] [Qemu-devel] [PATCH v3 2/4] block: auto-generated node-names,
Markus Armbruster <=
Re: [Qemu-block] [PATCH v3 0/4], Kevin Wolf, 2015/10/15