On Thu, 06/16 09:58, Denis V. Lunev wrote: > The request is area is specified in bytes, not in sectors.
s/request is/request/ > > Signed-off-by: Denis V. Lunev <[email protected]> > CC: Vladimir Sementsov-Ogievskiy<[email protected]> > CC: Stefan Hajnoczi <[email protected]> > CC: Fam Zheng <[email protected]> > CC: Kevin Wolf <[email protected]> > CC: Max Reitz <[email protected]> > --- > block/io.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/block/io.c b/block/io.c > index afb99c4..b51f681 100644 > --- a/block/io.c > +++ b/block/io.c > @@ -2263,8 +2263,8 @@ int coroutine_fn bdrv_co_discard(BlockDriverState *bs, > int64_t sector_num, > return 0; > } > > - tracked_request_begin(&req, bs, NULL, sector_num, nb_sectors, > - BDRV_TRACKED_DISCARD); > + tracked_request_begin(&req, bs, NULL, sector_num << BDRV_SECTOR_BITS, > + nb_sectors << BDRV_SECTOR_BITS, > BDRV_TRACKED_DISCARD); > bdrv_set_dirty(bs, sector_num, nb_sectors); > > max_discard = MIN_NON_ZERO(bs->bl.max_discard, BDRV_REQUEST_MAX_SECTORS); > -- > 2.5.0 > Reviewed-by: Fam Zheng <[email protected]>
