From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:53292) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XClEt-000791-8K for qemu-devel@nongnu.org; Thu, 31 Jul 2014 03:57:33 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XClEm-0008GC-VM for qemu-devel@nongnu.org; Thu, 31 Jul 2014 03:57:27 -0400 Received: from lputeaux-656-01-25-125.w80-12.abo.wanadoo.fr ([80.12.84.125]:56484 helo=paradis.irqsave.net) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XClEm-0008Fl-P1 for qemu-devel@nongnu.org; Thu, 31 Jul 2014 03:57:20 -0400 Date: Thu, 31 Jul 2014 09:56:34 +0200 From: =?iso-8859-1?Q?Beno=EEt?= Canet Message-ID: <20140731075634.GF707@irqsave.net> References: <1406402531-9278-1-git-send-email-mreitz@redhat.com> <1406402531-9278-3-git-send-email-mreitz@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline In-Reply-To: <1406402531-9278-3-git-send-email-mreitz@redhat.com> Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH alt 2/7] qemu-img: Add progress output for amend List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Max Reitz Cc: Kevin Wolf , qemu-devel@nongnu.org, Stefan Hajnoczi The Saturday 26 Jul 2014 =E0 21:22:06 (+0200), Max Reitz wrote : > Now that bdrv_amend_options() supports a status callback, use it to > display a progress report. >=20 > Signed-off-by: Max Reitz > --- > qemu-img.c | 26 +++++++++++++++++++++++--- > 1 file changed, 23 insertions(+), 3 deletions(-) >=20 > 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; > } > =20 > +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 =3D 0; > @@ -2740,12 +2747,12 @@ static int img_amend(int argc, char **argv) > QemuOpts *opts =3D NULL; > const char *fmt =3D NULL, *filename, *cache; > int flags; > - bool quiet =3D false; > + bool quiet =3D false, progress =3D false; > BlockDriverState *bs =3D NULL; > =20 > cache =3D BDRV_DEFAULT_CACHE; > for (;;) { > - c =3D getopt(argc, argv, "ho:f:t:q"); > + c =3D getopt(argc, argv, "ho:f:t:pq"); > if (c =3D=3D -1) { > break; > } > @@ -2775,6 +2782,9 @@ static int img_amend(int argc, char **argv) > case 't': > cache =3D optarg; > break; > + case 'p': > + progress =3D true; > + break; > case 'q': > quiet =3D true; > break; > @@ -2785,6 +2795,11 @@ static int img_amend(int argc, char **argv) > error_exit("Must specify options (-o)"); > } > =20 > + if (quiet) { > + progress =3D false; > + } > + qemu_progress_init(progress, 1.0); > + > filename =3D (optind =3D=3D argc - 1) ? argv[argc - 1] : NULL; > if (fmt && has_help_option(options)) { > /* If a format is explicitly specified (and possibly no filena= me is > @@ -2827,13 +2842,18 @@ static int img_amend(int argc, char **argv) > goto out; > } > =20 > - ret =3D bdrv_amend_options(bs, opts, NULL); > + /* In case the driver does not call amend_status_cb() */ > + qemu_progress_print(0.f, 0); > + ret =3D 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; > } > =20 > out: > + qemu_progress_end(); > + > if (bs) { > bdrv_unref(bs); > } > --=20 > 2.0.3 >=20 >=20