[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH alt 2/7] qemu-img: Add progress output for amend
From: |
Benoît Canet |
Subject: |
Re: [Qemu-devel] [PATCH alt 2/7] qemu-img: Add progress output for amend |
Date: |
Thu, 31 Jul 2014 09:56:34 +0200 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
The Saturday 26 Jul 2014 à 21:22:06 (+0200), Max Reitz wrote :
> Now that bdrv_amend_options() supports a status callback, use it to
> display a progress report.
>
> Signed-off-by: Max Reitz <address@hidden>
> ---
> qemu-img.c | 26 +++++++++++++++++++++++---
> 1 file changed, 23 insertions(+), 3 deletions(-)
>
> diff --git a/qemu-img.c b/qemu-img.c
> index 90d6b79..a06f425 100644
> --- a/qemu-img.c
> +++ b/qemu-img.c
> @@ -2732,6 +2732,13 @@ out:
> return 0;
> }
>
> +static void amend_status_cb(BlockDriverState *bs,
> + int64_t offset, int64_t total_work_size)
> +{
> + (void)bs;
This cast also look like a compiler pleasing thing.
Is it really required ?
> + qemu_progress_print(100.f * offset / total_work_size, 0);
> +}
> +
> static int img_amend(int argc, char **argv)
> {
> int c, ret = 0;
> @@ -2740,12 +2747,12 @@ static int img_amend(int argc, char **argv)
> QemuOpts *opts = NULL;
> const char *fmt = NULL, *filename, *cache;
> int flags;
> - bool quiet = false;
> + bool quiet = false, progress = false;
> BlockDriverState *bs = NULL;
>
> cache = BDRV_DEFAULT_CACHE;
> for (;;) {
> - c = getopt(argc, argv, "ho:f:t:q");
> + c = getopt(argc, argv, "ho:f:t:pq");
> if (c == -1) {
> break;
> }
> @@ -2775,6 +2782,9 @@ static int img_amend(int argc, char **argv)
> case 't':
> cache = optarg;
> break;
> + case 'p':
> + progress = true;
> + break;
> case 'q':
> quiet = true;
> break;
> @@ -2785,6 +2795,11 @@ static int img_amend(int argc, char **argv)
> error_exit("Must specify options (-o)");
> }
>
> + if (quiet) {
> + progress = false;
> + }
> + qemu_progress_init(progress, 1.0);
> +
> filename = (optind == argc - 1) ? argv[argc - 1] : NULL;
> if (fmt && has_help_option(options)) {
> /* If a format is explicitly specified (and possibly no filename is
> @@ -2827,13 +2842,18 @@ static int img_amend(int argc, char **argv)
> goto out;
> }
>
> - ret = bdrv_amend_options(bs, opts, NULL);
> + /* In case the driver does not call amend_status_cb() */
> + qemu_progress_print(0.f, 0);
> + ret = bdrv_amend_options(bs, opts, &amend_status_cb);
> + qemu_progress_print(100.f, 0);
> if (ret < 0) {
> error_report("Error while amending options: %s", strerror(-ret));
> goto out;
> }
>
> out:
> + qemu_progress_end();
> +
> if (bs) {
> bdrv_unref(bs);
> }
> --
> 2.0.3
>
>
- [Qemu-devel] [PATCH alt 0/7] block/qcow2: Improve zero cluster expansion, Max Reitz, 2014/07/26
- [Qemu-devel] [PATCH alt 1/7] block: Add status callback to bdrv_amend_options(), Max Reitz, 2014/07/26
- [Qemu-devel] [PATCH alt 2/7] qemu-img: Add progress output for amend, Max Reitz, 2014/07/26
- Re: [Qemu-devel] [PATCH alt 2/7] qemu-img: Add progress output for amend,
Benoît Canet <=
- [Qemu-devel] [PATCH alt 3/7] qemu-img: Fix insignifcant memleak, Max Reitz, 2014/07/26
- [Qemu-devel] [PATCH alt 4/7] block/qcow2: Implement status CB for amend, Max Reitz, 2014/07/26
- [Qemu-devel] [PATCH alt 5/7] block/qcow2: Make get_refcount() global, Max Reitz, 2014/07/26
- [Qemu-devel] [PATCH alt 6/7] block/qcow2: Simplify shared L2 handling in amend, Max Reitz, 2014/07/26