On (Thu) 19 May 2011 [16:05:39], Markus Armbruster wrote:
> Amit Shah <amit.s...@redhat.com> writes:
>
> > On (Thu) 19 May 2011 [13:37:16], Markus Armbruster wrote:
> >> diff --git a/hw/virtio-serial-bus.c b/hw/virtio-serial-bus.c
> >> index bd3121e..a7d6b2b 100644
> >> --- a/hw/virtio-serial-bus.c
> >> +++ b/hw/virtio-serial-bus.c
> >> @@ -642,6 +642,11 @@ static struct BusInfo virtser_bus_info = {
> >> .name = "virtio-serial-bus",
> >> .size = sizeof(VirtIOSerialBus),
> >> .print_dev = virtser_bus_dev_print,
> >> + .props = (Property[]) {
> >> + DEFINE_PROP_UINT32("nr", VirtIOSerialPort, id,
> >> VIRTIO_CONSOLE_BAD_ID),
> >> + DEFINE_PROP_STRING("name", VirtIOSerialPort, name),
> >> + DEFINE_PROP_END_OF_LIST()
> >> + }
> >
> > 'name' isn't a must-have for all devices, but this is change OK.
>
> The property is a must have, even though its value may remain null.
> Correct?
Ah; must have for the device (ie qdev purposes)? Yes. It's not a
must-have for guest-host communication, that's what I meant.
Amit
