Commit fbc8e1b7 authored by Max Reitz's avatar Max Reitz
Browse files

vpc: Do not return RAW from block_status 



vpc is not really a passthrough driver, even when using the fixed
subformat (where host and guest offsets are equal).  It should handle
preallocation like all other drivers do, namely by returning
DATA | RECURSE instead of RAW.

There is no tangible difference but the fact that bdrv_is_allocated() no
longer falls through to the protocol layer.

Signed-off-by: default avatarMax Reitz <mreitz@redhat.com>
Message-id: 20190725155512.9827-4-mreitz@redhat.com
Reviewed-by: default avatarJohn Snow <jsnow@redhat.com>
Signed-off-by: default avatarMax Reitz <mreitz@redhat.com>
parent 4dd84ac9

block/vpc.c

+1 −1
Original line number Diff line number Diff line
@@ -737,7 +737,7 @@ static int coroutine_fn vpc_co_block_status(BlockDriverState *bs, 
        *pnum = bytes;

        *map = offset;

        *file = bs->file->bs;

        return BDRV_BLOCK_RAW | BDRV_BLOCK_OFFSET_VALID;

        return BDRV_BLOCK_DATA | BDRV_BLOCK_OFFSET_VALID | BDRV_BLOCK_RECURSE;

    }



    qemu_co_mutex_lock(&s->lock);