[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v4 10/14] qemu-img: Switch to byte-based block acces
From: |
Eric Blake |
Subject: |
[Qemu-devel] [PATCH v4 10/14] qemu-img: Switch to byte-based block access |
Date: |
Fri, 29 Apr 2016 14:08:32 -0600 |
Sector-based blk_write() should die; switch to byte-based
blk_pwrite() instead. Likewise for blk_read().
Signed-off-by: Eric Blake <address@hidden>
---
qemu-img.c | 28 +++++++++++++++++++---------
1 file changed, 19 insertions(+), 9 deletions(-)
diff --git a/qemu-img.c b/qemu-img.c
index 46f2a6d..c19f9d4 100644
--- a/qemu-img.c
+++ b/qemu-img.c
@@ -1088,7 +1088,8 @@ static int check_empty_sectors(BlockBackend *blk, int64_t
sect_num,
uint8_t *buffer, bool quiet)
{
int pnum, ret = 0;
- ret = blk_read(blk, sect_num, buffer, sect_count);
+ ret = blk_pread(blk, sect_num << BDRV_SECTOR_BITS, buffer,
+ sect_count << BDRV_SECTOR_BITS);
if (ret < 0) {
error_report("Error while reading offset %" PRId64 " of %s: %s",
sectors_to_bytes(sect_num), filename, strerror(-ret));
@@ -1301,7 +1302,8 @@ static int img_compare(int argc, char **argv)
nb_sectors = MIN(pnum1, pnum2);
} else if (allocated1 == allocated2) {
if (allocated1) {
- ret = blk_read(blk1, sector_num, buf1, nb_sectors);
+ ret = blk_pread(blk1, sector_num << BDRV_SECTOR_BITS, buf1,
+ nb_sectors << BDRV_SECTOR_BITS);
if (ret < 0) {
error_report("Error while reading offset %" PRId64 " of
%s:"
" %s", sectors_to_bytes(sector_num),
filename1,
@@ -1309,7 +1311,8 @@ static int img_compare(int argc, char **argv)
ret = 4;
goto out;
}
- ret = blk_read(blk2, sector_num, buf2, nb_sectors);
+ ret = blk_pread(blk2, sector_num << BDRV_SECTOR_BITS, buf2,
+ nb_sectors << BDRV_SECTOR_BITS);
if (ret < 0) {
error_report("Error while reading offset %" PRId64
" of %s: %s", sectors_to_bytes(sector_num),
@@ -1522,7 +1525,9 @@ static int convert_read(ImgConvertState *s, int64_t
sector_num, int nb_sectors,
bs_sectors = s->src_sectors[s->src_cur];
n = MIN(nb_sectors, bs_sectors - (sector_num - s->src_cur_offset));
- ret = blk_read(blk, sector_num - s->src_cur_offset, buf, n);
+ ret = blk_pread(blk,
+ (sector_num - s->src_cur_offset) << BDRV_SECTOR_BITS,
+ buf, n << BDRV_SECTOR_BITS);
if (ret < 0) {
return ret;
}
@@ -1577,7 +1582,8 @@ static int convert_write(ImgConvertState *s, int64_t
sector_num, int nb_sectors,
if (!s->min_sparse ||
is_allocated_sectors_min(buf, n, &n, s->min_sparse))
{
- ret = blk_write(s->target, sector_num, buf, n);
+ ret = blk_pwrite(s->target, sector_num << BDRV_SECTOR_BITS,
+ buf, n << BDRV_SECTOR_BITS, 0);
if (ret < 0) {
return ret;
}
@@ -3023,7 +3029,8 @@ static int img_rebase(int argc, char **argv)
n = old_backing_num_sectors - sector;
}
- ret = blk_read(blk_old_backing, sector, buf_old, n);
+ ret = blk_pread(blk_old_backing, sector << BDRV_SECTOR_BITS,
+ buf_old, n << BDRV_SECTOR_BITS);
if (ret < 0) {
error_report("error while reading from old backing file");
goto out;
@@ -3037,7 +3044,8 @@ static int img_rebase(int argc, char **argv)
n = new_backing_num_sectors - sector;
}
- ret = blk_read(blk_new_backing, sector, buf_new, n);
+ ret = blk_pread(blk_new_backing, sector << BDRV_SECTOR_BITS,
+ buf_new, n << BDRV_SECTOR_BITS);
if (ret < 0) {
error_report("error while reading from new backing file");
goto out;
@@ -3053,8 +3061,10 @@ static int img_rebase(int argc, char **argv)
if (compare_sectors(buf_old + written * 512,
buf_new + written * 512, n - written, &pnum))
{
- ret = blk_write(blk, sector + written,
- buf_old + written * 512, pnum);
+ ret = blk_pwrite(blk,
+ (sector + written) << BDRV_SECTOR_BITS,
+ buf_old + written * 512,
+ pnum << BDRV_SECTOR_BITS, 0);
if (ret < 0) {
error_report("Error while writing to COW image: %s",
strerror(-ret));
--
2.5.5
- [Qemu-devel] [PATCH v4 06/14] sd: Switch to byte-based block access, (continued)
- [Qemu-devel] [PATCH v4 06/14] sd: Switch to byte-based block access, Eric Blake, 2016/04/29
- [Qemu-devel] [PATCH v4 09/14] nbd: Switch to byte-based block access, Eric Blake, 2016/04/29
- [Qemu-devel] [PATCH v4 02/14] fdc: Switch to byte-based block access, Eric Blake, 2016/04/29
- [Qemu-devel] [PATCH v4 07/14] m25p80: Switch to byte-based block access, Eric Blake, 2016/04/29
- [Qemu-devel] [PATCH v4 03/14] nand: Switch to byte-based block access, Eric Blake, 2016/04/29
- [Qemu-devel] [PATCH v4 05/14] pflash: Switch to byte-based block access, Eric Blake, 2016/04/29
- [Qemu-devel] [PATCH v4 13/14] block: Switch blk_write_zeroes() to byte interface, Eric Blake, 2016/04/29
- [Qemu-devel] [PATCH v4 14/14] block: Kill blk_write(), blk_read(), Eric Blake, 2016/04/29
- [Qemu-devel] [PATCH v4 04/14] onenand: Switch to byte-based block access, Eric Blake, 2016/04/29
- [Qemu-devel] [PATCH v4 08/14] atapi: Switch to byte-based block access, Eric Blake, 2016/04/29
- [Qemu-devel] [PATCH v4 10/14] qemu-img: Switch to byte-based block access,
Eric Blake <=
- [Qemu-devel] [PATCH v4 12/14] block: Switch blk_read_unthrottled() to byte interface, Eric Blake, 2016/04/29
- [Qemu-devel] [PATCH v4 01/14] block: Allow BDRV_REQ_FUA through blk_pwrite(), Eric Blake, 2016/04/29
- [Qemu-devel] [PATCH v4 11/14] qemu-io: Switch to byte-based block access, Eric Blake, 2016/04/29