[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v5 17/22] blkverify: Allow command-line configur
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH v5 17/22] blkverify: Allow command-line configuration |
Date: |
Fri, 13 Dec 2013 21:46:47 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Am 13.12.2013 um 18:10 hat Max Reitz geschrieben:
> Introduce the "test" and "raw" options for specifying images.
>
> Signed-off-by: Max Reitz <address@hidden>
> ---
> block/blkverify.c | 59
> ++++++++++++++++++++++++++++++++++++-------------------
> 1 file changed, 39 insertions(+), 20 deletions(-)
>
> diff --git a/block/blkverify.c b/block/blkverify.c
> index c6eb287..8bf81b5 100644
> --- a/block/blkverify.c
> +++ b/block/blkverify.c
> @@ -116,13 +116,46 @@ static QemuOptsList runtime_opts = {
> },
> };
>
> +static int open_image(BlockDriverState **pbs, const char *fname, QDict
> *options,
> + const char *bdref_key, int flags, Error **errp)
> +{
> + QDict *image_options;
> + int ret;
> + char *bdref_key_dot;
> +
> + bdref_key_dot = g_strdup_printf("%s.", bdref_key);
> + qdict_extract_subqdict(options, &image_options, bdref_key_dot);
> + g_free(bdref_key_dot);
> +
> + if (fname) {
> + /* If a filename is given, use bdrv_open() in order to use the
> correct
> + block driver (instead of just opening the raw image). */
> +
> + if (qdict_get_try_str(options, bdref_key)) {
> + error_setg(errp, "Cannot reference an existing block device
> while "
> + "giving a filename");
> + ret = -EINVAL;
> + goto fail;
> + }
> +
> + *pbs = bdrv_new("");
> + ret = bdrv_open(*pbs, fname, image_options, flags, NULL, errp);
> + } else {
> + ret = bdrv_file_open(pbs, NULL, qdict_get_try_str(options,
> bdref_key),
> + image_options, flags, errp);
> + }
> +
> +fail:
> + qdict_del(options, bdref_key);
> + return ret;
> +}
> +
> static int blkverify_open(BlockDriverState *bs, QDict *options, int flags,
> Error **errp)
> {
> BDRVBlkverifyState *s = bs->opaque;
> QemuOpts *opts;
> Error *local_err = NULL;
> - const char *filename, *raw;
> int ret;
>
> opts = qemu_opts_create_nofail(&runtime_opts);
> @@ -133,33 +166,19 @@ static int blkverify_open(BlockDriverState *bs, QDict
> *options, int flags,
> goto fail;
> }
>
> - /* Parse the raw image filename */
> - raw = qemu_opt_get(opts, "x-raw");
> - if (raw == NULL) {
> - error_setg(errp, "Could not retrieve raw image filename");
> - ret = -EINVAL;
> - goto fail;
> - }
> -
> - ret = bdrv_file_open(&bs->file, raw, NULL, NULL, flags, &local_err);
> + /* Open the raw file */
> + ret = open_image(&bs->file, qemu_opt_get(opts, "x-raw"), options, "raw",
> + flags, &local_err);
This changes the behaviour: We now do format probing, and probably
insert a raw block driver between blkverify and the protocol driver.
If you had a generalised open_image() that is shared with blkdebug, you
could specify no probing here, and enable probing for the main image.
Kevin
[Qemu-devel] [PATCH v5 18/22] blkverify: Don't require protocol filename, Max Reitz, 2013/12/13
[Qemu-devel] [PATCH v5 17/22] blkverify: Allow command-line configuration, Max Reitz, 2013/12/13
[Qemu-devel] [PATCH v5 19/22] blkdebug: Alias "errno" as "error", Max Reitz, 2013/12/13
[Qemu-devel] [PATCH v5 20/22] qapi: QMP interface for blkdebug and blkverify, Max Reitz, 2013/12/13