diff options
author | Andreas Färber <afaerber@suse.de> | 2014-03-13 17:12:53 +0100 |
---|---|---|
committer | Andreas Färber <afaerber@suse.de> | 2014-03-13 21:12:07 +0100 |
commit | be21c336162f5f4269131faffb1c6461cf57b74a (patch) | |
tree | bb23ee9148fdc4f8dc117eda1684a0ba91353b90 /hw/char/virtio-console.c | |
parent | bbbd67f0ccdba93702e58879997c1d2ca67311b1 (diff) | |
download | qemu-be21c336162f5f4269131faffb1c6461cf57b74a.zip qemu-be21c336162f5f4269131faffb1c6461cf57b74a.tar.gz qemu-be21c336162f5f4269131faffb1c6461cf57b74a.tar.bz2 |
virtio-console: Fix VIRTIO_CONSOLE() cast macro
Commit 0399a3819b27083ba69b88a9baa9025facab85bd (virtio-console: QOM
cast cleanup for VirtConsole) broke virtserialport since it shares
functions and state struct with virtconsole. Let virtconsole inherit
from virtserialport, and use virtserialport type for casting.
Note that virtio-serial-port is the abstract base type in
virtio-serial-bus.c, whereas virtserialport is the user-instantiatable
type in virtio-console.c. Therefore using TYPE_VIRTIO_CONSOLE_SERIAL_PORT.
Reported-by: Richard W.M. Jones <rjones@redhat.com>
Acked-by: Richard W.M. Jones <rjones@redhat.com>
Signed-off-by: Andreas Färber <afaerber@suse.de>
Diffstat (limited to 'hw/char/virtio-console.c')
-rw-r--r-- | hw/char/virtio-console.c | 24 |
1 files changed, 6 insertions, 18 deletions
diff --git a/hw/char/virtio-console.c b/hw/char/virtio-console.c index ffd29a8..6c8be0f 100644 --- a/hw/char/virtio-console.c +++ b/hw/char/virtio-console.c @@ -15,9 +15,9 @@ #include "trace.h" #include "hw/virtio/virtio-serial.h" -#define TYPE_VIRTIO_CONSOLE "virtconsole" +#define TYPE_VIRTIO_CONSOLE_SERIAL_PORT "virtserialport" #define VIRTIO_CONSOLE(obj) \ - OBJECT_CHECK(VirtConsole, (obj), TYPE_VIRTIO_CONSOLE) + OBJECT_CHECK(VirtConsole, (obj), TYPE_VIRTIO_CONSOLE_SERIAL_PORT) typedef struct VirtConsole { VirtIOSerialPort parent_obj; @@ -154,28 +154,16 @@ static void virtconsole_unrealize(DeviceState *dev, Error **errp) } } -static Property virtconsole_properties[] = { - DEFINE_PROP_CHR("chardev", VirtConsole, chr), - DEFINE_PROP_END_OF_LIST(), -}; - static void virtconsole_class_init(ObjectClass *klass, void *data) { - DeviceClass *dc = DEVICE_CLASS(klass); VirtIOSerialPortClass *k = VIRTIO_SERIAL_PORT_CLASS(klass); k->is_console = true; - k->realize = virtconsole_realize; - k->unrealize = virtconsole_unrealize; - k->have_data = flush_buf; - k->set_guest_connected = set_guest_connected; - dc->props = virtconsole_properties; } static const TypeInfo virtconsole_info = { - .name = TYPE_VIRTIO_CONSOLE, - .parent = TYPE_VIRTIO_SERIAL_PORT, - .instance_size = sizeof(VirtConsole), + .name = "virtconsole", + .parent = TYPE_VIRTIO_CONSOLE_SERIAL_PORT, .class_init = virtconsole_class_init, }; @@ -197,7 +185,7 @@ static void virtserialport_class_init(ObjectClass *klass, void *data) } static const TypeInfo virtserialport_info = { - .name = "virtserialport", + .name = TYPE_VIRTIO_CONSOLE_SERIAL_PORT, .parent = TYPE_VIRTIO_SERIAL_PORT, .instance_size = sizeof(VirtConsole), .class_init = virtserialport_class_init, @@ -205,8 +193,8 @@ static const TypeInfo virtserialport_info = { static void virtconsole_register_types(void) { - type_register_static(&virtconsole_info); type_register_static(&virtserialport_info); + type_register_static(&virtconsole_info); } type_init(virtconsole_register_types) |