qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 53/54] block: Add Error parameter to bdrv_set_ba


From: Max Reitz
Subject: Re: [Qemu-devel] [PATCH 53/54] block: Add Error parameter to bdrv_set_backing_hd()
Date: Sat, 25 Feb 2017 16:36:27 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.7.0

On 21.02.2017 15:58, Kevin Wolf wrote:
> Not all callers of bdrv_set_backing_hd() know for sure that attaching
> the backing file will be allowed by the permission system. Return the
> error from the function rather than aborting.
> 
> Signed-off-by: Kevin Wolf <address@hidden>
> ---
>  block.c               | 27 ++++++++++++++++++++-------
>  block/commit.c        | 14 +++++++-------
>  block/mirror.c        | 16 +++++++++++++++-
>  block/stream.c        |  9 ++++++++-
>  block/vvfat.c         |  2 +-
>  include/block/block.h |  3 ++-
>  6 files changed, 53 insertions(+), 18 deletions(-)

It's a bit unfortunate that bdrv_set_backing_hd() is not atomic, that
is, that the node will no longer have a backing file if the command
fails. Oh well.

Reviewed-by: Max Reitz <address@hidden>

Attachment: signature.asc
Description: OpenPGP digital signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]