diff options
| author | Hanna Reitz <hreitz@redhat.com> | 2022-07-14 15:28:00 +0200 |
|---|---|---|
| committer | Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru> | 2022-07-26 22:04:59 +0300 |
| commit | eba088f91d21d9e42a81bef847173da30c5da961 (patch) | |
| tree | 1321b01adcf2f6983454803445513452dc3caa2e /block | |
| parent | f6cce6bcb2ef959cdd4da0e368f7c72045f21d6d (diff) | |
| download | qemu-eba088f91d21d9e42a81bef847173da30c5da961.zip qemu-eba088f91d21d9e42a81bef847173da30c5da961.tar.gz qemu-eba088f91d21d9e42a81bef847173da30c5da961.tar.bz2 | |
block/parallels: Fix buffer-based write call
Commit a4072543ccdddbd241d5962d9237b8b41fd006bf has changed the I/O here
from working on a local one-element I/O vector to just using the buffer
directly (using the bdrv_co_pread()/bdrv_co_pwrite() helper functions
introduced shortly before).
However, it only changed the bdrv_co_preadv() call to bdrv_co_pread() -
the subsequent bdrv_co_pwritev() call stayed this way, and so still
expects a QEMUIOVector pointer instead of a plain buffer. We must
change that to be a bdrv_co_pwrite() call.
Fixes: a4072543ccdddbd241d5962d ("block/parallels: use buffer-based io")
Signed-off-by: Hanna Reitz <hreitz@redhat.com>
Reviewed-by: Denis V. Lunev <den@openvz.org>
Reviewed-by: Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>
Message-Id: <20220714132801.72464-2-hreitz@redhat.com>
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>
Diffstat (limited to 'block')
| -rw-r--r-- | block/parallels.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/block/parallels.c b/block/parallels.c index 8b235b9..a229c06 100644 --- a/block/parallels.c +++ b/block/parallels.c @@ -241,8 +241,8 @@ static int64_t allocate_clusters(BlockDriverState *bs, int64_t sector_num, return ret; } - ret = bdrv_co_pwritev(bs->file, s->data_end * BDRV_SECTOR_SIZE, - nb_cow_bytes, buf, 0); + ret = bdrv_co_pwrite(bs->file, s->data_end * BDRV_SECTOR_SIZE, + nb_cow_bytes, buf, 0); qemu_vfree(buf); if (ret < 0) { return ret; |