vl: remove unnecessary duplicate call to tpm_cleanup
authorPaolo Bonzini <pbonzini@redhat.com>
Thu, 16 Jun 2016 12:25:04 +0000 (14:25 +0200)
committerMichael Tokarev <mjt@gandalf>
Tue, 13 Sep 2016 15:12:34 +0000 (18:12 +0300)
tpm_cleanup is called from main() and also registered with atexit from
tpm_init.  The function only visits the tpm_backends linked list, and the
atexit registration happens right after tpm_init fills in the list from
-tpmdev options.  Therefore, the direct call is unnecessary.  Remove it.

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
vl.c

diff --git a/vl.c b/vl.c
index ee557a1..82cf85b 100644 (file)
--- a/vl.c
+++ b/vl.c
@@ -4620,9 +4620,6 @@ int main(int argc, char **argv, char **envp)
     bdrv_close_all();
     pause_all_vcpus();
     res_free();
-#ifdef CONFIG_TPM
-    tpm_cleanup();
-#endif
 
     /* vhost-user must be cleaned up before chardevs.  */
     net_cleanup();