[PULL 3/9] migration/colo: Tidy up bql_unlock() around bdrv_activate_all()

[Fabiano Rosas]

From: Li Zhijian <lizhij...@fujitsu.com>

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>
[fixed mangled author email address]
Signed-off-by: Fabiano Rosas <faro...@suse.de>
---
 migration/colo.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/migration/colo.c b/migration/colo.c
index ca37b932ac..f96c2ee069 100644
--- a/migration/colo.c
+++ b/migration/colo.c
@@ -837,12 +837,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();
 
-- 
2.35.3