Re: [PATCH] virtio: add Virtio*BusClass sizes

[Halil Pasic <pasic@linux.ibm.com>]

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,
>  };
>