diff options
| author | Kevin Wolf <kwolf@redhat.com> | 2012-12-07 18:08:46 +0100 |
|---|---|---|
| committer | Kevin Wolf <kwolf@redhat.com> | 2012-12-13 15:37:59 +0100 |
| commit | f50f88b9fea09fef12cc293126cf45dcf0ef600b (patch) | |
| tree | 2060b173e0069405cca56846cce4fe44e793952c /block/qcow2.h | |
| parent | 060bee8943c27d4d53f65570fafaa2559fcd87c3 (diff) | |
qcow2: Allocate l2meta only for cluster allocations
Even for writes to already allocated clusters, an l2meta is allocated,
though it stays effectively unused. After this patch, only allocating
requests still have one. Each l2meta now describes an in-flight request
that writes to clusters that are not yet hooked up in the L2 table.
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Diffstat (limited to 'block/qcow2.h')
| -rw-r--r-- | block/qcow2.h | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/block/qcow2.h b/block/qcow2.h index 24f100125c..6dc79b5af4 100644 --- a/block/qcow2.h +++ b/block/qcow2.h @@ -207,7 +207,10 @@ typedef struct Qcow2COWRegion { int nb_sectors; } Qcow2COWRegion; -/* XXX This could be private for qcow2-cluster.c */ +/** + * Describes an in-flight (part of a) write request that writes to clusters + * that are not referenced in their L2 table yet. + */ typedef struct QCowL2Meta { /** Guest offset of the first newly allocated cluster */ @@ -333,7 +336,7 @@ void qcow2_encrypt_sectors(BDRVQcowState *s, int64_t sector_num, int qcow2_get_cluster_offset(BlockDriverState *bs, uint64_t offset, int *num, uint64_t *cluster_offset); int qcow2_alloc_cluster_offset(BlockDriverState *bs, uint64_t offset, - int n_start, int n_end, int *num, uint64_t *host_offset, QCowL2Meta *m); + int n_start, int n_end, int *num, uint64_t *host_offset, QCowL2Meta **m); uint64_t qcow2_alloc_compressed_cluster_offset(BlockDriverState *bs, uint64_t offset, int compressed_size); |