diff options
author | 2009-09-09 17:53:37 +0200 | |
---|---|---|
committer | 2009-09-11 10:18:06 -0500 | |
commit | 40b4f539678266160badd5ac4afa9833f9089154 (patch) | |
tree | c54e3455975d74e8fefdfd349c9b7a854922abbf /block.h | |
parent | Revert "don't call cpu_sychronize_state from reset handlers" (diff) | |
download | qemu-kvm-40b4f539678266160badd5ac4afa9833f9089154.tar.gz qemu-kvm-40b4f539678266160badd5ac4afa9833f9089154.tar.bz2 qemu-kvm-40b4f539678266160badd5ac4afa9833f9089154.zip |
Add bdrv_aio_multiwrite
One performance problem of qcow2 during the initial image growth are
sequential writes that are not cluster aligned. In this case, when a first
requests requires to allocate a new cluster but writes only to the first
couple of sectors in that cluster, the rest of the cluster is zeroed - just
to be overwritten by the following second request that fills up the cluster.
Let's try to merge sequential write requests to the same cluster, so we can
avoid to write the zero padding to the disk in the first place.
As a nice side effect, also other formats take advantage of dealing with less
and larger requests.
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
Diffstat (limited to 'block.h')
-rw-r--r-- | block.h | 15 |
1 files changed, 15 insertions, 0 deletions
@@ -87,6 +87,21 @@ BlockDriverAIOCB *bdrv_aio_writev(BlockDriverState *bs, int64_t sector_num, BlockDriverCompletionFunc *cb, void *opaque); void bdrv_aio_cancel(BlockDriverAIOCB *acb); +typedef struct BlockRequest { + /* Fields to be filled by multiwrite caller */ + int64_t sector; + int nb_sectors; + QEMUIOVector *qiov; + BlockDriverCompletionFunc *cb; + void *opaque; + + /* Filled by multiwrite implementation */ + int error; +} BlockRequest; + +int bdrv_aio_multiwrite(BlockDriverState *bs, BlockRequest *reqs, + int num_reqs); + /* sg packet commands */ int bdrv_ioctl(BlockDriverState *bs, unsigned long int req, void *buf); BlockDriverAIOCB *bdrv_aio_ioctl(BlockDriverState *bs, |