Re: [PATCH v6 20/33] block: rename bdrv_invalidate_cache_all, blk_invalidate_cache and test_sync_op_invalidate_cache

[Paolo Bonzini]

On 1/27/22 10:18, Emanuele Giuseppe Esposito wrote:
> > >           /* Make sure all file formats flush their mutable metadata.
> > >            * If we get an error here, just don't restart the VM yet. */
> > > -        bdrv_invalidate_cache_all(&local_err);
> > > +        bdrv_activate_all(&local_err);
> > I guess that we can change the comment here, it just looks weird the
> > comment saying flush() and the function nawed _activate()
> Do you think it's enough to replace "flush" with "activate"? I am not
> sure whether "activate their mutable metadata" is meaningful.

"Activation" consists of throwing away mutable metadata.

That's what "flush" means in this case, but it is often interpreted as 
"persisting" the metadata instead.  So perhaps s/flush/throw away/.

Paolo