ping :)
10.06.2020 15:04, Vladimir Sementsov-Ogievskiy wrote:
> v5: rebase on coroutine-wrappers series, 02 changed correspondingly
>
> Based on series "[PATCH v7 0/7] coroutines: generate wrapper code", or
> in other words:
> Based-on: <20200610100336.23451-1-vsementsov@virtuozzo.com>
>
> Hi all!
>
> These series are here to address the following problem:
> block-status-above functions may consider space after EOF of
> intermediate backing files as unallocated, which is wrong, as these
> backing files are the reason of producing zeroes, we never go further by
> backing chain after a short backing file. So, if such short-backing file
> is _inside_ requested sub-chain of the backing chain, we should never
> report space after its EOF as unallocated.
>
> See patches 01,04,05 for details.
>
> Note, that this series leaves for another day the general problem
> around block-status: misuse of BDRV_BLOCK_ALLOCATED as is-fs-allocated
> vs go-to-backing.
> Audit for this problem is done here:
> "backing chain & block status & filters"
> https://lists.gnu.org/archive/html/qemu-devel/2020-04/msg04706.html
> And I'm going to prepare series to address this problem.
>
> Also, get_block_status func have same disease, but remains unfixed here:
> I want to make separate series for it, as it need some more refactoring,
> which should be based on series
> "[PATCH v5 0/7] coroutines: generate wrapper code"
>
> Vladimir Sementsov-Ogievskiy (5):
> block/io: fix bdrv_co_block_status_above
> block/io: bdrv_common_block_status_above: support include_base
> block/io: bdrv_common_block_status_above: support bs == base
> block/io: fix bdrv_is_allocated_above
> iotests: add commit top->base cases to 274
>
> block/coroutines.h | 2 +
> block/io.c | 100 ++++++++++++++++++-------------------
> block/qcow2.c | 16 +++++-
> tests/qemu-iotests/274 | 20 ++++++++
> tests/qemu-iotests/274.out | 65 ++++++++++++++++++++++++
> 5 files changed, 150 insertions(+), 53 deletions(-)
>
--
Best regards,
Vladimir