On 05/08/2015 11:21 AM, Kevin Wolf wrote:
> Instead of letting every caller of bdrv_open() determine the right flags
> for its child node manually and pass them to the function, pass the
> parent node and the role of the newly opened child (like backing file,
> protocol layer, etc.).
> 
> Signed-off-by: Kevin Wolf <kw...@redhat.com>
> ---
>  block.c                   | 74 
> ++++++++++++++++++++++++++++++++++++++---------
>  block/blkdebug.c          |  2 +-
>  block/blkverify.c         |  4 +--
>  block/quorum.c            |  4 +--
>  block/vmdk.c              |  5 ++--
>  include/block/block.h     |  4 ++-
>  include/block/block_int.h |  7 +++++
>  7 files changed, 78 insertions(+), 22 deletions(-)
> 

Took me a while to follow the flow of information, but looks correct.

Reviewed-by: Eric Blake <ebl...@redhat.com>

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to