* [PATCH v4 0/1] qemu-img: Add --target-is-zero to indicate that a target is blank
@ 2020-02-05 11:02 David Edmondson
2020-02-05 11:02 ` [PATCH v4 1/1] qemu-img: Add --target-is-zero to convert David Edmondson
0 siblings, 1 reply; 5+ messages in thread
From: David Edmondson @ 2020-02-05 11:02 UTC (permalink / raw)
To: qemu-devel; +Cc: David Edmondson, qemu-block
qemu-img: Add --target-is-zero to indicate that a target is blank
v4:
- Wording in the doc and error message.
v3:
- Merge with the rST docs.
- No more need to fix @var -> @code.
v2:
- Remove target_is_zero, preferring to set has_zero_init
directly (Mark Kanda).
- Disallow --target-is-zero in the presence of a backing file (Max
Reitz).
- Add relevant documentation (Max Reitz).
- @var -> @code for options in qemu-img.texi.
David Edmondson (1):
qemu-img: Add --target-is-zero to convert
docs/interop/qemu-img.rst | 9 ++++++++-
qemu-img-cmds.hx | 4 ++--
qemu-img.c | 26 +++++++++++++++++++++++---
3 files changed, 33 insertions(+), 6 deletions(-)
--
2.24.1
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH v4 1/1] qemu-img: Add --target-is-zero to convert
2020-02-05 11:02 [PATCH v4 0/1] qemu-img: Add --target-is-zero to indicate that a target is blank David Edmondson
@ 2020-02-05 11:02 ` David Edmondson
2020-02-05 11:25 ` Vladimir Sementsov-Ogievskiy
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: David Edmondson @ 2020-02-05 11:02 UTC (permalink / raw)
To: qemu-devel; +Cc: David Edmondson, qemu-block
In many cases the target of a convert operation is a newly provisioned
target that the user knows is blank (reads as zero). In this situation
there is no requirement for qemu-img to wastefully zero out the entire
device.
Add a new option, --target-is-zero, allowing the user to indicate that
an existing target device will return zeros for all reads.
Signed-off-by: David Edmondson <david.edmondson@oracle.com>
---
docs/interop/qemu-img.rst | 9 ++++++++-
qemu-img-cmds.hx | 4 ++--
qemu-img.c | 26 +++++++++++++++++++++++---
3 files changed, 33 insertions(+), 6 deletions(-)
diff --git a/docs/interop/qemu-img.rst b/docs/interop/qemu-img.rst
index fa27e5c7b453..763036857451 100644
--- a/docs/interop/qemu-img.rst
+++ b/docs/interop/qemu-img.rst
@@ -214,6 +214,13 @@ Parameters to convert subcommand:
will still be printed. Areas that cannot be read from the source will be
treated as containing only zeroes.
+.. option:: --target-is-zero
+
+ Assume that reading the destination image will always return
+ zeros. This parameter is mutually exclusive with a destination image
+ that has a backing file. It is required to also use the ``-n``
+ parameter to skip image creation.
+
Parameters to dd subcommand:
.. program:: qemu-img-dd
@@ -366,7 +373,7 @@ Command description:
4
Error on reading data
-.. option:: convert [--object OBJECTDEF] [--image-opts] [--target-image-opts] [-U] [-C] [-c] [-p] [-q] [-n] [-f FMT] [-t CACHE] [-T SRC_CACHE] [-O OUTPUT_FMT] [-B BACKING_FILE] [-o OPTIONS] [-l SNAPSHOT_PARAM] [-S SPARSE_SIZE] [-m NUM_COROUTINES] [-W] FILENAME [FILENAME2 [...]] OUTPUT_FILENAME
+.. option:: convert [--object OBJECTDEF] [--image-opts] [--target-image-opts] [--target-is-zero] [-U] [-C] [-c] [-p] [-q] [-n] [-f FMT] [-t CACHE] [-T SRC_CACHE] [-O OUTPUT_FMT] [-B BACKING_FILE] [-o OPTIONS] [-l SNAPSHOT_PARAM] [-S SPARSE_SIZE] [-m NUM_COROUTINES] [-W] FILENAME [FILENAME2 [...]] OUTPUT_FILENAME
Convert the disk image *FILENAME* or a snapshot *SNAPSHOT_PARAM*
to disk image *OUTPUT_FILENAME* using format *OUTPUT_FMT*. It can
diff --git a/qemu-img-cmds.hx b/qemu-img-cmds.hx
index 3fd836ca9090..e6f98b75473f 100644
--- a/qemu-img-cmds.hx
+++ b/qemu-img-cmds.hx
@@ -39,9 +39,9 @@ SRST
ERST
DEF("convert", img_convert,
- "convert [--object objectdef] [--image-opts] [--target-image-opts] [-U] [-C] [-c] [-p] [-q] [-n] [-f fmt] [-t cache] [-T src_cache] [-O output_fmt] [-B backing_file] [-o options] [-l snapshot_param] [-S sparse_size] [-m num_coroutines] [-W] [--salvage] filename [filename2 [...]] output_filename")
+ "convert [--object objectdef] [--image-opts] [--target-image-opts] [--target-is-zero] [-U] [-C] [-c] [-p] [-q] [-n] [-f fmt] [-t cache] [-T src_cache] [-O output_fmt] [-B backing_file] [-o options] [-l snapshot_param] [-S sparse_size] [-m num_coroutines] [-W] [--salvage] filename [filename2 [...]] output_filename")
SRST
-.. option:: convert [--object OBJECTDEF] [--image-opts] [--target-image-opts] [-U] [-C] [-c] [-p] [-q] [-n] [-f FMT] [-t CACHE] [-T SRC_CACHE] [-O OUTPUT_FMT] [-B BACKING_FILE] [-o OPTIONS] [-l SNAPSHOT_PARAM] [-S SPARSE_SIZE] [-m NUM_COROUTINES] [-W] [--salvage] FILENAME [FILENAME2 [...]] OUTPUT_FILENAME
+.. option:: convert [--object OBJECTDEF] [--image-opts] [--target-image-opts] [--target-is-zero] [-U] [-C] [-c] [-p] [-q] [-n] [-f FMT] [-t CACHE] [-T SRC_CACHE] [-O OUTPUT_FMT] [-B BACKING_FILE] [-o OPTIONS] [-l SNAPSHOT_PARAM] [-S SPARSE_SIZE] [-m NUM_COROUTINES] [-W] [--salvage] FILENAME [FILENAME2 [...]] OUTPUT_FILENAME
ERST
DEF("create", img_create,
diff --git a/qemu-img.c b/qemu-img.c
index 2b4562b9d9f2..0faf2cd2f530 100644
--- a/qemu-img.c
+++ b/qemu-img.c
@@ -70,6 +70,7 @@ enum {
OPTION_PREALLOCATION = 265,
OPTION_SHRINK = 266,
OPTION_SALVAGE = 267,
+ OPTION_TARGET_IS_ZERO = 268,
};
typedef enum OutputFormat {
@@ -1984,10 +1985,9 @@ static int convert_do_copy(ImgConvertState *s)
int64_t sector_num = 0;
/* Check whether we have zero initialisation or can get it efficiently */
- if (s->target_is_new && s->min_sparse && !s->target_has_backing) {
+ if (!s->has_zero_init && s->target_is_new && s->min_sparse &&
+ !s->target_has_backing) {
s->has_zero_init = bdrv_has_zero_init(blk_bs(s->target));
- } else {
- s->has_zero_init = false;
}
if (!s->has_zero_init && !s->target_has_backing &&
@@ -2086,6 +2086,7 @@ static int img_convert(int argc, char **argv)
{"force-share", no_argument, 0, 'U'},
{"target-image-opts", no_argument, 0, OPTION_TARGET_IMAGE_OPTS},
{"salvage", no_argument, 0, OPTION_SALVAGE},
+ {"target-is-zero", no_argument, 0, OPTION_TARGET_IS_ZERO},
{0, 0, 0, 0}
};
c = getopt_long(argc, argv, ":hf:O:B:Cco:l:S:pt:T:qnm:WU",
@@ -2209,6 +2210,14 @@ static int img_convert(int argc, char **argv)
case OPTION_TARGET_IMAGE_OPTS:
tgt_image_opts = true;
break;
+ case OPTION_TARGET_IS_ZERO:
+ /*
+ * The user asserting that the target is blank has the
+ * same effect as the target driver supporting zero
+ * initialisation.
+ */
+ s.has_zero_init = true;
+ break;
}
}
@@ -2247,6 +2256,11 @@ static int img_convert(int argc, char **argv)
warn_report("This will become an error in future QEMU versions.");
}
+ if (s.has_zero_init && !skip_create) {
+ error_report("--target-is-zero requires use of -n flag");
+ goto fail_getopt;
+ }
+
s.src_num = argc - optind - 1;
out_filename = s.src_num >= 1 ? argv[argc - 1] : NULL;
@@ -2380,6 +2394,12 @@ static int img_convert(int argc, char **argv)
}
s.target_has_backing = (bool) out_baseimg;
+ if (s.has_zero_init && s.target_has_backing) {
+ error_report("Cannot use --target-is-zero when the destination "
+ "image has a backing file");
+ goto out;
+ }
+
if (s.src_num > 1 && out_baseimg) {
error_report("Having a backing file for the target makes no sense when "
"concatenating multiple input images");
--
2.24.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v4 1/1] qemu-img: Add --target-is-zero to convert
2020-02-05 11:02 ` [PATCH v4 1/1] qemu-img: Add --target-is-zero to convert David Edmondson
@ 2020-02-05 11:25 ` Vladimir Sementsov-Ogievskiy
2020-02-06 1:45 ` Eric Blake
2020-02-07 10:54 ` Max Reitz
2 siblings, 0 replies; 5+ messages in thread
From: Vladimir Sementsov-Ogievskiy @ 2020-02-05 11:25 UTC (permalink / raw)
To: David Edmondson, qemu-devel; +Cc: qemu-block
05.02.2020 14:02, David Edmondson wrote:
> In many cases the target of a convert operation is a newly provisioned
> target that the user knows is blank (reads as zero). In this situation
> there is no requirement for qemu-img to wastefully zero out the entire
> device.
>
> Add a new option, --target-is-zero, allowing the user to indicate that
> an existing target device will return zeros for all reads.
>
> Signed-off-by: David Edmondson<david.edmondson@oracle.com>
Reviewed-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
--
Best regards,
Vladimir
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v4 1/1] qemu-img: Add --target-is-zero to convert
2020-02-05 11:02 ` [PATCH v4 1/1] qemu-img: Add --target-is-zero to convert David Edmondson
2020-02-05 11:25 ` Vladimir Sementsov-Ogievskiy
@ 2020-02-06 1:45 ` Eric Blake
2020-02-07 10:54 ` Max Reitz
2 siblings, 0 replies; 5+ messages in thread
From: Eric Blake @ 2020-02-06 1:45 UTC (permalink / raw)
To: David Edmondson, qemu-devel; +Cc: qemu-block
On 2/5/20 5:02 AM, David Edmondson wrote:
> In many cases the target of a convert operation is a newly provisioned
> target that the user knows is blank (reads as zero). In this situation
> there is no requirement for qemu-img to wastefully zero out the entire
> device.
>
> Add a new option, --target-is-zero, allowing the user to indicate that
> an existing target device will return zeros for all reads.
>
> Signed-off-by: David Edmondson <david.edmondson@oracle.com>
> ---
> docs/interop/qemu-img.rst | 9 ++++++++-
> qemu-img-cmds.hx | 4 ++--
> qemu-img.c | 26 +++++++++++++++++++++++---
> 3 files changed, 33 insertions(+), 6 deletions(-)
Reviewed-by: Eric Blake <eblake@redhat.com>
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3226
Virtualization: qemu.org | libvirt.org
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v4 1/1] qemu-img: Add --target-is-zero to convert
2020-02-05 11:02 ` [PATCH v4 1/1] qemu-img: Add --target-is-zero to convert David Edmondson
2020-02-05 11:25 ` Vladimir Sementsov-Ogievskiy
2020-02-06 1:45 ` Eric Blake
@ 2020-02-07 10:54 ` Max Reitz
2 siblings, 0 replies; 5+ messages in thread
From: Max Reitz @ 2020-02-07 10:54 UTC (permalink / raw)
To: David Edmondson, qemu-devel; +Cc: qemu-block
[-- Attachment #1.1: Type: text/plain, Size: 817 bytes --]
On 05.02.20 12:02, David Edmondson wrote:
> In many cases the target of a convert operation is a newly provisioned
> target that the user knows is blank (reads as zero). In this situation
> there is no requirement for qemu-img to wastefully zero out the entire
> device.
>
> Add a new option, --target-is-zero, allowing the user to indicate that
> an existing target device will return zeros for all reads.
>
> Signed-off-by: David Edmondson <david.edmondson@oracle.com>
> ---
> docs/interop/qemu-img.rst | 9 ++++++++-
> qemu-img-cmds.hx | 4 ++--
> qemu-img.c | 26 +++++++++++++++++++++++---
> 3 files changed, 33 insertions(+), 6 deletions(-)
Thanks, I’ve applied the patch to my block branch:
https://git.xanclic.moe/XanClic/qemu/commits/branch/block
Max
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2020-02-07 10:55 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-02-05 11:02 [PATCH v4 0/1] qemu-img: Add --target-is-zero to indicate that a target is blank David Edmondson
2020-02-05 11:02 ` [PATCH v4 1/1] qemu-img: Add --target-is-zero to convert David Edmondson
2020-02-05 11:25 ` Vladimir Sementsov-Ogievskiy
2020-02-06 1:45 ` Eric Blake
2020-02-07 10:54 ` Max Reitz
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).