[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v4 09/15] qcow2: truncate preallocated space
From: |
Anton Nefedov |
Subject: |
[Qemu-devel] [PATCH v4 09/15] qcow2: truncate preallocated space |
Date: |
Tue, 1 Aug 2017 17:19:06 +0300 |
From: "Denis V. Lunev" <address@hidden>
This could be done after calculation of the end of data and metadata in
the qcow2 image.
Signed-off-by: Denis V. Lunev <address@hidden>
Signed-off-by: Anton Nefedov <address@hidden>
---
block/qcow2.h | 3 +++
block/qcow2-cluster.c | 9 +++++++++
block/qcow2-refcount.c | 7 +++++++
block/qcow2.c | 7 +++++++
4 files changed, 26 insertions(+)
diff --git a/block/qcow2.h b/block/qcow2.h
index ebbb9cf..595ed9c 100644
--- a/block/qcow2.h
+++ b/block/qcow2.h
@@ -330,6 +330,7 @@ typedef struct BDRVQcow2State {
char *image_backing_format;
uint64_t prealloc_size;
+ uint64_t data_end;
} BDRVQcow2State;
typedef struct Qcow2COWRegion {
@@ -669,4 +670,6 @@ void qcow2_remove_persistent_dirty_bitmap(BlockDriverState
*bs,
const char *name,
Error **errp);
+void qcow2_update_data_end(BlockDriverState *bs, uint64_t off);
+
#endif
diff --git a/block/qcow2-cluster.c b/block/qcow2-cluster.c
index 0185986..75baaf4 100644
--- a/block/qcow2-cluster.c
+++ b/block/qcow2-cluster.c
@@ -2014,3 +2014,12 @@ fail:
g_free(l1_table);
return ret;
}
+
+void qcow2_update_data_end(BlockDriverState *bs, uint64_t off)
+{
+ BDRVQcow2State *s = bs->opaque;
+
+ if (s->data_end < off) {
+ s->data_end = off;
+ }
+}
diff --git a/block/qcow2-refcount.c b/block/qcow2-refcount.c
index c9b0dcb..d741a92 100644
--- a/block/qcow2-refcount.c
+++ b/block/qcow2-refcount.c
@@ -833,6 +833,9 @@ static int QEMU_WARN_UNUSED_RESULT
update_refcount(BlockDriverState *bs,
ret = alloc_refcount_block(bs, cluster_index, &refcount_block);
if (ret < 0) {
goto fail;
+ } else {
+ qcow2_update_data_end(bs, s->refcount_table_offset +
+ s->refcount_table_size * sizeof(uint64_t));
}
}
old_table_index = table_index;
@@ -954,6 +957,8 @@ retry:
s->free_cluster_index - 1 > (INT64_MAX >> s->cluster_bits))
{
return -EFBIG;
+ } else {
+ qcow2_update_data_end(bs, s->free_cluster_index << s->cluster_bits);
}
#ifdef DEBUG_ALLOC2
@@ -1018,6 +1023,8 @@ int64_t qcow2_alloc_clusters_at(BlockDriverState *bs,
uint64_t offset,
if (ret < 0) {
return ret;
+ } else {
+ qcow2_update_data_end(bs, offset + (nb_clusters << s->cluster_bits));
}
return i;
diff --git a/block/qcow2.c b/block/qcow2.c
index 2a1d2f2..4696106 100644
--- a/block/qcow2.c
+++ b/block/qcow2.c
@@ -1498,6 +1498,8 @@ static int qcow2_do_open(BlockDriverState *bs, QDict
*options, int flags,
}
}
+ s->data_end = bdrv_getlength(bs->file->bs);
+
#ifdef DEBUG_ALLOC
{
BdrvCheckResult result = {0};
@@ -2138,6 +2140,11 @@ static int qcow2_inactivate(BlockDriverState *bs)
if (result == 0) {
qcow2_mark_clean(bs);
+
+ /* truncate preallocated space */
+ if (!bs->read_only && s->data_end < bdrv_getlength(bs->file->bs)) {
+ bdrv_truncate(bs->file, s->data_end, PREALLOC_MODE_OFF, NULL);
+ }
s->flags |= BDRV_O_INACTIVE;
}
--
2.7.4
- [Qemu-devel] [PATCH v4 08/15] qcow2: set inactive flag, (continued)
- [Qemu-devel] [PATCH v4 08/15] qcow2: set inactive flag, Anton Nefedov, 2017/08/01
- [Qemu-devel] [PATCH v4 07/15] qcow2: preallocation at image expand, Anton Nefedov, 2017/08/01
- [Qemu-devel] [PATCH v4 01/15] mirror: inherit supported write/zero flags, Anton Nefedov, 2017/08/01
- [Qemu-devel] [PATCH v4 06/15] block: support BDRV_REQ_ALLOCATE in passthrough drivers, Anton Nefedov, 2017/08/01
- [Qemu-devel] [PATCH v4 05/15] file-posix: support BDRV_REQ_ALLOCATE, Anton Nefedov, 2017/08/01
- [Qemu-devel] [PATCH v4 09/15] qcow2: truncate preallocated space,
Anton Nefedov <=
- [Qemu-devel] [PATCH v4 03/15] block: introduce BDRV_REQ_ALLOCATE flag, Anton Nefedov, 2017/08/01
- [Qemu-devel] [PATCH v4 02/15] blkverify: set supported write/zero flags, Anton Nefedov, 2017/08/01
- [Qemu-devel] [PATCH v4 10/15] qcow2: check space leak at the end of the image, Anton Nefedov, 2017/08/01
- [Qemu-devel] [PATCH v4 15/15] iotest 134: test cluster-misaligned encrypted write, Anton Nefedov, 2017/08/01
- [Qemu-devel] [PATCH v4 13/15] qcow2: allocate image space by-cluster, Anton Nefedov, 2017/08/01
- [Qemu-devel] [PATCH v4 11/15] qcow2: move is_zero() up, Anton Nefedov, 2017/08/01