commit 9caa6f3dbe20f2c506df6698386fce941fc6238a
author Paolo Bonzini <pbonzini@redhat.com> Mon Jun 05 14:39:07 2017 +0200
committer Fam Zheng <famz@redhat.com> Fri Jun 16 07:55:00 2017 +0800
tree 0d2cceb60dbfe4f1d01cef4c23f69edbabcd7dac
parent 39c1b4254e6200519d747f6cb8c888004e2446e2

block: split BlockAcctStats creation and setup

block_acct_destroy is called unconditionally in blk_delete, but there is
no BlockAcctStats function that is called unconditionally in blk_new.
Split block_acct_init in two, so that it will be possible to create a
QemuMutex in block_acct_init and destroy it in block_acct_cleanup.

Cc: Alberto Garcia <berto@igalia.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Message-Id: <20170605123908.18777-19-pbonzini@redhat.com>
Reviewed-by: Alberto Garcia <berto@igalia.com>
Signed-off-by: Fam Zheng <famz@redhat.com>

blockdev.c

diff --git a/blockdev.c b/blockdev.c
index 39f289b..39d6b97 100644
--- a/blockdev.c
+++ b/blockdev.c
@@ -595,7 +595,7 @@
             autostart = 0;
         }
 
-        block_acct_init(blk_get_stats(blk), account_invalid, account_failed);
+        block_acct_setup(blk_get_stats(blk), account_invalid, account_failed);
 
         if (!parse_stats_intervals(blk_get_stats(blk), interval_list, errp)) {
             blk_unref(blk);