Re: [Qemu-block] [PATCH v3 1/2] block: sync bdrv_co_get_block_status_above() with bdrv_is_allocated_above()

2016-09-20 Thread Jeff Cody
On Tue, Sep 20, 2016 at 01:18:12AM +0200, Max Reitz wrote: > On 2016-09-15 at 18:34, Denis V. Lunev wrote: > >They should work very similar, covering same areas if backing store is > >shorter than the image. This change is necessary for the followup patch > >switching to

Re: [Qemu-block] [PATCH v3 1/2] block: sync bdrv_co_get_block_status_above() with bdrv_is_allocated_above()

2016-09-19 Thread Max Reitz
On 2016-09-15 at 18:34, Denis V. Lunev wrote: They should work very similar, covering same areas if backing store is shorter than the image. This change is necessary for the followup patch switching to bdrv_get_block_status_above() in mirror to avoid assert in check_block. This change should be

[Qemu-block] [PATCH v3 1/2] block: sync bdrv_co_get_block_status_above() with bdrv_is_allocated_above()

2016-09-15 Thread Denis V. Lunev
They should work very similar, covering same areas if backing store is shorter than the image. This change is necessary for the followup patch switching to bdrv_get_block_status_above() in mirror to avoid assert in check_block. This change should be made very carefully. Let us assume that we have