[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v6 15/18] virtio-serial: Apps should consume all dat
From: |
Amit Shah |
Subject: |
[Qemu-devel] [PATCH v6 15/18] virtio-serial: Apps should consume all data that guest sends out / Fix virtio api abuse |
Date: |
Tue, 27 Apr 2010 18:04:09 +0530 |
We cannot indicate to the guest how much data was consumed by an app for
out_bufs. So we just have to assume the apps will consume all the data
that are handed over to them.
Fix the virtio api abuse in control_out() and handle_output().
Signed-off-by: Amit Shah <address@hidden>
---
hw/virtio-console.c | 7 ++-----
hw/virtio-serial-bus.c | 6 +++---
hw/virtio-serial.h | 6 +++---
3 files changed, 8 insertions(+), 11 deletions(-)
diff --git a/hw/virtio-console.c b/hw/virtio-console.c
index bbbb6b8..caea11f 100644
--- a/hw/virtio-console.c
+++ b/hw/virtio-console.c
@@ -20,14 +20,11 @@ typedef struct VirtConsole {
/* Callback function that's called when the guest sends us data */
-static size_t flush_buf(VirtIOSerialPort *port, const uint8_t *buf, size_t len)
+static void flush_buf(VirtIOSerialPort *port, const uint8_t *buf, size_t len)
{
VirtConsole *vcon = DO_UPCAST(VirtConsole, port, port);
- ssize_t ret;
- ret = qemu_chr_write(vcon->chr, buf, len);
-
- return ret < 0 ? 0 : ret;
+ qemu_chr_write(vcon->chr, buf, len);
}
/* Readiness of the guest to accept data on a port */
diff --git a/hw/virtio-serial-bus.c b/hw/virtio-serial-bus.c
index 3053a35..ad44127 100644
--- a/hw/virtio-serial-bus.c
+++ b/hw/virtio-serial-bus.c
@@ -335,7 +335,7 @@ static void control_out(VirtIODevice *vdev, VirtQueue *vq)
copied = iov_to_buf(elem.out_sg, elem.out_num, buf, 0, len);
handle_control_message(vser, buf, copied);
- virtqueue_push(vq, &elem, copied);
+ virtqueue_push(vq, &elem, 0);
}
qemu_free(buf);
virtio_notify(vdev, vq);
@@ -379,11 +379,11 @@ static void handle_output(VirtIODevice *vdev, VirtQueue
*vq)
buf = qemu_malloc(buf_size);
ret = iov_to_buf(elem.out_sg, elem.out_num, buf, 0, buf_size);
- ret = port->info->have_data(port, buf, ret);
+ port->info->have_data(port, buf, ret);
qemu_free(buf);
next_buf:
- virtqueue_push(vq, &elem, ret);
+ virtqueue_push(vq, &elem, 0);
}
virtio_notify(vdev, vq);
}
diff --git a/hw/virtio-serial.h b/hw/virtio-serial.h
index f023873..62d76a2 100644
--- a/hw/virtio-serial.h
+++ b/hw/virtio-serial.h
@@ -136,10 +136,10 @@ struct VirtIOSerialPortInfo {
/*
* Guest wrote some data to the port. This data is handed over to
- * the app via this callback. The app should return the number of
- * bytes it successfully consumed.
+ * the app via this callback. The app is supposed to consume all
+ * the data that is presented to it.
*/
- size_t (*have_data)(VirtIOSerialPort *port, const uint8_t *buf, size_t
len);
+ void (*have_data)(VirtIOSerialPort *port, const uint8_t *buf, size_t len);
};
/* Interface to the virtio-serial bus */
--
1.6.2.5
- [Qemu-devel] [PATCH v6 05/18] virtio-serial: Use control messages to notify guest of new ports, (continued)
- [Qemu-devel] [PATCH v6 05/18] virtio-serial: Use control messages to notify guest of new ports, Amit Shah, 2010/04/27
- [Qemu-devel] [PATCH v6 06/18] virtio-serial: whitespace: match surrounding code, Amit Shah, 2010/04/27
- [Qemu-devel] [PATCH v6 07/18] virtio-serial: Remove redundant check for 0-sized write request, Amit Shah, 2010/04/27
- [Qemu-devel] [PATCH v6 08/18] virtio-serial: Update copyright year to 2010, Amit Shah, 2010/04/27
- [Qemu-devel] [PATCH v6 09/18] virtio-serial: Propagate errors in initialising ports / devices in guest, Amit Shah, 2010/04/27
- [Qemu-devel] [PATCH v6 10/18] virtio-serial: Send out guest data to ports only if port is opened, Amit Shah, 2010/04/27
- [Qemu-devel] [PATCH v6 11/18] iov: Introduce a new file for helpers around iovs, add iov_from_buf(), Amit Shah, 2010/04/27
- [Qemu-devel] [PATCH v6 12/18] iov: Add iov_to_buf and iov_size helpers, Amit Shah, 2010/04/27
- [Qemu-devel] [PATCH v6 13/18] virtio-serial: Handle scatter-gather buffers for control messages, Amit Shah, 2010/04/27
- [Qemu-devel] [PATCH v6 14/18] virtio-serial: Handle scatter/gather input from the guest, Amit Shah, 2010/04/27
- [Qemu-devel] [PATCH v6 15/18] virtio-serial: Apps should consume all data that guest sends out / Fix virtio api abuse,
Amit Shah <=
- [Qemu-devel] [PATCH v6 16/18] virtio-serial: Discard data that guest sends us when ports aren't connected, Amit Shah, 2010/04/27
- [Qemu-devel] [PATCH v6 17/18] virtio-serial: Implement flow control for individual ports, Amit Shah, 2010/04/27
- [Qemu-devel] [PATCH v6 18/18] virtio-serial-bus: wake up iothread upon guest read notification, Amit Shah, 2010/04/27
- [Qemu-devel] Re: [PATCH v6 18/18] virtio-serial-bus: wake up iothread upon guest read notification, Anthony Liguori, 2010/04/27
- [Qemu-devel] Re: [PATCH v6 18/18] virtio-serial-bus: wake up iothread upon guest read notification, Marcelo Tosatti, 2010/04/27
- [Qemu-devel] Re: [PATCH v6 18/18] virtio-serial-bus: wake up iothread upon guest read notification, Anthony Liguori, 2010/04/27
- [Qemu-devel] Re: [PATCH v6 18/18] virtio-serial-bus: wake up iothread upon guest read notification, Amit Shah, 2010/04/28
- [Qemu-devel] Re: [PATCH v6 18/18] virtio-serial-bus: wake up iothread upon guest read notification, Anthony Liguori, 2010/04/28
- [Qemu-devel] Re: [PATCH v6 18/18] virtio-serial-bus: wake up iothread upon guest read notification, Amit Shah, 2010/04/28
- [Qemu-devel] Re: [PATCH v6 05/18] virtio-serial: Use control messages to notify guest of new ports, Anthony Liguori, 2010/04/27