diff options
| author | Eric Blake | 2017-07-21 20:32:42 +0200 |
|---|---|---|
| committer | Kevin Wolf | 2017-07-24 15:06:04 +0200 |
| commit | 6c98c57af3f4fab85bdf5f01616c91322bd4312a (patch) | |
| tree | b2b6304d43d43afc327a4570097e760fa657c8e4 /block | |
| parent | iotests: Remove a few tests from 'quick' group (diff) | |
| download | qemu-6c98c57af3f4fab85bdf5f01616c91322bd4312a.tar.gz qemu-6c98c57af3f4fab85bdf5f01616c91322bd4312a.tar.xz qemu-6c98c57af3f4fab85bdf5f01616c91322bd4312a.zip | |
dirty-bitmap: Report BlockDirtyInfo.count in bytes, as documented
We've been documenting the value in bytes since its introduction
in commit b9a9b3a4 (v1.3), where it was actually reported in bytes.
Commit e4654d2 (v2.0) then removed things from block/qapi.c, in
preparation for a rewrite to a list of dirty sectors in the next
commit 21b5683 in block.c, but the new code mistakenly started
reporting in sectors.
Fixes: https://bugzilla.redhat.com/1441460
CC: qemu-stable@nongnu.org
Signed-off-by: Eric Blake <eblake@redhat.com>
Reviewed-by: John Snow <jsnow@redhat.com>
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Diffstat (limited to 'block')
| -rw-r--r-- | block/dirty-bitmap.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/block/dirty-bitmap.c b/block/dirty-bitmap.c index 543bddb9b5..30462d4f9a 100644 --- a/block/dirty-bitmap.c +++ b/block/dirty-bitmap.c @@ -461,7 +461,7 @@ BlockDirtyInfoList *bdrv_query_dirty_bitmaps(BlockDriverState *bs) QLIST_FOREACH(bm, &bs->dirty_bitmaps, list) { BlockDirtyInfo *info = g_new0(BlockDirtyInfo, 1); BlockDirtyInfoList *entry = g_new0(BlockDirtyInfoList, 1); - info->count = bdrv_get_dirty_count(bm); + info->count = bdrv_get_dirty_count(bm) << BDRV_SECTOR_BITS; info->granularity = bdrv_dirty_bitmap_granularity(bm); info->has_name = !!bm->name; info->name = g_strdup(bm->name); |