[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-block] [PATCH 2/5] vdi: Replace bdrv_{read, write}() with bdrv_{pr
From: |
Alberto Garcia |
Subject: |
[Qemu-block] [PATCH 2/5] vdi: Replace bdrv_{read, write}() with bdrv_{pread, pwrite}() |
Date: |
Mon, 29 Apr 2019 20:42:37 +0200 |
There's only a couple of bdrv_read() and bdrv_write() calls left in
the vdi code, and they can be trivially replaced with the byte-based
bdrv_pread() and bdrv_pwrite().
Signed-off-by: Alberto Garcia <address@hidden>
---
block/vdi.c | 11 ++++++-----
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/block/vdi.c b/block/vdi.c
index e1c42ad732..8d849b2754 100644
--- a/block/vdi.c
+++ b/block/vdi.c
@@ -384,7 +384,7 @@ static int vdi_open(BlockDriverState *bs, QDict *options,
int flags,
logout("\n");
- ret = bdrv_read(bs->file, 0, (uint8_t *)&header, 1);
+ ret = bdrv_pread(bs->file, 0, (uint8_t *)&header, sizeof(header));
if (ret < 0) {
goto fail;
}
@@ -484,8 +484,8 @@ static int vdi_open(BlockDriverState *bs, QDict *options,
int flags,
goto fail;
}
- ret = bdrv_read(bs->file, s->bmap_sector, (uint8_t *)s->bmap,
- bmap_size);
+ ret = bdrv_pread(bs->file, header.offset_bmap, (uint8_t *)s->bmap,
+ bmap_size * SECTOR_SIZE);
if (ret < 0) {
goto fail_free_bmap;
}
@@ -704,7 +704,7 @@ nonallocating_write:
assert(VDI_IS_ALLOCATED(bmap_first));
*header = s->header;
vdi_header_to_le(header);
- ret = bdrv_write(bs->file, 0, block, 1);
+ ret = bdrv_pwrite(bs->file, 0, block, sizeof(*block));
g_free(block);
block = NULL;
@@ -722,7 +722,8 @@ nonallocating_write:
base = ((uint8_t *)&s->bmap[0]) + bmap_first * SECTOR_SIZE;
logout("will write %u block map sectors starting from entry %u\n",
n_sectors, bmap_first);
- ret = bdrv_write(bs->file, offset, base, n_sectors);
+ ret = bdrv_pwrite(bs->file, offset * SECTOR_SIZE, base,
+ n_sectors * SECTOR_SIZE);
}
return ret;
--
2.11.0
- [Qemu-block] [PATCH 0/5] Remove bdrv_read() and bdrv_write(), Alberto Garcia, 2019/04/29
- [Qemu-block] [PATCH 1/5] qcow2: Replace bdrv_write() with bdrv_pwrite(), Alberto Garcia, 2019/04/29
- [Qemu-block] [PATCH 2/5] vdi: Replace bdrv_{read, write}() with bdrv_{pread, pwrite}(),
Alberto Garcia <=
- [Qemu-block] [PATCH 3/5] vvfat: Replace bdrv_{read, write}() with bdrv_{pread, pwrite}(), Alberto Garcia, 2019/04/29
- [Qemu-block] [PATCH 4/5] block: Remove bdrv_read() and bdrv_write(), Alberto Garcia, 2019/04/29
- [Qemu-block] [PATCH 5/5] qcow2: Remove BDRVQcow2State.cluster_sectors, Alberto Garcia, 2019/04/29
- Re: [Qemu-block] [PATCH 0/5] Remove bdrv_read() and bdrv_write(), Eric Blake, 2019/04/29