On 01/13/2017 02:52 PM, Max Reitz wrote: > bdrv_refresh_filename() can do the same and it has some checks whether > the filename can actually be inherited or not, so we can let it do its > job in bdrv_open_inherit() after bdrv_open_common() has been called. > > The only thing we need to set in bdrv_open_common() is the > exact_filename of a BDS without an underlying file, for two reasons: > (1) It cannot be inherited from an underlying file BDS, so it has to be > set somewhere. > (2) The driver may need the filename in its bdrv_file_open() > implementation (format drivers do not need their own filename, > though they may need their file BDS's name). > > Signed-off-by: Max Reitz <mre...@redhat.com> > --- > block.c | 7 +++---- > 1 file changed, 3 insertions(+), 4 deletions(-) >
Reviewed-by: Eric Blake <ebl...@redhat.com> -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
