[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 21/22] block: Make bdrv_pread() a bdrv_prwv_co() wra
From: |
Kevin Wolf |
Subject: |
[Qemu-devel] [PATCH 21/22] block: Make bdrv_pread() a bdrv_prwv_co() wrapper |
Date: |
Wed, 11 Dec 2013 22:08:28 +0100 |
Instead of implementing the alignment adjustment here, use the now
existing functionality of bdrv_co_do_preadv().
Signed-off-by: Kevin Wolf <address@hidden>
---
block.c | 49 +++++++++++++------------------------------------
1 file changed, 13 insertions(+), 36 deletions(-)
diff --git a/block.c b/block.c
index 5e2e092..e8653e3 100644
--- a/block.c
+++ b/block.c
@@ -2523,49 +2523,26 @@ int bdrv_make_zero(BlockDriverState *bs,
BdrvRequestFlags flags)
}
}
-int bdrv_pread(BlockDriverState *bs, int64_t offset,
- void *buf, int count1)
+int bdrv_pread(BlockDriverState *bs, int64_t offset, void *buf, int bytes)
{
- uint8_t tmp_buf[BDRV_SECTOR_SIZE];
- int len, nb_sectors, count;
- int64_t sector_num;
+ QEMUIOVector qiov;
+ struct iovec iov = {
+ .iov_base = (void *)buf,
+ .iov_len = bytes,
+ };
int ret;
- count = count1;
- /* first read to align to sector start */
- len = (BDRV_SECTOR_SIZE - offset) & (BDRV_SECTOR_SIZE - 1);
- if (len > count)
- len = count;
- sector_num = offset >> BDRV_SECTOR_BITS;
- if (len > 0) {
- if ((ret = bdrv_read(bs, sector_num, tmp_buf, 1)) < 0)
- return ret;
- memcpy(buf, tmp_buf + (offset & (BDRV_SECTOR_SIZE - 1)), len);
- count -= len;
- if (count == 0)
- return count1;
- sector_num++;
- buf += len;
+ if (bytes < 0) {
+ return -EINVAL;
}
- /* read the sectors "in place" */
- nb_sectors = count >> BDRV_SECTOR_BITS;
- if (nb_sectors > 0) {
- if ((ret = bdrv_read(bs, sector_num, buf, nb_sectors)) < 0)
- return ret;
- sector_num += nb_sectors;
- len = nb_sectors << BDRV_SECTOR_BITS;
- buf += len;
- count -= len;
+ qemu_iovec_init_external(&qiov, &iov, 1);
+ ret = bdrv_prwv_co(bs, offset, &qiov, false, 0);
+ if (ret < 0) {
+ return ret;
}
- /* add data from the last sector */
- if (count > 0) {
- if ((ret = bdrv_read(bs, sector_num, tmp_buf, 1)) < 0)
- return ret;
- memcpy(buf, tmp_buf, count);
- }
- return count1;
+ return bytes;
}
int bdrv_pwritev(BlockDriverState *bs, int64_t offset, QEMUIOVector *qiov)
--
1.8.1.4
- [Qemu-devel] [PATCH 11/22] block: Introduce bdrv_aligned_pwritev(), (continued)
- [Qemu-devel] [PATCH 11/22] block: Introduce bdrv_aligned_pwritev(), Kevin Wolf, 2013/12/11
- [Qemu-devel] [PATCH 12/22] block: write: Handle COR dependency after I/O throttling, Kevin Wolf, 2013/12/11
- [Qemu-devel] [PATCH 13/22] block: Introduce bdrv_co_do_pwritev(), Kevin Wolf, 2013/12/11
- [Qemu-devel] [PATCH 14/22] block: Switch BdrvTrackedRequest to byte granularity, Kevin Wolf, 2013/12/11
- [Qemu-devel] [PATCH 16/22] block: Make zero-after-EOF work with larger alignment, Kevin Wolf, 2013/12/11
- [Qemu-devel] [PATCH 15/22] block: Allow waiting for overlapping requests between begin/end, Kevin Wolf, 2013/12/11
- [Qemu-devel] [PATCH 17/22] block: Generalise and optimise COR serialisation, Kevin Wolf, 2013/12/11
- [Qemu-devel] [PATCH 18/22] block: Make overlap range for serialisation dynamic, Kevin Wolf, 2013/12/11
- [Qemu-devel] [PATCH 19/22] block: Align requests in bdrv_co_do_pwritev(), Kevin Wolf, 2013/12/11
- [Qemu-devel] [PATCH 20/22] block: Change coroutine wrapper to byte granularity, Kevin Wolf, 2013/12/11
- [Qemu-devel] [PATCH 21/22] block: Make bdrv_pread() a bdrv_prwv_co() wrapper,
Kevin Wolf <=
- [Qemu-devel] [PATCH 22/22] block: Make bdrv_pwrite() a bdrv_prwv_co() wrapper, Kevin Wolf, 2013/12/11
- Re: [Qemu-devel] [PATCH 00/22] block: Support for 512b-on-4k emulation, Peter Lieven, 2013/12/12