[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 03/23] qcow2: Fix "total clusters" number in bdrv_ch
From: |
Stefan Hajnoczi |
Subject: |
[Qemu-devel] [PATCH 03/23] qcow2: Fix "total clusters" number in bdrv_check |
Date: |
Thu, 28 Mar 2013 17:40:17 +0100 |
From: Kevin Wolf <address@hidden>
This should be based on the virtual disk size, not on the size of the
image.
Interesting observation: With some VM state stored in the image file,
percentages higher than 100% are possible, even though snapshots
themselves are ignored. This is a qcow2 bug to be fixed another day: The
VM state should be discarded in the active L2 tables after completing
the snapshot creation.
Signed-off-by: Kevin Wolf <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>
---
block/qcow2-refcount.c | 4 +++-
tests/qemu-iotests/044.out | 2 +-
2 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/block/qcow2-refcount.c b/block/qcow2-refcount.c
index 9bfb390..c38e970 100644
--- a/block/qcow2-refcount.c
+++ b/block/qcow2-refcount.c
@@ -1152,9 +1152,11 @@ int qcow2_check_refcounts(BlockDriverState *bs,
BdrvCheckResult *res,
size = bdrv_getlength(bs->file);
nb_clusters = size_to_clusters(s, size);
- res->bfi.total_clusters = nb_clusters;
refcount_table = g_malloc0(nb_clusters * sizeof(uint16_t));
+ res->bfi.total_clusters =
+ size_to_clusters(s, bs->total_sectors * BDRV_SECTOR_SIZE);
+
/* header */
inc_refcounts(bs, res, refcount_table, nb_clusters,
0, s->cluster_size);
diff --git a/tests/qemu-iotests/044.out b/tests/qemu-iotests/044.out
index 5eed3f8..34c25c7 100644
--- a/tests/qemu-iotests/044.out
+++ b/tests/qemu-iotests/044.out
@@ -1,5 +1,5 @@
No errors were found on the image.
-7292415/8391499= 86.90% allocated, 0.00% fragmented, 0.00% compressed clusters
+7292415/33554432 = 21.73% allocated, 0.00% fragmented, 0.00% compressed
clusters
Image end offset: 4296447488
.
----------------------------------------------------------------------
--
1.8.1.4
- [Qemu-devel] [PULL 00/23] Block patches, Stefan Hajnoczi, 2013/03/28
- [Qemu-devel] [PATCH 02/23] qemu-iotests: More concurrent allocation scenarios, Stefan Hajnoczi, 2013/03/28
- [Qemu-devel] [PATCH 01/23] vl.c: call bdrv_init_with_whitelist() before cmdline parsing, Stefan Hajnoczi, 2013/03/28
- [Qemu-devel] [PATCH 05/23] qcow2: Handle dependencies earlier, Stefan Hajnoczi, 2013/03/28
- [Qemu-devel] [PATCH 03/23] qcow2: Fix "total clusters" number in bdrv_check,
Stefan Hajnoczi <=
- [Qemu-devel] [PATCH 04/23] qcow2: Remove bogus unlock of s->lock, Stefan Hajnoczi, 2013/03/28
- [Qemu-devel] [PATCH 06/23] qcow2: Improve check for overlapping allocations, Stefan Hajnoczi, 2013/03/28
- [Qemu-devel] [PATCH 08/23] qcow2: Decouple cluster allocation from cluster reuse code, Stefan Hajnoczi, 2013/03/28
- [Qemu-devel] [PATCH 07/23] qcow2: Change handle_dependency to byte granularity, Stefan Hajnoczi, 2013/03/28
- [Qemu-devel] [PATCH 09/23] qcow2: Factor out handle_alloc(), Stefan Hajnoczi, 2013/03/28
- [Qemu-devel] [PATCH 11/23] qcow2: handle_alloc(): Get rid of keep_clusters parameter, Stefan Hajnoczi, 2013/03/28
- [Qemu-devel] [PATCH 10/23] qcow2: handle_alloc(): Get rid of nb_clusters parameter, Stefan Hajnoczi, 2013/03/28
- [Qemu-devel] [PATCH 12/23] qcow2: Finalise interface of handle_alloc(), Stefan Hajnoczi, 2013/03/28
- [Qemu-devel] [PATCH 13/23] qcow2: Clean up handle_alloc(), Stefan Hajnoczi, 2013/03/28
- [Qemu-devel] [PATCH 14/23] qcow2: Factor out handle_copied(), Stefan Hajnoczi, 2013/03/28