Re: [PATCH 1/8] block/vdi: return ZERO block-status when appropriate

2020-05-07 Thread Vladimir Sementsov-Ogievskiy
06.05.2020 16:57, Eric Blake wrote: On 5/6/20 4:25 AM, Vladimir Sementsov-Ogievskiy wrote: In case of !VDI_IS_ALLOCATED[], we do zero out the corresponding chunk of qiov. So, this should be reported as ZERO. This part makes sense outright, since vdi does not allow backing files, so all reads

Re: [PATCH 1/8] block/vdi: return ZERO block-status when appropriate

2020-05-06 Thread Vladimir Sementsov-Ogievskiy
06.05.2020 16:57, Eric Blake wrote: On 5/6/20 4:25 AM, Vladimir Sementsov-Ogievskiy wrote: In case of !VDI_IS_ALLOCATED[], we do zero out the corresponding chunk of qiov. So, this should be reported as ZERO. This part makes sense outright, since vdi does not allow backing files, so all reads

Re: [PATCH 1/8] block/vdi: return ZERO block-status when appropriate

2020-05-06 Thread Eric Blake
On 5/6/20 4:25 AM, Vladimir Sementsov-Ogievskiy wrote: In case of !VDI_IS_ALLOCATED[], we do zero out the corresponding chunk of qiov. So, this should be reported as ZERO. This part makes sense outright, since vdi does not allow backing files, so all reads of a VDI disk result in data rather t

[PATCH 1/8] block/vdi: return ZERO block-status when appropriate

2020-05-06 Thread Vladimir Sementsov-Ogievskiy
In case of !VDI_IS_ALLOCATED[], we do zero out the corresponding chunk of qiov. So, this should be reported as ZERO. After block-status update, it never reports 0, so setting unallocated_blocks_are_zero doesn't make sense. Drop it. Signed-off-by: Vladimir Sementsov-Ogievskiy --- block/vdi.c | 3