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 > --- > 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 -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org