[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v4 4/6] qemu-img: Enable progress output for commit
From: |
Max Reitz |
Subject: |
[Qemu-devel] [PATCH v4 4/6] qemu-img: Enable progress output for commit |
Date: |
Sat, 12 Apr 2014 20:57:38 +0200 |
Implement progress output for the commit command by querying the
progress of the block job.
Signed-off-by: Max Reitz <address@hidden>
---
qemu-img-cmds.hx | 4 ++--
qemu-img.c | 44 ++++++++++++++++++++++++++++++++++++++++++--
qemu-img.texi | 2 +-
3 files changed, 45 insertions(+), 5 deletions(-)
diff --git a/qemu-img-cmds.hx b/qemu-img-cmds.hx
index d029609..8bc55cd 100644
--- a/qemu-img-cmds.hx
+++ b/qemu-img-cmds.hx
@@ -22,9 +22,9 @@ STEXI
ETEXI
DEF("commit", img_commit,
- "commit [-q] [-f fmt] [-t cache] filename")
+ "commit [-q] [-f fmt] [-t cache] [-p] filename")
STEXI
address@hidden commit [-q] [-f @var{fmt}] [-t @var{cache}] @var{filename}
address@hidden commit [-q] [-f @var{fmt}] [-t @var{cache}] [-p] @var{filename}
ETEXI
DEF("compare", img_compare,
diff --git a/qemu-img.c b/qemu-img.c
index 9fe6384..50d7519 100644
--- a/qemu-img.c
+++ b/qemu-img.c
@@ -686,6 +686,9 @@ fail:
struct CommonBlockJobCBInfo {
Error **errp;
bool done;
+ /* If the blockjob works on several sectors at once, this field has to
+ * contain that granularity in bytes */
+ uint64_t granularity;
};
static void common_block_job_cb(void *opaque, int ret)
@@ -702,12 +705,34 @@ static void common_block_job_cb(void *opaque, int ret)
static void run_block_job(BlockJob *job, struct CommonBlockJobCBInfo *cbi)
{
BlockJobInfo *info;
+ uint64_t mod_offset = 0;
do {
qemu_aio_wait();
info = block_job_query(job);
+ if (info->offset) {
+ if (!mod_offset) {
+ /* Some block jobs (at least "commit") will only work on a
+ * subset of the image file and therefore basically skip many
+ * sectors at the start (processing them apparently
+ * instantaneously). These sectors should be ignored when
+ * calculating the progress.
+ * If the blockjob processes several sectors at once, the first
+ * progress report is likely to come after one such sector
group
+ * (whose size is cbi->granularity), therefore subtract it from
+ * the current offset in order to obtain a more probable
+ * starting offset. */
+ mod_offset = info->offset > cbi->granularity
+ ? info->offset - cbi->granularity
+ : 0;
+ }
+
+ qemu_progress_print((float)(info->offset - mod_offset) /
+ (info->len - mod_offset) * 100.f, 0);
+ }
+
if (!info->busy && info->offset < info->len) {
block_job_resume(job);
}
@@ -724,13 +749,13 @@ static int img_commit(int argc, char **argv)
int c, ret, flags;
const char *filename, *fmt, *cache;
BlockDriverState *bs, *base_bs;
- bool quiet = false;
+ bool progress = false, quiet = false;
Error *local_err = NULL;
fmt = NULL;
cache = BDRV_DEFAULT_CACHE;
for(;;) {
- c = getopt(argc, argv, "f:ht:q");
+ c = getopt(argc, argv, "f:ht:qp");
if (c == -1) {
break;
}
@@ -745,11 +770,20 @@ static int img_commit(int argc, char **argv)
case 't':
cache = optarg;
break;
+ case 'p':
+ progress = true;
+ break;
case 'q':
quiet = true;
break;
}
}
+
+ /* Progress is not shown in Quiet mode */
+ if (quiet) {
+ progress = false;
+ }
+
if (optind != argc - 1) {
help();
}
@@ -767,6 +801,9 @@ static int img_commit(int argc, char **argv)
return 1;
}
+ qemu_progress_init(progress, 1.f);
+ qemu_progress_print(0.f, 100);
+
/* This is different from QMP, which by default uses the deepest file in
the
* backing chain (i.e., the very base); however, the traditional behavior
of
* qemu-img commit is using the immediate backing file. */
@@ -778,6 +815,7 @@ static int img_commit(int argc, char **argv)
struct CommonBlockJobCBInfo cbi = {
.errp = &local_err,
+ .granularity = COMMIT_BUF_SECTORS << BDRV_SECTOR_BITS,
};
commit_active_start(bs, base_bs, 0, COMMIT_BUF_SECTORS << BDRV_SECTOR_BITS,
@@ -790,6 +828,8 @@ static int img_commit(int argc, char **argv)
run_block_job(bs->job, &cbi);
done:
+ qemu_progress_end();
+
bdrv_unref(bs);
if (local_err) {
diff --git a/qemu-img.texi b/qemu-img.texi
index f84590e..1a9c08f 100644
--- a/qemu-img.texi
+++ b/qemu-img.texi
@@ -140,7 +140,7 @@ this case. @var{backing_file} will never be modified unless
you use the
The size can also be specified using the @var{size} option with @code{-o},
it doesn't need to be specified separately in this case.
address@hidden commit [-f @var{fmt}] [-t @var{cache}] @var{filename}
address@hidden commit [-q] [-f @var{fmt}] [-t @var{cache}] [-p] @var{filename}
Commit the changes recorded in @var{filename} in its base image or backing
file.
If the backing file is smaller than the snapshot, then the backing file will be
--
1.9.2
- Re: [Qemu-devel] [PATCH v4 1/6] block-commit: Expose granularity, (continued)
- [Qemu-devel] [PATCH v4 3/6] qemu-img: Implement commit like QMP, Max Reitz, 2014/04/12
- [Qemu-devel] [PATCH v4 4/6] qemu-img: Enable progress output for commit,
Max Reitz <=
- Re: [Qemu-devel] [PATCH v4 4/6] qemu-img: Enable progress output for commit, Kevin Wolf, 2014/04/16
- Re: [Qemu-devel] [PATCH v4 4/6] qemu-img: Enable progress output for commit, Max Reitz, 2014/04/16
- Re: [Qemu-devel] [PATCH v4 4/6] qemu-img: Enable progress output for commit, Max Reitz, 2014/04/16
- Re: [Qemu-devel] [PATCH v4 4/6] qemu-img: Enable progress output for commit, Kevin Wolf, 2014/04/17
- Re: [Qemu-devel] [PATCH v4 4/6] qemu-img: Enable progress output for commit, Fam Zheng, 2014/04/17
- Re: [Qemu-devel] [PATCH v4 4/6] qemu-img: Enable progress output for commit, Eric Blake, 2014/04/17
- Re: [Qemu-devel] [PATCH v4 4/6] qemu-img: Enable progress output for commit, Max Reitz, 2014/04/17
[Qemu-devel] [PATCH v4 5/6] qemu-img: Specify backing file for commit, Max Reitz, 2014/04/12
[Qemu-devel] [PATCH v4 6/6] iotests: Commit tests for two-layer backing chains, Max Reitz, 2014/04/12