| Message ID | 20200824122051.99432-1-cohuck@redhat.com |
|---|---|
| State | New |
| Headers | show |
| Series | virtio: add Virtio*BusClass sizes | expand |
On Mon, 24 Aug 2020 14:20:51 +0200 Cornelia Huck <cohuck@redhat.com> wrote: > Both VirtioPCIBusClass and VirtioCcwBusClass are typedefs of > VirtioBusClass, but set .class_size in the TypeInfo anyway > to be safe if that changes in the future. > > Reported-by: Eduardo Habkost <ehabkost@redhat.com> > Signed-off-by: Cornelia Huck <cohuck@redhat.com> Reviewed-by: Halil Pasic <pasic@linux.ibm.com> > --- > hw/s390x/virtio-ccw.c | 1 + > hw/virtio/virtio-pci.c | 1 + > 2 files changed, 2 insertions(+) > > diff --git a/hw/s390x/virtio-ccw.c b/hw/s390x/virtio-ccw.c > index 0e602702971b..8feb3451a098 100644 > --- a/hw/s390x/virtio-ccw.c > +++ b/hw/s390x/virtio-ccw.c > @@ -1237,6 +1237,7 @@ static const TypeInfo virtio_ccw_bus_info = { > .name = TYPE_VIRTIO_CCW_BUS, > .parent = TYPE_VIRTIO_BUS, > .instance_size = sizeof(VirtioCcwBusState), > + .class_size = sizeof(VirtioCcwBusClass), > .class_init = virtio_ccw_bus_class_init, > }; > > diff --git a/hw/virtio/virtio-pci.c b/hw/virtio/virtio-pci.c > index ccdf54e81c78..45e7b65a8d24 100644 > --- a/hw/virtio/virtio-pci.c > +++ b/hw/virtio/virtio-pci.c > @@ -2101,6 +2101,7 @@ static const TypeInfo virtio_pci_bus_info = { > .name = TYPE_VIRTIO_PCI_BUS, > .parent = TYPE_VIRTIO_BUS, > .instance_size = sizeof(VirtioPCIBusState), > + .class_size = sizeof(VirtioPCIBusClass), > .class_init = virtio_pci_bus_class_init, > }; >
diff --git a/hw/s390x/virtio-ccw.c b/hw/s390x/virtio-ccw.c index 0e602702971b..8feb3451a098 100644 --- a/hw/s390x/virtio-ccw.c +++ b/hw/s390x/virtio-ccw.c @@ -1237,6 +1237,7 @@ static const TypeInfo virtio_ccw_bus_info = { .name = TYPE_VIRTIO_CCW_BUS, .parent = TYPE_VIRTIO_BUS, .instance_size = sizeof(VirtioCcwBusState), + .class_size = sizeof(VirtioCcwBusClass), .class_init = virtio_ccw_bus_class_init, }; diff --git a/hw/virtio/virtio-pci.c b/hw/virtio/virtio-pci.c index ccdf54e81c78..45e7b65a8d24 100644 --- a/hw/virtio/virtio-pci.c +++ b/hw/virtio/virtio-pci.c @@ -2101,6 +2101,7 @@ static const TypeInfo virtio_pci_bus_info = { .name = TYPE_VIRTIO_PCI_BUS, .parent = TYPE_VIRTIO_BUS, .instance_size = sizeof(VirtioPCIBusState), + .class_size = sizeof(VirtioPCIBusClass), .class_init = virtio_pci_bus_class_init, };
Both VirtioPCIBusClass and VirtioCcwBusClass are typedefs of VirtioBusClass, but set .class_size in the TypeInfo anyway to be safe if that changes in the future. Reported-by: Eduardo Habkost <ehabkost@redhat.com> Signed-off-by: Cornelia Huck <cohuck@redhat.com> --- hw/s390x/virtio-ccw.c | 1 + hw/virtio/virtio-pci.c | 1 + 2 files changed, 2 insertions(+)