[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/2] virtio-console: no need to remove char hand
From: |
Amit Shah |
Subject: |
Re: [Qemu-devel] [PATCH 2/2] virtio-console: no need to remove char handlers explicitly |
Date: |
Fri, 13 Jan 2012 15:28:05 +0530 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On (Fri) 13 Jan 2012 [15:24:59], Amit Shah wrote:
> qdev is now equipped (thanks to the last commit) to disassociate
> chardevs from the qdev devices on the devices going away. So doing it
> in the virtio-console driver is not necessary.
>
> Since that was the only thing being done in the qdev exit method, drop
> it entirely.
Commit message is correct, patch is not. Updated patch coming soon..
> diff --git a/hw/virtio-console.c b/hw/virtio-console.c
> index 73d866a..9275fd9 100644
> --- a/hw/virtio-console.c
> +++ b/hw/virtio-console.c
> @@ -127,16 +127,6 @@ static int virtconsole_initfn(VirtIOSerialPort *port)
>
> static int virtconsole_exitfn(VirtIOSerialPort *port)
> {
> - VirtConsole *vcon = DO_UPCAST(VirtConsole, port, port);
> -
> - if (vcon->chr) {
> - /*
> - * Instead of closing the chardev, free it so it can be used
> - * for other purposes.
> - */
> - qemu_chr_add_handlers(vcon->chr, NULL, NULL, NULL, NULL);
> - }
> -
> return 0;
> }
Amit