On Thu, 12/07 14:30, Eric Blake wrote: > We are gradually moving away from sector-based interfaces, towards > byte-based. Update the vpc driver accordingly. Drop the now-unused > get_sector_offset().
No get_sector_offset change in the patch any more, it was removed by 778b087e513ea6fdc525c5a194ff7c9b8d3f53cb. > > Signed-off-by: Eric Blake <ebl...@redhat.com> > > --- > v5: fix incorrect rounding in 'map' and bad loop condition [Vladimir] > v4: rebase to interface tweak > v3: rebase to master > v2: drop get_sector_offset() [Kevin], rebase to mapping flag > --- > block/vpc.c | 43 ++++++++++++++++++++++--------------------- > 1 file changed, 22 insertions(+), 21 deletions(-) > > diff --git a/block/vpc.c b/block/vpc.c > index 1576d7b595..0000b5b9b4 100644 > --- a/block/vpc.c > +++ b/block/vpc.c > @@ -705,53 +705,54 @@ fail: > return ret; > } > > -static int64_t coroutine_fn vpc_co_get_block_status(BlockDriverState *bs, > - int64_t sector_num, int nb_sectors, int *pnum, BlockDriverState > **file) > +static int coroutine_fn vpc_co_block_status(BlockDriverState *bs, > + bool want_zero, > + int64_t offset, int64_t bytes, > + int64_t *pnum, int64_t *map, > + BlockDriverState **file) > { > BDRVVPCState *s = bs->opaque; > VHDFooter *footer = (VHDFooter*) s->footer_buf; > - int64_t start, offset; > + int64_t image_offset; > bool allocated; > - int64_t ret; > + int ret; > int n; > > if (be32_to_cpu(footer->type) == VHD_FIXED) { > - *pnum = nb_sectors; > + *pnum = bytes; > + *map = offset; > *file = bs->file->bs; > - return BDRV_BLOCK_RAW | BDRV_BLOCK_OFFSET_VALID | > - (sector_num << BDRV_SECTOR_BITS); > + return BDRV_BLOCK_RAW | BDRV_BLOCK_OFFSET_VALID; > } > > qemu_co_mutex_lock(&s->lock); > > - offset = get_image_offset(bs, sector_num << BDRV_SECTOR_BITS, false, > NULL); > - start = offset; > - allocated = (offset != -1); > + image_offset = get_image_offset(bs, offset, false, NULL); > + allocated = (image_offset != -1); > *pnum = 0; > ret = 0; > > do { > /* All sectors in a block are contiguous (without using the bitmap) > */ > - n = ROUND_UP(sector_num + 1, s->block_size / BDRV_SECTOR_SIZE) > - - sector_num; > - n = MIN(n, nb_sectors); > + n = ROUND_UP(offset + 1, s->block_size) - offset; > + n = MIN(n, bytes); Should 'n' be updated to int64_t to match the types of offset and bytes? > > *pnum += n; > - sector_num += n; > - nb_sectors -= n; > + offset += n; > + bytes -= n; > /* *pnum can't be greater than one block for allocated > * sectors since there is always a bitmap in between. */ > if (allocated) { > *file = bs->file->bs; > - ret = BDRV_BLOCK_DATA | BDRV_BLOCK_OFFSET_VALID | start; > + *map = image_offset; > + ret = BDRV_BLOCK_DATA | BDRV_BLOCK_OFFSET_VALID; > break; > } > - if (nb_sectors == 0) { > + if (bytes == 0) { > break; > } > - offset = get_image_offset(bs, sector_num << BDRV_SECTOR_BITS, false, > - NULL); > - } while (offset == -1); > + image_offset = get_image_offset(bs, offset, false, NULL); > + } while (image_offset == -1); > > qemu_co_mutex_unlock(&s->lock); > return ret;