On Fri 28 Jun 2019 12:32:51 AM CEST, Max Reitz wrote: > @@ -4416,6 +4416,14 @@ int bdrv_freeze_backing_chain(BlockDriverState *bs, > BlockDriverState *base, > return -EPERM; > } > > + for (i = bs; i != base; i = backing_bs(i)) { > + if (i->backing && backing_bs(i)->never_freeze) { > + error_setg(errp, "Cannot freeze '%s' link to '%s'", > + i->backing->name, backing_bs(i)->node_name); > + return -EPERM; > + } > + }
How about adding this to bdrv_is_backing_chain_frozen() instead? Berto