[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v6 3/5] block/io: bdrv_common_block_status_above: support bs
From: |
Alberto Garcia |
Subject: |
Re: [PATCH v6 3/5] block/io: bdrv_common_block_status_above: support bs == base |
Date: |
Wed, 23 Sep 2020 18:22:52 +0200 |
User-agent: |
Notmuch/0.18.2 (http://notmuchmail.org) Emacs/24.4.1 (i586-pc-linux-gnu) |
On Wed 16 Sep 2020 02:20:06 PM CEST, Vladimir Sementsov-Ogievskiy wrote:
> We are going to reuse bdrv_common_block_status_above in
> bdrv_is_allocated_above. bdrv_is_allocated_above may be called with
> include_base == false and still bs == base (for ex. from img_rebase()).
>
> So, support this corner case.
>
> Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
> Reviewed-by: Kevin Wolf <kwolf@redhat.com>
> Reviewed-by: Eric Blake <eblake@redhat.com>
Reviewed-by: Alberto Garcia <berto@igalia.com>
(my question about include_base remains, but otherwise this patch is
correct)
Berto
- [PATCH v6 0/5] fix & merge block_status_above and is_allocated_above, Vladimir Sementsov-Ogievskiy, 2020/09/16
- [PATCH v6 4/5] block/io: fix bdrv_is_allocated_above, Vladimir Sementsov-Ogievskiy, 2020/09/16
- [PATCH v6 1/5] block/io: fix bdrv_co_block_status_above, Vladimir Sementsov-Ogievskiy, 2020/09/16
- [PATCH v6 5/5] iotests: add commit top->base cases to 274, Vladimir Sementsov-Ogievskiy, 2020/09/16
- [PATCH v6 2/5] block/io: bdrv_common_block_status_above: support include_base, Vladimir Sementsov-Ogievskiy, 2020/09/16
- [PATCH v6 3/5] block/io: bdrv_common_block_status_above: support bs == base, Vladimir Sementsov-Ogievskiy, 2020/09/16
- Re: [PATCH v6 3/5] block/io: bdrv_common_block_status_above: support bs == base,
Alberto Garcia <=