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

2022-01-31 Thread 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(_err); +bdrv_activate_all(_err); I guess that we can

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

2022-01-27 Thread Emanuele Giuseppe Esposito
On 24/01/2022 11:44, Juan Quintela wrote: >> diff --git a/migration/migration.c b/migration/migration.c >> index 0652165610..1f06fd2d18 100644 >> --- a/migration/migration.c >> +++ b/migration/migration.c >> @@ -499,7 +499,7 @@ static void process_incoming_migration_bh(void *opaque) >>

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

2022-01-26 Thread Hanna Reitz
On 21.01.22 18:05, Emanuele Giuseppe Esposito wrote: Following the bdrv_activate renaming, change also the name of the respective callers. bdrv_invalidate_cache_all -> bdrv_activate_all blk_invalidate_cache -> blk_activate test_sync_op_invalidate_cache -> test_sync_op_activate No functional

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

2022-01-24 Thread Juan Quintela
Emanuele Giuseppe Esposito wrote: > Following the bdrv_activate renaming, change also the name > of the respective callers. > > bdrv_invalidate_cache_all -> bdrv_activate_all > blk_invalidate_cache -> blk_activate > test_sync_op_invalidate_cache -> test_sync_op_activate > > No functional change

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

2022-01-21 Thread Emanuele Giuseppe Esposito
Following the bdrv_activate renaming, change also the name of the respective callers. bdrv_invalidate_cache_all -> bdrv_activate_all blk_invalidate_cache -> blk_activate test_sync_op_invalidate_cache -> test_sync_op_activate No functional change intended. Signed-off-by: Emanuele Giuseppe