Re: [PATCH 1/1] virtio: console: tell host of open ports after resume from s3/s4

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Apr 25, 2012 at 02:40:39PM +0530, Amit Shah wrote:
> If a port was open before going into one of the sleep states, the port
> can continue normal operation after restore.  However, the host has to
> be told that the guest side of the connection is open to restore
> pre-suspend state.
> 
> This wasn't noticed so far due to a bug in qemu that was fixed recently
> (which marked the guest-side connection as always open).
> 
> CC: stable@xxxxxxxxxxxxxxx   # Only for 3.3
> 
> Signed-off-by: Amit Shah <amit.shah@xxxxxxxxxx>

Applied and queued for the next 3.4 pull, thanks.

> ---
>  drivers/char/virtio_console.c |    7 +++++++
>  1 files changed, 7 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/char/virtio_console.c b/drivers/char/virtio_console.c
> index ddf86b6..cdf2f54 100644
> --- a/drivers/char/virtio_console.c
> +++ b/drivers/char/virtio_console.c
> @@ -1895,6 +1895,13 @@ static int virtcons_restore(struct virtio_device *vdev)
>  
>  		/* Get port open/close status on the host */
>  		send_control_msg(port, VIRTIO_CONSOLE_PORT_READY, 1);
> +
> +		/*
> +		 * If a port was open at the time of suspending, we
> +		 * have to let the host know that it's still open.
> +		 */
> +		if (port->guest_connected)
> +			send_control_msg(port, VIRTIO_CONSOLE_PORT_OPEN, 1);
>  	}
>  	return 0;
>  }
> -- 
> 1.7.7.6
_______________________________________________
Virtualization mailing list
Virtualization@xxxxxxxxxxxxxxxxxxxxxxxxxx
https://lists.linuxfoundation.org/mailman/listinfo/virtualization


[Index of Archives]     [KVM Development]     [Libvirt Development]     [Libvirt Users]     [CentOS Virtualization]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux