qemu-img: Switch to byte-based block access
Sector-based blk_write() should die; switch to byte-based blk_pwrite() instead. Likewise for blk_read(). Signed-off-by: Eric Blake <eblake@redhat.com> Signed-off-by: Kevin Wolf <kwolf@redhat.com>
This commit is contained in:
parent
bd31c214c3
commit
9166920a0b
28
qemu-img.c
28
qemu-img.c
|
@ -1088,7 +1088,8 @@ static int check_empty_sectors(BlockBackend *blk, int64_t sect_num,
|
||||||
uint8_t *buffer, bool quiet)
|
uint8_t *buffer, bool quiet)
|
||||||
{
|
{
|
||||||
int pnum, ret = 0;
|
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) {
|
if (ret < 0) {
|
||||||
error_report("Error while reading offset %" PRId64 " of %s: %s",
|
error_report("Error while reading offset %" PRId64 " of %s: %s",
|
||||||
sectors_to_bytes(sect_num), filename, strerror(-ret));
|
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);
|
nb_sectors = MIN(pnum1, pnum2);
|
||||||
} else if (allocated1 == allocated2) {
|
} else if (allocated1 == allocated2) {
|
||||||
if (allocated1) {
|
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) {
|
if (ret < 0) {
|
||||||
error_report("Error while reading offset %" PRId64 " of %s:"
|
error_report("Error while reading offset %" PRId64 " of %s:"
|
||||||
" %s", sectors_to_bytes(sector_num), filename1,
|
" %s", sectors_to_bytes(sector_num), filename1,
|
||||||
|
@ -1309,7 +1311,8 @@ static int img_compare(int argc, char **argv)
|
||||||
ret = 4;
|
ret = 4;
|
||||||
goto out;
|
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) {
|
if (ret < 0) {
|
||||||
error_report("Error while reading offset %" PRId64
|
error_report("Error while reading offset %" PRId64
|
||||||
" of %s: %s", sectors_to_bytes(sector_num),
|
" 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];
|
bs_sectors = s->src_sectors[s->src_cur];
|
||||||
|
|
||||||
n = MIN(nb_sectors, bs_sectors - (sector_num - s->src_cur_offset));
|
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) {
|
if (ret < 0) {
|
||||||
return ret;
|
return ret;
|
||||||
}
|
}
|
||||||
|
@ -1577,7 +1582,8 @@ static int convert_write(ImgConvertState *s, int64_t sector_num, int nb_sectors,
|
||||||
if (!s->min_sparse ||
|
if (!s->min_sparse ||
|
||||||
is_allocated_sectors_min(buf, n, &n, 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) {
|
if (ret < 0) {
|
||||||
return ret;
|
return ret;
|
||||||
}
|
}
|
||||||
|
@ -3024,7 +3030,8 @@ static int img_rebase(int argc, char **argv)
|
||||||
n = old_backing_num_sectors - sector;
|
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) {
|
if (ret < 0) {
|
||||||
error_report("error while reading from old backing file");
|
error_report("error while reading from old backing file");
|
||||||
goto out;
|
goto out;
|
||||||
|
@ -3038,7 +3045,8 @@ static int img_rebase(int argc, char **argv)
|
||||||
n = new_backing_num_sectors - sector;
|
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) {
|
if (ret < 0) {
|
||||||
error_report("error while reading from new backing file");
|
error_report("error while reading from new backing file");
|
||||||
goto out;
|
goto out;
|
||||||
|
@ -3054,8 +3062,10 @@ static int img_rebase(int argc, char **argv)
|
||||||
if (compare_sectors(buf_old + written * 512,
|
if (compare_sectors(buf_old + written * 512,
|
||||||
buf_new + written * 512, n - written, &pnum))
|
buf_new + written * 512, n - written, &pnum))
|
||||||
{
|
{
|
||||||
ret = blk_write(blk, sector + written,
|
ret = blk_pwrite(blk,
|
||||||
buf_old + written * 512, pnum);
|
(sector + written) << BDRV_SECTOR_BITS,
|
||||||
|
buf_old + written * 512,
|
||||||
|
pnum << BDRV_SECTOR_BITS, 0);
|
||||||
if (ret < 0) {
|
if (ret < 0) {
|
||||||
error_report("Error while writing to COW image: %s",
|
error_report("Error while writing to COW image: %s",
|
||||||
strerror(-ret));
|
strerror(-ret));
|
||||||
|
|
Loading…
Reference in New Issue