[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v6 1/6] virtio-console: Factor out common init betwe
From: |
Amit Shah |
Subject: |
[Qemu-devel] [PATCH v6 1/6] virtio-console: Factor out common init between console and generic ports |
Date: |
Tue, 4 May 2010 23:53:54 +0530 |
The initialisation for generic ports and console ports is similar.
Factor out the parts that are the same in a different function that can
be called from each of the initfns.
Signed-off-by: Amit Shah <address@hidden>
---
hw/virtio-console.c | 31 ++++++++++++++-----------------
1 files changed, 14 insertions(+), 17 deletions(-)
diff --git a/hw/virtio-console.c b/hw/virtio-console.c
index caea11f..d7fe68b 100644
--- a/hw/virtio-console.c
+++ b/hw/virtio-console.c
@@ -58,24 +58,28 @@ static void chr_event(void *opaque, int event)
}
}
-/* Virtio Console Ports */
-static int virtconsole_initfn(VirtIOSerialDevice *dev)
+static int generic_port_init(VirtConsole *vcon, VirtIOSerialDevice *dev)
{
- VirtIOSerialPort *port = DO_UPCAST(VirtIOSerialPort, dev, &dev->qdev);
- VirtConsole *vcon = DO_UPCAST(VirtConsole, port, port);
-
- port->info = dev->info;
-
- port->is_console = true;
+ vcon->port.info = dev->info;
if (vcon->chr) {
qemu_chr_add_handlers(vcon->chr, chr_can_read, chr_read, chr_event,
vcon);
- port->info->have_data = flush_buf;
+ vcon->port.info->have_data = flush_buf;
}
return 0;
}
+/* Virtio Console Ports */
+static int virtconsole_initfn(VirtIOSerialDevice *dev)
+{
+ VirtIOSerialPort *port = DO_UPCAST(VirtIOSerialPort, dev, &dev->qdev);
+ VirtConsole *vcon = DO_UPCAST(VirtConsole, port, port);
+
+ port->is_console = true;
+ return generic_port_init(vcon, dev);
+}
+
static int virtconsole_exitfn(VirtIOSerialDevice *dev)
{
VirtIOSerialPort *port = DO_UPCAST(VirtIOSerialPort, dev, &dev->qdev);
@@ -115,14 +119,7 @@ static int virtserialport_initfn(VirtIOSerialDevice *dev)
VirtIOSerialPort *port = DO_UPCAST(VirtIOSerialPort, dev, &dev->qdev);
VirtConsole *vcon = DO_UPCAST(VirtConsole, port, port);
- port->info = dev->info;
-
- if (vcon->chr) {
- qemu_chr_add_handlers(vcon->chr, chr_can_read, chr_read, chr_event,
- vcon);
- port->info->have_data = flush_buf;
- }
- return 0;
+ return generic_port_init(vcon, dev);
}
static VirtIOSerialPortInfo virtserialport_info = {
--
1.6.2.5
- [Qemu-devel] [PATCH v6 0/6] char: non-blocking writes, virtio-console flow control, Amit Shah, 2010/05/04
- [Qemu-devel] [PATCH v6 1/6] virtio-console: Factor out common init between console and generic ports,
Amit Shah <=
- [Qemu-devel] [PATCH v6 2/6] char: Add a QemuChrHandlers struct to initialise chardev handlers, Amit Shah, 2010/05/04
- [Qemu-devel] [PATCH v6 3/6] char: Let writers know how much data was written in case of errors, Amit Shah, 2010/05/04
- [Qemu-devel] [PATCH v6 4/6] char: Add qemu_chr_write_nb() for nonblocking writes, Amit Shah, 2010/05/04
- [Qemu-devel] [PATCH v6 5/6] char: unix/tcp: Add a non-blocking write handler, Amit Shah, 2010/05/04
- [Qemu-devel] [PATCH v6 6/6] virtio-console: Throttle virtio-serial-bus if we can't consume any more guest data, Amit Shah, 2010/05/04
- [Qemu-devel] Re: [PATCH v6 6/6] virtio-console: Throttle virtio-serial-bus if we can't consume any more guest data, Juan Quintela, 2010/05/04
- [Qemu-devel] Re: [PATCH v6 5/6] char: unix/tcp: Add a non-blocking write handler, Juan Quintela, 2010/05/04
- [Qemu-devel] Re: [PATCH v6 5/6] char: unix/tcp: Add a non-blocking write handler, Amit Shah, 2010/05/04
[Qemu-devel] Re: [PATCH v6 0/6] char: non-blocking writes, virtio-console flow control, Gerd Hoffmann, 2010/05/04