[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH] qmp: fill in the image field in BlockDeviceInfo
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-block] [PATCH] qmp: fill in the image field in BlockDeviceInfo |
Date: |
Wed, 22 Apr 2015 11:18:24 +0100 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Fri, Apr 17, 2015 at 02:52:43PM +0300, Alberto Garcia wrote:
> The image field in BlockDeviceInfo is supposed to contain an ImageInfo
> object. However that is being filled in by bdrv_query_info(), not by
> bdrv_block_device_info(), which is where BlockDeviceInfo is actually
> created.
>
> Anyone calling bdrv_block_device_info() directly will get a null image
> field. As a consequence of this, the HMP command 'info block -n -v'
> crashes QEMU.
>
> This patch moves the code that fills in that field from
> bdrv_query_info() to bdrv_block_device_info().
>
> Signed-off-by: Alberto Garcia <address@hidden>
> ---
> block.c | 9 +++++++--
> block/qapi.c | 46 +++++++++++++++++++++++++---------------------
> blockdev.c | 2 +-
> include/block/block.h | 2 +-
> include/block/qapi.h | 2 +-
> 5 files changed, 35 insertions(+), 26 deletions(-)
Reverted the QEMU 2.3 fix and replaced it with this commit.
Thanks, applied to my block-next tree:
https://github.com/stefanha/qemu/commits/block-next
Stefan
pgpruUvwZGiFl.pgp
Description: PGP signature