block/qcow2: refactor qcow2_co_preadv to use buffer-based io
Use buffer based io in encrypted case. Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com> Acked-by: Stefan Hajnoczi <stefanha@redhat.com> Message-id: 20190604161514.262241-11-vsementsov@virtuozzo.com Message-Id: <20190604161514.262241-11-vsementsov@virtuozzo.com> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
This commit is contained in:
parent
1acc3466a2
commit
00721a3529
@ -2059,19 +2059,15 @@ static coroutine_fn int qcow2_co_preadv(BlockDriverState *bs, uint64_t offset,
|
|||||||
}
|
}
|
||||||
|
|
||||||
assert(cur_bytes <= QCOW_MAX_CRYPT_CLUSTERS * s->cluster_size);
|
assert(cur_bytes <= QCOW_MAX_CRYPT_CLUSTERS * s->cluster_size);
|
||||||
qemu_iovec_reset(&hd_qiov);
|
|
||||||
qemu_iovec_add(&hd_qiov, cluster_data, cur_bytes);
|
|
||||||
}
|
|
||||||
|
|
||||||
BLKDBG_EVENT(bs->file, BLKDBG_READ_AIO);
|
BLKDBG_EVENT(bs->file, BLKDBG_READ_AIO);
|
||||||
ret = bdrv_co_preadv(s->data_file,
|
ret = bdrv_co_pread(s->data_file,
|
||||||
cluster_offset + offset_in_cluster,
|
cluster_offset + offset_in_cluster,
|
||||||
cur_bytes, &hd_qiov, 0);
|
cur_bytes, cluster_data, 0);
|
||||||
if (ret < 0) {
|
if (ret < 0) {
|
||||||
goto fail;
|
goto fail;
|
||||||
}
|
}
|
||||||
if (bs->encrypted) {
|
|
||||||
assert(s->crypto);
|
|
||||||
assert((offset & (BDRV_SECTOR_SIZE - 1)) == 0);
|
assert((offset & (BDRV_SECTOR_SIZE - 1)) == 0);
|
||||||
assert((cur_bytes & (BDRV_SECTOR_SIZE - 1)) == 0);
|
assert((cur_bytes & (BDRV_SECTOR_SIZE - 1)) == 0);
|
||||||
if (qcow2_co_decrypt(bs, cluster_offset, offset,
|
if (qcow2_co_decrypt(bs, cluster_offset, offset,
|
||||||
@ -2080,6 +2076,14 @@ static coroutine_fn int qcow2_co_preadv(BlockDriverState *bs, uint64_t offset,
|
|||||||
goto fail;
|
goto fail;
|
||||||
}
|
}
|
||||||
qemu_iovec_from_buf(qiov, bytes_done, cluster_data, cur_bytes);
|
qemu_iovec_from_buf(qiov, bytes_done, cluster_data, cur_bytes);
|
||||||
|
} else {
|
||||||
|
BLKDBG_EVENT(bs->file, BLKDBG_READ_AIO);
|
||||||
|
ret = bdrv_co_preadv(s->data_file,
|
||||||
|
cluster_offset + offset_in_cluster,
|
||||||
|
cur_bytes, &hd_qiov, 0);
|
||||||
|
if (ret < 0) {
|
||||||
|
goto fail;
|
||||||
|
}
|
||||||
}
|
}
|
||||||
break;
|
break;
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user