[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [Qemu-devel] [PATCH v5 2/3] block/io: bdrv_pdiscard: su
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-block] [Qemu-devel] [PATCH v5 2/3] block/io: bdrv_pdiscard: support int64_t bytes parameter |
Date: |
Tue, 30 Apr 2019 13:09:18 +0200 |
User-agent: |
Mutt/1.11.3 (2019-02-01) |
Am 30.04.2019 um 12:03 hat Vladimir Sementsov-Ogievskiy geschrieben:
> 30.04.2019 12:24, Stefano Garzarella wrote:
> > On Tue, Apr 23, 2019 at 03:57:05PM +0300, Vladimir Sementsov-Ogievskiy
> > wrote:
> >> This fixes at least one overflow in qcow2_process_discards, which
> >> passes 64bit region length to bdrv_pdiscard where bytes (or sectors in
> >> the past) parameter is int since its introduction in 0b919fae.
> >>
> >> Signed-off-by: Vladimir Sementsov-Ogievskiy <address@hidden>
> >> ---
> >> include/block/block.h | 4 ++--
> >> block/io.c | 16 ++++++++--------
> >> 2 files changed, 10 insertions(+), 10 deletions(-)
> >>
> >> diff --git a/include/block/block.h b/include/block/block.h
> >> index c7a26199aa..69fa18867e 100644
> >> --- a/include/block/block.h
> >> +++ b/include/block/block.h
> >> @@ -432,8 +432,8 @@ void bdrv_drain_all(void);
> >> AIO_WAIT_WHILE(bdrv_get_aio_context(bs_), \
> >> cond); })
> >>
> >> -int bdrv_pdiscard(BdrvChild *child, int64_t offset, int bytes);
> >> -int bdrv_co_pdiscard(BdrvChild *child, int64_t offset, int bytes);
> >> +int bdrv_pdiscard(BdrvChild *child, int64_t offset, int64_t bytes);
> >> +int bdrv_co_pdiscard(BdrvChild *child, int64_t offset, int64_t bytes);
> >> int bdrv_has_zero_init_1(BlockDriverState *bs);
> >> int bdrv_has_zero_init(BlockDriverState *bs);
> >> bool bdrv_unallocated_blocks_are_zero(BlockDriverState *bs);
> >> diff --git a/block/io.c b/block/io.c
> >> index dfc153b8d8..16b6c5d855 100644
> >> --- a/block/io.c
> >> +++ b/block/io.c
> >> @@ -2653,7 +2653,7 @@ int bdrv_flush(BlockDriverState *bs)
> >> typedef struct DiscardCo {
> >> BdrvChild *child;
> >> int64_t offset;
> >> - int bytes;
> >> + int64_t bytes;
> >> int ret;
> >> } DiscardCo;
> >> static void coroutine_fn bdrv_pdiscard_co_entry(void *opaque)
> >> @@ -2664,14 +2664,15 @@ static void coroutine_fn
> >> bdrv_pdiscard_co_entry(void *opaque)
> >> aio_wait_kick();
> >> }
> >>
> >> -int coroutine_fn bdrv_co_pdiscard(BdrvChild *child, int64_t offset, int
> >> bytes)
> >> +int coroutine_fn bdrv_co_pdiscard(BdrvChild *child, int64_t offset,
> >> + int64_t bytes)
> >> {
> >> BdrvTrackedRequest req;
> >> int max_pdiscard, ret;
> >> int head, tail, align;
> >> BlockDriverState *bs = child->bs;
> >>
> >> - if (!bs || !bs->drv) {
> >> + if (!bs || !bs->drv || !bdrv_is_inserted(bs)) {
> >
> > Should we describe this change in the commit message?
>
> Honestly, don't want to resend the series for this.
I haven't reviewed the patches yet, but if this remains the only thing
to change, it can be updated while applying the series if we have a
specific proposal for a new commit message.
Kevin
[Qemu-block] [PATCH v5 3/3] iotests: test big qcow2 shrink, Vladimir Sementsov-Ogievskiy, 2019/04/23