On Fri 25 Aug 2017 03:23:26 PM CEST, Manos Pitsidianakis wrote: > +static inline BlockDriverState *child_bs(BlockDriverState *bs) > +{ > + BlockDriverState *backing = backing_bs(bs); > + BlockDriverState *file = file_bs(bs); > + assert(!(file && backing)); > + return backing ?: file; > +}
I'm still not sure how useful it is to have a separate function for this, instead of putting it inside bdrv_get_first_explicit(), but anyway Reviewed-by: Alberto Garcia <be...@igalia.com> Berto