16.05.2024 06:45, Li Zhijian wrote:
Make the code more tight.
Suggested-by: Michael Tokarev <m...@tls.msk.ru>
Reviewed-by: Peter Xu <pet...@redhat.com>
Reviewed-by: Zhang Chen <chen.zh...@intel.com>
Signed-off-by: Li Zhijian <lizhij...@fujitsu.com>
Signed-off-by: Michael Tokarev <m...@tls.msk.ru>
/mjt
---
V2: Collected reviewed-by tags
This change/comment suggested by "Michael Tokarev <m...@tls.msk.ru>" came
a bit late at that time, let's update it together in these minor set
this time.
---
migration/colo.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/migration/colo.c b/migration/colo.c
index 991806c06a..1b6d9da1c8 100644
--- a/migration/colo.c
+++ b/migration/colo.c
@@ -838,12 +838,11 @@ static void *colo_process_incoming_thread(void *opaque)
/* Make sure all file formats throw away their mutable metadata */
bql_lock();
bdrv_activate_all(&local_err);
+ bql_unlock();
if (local_err) {
- bql_unlock();
error_report_err(local_err);
return NULL;
}
- bql_unlock();
failover_init_state();