Am 06.01.2012 15:01, schrieb Stefan Hajnoczi:
> From: Marcelo Tosatti <mtosa...@redhat.com>
>
> Add bdrv_find_backing_image: given a BlockDriverState pointer, and an id,
> traverse the backing image chain to locate the id.
>
> Signed-off-by: Marcelo Tosatti <mtosa...@redhat.com>
> Signed-off-by: Stefan Hajnoczi <stefa...@linux.vnet.ibm.com>
> ---
> block.c | 17 +++++++++++++++++
> block.h | 1 +
> 2 files changed, 18 insertions(+), 0 deletions(-)
>
> diff --git a/block.c b/block.c
> index 5bfaa3a..9b688a0 100644
> --- a/block.c
> +++ b/block.c
> @@ -2614,6 +2614,23 @@ int bdrv_snapshot_load_tmp(BlockDriverState *bs,
> return -ENOTSUP;
> }
>
> +BlockDriverState *bdrv_find_backing_image(BlockDriverState *bs, const char
> *id)
> +{
> + if (!bs->drv) {
> + return NULL;
> + }
> +
> + if (bs->backing_hd) {
> + if (strcmp(bs->backing_file, id) == 0) {
> + return bs->backing_hd;
So it's not really just some id, but the backing file name? I would find
it clearer to reflect that in the parameter name and the QMP error in
the next patch.
Kevin
