[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 6/8] qcow2: Enable dirty flag in qcow2_alloc_cluster
From: |
Kevin Wolf |
Subject: |
[Qemu-devel] [PATCH 6/8] qcow2: Enable dirty flag in qcow2_alloc_cluster_link_l2 |
Date: |
Fri, 7 Dec 2012 18:08:47 +0100 |
This is closer to where the dirty flag is really needed, and it avoids
having checks for special cases related to cluster allocation directly
in the writev loop.
Signed-off-by: Kevin Wolf <address@hidden>
---
block/qcow2-cluster.c | 5 ++++-
block/qcow2.c | 7 +------
block/qcow2.h | 2 ++
3 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/block/qcow2-cluster.c b/block/qcow2-cluster.c
index c2b59e7..7a038ac 100644
--- a/block/qcow2-cluster.c
+++ b/block/qcow2-cluster.c
@@ -668,11 +668,14 @@ int qcow2_alloc_cluster_link_l2(BlockDriverState *bs,
QCowL2Meta *m)
}
/* Update L2 table. */
-
+ if (s->compatible_features & QCOW2_COMPAT_LAZY_REFCOUNTS) {
+ qcow2_mark_dirty(bs);
+ }
if (qcow2_need_accurate_refcounts(s)) {
qcow2_cache_set_dependency(bs, s->l2_table_cache,
s->refcount_block_cache);
}
+
ret = get_cluster_table(bs, m->offset, &l2_table, &l2_index);
if (ret < 0) {
goto err;
diff --git a/block/qcow2.c b/block/qcow2.c
index 08d92cc..dbcc0ff 100644
--- a/block/qcow2.c
+++ b/block/qcow2.c
@@ -222,7 +222,7 @@ static void report_unsupported_feature(BlockDriverState *bs,
* updated successfully. Therefore it is not required to check the return
* value of this function.
*/
-static int qcow2_mark_dirty(BlockDriverState *bs)
+int qcow2_mark_dirty(BlockDriverState *bs)
{
BDRVQcowState *s = bs->opaque;
uint64_t val;
@@ -803,11 +803,6 @@ static coroutine_fn int qcow2_co_writev(BlockDriverState
*bs,
goto fail;
}
- if (l2meta->nb_clusters > 0 &&
- (s->compatible_features & QCOW2_COMPAT_LAZY_REFCOUNTS)) {
- qcow2_mark_dirty(bs);
- }
-
assert((cluster_offset & 511) == 0);
qemu_iovec_reset(&hd_qiov);
diff --git a/block/qcow2.h b/block/qcow2.h
index 6dc79b5..a60fcb4 100644
--- a/block/qcow2.h
+++ b/block/qcow2.h
@@ -303,6 +303,8 @@ static inline bool
qcow2_need_accurate_refcounts(BDRVQcowState *s)
/* qcow2.c functions */
int qcow2_backing_read1(BlockDriverState *bs, QEMUIOVector *qiov,
int64_t sector_num, int nb_sectors);
+
+int qcow2_mark_dirty(BlockDriverState *bs);
int qcow2_update_header(BlockDriverState *bs);
/* qcow2-refcount.c functions */
--
1.7.6.5
- [Qemu-devel] [PATCH 0/8] qcow2: Cleanups and refactoring, Kevin Wolf, 2012/12/07
- [Qemu-devel] [PATCH 3/8] qcow2: Allocate l2meta dynamically, Kevin Wolf, 2012/12/07
- [Qemu-devel] [PATCH 1/8] qcow2: Round QCowL2Meta.offset down to cluster boundary, Kevin Wolf, 2012/12/07
- [Qemu-devel] [PATCH 2/8] qcow2: Introduce Qcow2COWRegion, Kevin Wolf, 2012/12/07
- [Qemu-devel] [PATCH 8/8] qcow2: Factor out handle_dependencies(), Kevin Wolf, 2012/12/07
- [Qemu-devel] [PATCH 6/8] qcow2: Enable dirty flag in qcow2_alloc_cluster_link_l2,
Kevin Wolf <=
- [Qemu-devel] [PATCH 5/8] qcow2: Allocate l2meta only for cluster allocations, Kevin Wolf, 2012/12/07
- [Qemu-devel] [PATCH 4/8] qcow2: Drop l2meta.cluster_offset, Kevin Wolf, 2012/12/07
- [Qemu-devel] [PATCH 7/8] qcow2: Execute run_dependent_requests() without lock, Kevin Wolf, 2012/12/07