[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/7] virtio-console: Also throttle when less was
From: |
Amit Shah |
Subject: |
Re: [Qemu-devel] [PATCH 1/7] virtio-console: Also throttle when less was written then requested |
Date: |
Fri, 29 Mar 2013 15:01:14 +0530 |
On (Thu) 28 Mar 2013 [14:28:11], Hans de Goede wrote:
> This is necessary so that we get properly woken up to write the rest.
>
> Signed-off-by: Hans de Goede <address@hidden>
> Acked-by: Amit Shah <address@hidden>
> ---
> hw/virtio-console.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/hw/virtio-console.c b/hw/virtio-console.c
> index 284180f..015947c 100644
> --- a/hw/virtio-console.c
> +++ b/hw/virtio-console.c
> @@ -47,7 +47,7 @@ static ssize_t flush_buf(VirtIOSerialPort *port, const
> uint8_t *buf, size_t len)
> ret = qemu_chr_fe_write(vcon->chr, buf, len);
> trace_virtio_console_flush_buf(port->id, len, ret);
>
> - if (ret <= 0) {
> + if (ret < len) {
Oops, there's a conversion bug here: len is size_t, and ret is
ssize_t. Both need to be the same type.
Since we're not returning negative, ret should be changed to size_t.
The function signature changes too, but that's alright.
> VirtIOSerialPortClass *k = VIRTIO_SERIAL_PORT_GET_CLASS(port);
>
> /*
> @@ -56,7 +56,8 @@ static ssize_t flush_buf(VirtIOSerialPort *port, const
> uint8_t *buf, size_t len)
> * we had a finer-grained message, like -EPIPE, we could close
> * this connection.
> */
> - ret = 0;
> + if (ret < 0)
> + ret = 0;
And there need to be braces here.
Amit
- [Qemu-devel] [PATCH 0/7] Spice / usb-redir chardev flowcontrol patches v3, Hans de Goede, 2013/03/28
- [Qemu-devel] [PATCH 1/7] virtio-console: Also throttle when less was written then requested, Hans de Goede, 2013/03/28
- Re: [Qemu-devel] [PATCH 1/7] virtio-console: Also throttle when less was written then requested,
Amit Shah <=
- [Qemu-devel] [PATCH 5/7] spice-qemu-char: Remove intermediate buffer, Hans de Goede, 2013/03/28
- [Qemu-devel] [PATCH 3/7] spice-qemu-char: Remove #ifdef-ed code for old spice-server compat, Hans de Goede, 2013/03/28
- [Qemu-devel] [PATCH 4/7] spice-qemu-char: Add watch support, Hans de Goede, 2013/03/28
- [Qemu-devel] [PATCH 6/7] usb-redir: Add flow control support, Hans de Goede, 2013/03/28
- [Qemu-devel] [PATCH 2/7] virtio-console: Remove any pending watches on close, Hans de Goede, 2013/03/28
- [Qemu-devel] [PATCH 7/7] usb-serial: Remove double call to qemu_chr_add_handlers( NULL ), Hans de Goede, 2013/03/28