Re: [Qemu-devel] [PATCH 3/4] block/raw: add .bdrv_get_info

[Stefan Hajnoczi <stefanha@gmail.com>]

On Mon, Jul 15, 2013 at 12:49:34PM +0200, Peter Lieven wrote:
> Signed-off-by: Peter Lieven <pl@kamp.de>
> ---
>  block/raw.c |    6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/block/raw.c b/block/raw.c
> index 8c81de9..f1682d4 100644
> --- a/block/raw.c
> +++ b/block/raw.c
> @@ -121,6 +121,11 @@ static int raw_has_zero_init(BlockDriverState *bs)
>      return bdrv_has_zero_init(bs->file);
>  }
>  
> +static int raw_get_info(BlockDriverState *bs, BlockDriverInfo *bdi)
> +{
> +    return bdrv_get_info(bs->file, bdi);
> +}
> +
>  static BlockDriver bdrv_raw = {
>      .format_name        = "raw",
>  
> @@ -140,6 +145,7 @@ static BlockDriver bdrv_raw = {
>  
>      .bdrv_probe         = raw_probe,
>      .bdrv_getlength     = raw_getlength,
> +    .bdrv_get_info      = raw_get_info,

I checked BlockDriverInfo to make sure the fields still make sense for
the raw BlockDriverState.  The vm_state_offset field is questionable,
since the raw BDS doesn't know about vmstate and you certainly cannot
write to it.  If protocols start supporting vmstate we might have to
rework that anyway, so I'm happy with this patch.

Stefan