Re: [Qemu-devel] [PATCH v10 9/9] tpm: Move tpm_cleanup() to right place

[Stefan Berger <stefanb@linux.vnet.ibm.com>]

On 09/29/2017 07:10 AM, Amarnath Valluri wrote:
> As Emulator TPM backend uses chardev, tpm cleanup should happen before chardev
> similar to other vhost-users.
>
> Signed-off-by: Amarnath Valluri <amarnath.valluri@intel.com>

Reviewed-by: Stefan Berger <stefanb@linux.vnet.ibm.com>

> ---
>   tpm.c | 1 -
>   vl.c  | 1 +
>   2 files changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tpm.c b/tpm.c
> index 3b8c7ed..3122227 100644
> --- a/tpm.c
> +++ b/tpm.c
> @@ -172,7 +172,6 @@ int tpm_init(void)
>           return -1;
>       }
>
> -    atexit(tpm_cleanup);
>       return 0;
>   }
>
> diff --git a/vl.c b/vl.c
> index 9bb5058..ed86209 100644
> --- a/vl.c
> +++ b/vl.c
> @@ -4893,6 +4893,7 @@ int main(int argc, char **argv, char **envp)
>       res_free();
>
>       /* vhost-user must be cleaned up before chardevs.  */
> +    tpm_cleanup();
>       net_cleanup();
>       audio_cleanup();
>       monitor_cleanup();