[prev in list] [next in list] [prev in thread] [next in thread] 

List:       qemu-s390x
Subject:    Re: [PATCH] Mark remaining global TypeInfo instances as const
From:       Bernhard Beschow <shentey () gmail ! com>
Date:       2022-01-17 15:08:09
Message-ID: CAG4p6K7AZ4iixONiX--cnYWJvhyjwjiBwZSFLPhLz2+ejMTT_g () mail ! gmail ! com
[Download RAW message or body]

Hi Philippe,

Thanks for your review. I've now sent v2 of the patch with your comments
addressed.

BTW: I almost missed your review because apparently I wasn't in the CC.
That is also the reason I'm not responding to your mail directly.

Bernhard.

On Thu, Jan 13, 2022 at 6:10 PM Bernhard Beschow <shentey@gmail.com> wrote:

> More than 1k of TypeInfo instances are already marked as const. Mark the
> remaining ones, too.
>
> Signed-off-by: Bernhard Beschow <shentey@gmail.com>
> ---
>  hw/core/generic-loader.c   | 2 +-
>  hw/core/guest-loader.c     | 2 +-
>  hw/display/bcm2835_fb.c    | 2 +-
>  hw/display/i2c-ddc.c       | 2 +-
>  hw/display/macfb.c         | 4 ++--
>  hw/display/virtio-vga.c    | 2 +-
>  hw/dma/bcm2835_dma.c       | 2 +-
>  hw/i386/pc_piix.c          | 2 +-
>  hw/i386/sgx-epc.c          | 2 +-
>  hw/intc/bcm2835_ic.c       | 2 +-
>  hw/intc/bcm2836_control.c  | 2 +-
>  hw/ipmi/ipmi.c             | 4 ++--
>  hw/mem/nvdimm.c            | 2 +-
>  hw/mem/pc-dimm.c           | 2 +-
>  hw/misc/bcm2835_mbox.c     | 2 +-
>  hw/misc/bcm2835_powermgt.c | 2 +-
>  hw/misc/bcm2835_property.c | 2 +-
>  hw/misc/bcm2835_rng.c      | 2 +-
>  hw/misc/pvpanic-isa.c      | 2 +-
>  hw/misc/pvpanic-pci.c      | 2 +-
>  hw/net/fsl_etsec/etsec.c   | 2 +-
>  hw/ppc/prep_systemio.c     | 2 +-
>  hw/ppc/spapr_iommu.c       | 2 +-
>  hw/s390x/s390-pci-bus.c    | 2 +-
>  hw/s390x/sclp.c            | 2 +-
>  hw/s390x/tod-kvm.c         | 2 +-
>  hw/s390x/tod-tcg.c         | 2 +-
>  hw/s390x/tod.c             | 2 +-
>  hw/scsi/lsi53c895a.c       | 2 +-
>  hw/sd/allwinner-sdhost.c   | 2 +-
>  hw/sd/aspeed_sdhci.c       | 2 +-
>  hw/sd/bcm2835_sdhost.c     | 2 +-
>  hw/sd/cadence_sdhci.c      | 2 +-
>  hw/sd/npcm7xx_sdhci.c      | 2 +-
>  hw/usb/dev-mtp.c           | 2 +-
>  hw/usb/host-libusb.c       | 2 +-
>  hw/vfio/igd.c              | 2 +-
>  hw/virtio/virtio-pmem.c    | 2 +-
>  qom/object.c               | 4 ++--
>  39 files changed, 42 insertions(+), 42 deletions(-)
>
> diff --git a/hw/core/generic-loader.c b/hw/core/generic-loader.c
> index 9a24ffb880..eaafc416f4 100644
> --- a/hw/core/generic-loader.c
> +++ b/hw/core/generic-loader.c
> @@ -207,7 +207,7 @@ static void generic_loader_class_init(ObjectClass
> *klass, void *data)
>      set_bit(DEVICE_CATEGORY_MISC, dc->categories);
>  }
>
> -static TypeInfo generic_loader_info = {
> +static const TypeInfo generic_loader_info = {
>      .name = TYPE_GENERIC_LOADER,
>      .parent = TYPE_DEVICE,
>      .instance_size = sizeof(GenericLoaderState),
> diff --git a/hw/core/guest-loader.c b/hw/core/guest-loader.c
> index d3f9d1a06e..391c875a29 100644
> --- a/hw/core/guest-loader.c
> +++ b/hw/core/guest-loader.c
> @@ -129,7 +129,7 @@ static void guest_loader_class_init(ObjectClass
> *klass, void *data)
>      set_bit(DEVICE_CATEGORY_MISC, dc->categories);
>  }
>
> -static TypeInfo guest_loader_info = {
> +static const TypeInfo guest_loader_info = {
>      .name = TYPE_GUEST_LOADER,
>      .parent = TYPE_DEVICE,
>      .instance_size = sizeof(GuestLoaderState),
> diff --git a/hw/display/bcm2835_fb.c b/hw/display/bcm2835_fb.c
> index 2be77bdd3a..088fc3d51c 100644
> --- a/hw/display/bcm2835_fb.c
> +++ b/hw/display/bcm2835_fb.c
> @@ -454,7 +454,7 @@ static void bcm2835_fb_class_init(ObjectClass *klass,
> void *data)
>      dc->vmsd = &vmstate_bcm2835_fb;
>  }
>
> -static TypeInfo bcm2835_fb_info = {
> +static const TypeInfo bcm2835_fb_info = {
>      .name          = TYPE_BCM2835_FB,
>      .parent        = TYPE_SYS_BUS_DEVICE,
>      .instance_size = sizeof(BCM2835FBState),
> diff --git a/hw/display/i2c-ddc.c b/hw/display/i2c-ddc.c
> index 13eb529fc1..146489518c 100644
> --- a/hw/display/i2c-ddc.c
> +++ b/hw/display/i2c-ddc.c
> @@ -113,7 +113,7 @@ static void i2c_ddc_class_init(ObjectClass *oc, void
> *data)
>      isc->send = i2c_ddc_tx;
>  }
>
> -static TypeInfo i2c_ddc_info = {
> +static const TypeInfo i2c_ddc_info = {
>      .name = TYPE_I2CDDC,
>      .parent = TYPE_I2C_SLAVE,
>      .instance_size = sizeof(I2CDDCState),
> diff --git a/hw/display/macfb.c b/hw/display/macfb.c
> index 4bd7c3ad6a..69c2ea2b6e 100644
> --- a/hw/display/macfb.c
> +++ b/hw/display/macfb.c
> @@ -783,14 +783,14 @@ static void macfb_nubus_class_init(ObjectClass
> *klass, void *data)
>      device_class_set_props(dc, macfb_nubus_properties);
>  }
>
> -static TypeInfo macfb_sysbus_info = {
> +static const TypeInfo macfb_sysbus_info = {
>      .name          = TYPE_MACFB,
>      .parent        = TYPE_SYS_BUS_DEVICE,
>      .instance_size = sizeof(MacfbSysBusState),
>      .class_init    = macfb_sysbus_class_init,
>  };
>
> -static TypeInfo macfb_nubus_info = {
> +static const TypeInfo macfb_nubus_info = {
>      .name          = TYPE_NUBUS_MACFB,
>      .parent        = TYPE_NUBUS_DEVICE,
>      .instance_size = sizeof(MacfbNubusState),
> diff --git a/hw/display/virtio-vga.c b/hw/display/virtio-vga.c
> index b23a75a04b..5a2f7a4540 100644
> --- a/hw/display/virtio-vga.c
> +++ b/hw/display/virtio-vga.c
> @@ -220,7 +220,7 @@ static void virtio_vga_base_class_init(ObjectClass
> *klass, void *data)
>                                     virtio_vga_set_big_endian_fb);
>  }
>
> -static TypeInfo virtio_vga_base_info = {
> +static const TypeInfo virtio_vga_base_info = {
>      .name          = TYPE_VIRTIO_VGA_BASE,
>      .parent        = TYPE_VIRTIO_PCI,
>      .instance_size = sizeof(VirtIOVGABase),
> diff --git a/hw/dma/bcm2835_dma.c b/hw/dma/bcm2835_dma.c
> index eb0002a2b9..5e9306110d 100644
> --- a/hw/dma/bcm2835_dma.c
> +++ b/hw/dma/bcm2835_dma.c
> @@ -394,7 +394,7 @@ static void bcm2835_dma_class_init(ObjectClass *klass,
> void *data)
>      dc->vmsd = &vmstate_bcm2835_dma;
>  }
>
> -static TypeInfo bcm2835_dma_info = {
> +static const TypeInfo bcm2835_dma_info = {
>      .name          = TYPE_BCM2835_DMA,
>      .parent        = TYPE_SYS_BUS_DEVICE,
>      .instance_size = sizeof(BCM2835DMAState),
> diff --git a/hw/i386/pc_piix.c b/hw/i386/pc_piix.c
> index 7c7790a5ce..f6557d3c21 100644
> --- a/hw/i386/pc_piix.c
> +++ b/hw/i386/pc_piix.c
> @@ -865,7 +865,7 @@ static void isa_bridge_class_init(ObjectClass *klass,
> void *data)
>      k->class_id     = PCI_CLASS_BRIDGE_ISA;
>  };
>
> -static TypeInfo isa_bridge_info = {
> +static const TypeInfo isa_bridge_info = {
>      .name          = "igd-passthrough-isa-bridge",
>      .parent        = TYPE_PCI_DEVICE,
>      .instance_size = sizeof(PCIDevice),
> diff --git a/hw/i386/sgx-epc.c b/hw/i386/sgx-epc.c
> index 96b2940d75..d664829d35 100644
> --- a/hw/i386/sgx-epc.c
> +++ b/hw/i386/sgx-epc.c
> @@ -167,7 +167,7 @@ static void sgx_epc_class_init(ObjectClass *oc, void
> *data)
>      mdc->fill_device_info = sgx_epc_md_fill_device_info;
>  }
>
> -static TypeInfo sgx_epc_info = {
> +static const TypeInfo sgx_epc_info = {
>      .name          = TYPE_SGX_EPC,
>      .parent        = TYPE_DEVICE,
>      .instance_size = sizeof(SGXEPCDevice),
> diff --git a/hw/intc/bcm2835_ic.c b/hw/intc/bcm2835_ic.c
> index 9000d995e8..4513fad16f 100644
> --- a/hw/intc/bcm2835_ic.c
> +++ b/hw/intc/bcm2835_ic.c
> @@ -227,7 +227,7 @@ static void bcm2835_ic_class_init(ObjectClass *klass,
> void *data)
>      dc->vmsd = &vmstate_bcm2835_ic;
>  }
>
> -static TypeInfo bcm2835_ic_info = {
> +static const TypeInfo bcm2835_ic_info = {
>      .name          = TYPE_BCM2835_IC,
>      .parent        = TYPE_SYS_BUS_DEVICE,
>      .instance_size = sizeof(BCM2835ICState),
> diff --git a/hw/intc/bcm2836_control.c b/hw/intc/bcm2836_control.c
> index 2ead76ffdc..b0589df188 100644
> --- a/hw/intc/bcm2836_control.c
> +++ b/hw/intc/bcm2836_control.c
> @@ -392,7 +392,7 @@ static void bcm2836_control_class_init(ObjectClass
> *klass, void *data)
>      dc->vmsd = &vmstate_bcm2836_control;
>  }
>
> -static TypeInfo bcm2836_control_info = {
> +static const TypeInfo bcm2836_control_info = {
>      .name          = TYPE_BCM2836_CONTROL,
>      .parent        = TYPE_SYS_BUS_DEVICE,
>      .instance_size = sizeof(BCM2836ControlState),
> diff --git a/hw/ipmi/ipmi.c b/hw/ipmi/ipmi.c
> index 8d35c9fdd6..bbb07b151e 100644
> --- a/hw/ipmi/ipmi.c
> +++ b/hw/ipmi/ipmi.c
> @@ -85,7 +85,7 @@ static void ipmi_interface_class_init(ObjectClass
> *class, void *data)
>      ik->do_hw_op = ipmi_do_hw_op;
>  }
>
> -static TypeInfo ipmi_interface_type_info = {
> +static const TypeInfo ipmi_interface_type_info = {
>      .name = TYPE_IPMI_INTERFACE,
>      .parent = TYPE_INTERFACE,
>      .class_size = sizeof(IPMIInterfaceClass),
> @@ -120,7 +120,7 @@ static void bmc_class_init(ObjectClass *oc, void *data)
>      device_class_set_props(dc, ipmi_bmc_properties);
>  }
>
> -static TypeInfo ipmi_bmc_type_info = {
> +static const TypeInfo ipmi_bmc_type_info = {
>      .name = TYPE_IPMI_BMC,
>      .parent = TYPE_DEVICE,
>      .instance_size = sizeof(IPMIBmc),
> diff --git a/hw/mem/nvdimm.c b/hw/mem/nvdimm.c
> index 7397b67156..933bc5e0c7 100644
> --- a/hw/mem/nvdimm.c
> +++ b/hw/mem/nvdimm.c
> @@ -248,7 +248,7 @@ static void nvdimm_class_init(ObjectClass *oc, void
> *data)
>      set_bit(DEVICE_CATEGORY_STORAGE, dc->categories);
>  }
>
> -static TypeInfo nvdimm_info = {
> +static const TypeInfo nvdimm_info = {
>      .name          = TYPE_NVDIMM,
>      .parent        = TYPE_PC_DIMM,
>      .class_size    = sizeof(NVDIMMClass),
> diff --git a/hw/mem/pc-dimm.c b/hw/mem/pc-dimm.c
> index 48b913aba6..28af0d71b2 100644
> --- a/hw/mem/pc-dimm.c
> +++ b/hw/mem/pc-dimm.c
> @@ -286,7 +286,7 @@ static void pc_dimm_class_init(ObjectClass *oc, void
> *data)
>      mdc->fill_device_info = pc_dimm_md_fill_device_info;
>  }
>
> -static TypeInfo pc_dimm_info = {
> +static const TypeInfo pc_dimm_info = {
>      .name          = TYPE_PC_DIMM,
>      .parent        = TYPE_DEVICE,
>      .instance_size = sizeof(PCDIMMDevice),
> diff --git a/hw/misc/bcm2835_mbox.c b/hw/misc/bcm2835_mbox.c
> index 9f73cbd5e4..1908c152f6 100644
> --- a/hw/misc/bcm2835_mbox.c
> +++ b/hw/misc/bcm2835_mbox.c
> @@ -324,7 +324,7 @@ static void bcm2835_mbox_class_init(ObjectClass
> *klass, void *data)
>      dc->vmsd = &vmstate_bcm2835_mbox;
>  }
>
> -static TypeInfo bcm2835_mbox_info = {
> +static const TypeInfo bcm2835_mbox_info = {
>      .name          = TYPE_BCM2835_MBOX,
>      .parent        = TYPE_SYS_BUS_DEVICE,
>      .instance_size = sizeof(BCM2835MboxState),
> diff --git a/hw/misc/bcm2835_powermgt.c b/hw/misc/bcm2835_powermgt.c
> index 25fa804cbd..976f3d34e5 100644
> --- a/hw/misc/bcm2835_powermgt.c
> +++ b/hw/misc/bcm2835_powermgt.c
> @@ -144,7 +144,7 @@ static void bcm2835_powermgt_class_init(ObjectClass
> *klass, void *data)
>      dc->vmsd = &vmstate_bcm2835_powermgt;
>  }
>
> -static TypeInfo bcm2835_powermgt_info = {
> +static const TypeInfo bcm2835_powermgt_info = {
>      .name          = TYPE_BCM2835_POWERMGT,
>      .parent        = TYPE_SYS_BUS_DEVICE,
>      .instance_size = sizeof(BCM2835PowerMgtState),
> diff --git a/hw/misc/bcm2835_property.c b/hw/misc/bcm2835_property.c
> index 76ea511d53..e94e951057 100644
> --- a/hw/misc/bcm2835_property.c
> +++ b/hw/misc/bcm2835_property.c
> @@ -421,7 +421,7 @@ static void bcm2835_property_class_init(ObjectClass
> *klass, void *data)
>      dc->vmsd = &vmstate_bcm2835_property;
>  }
>
> -static TypeInfo bcm2835_property_info = {
> +static const TypeInfo bcm2835_property_info = {
>      .name          = TYPE_BCM2835_PROPERTY,
>      .parent        = TYPE_SYS_BUS_DEVICE,
>      .instance_size = sizeof(BCM2835PropertyState),
> diff --git a/hw/misc/bcm2835_rng.c b/hw/misc/bcm2835_rng.c
> index d0c4e64e88..b3c80cf186 100644
> --- a/hw/misc/bcm2835_rng.c
> +++ b/hw/misc/bcm2835_rng.c
> @@ -131,7 +131,7 @@ static void bcm2835_rng_class_init(ObjectClass *klass,
> void *data)
>      dc->vmsd = &vmstate_bcm2835_rng;
>  }
>
> -static TypeInfo bcm2835_rng_info = {
> +static const TypeInfo bcm2835_rng_info = {
>      .name          = TYPE_BCM2835_RNG,
>      .parent        = TYPE_SYS_BUS_DEVICE,
>      .instance_size = sizeof(BCM2835RngState),
> diff --git a/hw/misc/pvpanic-isa.c b/hw/misc/pvpanic-isa.c
> index 7b66d58acc..a39fcdd1fc 100644
> --- a/hw/misc/pvpanic-isa.c
> +++ b/hw/misc/pvpanic-isa.c
> @@ -77,7 +77,7 @@ static void pvpanic_isa_class_init(ObjectClass *klass,
> void *data)
>      set_bit(DEVICE_CATEGORY_MISC, dc->categories);
>  }
>
> -static TypeInfo pvpanic_isa_info = {
> +static const TypeInfo pvpanic_isa_info = {
>      .name          = TYPE_PVPANIC_ISA_DEVICE,
>      .parent        = TYPE_ISA_DEVICE,
>      .instance_size = sizeof(PVPanicISAState),
> diff --git a/hw/misc/pvpanic-pci.c b/hw/misc/pvpanic-pci.c
> index af8cbe2830..62e1be68c1 100644
> --- a/hw/misc/pvpanic-pci.c
> +++ b/hw/misc/pvpanic-pci.c
> @@ -74,7 +74,7 @@ static void pvpanic_pci_class_init(ObjectClass *klass,
> void *data)
>      set_bit(DEVICE_CATEGORY_MISC, dc->categories);
>  }
>
> -static TypeInfo pvpanic_pci_info = {
> +static const TypeInfo pvpanic_pci_info = {
>      .name          = TYPE_PVPANIC_PCI_DEVICE,
>      .parent        = TYPE_PCI_DEVICE,
>      .instance_size = sizeof(PVPanicPCIState),
> diff --git a/hw/net/fsl_etsec/etsec.c b/hw/net/fsl_etsec/etsec.c
> index bd9d62b559..e7fc082518 100644
> --- a/hw/net/fsl_etsec/etsec.c
> +++ b/hw/net/fsl_etsec/etsec.c
> @@ -430,7 +430,7 @@ static void etsec_class_init(ObjectClass *klass, void
> *data)
>      dc->user_creatable = true;
>  }
>
> -static TypeInfo etsec_info = {
> +static const TypeInfo etsec_info = {
>      .name                  = TYPE_ETSEC_COMMON,
>      .parent                = TYPE_SYS_BUS_DEVICE,
>      .instance_size         = sizeof(eTSEC),
> diff --git a/hw/ppc/prep_systemio.c b/hw/ppc/prep_systemio.c
> index b2bd783248..8c9b8dd67b 100644
> --- a/hw/ppc/prep_systemio.c
> +++ b/hw/ppc/prep_systemio.c
> @@ -300,7 +300,7 @@ static void prep_systemio_class_initfn(ObjectClass
> *klass, void *data)
>      device_class_set_props(dc, prep_systemio_properties);
>  }
>
> -static TypeInfo prep_systemio800_info = {
> +static const TypeInfo prep_systemio800_info = {
>      .name          = TYPE_PREP_SYSTEMIO,
>      .parent        = TYPE_ISA_DEVICE,
>      .instance_size = sizeof(PrepSystemIoState),
> diff --git a/hw/ppc/spapr_iommu.c b/hw/ppc/spapr_iommu.c
> index db01071858..81e5a1aea3 100644
> --- a/hw/ppc/spapr_iommu.c
> +++ b/hw/ppc/spapr_iommu.c
> @@ -685,7 +685,7 @@ static void spapr_tce_table_class_init(ObjectClass
> *klass, void *data)
>      spapr_register_hypercall(H_STUFF_TCE, h_stuff_tce);
>  }
>
> -static TypeInfo spapr_tce_table_info = {
> +static const TypeInfo spapr_tce_table_info = {
>      .name = TYPE_SPAPR_TCE_TABLE,
>      .parent = TYPE_DEVICE,
>      .instance_size = sizeof(SpaprTceTable),
> diff --git a/hw/s390x/s390-pci-bus.c b/hw/s390x/s390-pci-bus.c
> index 01b58ebc70..4b2bdd94b3 100644
> --- a/hw/s390x/s390-pci-bus.c
> +++ b/hw/s390x/s390-pci-bus.c
> @@ -1392,7 +1392,7 @@ static const TypeInfo s390_pci_device_info = {
>      .class_init = s390_pci_device_class_init,
>  };
>
> -static TypeInfo s390_pci_iommu_info = {
> +static const TypeInfo s390_pci_iommu_info = {
>      .name = TYPE_S390_PCI_IOMMU,
>      .parent = TYPE_OBJECT,
>      .instance_size = sizeof(S390PCIIOMMU),
> diff --git a/hw/s390x/sclp.c b/hw/s390x/sclp.c
> index 89c30a8a91..eff74479f4 100644
> --- a/hw/s390x/sclp.c
> +++ b/hw/s390x/sclp.c
> @@ -460,7 +460,7 @@ static void sclp_class_init(ObjectClass *oc, void
> *data)
>      sc->service_interrupt = service_interrupt;
>  }
>
> -static TypeInfo sclp_info = {
> +static const TypeInfo sclp_info = {
>      .name = TYPE_SCLP,
>      .parent = TYPE_DEVICE,
>      .instance_init = sclp_init,
> diff --git a/hw/s390x/tod-kvm.c b/hw/s390x/tod-kvm.c
> index ec855811ae..9d0cbfbce2 100644
> --- a/hw/s390x/tod-kvm.c
> +++ b/hw/s390x/tod-kvm.c
> @@ -147,7 +147,7 @@ static void kvm_s390_tod_init(Object *obj)
>      td->stopped = false;
>  }
>
> -static TypeInfo kvm_s390_tod_info = {
> +static const TypeInfo kvm_s390_tod_info = {
>      .name = TYPE_KVM_S390_TOD,
>      .parent = TYPE_S390_TOD,
>      .instance_size = sizeof(S390TODState),
> diff --git a/hw/s390x/tod-tcg.c b/hw/s390x/tod-tcg.c
> index 9bb94ff72b..ad4993203d 100644
> --- a/hw/s390x/tod-tcg.c
> +++ b/hw/s390x/tod-tcg.c
> @@ -73,7 +73,7 @@ static void qemu_s390_tod_init(Object *obj)
>      }
>  }
>
> -static TypeInfo qemu_s390_tod_info = {
> +static const TypeInfo qemu_s390_tod_info = {
>      .name = TYPE_QEMU_S390_TOD,
>      .parent = TYPE_S390_TOD,
>      .instance_size = sizeof(S390TODState),
> diff --git a/hw/s390x/tod.c b/hw/s390x/tod.c
> index fd5a36bf24..c81b1c0338 100644
> --- a/hw/s390x/tod.c
> +++ b/hw/s390x/tod.c
> @@ -123,7 +123,7 @@ static void s390_tod_class_init(ObjectClass *oc, void
> *data)
>      dc->user_creatable = false;
>  }
>
> -static TypeInfo s390_tod_info = {
> +static const TypeInfo s390_tod_info = {
>      .name = TYPE_S390_TOD,
>      .parent = TYPE_DEVICE,
>      .instance_size = sizeof(S390TODState),
> diff --git a/hw/scsi/lsi53c895a.c b/hw/scsi/lsi53c895a.c
> index 4c431adb77..c8773f73f7 100644
> --- a/hw/scsi/lsi53c895a.c
> +++ b/hw/scsi/lsi53c895a.c
> @@ -2352,7 +2352,7 @@ static void lsi53c810_class_init(ObjectClass *klass,
> void *data)
>      k->device_id = PCI_DEVICE_ID_LSI_53C810;
>  }
>
> -static TypeInfo lsi53c810_info = {
> +static const TypeInfo lsi53c810_info = {
>      .name          = TYPE_LSI53C810,
>      .parent        = TYPE_LSI53C895A,
>      .class_init    = lsi53c810_class_init,
> diff --git a/hw/sd/allwinner-sdhost.c b/hw/sd/allwinner-sdhost.c
> index de5bc49e68..041e45c680 100644
> --- a/hw/sd/allwinner-sdhost.c
> +++ b/hw/sd/allwinner-sdhost.c
> @@ -835,7 +835,7 @@ static void
> allwinner_sdhost_sun5i_class_init(ObjectClass *klass, void *data)
>      sc->max_desc_size = 64 * KiB;
>  }
>
> -static TypeInfo allwinner_sdhost_info = {
> +static const TypeInfo allwinner_sdhost_info = {
>      .name          = TYPE_AW_SDHOST,
>      .parent        = TYPE_SYS_BUS_DEVICE,
>      .instance_init = allwinner_sdhost_init,
> diff --git a/hw/sd/aspeed_sdhci.c b/hw/sd/aspeed_sdhci.c
> index df1bdf1fa4..be8cafd65f 100644
> --- a/hw/sd/aspeed_sdhci.c
> +++ b/hw/sd/aspeed_sdhci.c
> @@ -198,7 +198,7 @@ static void aspeed_sdhci_class_init(ObjectClass
> *classp, void *data)
>      device_class_set_props(dc, aspeed_sdhci_properties);
>  }
>
> -static TypeInfo aspeed_sdhci_info = {
> +static const TypeInfo aspeed_sdhci_info = {
>      .name          = TYPE_ASPEED_SDHCI,
>      .parent        = TYPE_SYS_BUS_DEVICE,
>      .instance_size = sizeof(AspeedSDHCIState),
> diff --git a/hw/sd/bcm2835_sdhost.c b/hw/sd/bcm2835_sdhost.c
> index 088a7ac6ed..9431c35914 100644
> --- a/hw/sd/bcm2835_sdhost.c
> +++ b/hw/sd/bcm2835_sdhost.c
> @@ -436,7 +436,7 @@ static void bcm2835_sdhost_class_init(ObjectClass
> *klass, void *data)
>      dc->vmsd = &vmstate_bcm2835_sdhost;
>  }
>
> -static TypeInfo bcm2835_sdhost_info = {
> +static const TypeInfo bcm2835_sdhost_info = {
>      .name          = TYPE_BCM2835_SDHOST,
>      .parent        = TYPE_SYS_BUS_DEVICE,
>      .instance_size = sizeof(BCM2835SDHostState),
> diff --git a/hw/sd/cadence_sdhci.c b/hw/sd/cadence_sdhci.c
> index 56b8bae1c3..75db34befe 100644
> --- a/hw/sd/cadence_sdhci.c
> +++ b/hw/sd/cadence_sdhci.c
> @@ -175,7 +175,7 @@ static void cadence_sdhci_class_init(ObjectClass
> *classp, void *data)
>      dc->vmsd = &vmstate_cadence_sdhci;
>  }
>
> -static TypeInfo cadence_sdhci_info = {
> +static const TypeInfo cadence_sdhci_info = {
>      .name          = TYPE_CADENCE_SDHCI,
>      .parent        = TYPE_SYS_BUS_DEVICE,
>      .instance_size = sizeof(CadenceSDHCIState),
> diff --git a/hw/sd/npcm7xx_sdhci.c b/hw/sd/npcm7xx_sdhci.c
> index ef503365df..b2f5b4a542 100644
> --- a/hw/sd/npcm7xx_sdhci.c
> +++ b/hw/sd/npcm7xx_sdhci.c
> @@ -166,7 +166,7 @@ static void npcm7xx_sdhci_instance_init(Object *obj)
>                              TYPE_SYSBUS_SDHCI);
>  }
>
> -static TypeInfo npcm7xx_sdhci_info = {
> +static const TypeInfo npcm7xx_sdhci_info = {
>      .name = TYPE_NPCM7XX_SDHCI,
>      .parent = TYPE_SYS_BUS_DEVICE,
>      .instance_size = sizeof(NPCM7xxSDHCIState),
> diff --git a/hw/usb/dev-mtp.c b/hw/usb/dev-mtp.c
> index c1d1694fd0..1e6ac76bef 100644
> --- a/hw/usb/dev-mtp.c
> +++ b/hw/usb/dev-mtp.c
> @@ -2106,7 +2106,7 @@ static void usb_mtp_class_initfn(ObjectClass *klass,
> void *data)
>      device_class_set_props(dc, mtp_properties);
>  }
>
> -static TypeInfo mtp_info = {
> +static const TypeInfo mtp_info = {
>      .name          = TYPE_USB_MTP,
>      .parent        = TYPE_USB_DEVICE,
>      .instance_size = sizeof(MTPState),
> diff --git a/hw/usb/host-libusb.c b/hw/usb/host-libusb.c
> index d0d46dd0a4..2b35cb6cdd 100644
> --- a/hw/usb/host-libusb.c
> +++ b/hw/usb/host-libusb.c
> @@ -1801,7 +1801,7 @@ static void usb_host_class_initfn(ObjectClass
> *klass, void *data)
>      set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories);
>  }
>
> -static TypeInfo usb_host_dev_info = {
> +static const TypeInfo usb_host_dev_info = {
>      .name          = TYPE_USB_HOST_DEVICE,
>      .parent        = TYPE_USB_DEVICE,
>      .instance_size = sizeof(USBHostDevice),
> diff --git a/hw/vfio/igd.c b/hw/vfio/igd.c
> index d4685709a3..afe3fe7efc 100644
> --- a/hw/vfio/igd.c
> +++ b/hw/vfio/igd.c
> @@ -199,7 +199,7 @@ static void
> vfio_pci_igd_lpc_bridge_class_init(ObjectClass *klass, void *data)
>      k->class_id = PCI_CLASS_BRIDGE_ISA;
>  }
>
> -static TypeInfo vfio_pci_igd_lpc_bridge_info = {
> +static const TypeInfo vfio_pci_igd_lpc_bridge_info = {
>      .name = "vfio-pci-igd-lpc-bridge",
>      .parent = TYPE_PCI_DEVICE,
>      .class_init = vfio_pci_igd_lpc_bridge_class_init,
> diff --git a/hw/virtio/virtio-pmem.c b/hw/virtio/virtio-pmem.c
> index d1aeb90a31..5419dca75e 100644
> --- a/hw/virtio/virtio-pmem.c
> +++ b/hw/virtio/virtio-pmem.c
> @@ -182,7 +182,7 @@ static void virtio_pmem_class_init(ObjectClass *klass,
> void *data)
>      set_bit(DEVICE_CATEGORY_STORAGE, dc->categories);
>  }
>
> -static TypeInfo virtio_pmem_info = {
> +static const TypeInfo virtio_pmem_info = {
>      .name          = TYPE_VIRTIO_PMEM,
>      .parent        = TYPE_VIRTIO_DEVICE,
>      .class_size    = sizeof(VirtIOPMEMClass),
> diff --git a/qom/object.c b/qom/object.c
> index 4f0677cca9..9f7a33139d 100644
> --- a/qom/object.c
> +++ b/qom/object.c
> @@ -2793,13 +2793,13 @@ static void object_class_init(ObjectClass *klass,
> void *data)
>
>  static void register_types(void)
>  {
> -    static TypeInfo interface_info = {
> +    static const TypeInfo interface_info = {
>          .name = TYPE_INTERFACE,
>          .class_size = sizeof(InterfaceClass),
>          .abstract = true,
>      };
>
> -    static TypeInfo object_info = {
> +    static const TypeInfo object_info = {
>          .name = TYPE_OBJECT,
>          .instance_size = sizeof(Object),
>          .class_init = object_class_init,
> --
> 2.34.1
>
>

[Attachment #3 (text/html)]

<div dir="ltr"><div>Hi Philippe,</div><div><br></div><div>Thanks for your review. \
I&#39;ve now sent v2 of the patch with your comments \
addressed.</div><div><br></div><div>BTW: I almost missed your review  because \
apparently I wasn&#39;t in the CC. That is also the reason I&#39;m not responding to \
your mail directly.<br></div><div><br></div>Bernhard.<br></div><br><div \
class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jan 13, 2022 at 6:10 PM \
Bernhard Beschow &lt;<a href="mailto:shentey@gmail.com">shentey@gmail.com</a>&gt; \
wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px \
0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">More than 1k of \
TypeInfo instances are already marked as const. Mark the<br> remaining ones, too.<br>
<br>
Signed-off-by: Bernhard Beschow &lt;<a href="mailto:shentey@gmail.com" \
                target="_blank">shentey@gmail.com</a>&gt;<br>
---<br>
  hw/core/generic-loader.c     | 2 +-<br>
  hw/core/guest-loader.c        | 2 +-<br>
  hw/display/bcm2835_fb.c      | 2 +-<br>
  hw/display/i2c-ddc.c           | 2 +-<br>
  hw/display/macfb.c              | 4 ++--<br>
  hw/display/virtio-vga.c      | 2 +-<br>
  hw/dma/bcm2835_dma.c           | 2 +-<br>
  hw/i386/pc_piix.c               | 2 +-<br>
  hw/i386/sgx-epc.c               | 2 +-<br>
  hw/intc/bcm2835_ic.c           | 2 +-<br>
  hw/intc/bcm2836_control.c   | 2 +-<br>
  hw/ipmi/ipmi.c                    | 4 ++--<br>
  hw/mem/nvdimm.c                  | 2 +-<br>
  hw/mem/pc-dimm.c                 | 2 +-<br>
  hw/misc/bcm2835_mbox.c        | 2 +-<br>
  hw/misc/bcm2835_powermgt.c | 2 +-<br>
  hw/misc/bcm2835_property.c | 2 +-<br>
  hw/misc/bcm2835_rng.c         | 2 +-<br>
  hw/misc/pvpanic-isa.c         | 2 +-<br>
  hw/misc/pvpanic-pci.c         | 2 +-<br>
  hw/net/fsl_etsec/etsec.c     | 2 +-<br>
  hw/ppc/prep_systemio.c        | 2 +-<br>
  hw/ppc/spapr_iommu.c           | 2 +-<br>
  hw/s390x/s390-pci-bus.c      | 2 +-<br>
  hw/s390x/sclp.c                  | 2 +-<br>
  hw/s390x/tod-kvm.c              | 2 +-<br>
  hw/s390x/tod-tcg.c              | 2 +-<br>
  hw/s390x/tod.c                    | 2 +-<br>
  hw/scsi/lsi53c895a.c           | 2 +-<br>
  hw/sd/allwinner-sdhost.c     | 2 +-<br>
  hw/sd/aspeed_sdhci.c           | 2 +-<br>
  hw/sd/bcm2835_sdhost.c        | 2 +-<br>
  hw/sd/cadence_sdhci.c         | 2 +-<br>
  hw/sd/npcm7xx_sdhci.c         | 2 +-<br>
  hw/usb/dev-mtp.c                 | 2 +-<br>
  hw/usb/host-libusb.c           | 2 +-<br>
  hw/vfio/igd.c                     | 2 +-<br>
  hw/virtio/virtio-pmem.c      | 2 +-<br>
  qom/object.c                       | 4 ++--<br>
  39 files changed, 42 insertions(+), 42 deletions(-)<br>
<br>
diff --git a/hw/core/generic-loader.c b/hw/core/generic-loader.c<br>
index 9a24ffb880..eaafc416f4 100644<br>
--- a/hw/core/generic-loader.c<br>
+++ b/hw/core/generic-loader.c<br>
@@ -207,7 +207,7 @@ static void generic_loader_class_init(ObjectClass *klass, void \
*data)<br>  set_bit(DEVICE_CATEGORY_MISC, dc-&gt;categories);<br>
  }<br>
<br>
-static TypeInfo generic_loader_info = {<br>
+static const TypeInfo generic_loader_info = {<br>
        .name = TYPE_GENERIC_LOADER,<br>
        .parent = TYPE_DEVICE,<br>
        .instance_size = sizeof(GenericLoaderState),<br>
diff --git a/hw/core/guest-loader.c b/hw/core/guest-loader.c<br>
index d3f9d1a06e..391c875a29 100644<br>
--- a/hw/core/guest-loader.c<br>
+++ b/hw/core/guest-loader.c<br>
@@ -129,7 +129,7 @@ static void guest_loader_class_init(ObjectClass *klass, void \
*data)<br>  set_bit(DEVICE_CATEGORY_MISC, dc-&gt;categories);<br>
  }<br>
<br>
-static TypeInfo guest_loader_info = {<br>
+static const TypeInfo guest_loader_info = {<br>
        .name = TYPE_GUEST_LOADER,<br>
        .parent = TYPE_DEVICE,<br>
        .instance_size = sizeof(GuestLoaderState),<br>
diff --git a/hw/display/bcm2835_fb.c b/hw/display/bcm2835_fb.c<br>
index 2be77bdd3a..088fc3d51c 100644<br>
--- a/hw/display/bcm2835_fb.c<br>
+++ b/hw/display/bcm2835_fb.c<br>
@@ -454,7 +454,7 @@ static void bcm2835_fb_class_init(ObjectClass *klass, void \
*data)<br>  dc-&gt;vmsd = &amp;vmstate_bcm2835_fb;<br>
  }<br>
<br>
-static TypeInfo bcm2835_fb_info = {<br>
+static const TypeInfo bcm2835_fb_info = {<br>
        .name               = TYPE_BCM2835_FB,<br>
        .parent            = TYPE_SYS_BUS_DEVICE,<br>
        .instance_size = sizeof(BCM2835FBState),<br>
diff --git a/hw/display/i2c-ddc.c b/hw/display/i2c-ddc.c<br>
index 13eb529fc1..146489518c 100644<br>
--- a/hw/display/i2c-ddc.c<br>
+++ b/hw/display/i2c-ddc.c<br>
@@ -113,7 +113,7 @@ static void i2c_ddc_class_init(ObjectClass *oc, void *data)<br>
        isc-&gt;send = i2c_ddc_tx;<br>
  }<br>
<br>
-static TypeInfo i2c_ddc_info = {<br>
+static const TypeInfo i2c_ddc_info = {<br>
        .name = TYPE_I2CDDC,<br>
        .parent = TYPE_I2C_SLAVE,<br>
        .instance_size = sizeof(I2CDDCState),<br>
diff --git a/hw/display/macfb.c b/hw/display/macfb.c<br>
index 4bd7c3ad6a..69c2ea2b6e 100644<br>
--- a/hw/display/macfb.c<br>
+++ b/hw/display/macfb.c<br>
@@ -783,14 +783,14 @@ static void macfb_nubus_class_init(ObjectClass *klass, void \
*data)<br>  device_class_set_props(dc, macfb_nubus_properties);<br>
  }<br>
<br>
-static TypeInfo macfb_sysbus_info = {<br>
+static const TypeInfo macfb_sysbus_info = {<br>
        .name               = TYPE_MACFB,<br>
        .parent            = TYPE_SYS_BUS_DEVICE,<br>
        .instance_size = sizeof(MacfbSysBusState),<br>
        .class_init      = macfb_sysbus_class_init,<br>
  };<br>
<br>
-static TypeInfo macfb_nubus_info = {<br>
+static const TypeInfo macfb_nubus_info = {<br>
        .name               = TYPE_NUBUS_MACFB,<br>
        .parent            = TYPE_NUBUS_DEVICE,<br>
        .instance_size = sizeof(MacfbNubusState),<br>
diff --git a/hw/display/virtio-vga.c b/hw/display/virtio-vga.c<br>
index b23a75a04b..5a2f7a4540 100644<br>
--- a/hw/display/virtio-vga.c<br>
+++ b/hw/display/virtio-vga.c<br>
@@ -220,7 +220,7 @@ static void virtio_vga_base_class_init(ObjectClass *klass, void \
                *data)<br>
                                                      \
virtio_vga_set_big_endian_fb);<br>  }<br>
<br>
-static TypeInfo virtio_vga_base_info = {<br>
+static const TypeInfo virtio_vga_base_info = {<br>
        .name               = TYPE_VIRTIO_VGA_BASE,<br>
        .parent            = TYPE_VIRTIO_PCI,<br>
        .instance_size = sizeof(VirtIOVGABase),<br>
diff --git a/hw/dma/bcm2835_dma.c b/hw/dma/bcm2835_dma.c<br>
index eb0002a2b9..5e9306110d 100644<br>
--- a/hw/dma/bcm2835_dma.c<br>
+++ b/hw/dma/bcm2835_dma.c<br>
@@ -394,7 +394,7 @@ static void bcm2835_dma_class_init(ObjectClass *klass, void \
*data)<br>  dc-&gt;vmsd = &amp;vmstate_bcm2835_dma;<br>
  }<br>
<br>
-static TypeInfo bcm2835_dma_info = {<br>
+static const TypeInfo bcm2835_dma_info = {<br>
        .name               = TYPE_BCM2835_DMA,<br>
        .parent            = TYPE_SYS_BUS_DEVICE,<br>
        .instance_size = sizeof(BCM2835DMAState),<br>
diff --git a/hw/i386/pc_piix.c b/hw/i386/pc_piix.c<br>
index 7c7790a5ce..f6557d3c21 100644<br>
--- a/hw/i386/pc_piix.c<br>
+++ b/hw/i386/pc_piix.c<br>
@@ -865,7 +865,7 @@ static void isa_bridge_class_init(ObjectClass *klass, void \
*data)<br>  k-&gt;class_id        = PCI_CLASS_BRIDGE_ISA;<br>
  };<br>
<br>
-static TypeInfo isa_bridge_info = {<br>
+static const TypeInfo isa_bridge_info = {<br>
        .name               = &quot;igd-passthrough-isa-bridge&quot;,<br>
        .parent            = TYPE_PCI_DEVICE,<br>
        .instance_size = sizeof(PCIDevice),<br>
diff --git a/hw/i386/sgx-epc.c b/hw/i386/sgx-epc.c<br>
index 96b2940d75..d664829d35 100644<br>
--- a/hw/i386/sgx-epc.c<br>
+++ b/hw/i386/sgx-epc.c<br>
@@ -167,7 +167,7 @@ static void sgx_epc_class_init(ObjectClass *oc, void *data)<br>
        mdc-&gt;fill_device_info = sgx_epc_md_fill_device_info;<br>
  }<br>
<br>
-static TypeInfo sgx_epc_info = {<br>
+static const TypeInfo sgx_epc_info = {<br>
        .name               = TYPE_SGX_EPC,<br>
        .parent            = TYPE_DEVICE,<br>
        .instance_size = sizeof(SGXEPCDevice),<br>
diff --git a/hw/intc/bcm2835_ic.c b/hw/intc/bcm2835_ic.c<br>
index 9000d995e8..4513fad16f 100644<br>
--- a/hw/intc/bcm2835_ic.c<br>
+++ b/hw/intc/bcm2835_ic.c<br>
@@ -227,7 +227,7 @@ static void bcm2835_ic_class_init(ObjectClass *klass, void \
*data)<br>  dc-&gt;vmsd = &amp;vmstate_bcm2835_ic;<br>
  }<br>
<br>
-static TypeInfo bcm2835_ic_info = {<br>
+static const TypeInfo bcm2835_ic_info = {<br>
        .name               = TYPE_BCM2835_IC,<br>
        .parent            = TYPE_SYS_BUS_DEVICE,<br>
        .instance_size = sizeof(BCM2835ICState),<br>
diff --git a/hw/intc/bcm2836_control.c b/hw/intc/bcm2836_control.c<br>
index 2ead76ffdc..b0589df188 100644<br>
--- a/hw/intc/bcm2836_control.c<br>
+++ b/hw/intc/bcm2836_control.c<br>
@@ -392,7 +392,7 @@ static void bcm2836_control_class_init(ObjectClass *klass, void \
*data)<br>  dc-&gt;vmsd = &amp;vmstate_bcm2836_control;<br>
  }<br>
<br>
-static TypeInfo bcm2836_control_info = {<br>
+static const TypeInfo bcm2836_control_info = {<br>
        .name               = TYPE_BCM2836_CONTROL,<br>
        .parent            = TYPE_SYS_BUS_DEVICE,<br>
        .instance_size = sizeof(BCM2836ControlState),<br>
diff --git a/hw/ipmi/ipmi.c b/hw/ipmi/ipmi.c<br>
index 8d35c9fdd6..bbb07b151e 100644<br>
--- a/hw/ipmi/ipmi.c<br>
+++ b/hw/ipmi/ipmi.c<br>
@@ -85,7 +85,7 @@ static void ipmi_interface_class_init(ObjectClass *class, void \
*data)<br>  ik-&gt;do_hw_op = ipmi_do_hw_op;<br>
  }<br>
<br>
-static TypeInfo ipmi_interface_type_info = {<br>
+static const TypeInfo ipmi_interface_type_info = {<br>
        .name = TYPE_IPMI_INTERFACE,<br>
        .parent = TYPE_INTERFACE,<br>
        .class_size = sizeof(IPMIInterfaceClass),<br>
@@ -120,7 +120,7 @@ static void bmc_class_init(ObjectClass *oc, void *data)<br>
        device_class_set_props(dc, ipmi_bmc_properties);<br>
  }<br>
<br>
-static TypeInfo ipmi_bmc_type_info = {<br>
+static const TypeInfo ipmi_bmc_type_info = {<br>
        .name = TYPE_IPMI_BMC,<br>
        .parent = TYPE_DEVICE,<br>
        .instance_size = sizeof(IPMIBmc),<br>
diff --git a/hw/mem/nvdimm.c b/hw/mem/nvdimm.c<br>
index 7397b67156..933bc5e0c7 100644<br>
--- a/hw/mem/nvdimm.c<br>
+++ b/hw/mem/nvdimm.c<br>
@@ -248,7 +248,7 @@ static void nvdimm_class_init(ObjectClass *oc, void *data)<br>
        set_bit(DEVICE_CATEGORY_STORAGE, dc-&gt;categories);<br>
  }<br>
<br>
-static TypeInfo nvdimm_info = {<br>
+static const TypeInfo nvdimm_info = {<br>
        .name               = TYPE_NVDIMM,<br>
        .parent            = TYPE_PC_DIMM,<br>
        .class_size      = sizeof(NVDIMMClass),<br>
diff --git a/hw/mem/pc-dimm.c b/hw/mem/pc-dimm.c<br>
index 48b913aba6..28af0d71b2 100644<br>
--- a/hw/mem/pc-dimm.c<br>
+++ b/hw/mem/pc-dimm.c<br>
@@ -286,7 +286,7 @@ static void pc_dimm_class_init(ObjectClass *oc, void *data)<br>
        mdc-&gt;fill_device_info = pc_dimm_md_fill_device_info;<br>
  }<br>
<br>
-static TypeInfo pc_dimm_info = {<br>
+static const TypeInfo pc_dimm_info = {<br>
        .name               = TYPE_PC_DIMM,<br>
        .parent            = TYPE_DEVICE,<br>
        .instance_size = sizeof(PCDIMMDevice),<br>
diff --git a/hw/misc/bcm2835_mbox.c b/hw/misc/bcm2835_mbox.c<br>
index 9f73cbd5e4..1908c152f6 100644<br>
--- a/hw/misc/bcm2835_mbox.c<br>
+++ b/hw/misc/bcm2835_mbox.c<br>
@@ -324,7 +324,7 @@ static void bcm2835_mbox_class_init(ObjectClass *klass, void \
*data)<br>  dc-&gt;vmsd = &amp;vmstate_bcm2835_mbox;<br>
  }<br>
<br>
-static TypeInfo bcm2835_mbox_info = {<br>
+static const TypeInfo bcm2835_mbox_info = {<br>
        .name               = TYPE_BCM2835_MBOX,<br>
        .parent            = TYPE_SYS_BUS_DEVICE,<br>
        .instance_size = sizeof(BCM2835MboxState),<br>
diff --git a/hw/misc/bcm2835_powermgt.c b/hw/misc/bcm2835_powermgt.c<br>
index 25fa804cbd..976f3d34e5 100644<br>
--- a/hw/misc/bcm2835_powermgt.c<br>
+++ b/hw/misc/bcm2835_powermgt.c<br>
@@ -144,7 +144,7 @@ static void bcm2835_powermgt_class_init(ObjectClass *klass, void \
*data)<br>  dc-&gt;vmsd = &amp;vmstate_bcm2835_powermgt;<br>
  }<br>
<br>
-static TypeInfo bcm2835_powermgt_info = {<br>
+static const TypeInfo bcm2835_powermgt_info = {<br>
        .name               = TYPE_BCM2835_POWERMGT,<br>
        .parent            = TYPE_SYS_BUS_DEVICE,<br>
        .instance_size = sizeof(BCM2835PowerMgtState),<br>
diff --git a/hw/misc/bcm2835_property.c b/hw/misc/bcm2835_property.c<br>
index 76ea511d53..e94e951057 100644<br>
--- a/hw/misc/bcm2835_property.c<br>
+++ b/hw/misc/bcm2835_property.c<br>
@@ -421,7 +421,7 @@ static void bcm2835_property_class_init(ObjectClass *klass, void \
*data)<br>  dc-&gt;vmsd = &amp;vmstate_bcm2835_property;<br>
  }<br>
<br>
-static TypeInfo bcm2835_property_info = {<br>
+static const TypeInfo bcm2835_property_info = {<br>
        .name               = TYPE_BCM2835_PROPERTY,<br>
        .parent            = TYPE_SYS_BUS_DEVICE,<br>
        .instance_size = sizeof(BCM2835PropertyState),<br>
diff --git a/hw/misc/bcm2835_rng.c b/hw/misc/bcm2835_rng.c<br>
index d0c4e64e88..b3c80cf186 100644<br>
--- a/hw/misc/bcm2835_rng.c<br>
+++ b/hw/misc/bcm2835_rng.c<br>
@@ -131,7 +131,7 @@ static void bcm2835_rng_class_init(ObjectClass *klass, void \
*data)<br>  dc-&gt;vmsd = &amp;vmstate_bcm2835_rng;<br>
  }<br>
<br>
-static TypeInfo bcm2835_rng_info = {<br>
+static const TypeInfo bcm2835_rng_info = {<br>
        .name               = TYPE_BCM2835_RNG,<br>
        .parent            = TYPE_SYS_BUS_DEVICE,<br>
        .instance_size = sizeof(BCM2835RngState),<br>
diff --git a/hw/misc/pvpanic-isa.c b/hw/misc/pvpanic-isa.c<br>
index 7b66d58acc..a39fcdd1fc 100644<br>
--- a/hw/misc/pvpanic-isa.c<br>
+++ b/hw/misc/pvpanic-isa.c<br>
@@ -77,7 +77,7 @@ static void pvpanic_isa_class_init(ObjectClass *klass, void \
*data)<br>  set_bit(DEVICE_CATEGORY_MISC, dc-&gt;categories);<br>
  }<br>
<br>
-static TypeInfo pvpanic_isa_info = {<br>
+static const TypeInfo pvpanic_isa_info = {<br>
        .name               = TYPE_PVPANIC_ISA_DEVICE,<br>
        .parent            = TYPE_ISA_DEVICE,<br>
        .instance_size = sizeof(PVPanicISAState),<br>
diff --git a/hw/misc/pvpanic-pci.c b/hw/misc/pvpanic-pci.c<br>
index af8cbe2830..62e1be68c1 100644<br>
--- a/hw/misc/pvpanic-pci.c<br>
+++ b/hw/misc/pvpanic-pci.c<br>
@@ -74,7 +74,7 @@ static void pvpanic_pci_class_init(ObjectClass *klass, void \
*data)<br>  set_bit(DEVICE_CATEGORY_MISC, dc-&gt;categories);<br>
  }<br>
<br>
-static TypeInfo pvpanic_pci_info = {<br>
+static const TypeInfo pvpanic_pci_info = {<br>
        .name               = TYPE_PVPANIC_PCI_DEVICE,<br>
        .parent            = TYPE_PCI_DEVICE,<br>
        .instance_size = sizeof(PVPanicPCIState),<br>
diff --git a/hw/net/fsl_etsec/etsec.c b/hw/net/fsl_etsec/etsec.c<br>
index bd9d62b559..e7fc082518 100644<br>
--- a/hw/net/fsl_etsec/etsec.c<br>
+++ b/hw/net/fsl_etsec/etsec.c<br>
@@ -430,7 +430,7 @@ static void etsec_class_init(ObjectClass *klass, void *data)<br>
        dc-&gt;user_creatable = true;<br>
  }<br>
<br>
-static TypeInfo etsec_info = {<br>
+static const TypeInfo etsec_info = {<br>
        .name                           = TYPE_ETSEC_COMMON,<br>
        .parent                        = TYPE_SYS_BUS_DEVICE,<br>
        .instance_size              = sizeof(eTSEC),<br>
diff --git a/hw/ppc/prep_systemio.c b/hw/ppc/prep_systemio.c<br>
index b2bd783248..8c9b8dd67b 100644<br>
--- a/hw/ppc/prep_systemio.c<br>
+++ b/hw/ppc/prep_systemio.c<br>
@@ -300,7 +300,7 @@ static void prep_systemio_class_initfn(ObjectClass *klass, void \
*data)<br>  device_class_set_props(dc, prep_systemio_properties);<br>
  }<br>
<br>
-static TypeInfo prep_systemio800_info = {<br>
+static const TypeInfo prep_systemio800_info = {<br>
        .name               = TYPE_PREP_SYSTEMIO,<br>
        .parent            = TYPE_ISA_DEVICE,<br>
        .instance_size = sizeof(PrepSystemIoState),<br>
diff --git a/hw/ppc/spapr_iommu.c b/hw/ppc/spapr_iommu.c<br>
index db01071858..81e5a1aea3 100644<br>
--- a/hw/ppc/spapr_iommu.c<br>
+++ b/hw/ppc/spapr_iommu.c<br>
@@ -685,7 +685,7 @@ static void spapr_tce_table_class_init(ObjectClass *klass, void \
*data)<br>  spapr_register_hypercall(H_STUFF_TCE, h_stuff_tce);<br>
  }<br>
<br>
-static TypeInfo spapr_tce_table_info = {<br>
+static const TypeInfo spapr_tce_table_info = {<br>
        .name = TYPE_SPAPR_TCE_TABLE,<br>
        .parent = TYPE_DEVICE,<br>
        .instance_size = sizeof(SpaprTceTable),<br>
diff --git a/hw/s390x/s390-pci-bus.c b/hw/s390x/s390-pci-bus.c<br>
index 01b58ebc70..4b2bdd94b3 100644<br>
--- a/hw/s390x/s390-pci-bus.c<br>
+++ b/hw/s390x/s390-pci-bus.c<br>
@@ -1392,7 +1392,7 @@ static const TypeInfo s390_pci_device_info = {<br>
        .class_init = s390_pci_device_class_init,<br>
  };<br>
<br>
-static TypeInfo s390_pci_iommu_info = {<br>
+static const TypeInfo s390_pci_iommu_info = {<br>
        .name = TYPE_S390_PCI_IOMMU,<br>
        .parent = TYPE_OBJECT,<br>
        .instance_size = sizeof(S390PCIIOMMU),<br>
diff --git a/hw/s390x/sclp.c b/hw/s390x/sclp.c<br>
index 89c30a8a91..eff74479f4 100644<br>
--- a/hw/s390x/sclp.c<br>
+++ b/hw/s390x/sclp.c<br>
@@ -460,7 +460,7 @@ static void sclp_class_init(ObjectClass *oc, void *data)<br>
        sc-&gt;service_interrupt = service_interrupt;<br>
  }<br>
<br>
-static TypeInfo sclp_info = {<br>
+static const TypeInfo sclp_info = {<br>
        .name = TYPE_SCLP,<br>
        .parent = TYPE_DEVICE,<br>
        .instance_init = sclp_init,<br>
diff --git a/hw/s390x/tod-kvm.c b/hw/s390x/tod-kvm.c<br>
index ec855811ae..9d0cbfbce2 100644<br>
--- a/hw/s390x/tod-kvm.c<br>
+++ b/hw/s390x/tod-kvm.c<br>
@@ -147,7 +147,7 @@ static void kvm_s390_tod_init(Object *obj)<br>
        td-&gt;stopped = false;<br>
  }<br>
<br>
-static TypeInfo kvm_s390_tod_info = {<br>
+static const TypeInfo kvm_s390_tod_info = {<br>
        .name = TYPE_KVM_S390_TOD,<br>
        .parent = TYPE_S390_TOD,<br>
        .instance_size = sizeof(S390TODState),<br>
diff --git a/hw/s390x/tod-tcg.c b/hw/s390x/tod-tcg.c<br>
index 9bb94ff72b..ad4993203d 100644<br>
--- a/hw/s390x/tod-tcg.c<br>
+++ b/hw/s390x/tod-tcg.c<br>
@@ -73,7 +73,7 @@ static void qemu_s390_tod_init(Object *obj)<br>
        }<br>
  }<br>
<br>
-static TypeInfo qemu_s390_tod_info = {<br>
+static const TypeInfo qemu_s390_tod_info = {<br>
        .name = TYPE_QEMU_S390_TOD,<br>
        .parent = TYPE_S390_TOD,<br>
        .instance_size = sizeof(S390TODState),<br>
diff --git a/hw/s390x/tod.c b/hw/s390x/tod.c<br>
index fd5a36bf24..c81b1c0338 100644<br>
--- a/hw/s390x/tod.c<br>
+++ b/hw/s390x/tod.c<br>
@@ -123,7 +123,7 @@ static void s390_tod_class_init(ObjectClass *oc, void *data)<br>
        dc-&gt;user_creatable = false;<br>
  }<br>
<br>
-static TypeInfo s390_tod_info = {<br>
+static const TypeInfo s390_tod_info = {<br>
        .name = TYPE_S390_TOD,<br>
        .parent = TYPE_DEVICE,<br>
        .instance_size = sizeof(S390TODState),<br>
diff --git a/hw/scsi/lsi53c895a.c b/hw/scsi/lsi53c895a.c<br>
index 4c431adb77..c8773f73f7 100644<br>
--- a/hw/scsi/lsi53c895a.c<br>
+++ b/hw/scsi/lsi53c895a.c<br>
@@ -2352,7 +2352,7 @@ static void lsi53c810_class_init(ObjectClass *klass, void \
*data)<br>  k-&gt;device_id = PCI_DEVICE_ID_LSI_53C810;<br>
  }<br>
<br>
-static TypeInfo lsi53c810_info = {<br>
+static const TypeInfo lsi53c810_info = {<br>
        .name               = TYPE_LSI53C810,<br>
        .parent            = TYPE_LSI53C895A,<br>
        .class_init      = lsi53c810_class_init,<br>
diff --git a/hw/sd/allwinner-sdhost.c b/hw/sd/allwinner-sdhost.c<br>
index de5bc49e68..041e45c680 100644<br>
--- a/hw/sd/allwinner-sdhost.c<br>
+++ b/hw/sd/allwinner-sdhost.c<br>
@@ -835,7 +835,7 @@ static void allwinner_sdhost_sun5i_class_init(ObjectClass *klass, \
void *data)<br>  sc-&gt;max_desc_size = 64 * KiB;<br>
  }<br>
<br>
-static TypeInfo allwinner_sdhost_info = {<br>
+static const TypeInfo allwinner_sdhost_info = {<br>
        .name               = TYPE_AW_SDHOST,<br>
        .parent            = TYPE_SYS_BUS_DEVICE,<br>
        .instance_init = allwinner_sdhost_init,<br>
diff --git a/hw/sd/aspeed_sdhci.c b/hw/sd/aspeed_sdhci.c<br>
index df1bdf1fa4..be8cafd65f 100644<br>
--- a/hw/sd/aspeed_sdhci.c<br>
+++ b/hw/sd/aspeed_sdhci.c<br>
@@ -198,7 +198,7 @@ static void aspeed_sdhci_class_init(ObjectClass *classp, void \
*data)<br>  device_class_set_props(dc, aspeed_sdhci_properties);<br>
  }<br>
<br>
-static TypeInfo aspeed_sdhci_info = {<br>
+static const TypeInfo aspeed_sdhci_info = {<br>
        .name               = TYPE_ASPEED_SDHCI,<br>
        .parent            = TYPE_SYS_BUS_DEVICE,<br>
        .instance_size = sizeof(AspeedSDHCIState),<br>
diff --git a/hw/sd/bcm2835_sdhost.c b/hw/sd/bcm2835_sdhost.c<br>
index 088a7ac6ed..9431c35914 100644<br>
--- a/hw/sd/bcm2835_sdhost.c<br>
+++ b/hw/sd/bcm2835_sdhost.c<br>
@@ -436,7 +436,7 @@ static void bcm2835_sdhost_class_init(ObjectClass *klass, void \
*data)<br>  dc-&gt;vmsd = &amp;vmstate_bcm2835_sdhost;<br>
  }<br>
<br>
-static TypeInfo bcm2835_sdhost_info = {<br>
+static const TypeInfo bcm2835_sdhost_info = {<br>
        .name               = TYPE_BCM2835_SDHOST,<br>
        .parent            = TYPE_SYS_BUS_DEVICE,<br>
        .instance_size = sizeof(BCM2835SDHostState),<br>
diff --git a/hw/sd/cadence_sdhci.c b/hw/sd/cadence_sdhci.c<br>
index 56b8bae1c3..75db34befe 100644<br>
--- a/hw/sd/cadence_sdhci.c<br>
+++ b/hw/sd/cadence_sdhci.c<br>
@@ -175,7 +175,7 @@ static void cadence_sdhci_class_init(ObjectClass *classp, void \
*data)<br>  dc-&gt;vmsd = &amp;vmstate_cadence_sdhci;<br>
  }<br>
<br>
-static TypeInfo cadence_sdhci_info = {<br>
+static const TypeInfo cadence_sdhci_info = {<br>
        .name               = TYPE_CADENCE_SDHCI,<br>
        .parent            = TYPE_SYS_BUS_DEVICE,<br>
        .instance_size = sizeof(CadenceSDHCIState),<br>
diff --git a/hw/sd/npcm7xx_sdhci.c b/hw/sd/npcm7xx_sdhci.c<br>
index ef503365df..b2f5b4a542 100644<br>
--- a/hw/sd/npcm7xx_sdhci.c<br>
+++ b/hw/sd/npcm7xx_sdhci.c<br>
@@ -166,7 +166,7 @@ static void npcm7xx_sdhci_instance_init(Object *obj)<br>
                                            TYPE_SYSBUS_SDHCI);<br>
  }<br>
<br>
-static TypeInfo npcm7xx_sdhci_info = {<br>
+static const TypeInfo npcm7xx_sdhci_info = {<br>
        .name = TYPE_NPCM7XX_SDHCI,<br>
        .parent = TYPE_SYS_BUS_DEVICE,<br>
        .instance_size = sizeof(NPCM7xxSDHCIState),<br>
diff --git a/hw/usb/dev-mtp.c b/hw/usb/dev-mtp.c<br>
index c1d1694fd0..1e6ac76bef 100644<br>
--- a/hw/usb/dev-mtp.c<br>
+++ b/hw/usb/dev-mtp.c<br>
@@ -2106,7 +2106,7 @@ static void usb_mtp_class_initfn(ObjectClass *klass, void \
*data)<br>  device_class_set_props(dc, mtp_properties);<br>
  }<br>
<br>
-static TypeInfo mtp_info = {<br>
+static const TypeInfo mtp_info = {<br>
        .name               = TYPE_USB_MTP,<br>
        .parent            = TYPE_USB_DEVICE,<br>
        .instance_size = sizeof(MTPState),<br>
diff --git a/hw/usb/host-libusb.c b/hw/usb/host-libusb.c<br>
index d0d46dd0a4..2b35cb6cdd 100644<br>
--- a/hw/usb/host-libusb.c<br>
+++ b/hw/usb/host-libusb.c<br>
@@ -1801,7 +1801,7 @@ static void usb_host_class_initfn(ObjectClass *klass, void \
*data)<br>  set_bit(DEVICE_CATEGORY_BRIDGE, dc-&gt;categories);<br>
  }<br>
<br>
-static TypeInfo usb_host_dev_info = {<br>
+static const TypeInfo usb_host_dev_info = {<br>
        .name               = TYPE_USB_HOST_DEVICE,<br>
        .parent            = TYPE_USB_DEVICE,<br>
        .instance_size = sizeof(USBHostDevice),<br>
diff --git a/hw/vfio/igd.c b/hw/vfio/igd.c<br>
index d4685709a3..afe3fe7efc 100644<br>
--- a/hw/vfio/igd.c<br>
+++ b/hw/vfio/igd.c<br>
@@ -199,7 +199,7 @@ static void vfio_pci_igd_lpc_bridge_class_init(ObjectClass \
*klass, void *data)<br>  k-&gt;class_id = PCI_CLASS_BRIDGE_ISA;<br>
  }<br>
<br>
-static TypeInfo vfio_pci_igd_lpc_bridge_info = {<br>
+static const TypeInfo vfio_pci_igd_lpc_bridge_info = {<br>
        .name = &quot;vfio-pci-igd-lpc-bridge&quot;,<br>
        .parent = TYPE_PCI_DEVICE,<br>
        .class_init = vfio_pci_igd_lpc_bridge_class_init,<br>
diff --git a/hw/virtio/virtio-pmem.c b/hw/virtio/virtio-pmem.c<br>
index d1aeb90a31..5419dca75e 100644<br>
--- a/hw/virtio/virtio-pmem.c<br>
+++ b/hw/virtio/virtio-pmem.c<br>
@@ -182,7 +182,7 @@ static void virtio_pmem_class_init(ObjectClass *klass, void \
*data)<br>  set_bit(DEVICE_CATEGORY_STORAGE, dc-&gt;categories);<br>
  }<br>
<br>
-static TypeInfo virtio_pmem_info = {<br>
+static const TypeInfo virtio_pmem_info = {<br>
        .name               = TYPE_VIRTIO_PMEM,<br>
        .parent            = TYPE_VIRTIO_DEVICE,<br>
        .class_size      = sizeof(VirtIOPMEMClass),<br>
diff --git a/qom/object.c b/qom/object.c<br>
index 4f0677cca9..9f7a33139d 100644<br>
--- a/qom/object.c<br>
+++ b/qom/object.c<br>
@@ -2793,13 +2793,13 @@ static void object_class_init(ObjectClass *klass, void \
*data)<br> <br>
  static void register_types(void)<br>
  {<br>
-      static TypeInfo interface_info = {<br>
+      static const TypeInfo interface_info = {<br>
              .name = TYPE_INTERFACE,<br>
              .class_size = sizeof(InterfaceClass),<br>
              .abstract = true,<br>
        };<br>
<br>
-      static TypeInfo object_info = {<br>
+      static const TypeInfo object_info = {<br>
              .name = TYPE_OBJECT,<br>
              .instance_size = sizeof(Object),<br>
              .class_init = object_class_init,<br>
-- <br>
2.34.1<br>
<br>
</blockquote></div>



[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic