[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 21/32] char: clean up remaining chardevs when lea
From: |
Andrew Jones |
Subject: |
Re: [Qemu-devel] [PULL 21/32] char: clean up remaining chardevs when leaving |
Date: |
Fri, 1 Jul 2016 14:34:13 +0200 |
User-agent: |
Mutt/1.6.0.1 (2016-04-01) |
On Tue, Jun 28, 2016 at 07:33:51PM +0200, Paolo Bonzini wrote:
> From: Marc-André Lureau <address@hidden>
>
> This helps to remove various chardev resources leaks when leaving qemu.
>
> Signed-off-by: Marc-André Lureau <address@hidden>
> Message-Id: <address@hidden>
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
> qemu-char.c | 11 +++++++++++
> 1 file changed, 11 insertions(+)
>
> diff --git a/qemu-char.c b/qemu-char.c
> index 84f49ac..016badb 100644
> --- a/qemu-char.c
> +++ b/qemu-char.c
> @@ -4549,6 +4549,15 @@ void qmp_chardev_remove(const char *id, Error **errp)
> qemu_chr_delete(chr);
> }
>
> +static void qemu_chr_cleanup(void)
> +{
> + CharDriverState *chr;
> +
> + QTAILQ_FOREACH(chr, &chardevs, next) {
> + qemu_chr_delete(chr);
> + }
> +}
> +
> static void register_types(void)
> {
> register_char_driver("null", CHARDEV_BACKEND_KIND_NULL, NULL,
> @@ -4595,6 +4604,8 @@ static void register_types(void)
> * is specified
> */
> qemu_add_machine_init_done_notifier(&muxes_realize_notify);
> +
> + atexit(qemu_chr_cleanup);
> }
>
> type_init(register_types);
> --
> 2.7.4
>
>
>
With this patch, I now get
qemu: qemu_mutex_destroy: Device or resource busy
and a SIGABRT when my arm tcg kvm-unit-tests guest exits
(I haven't tried any other guest types yet)
Thanks,
drew
- Re: [Qemu-devel] [PULL 21/32] char: clean up remaining chardevs when leaving,
Andrew Jones <=