Hi,
This series ensure:
- qbus_new() and sysbus_init_mmio() are called *before*
a device is realized,
- sysbus_mmio_map() is called *after* it is realized.
First we fix some abuse, then we enforce in qdev/sysbus
core code.
There is still a failure in PXA2xx:
qemu-system-aarch64: sysbus_init_mmio(type:pxa2xx_pic) but object is realized
Apparently it cames from commit 3f6c925f37 ("Use i2c_slave_init() to
allocate the PXA (dummy) I2C slave"), which I presume was how to model
slave<->master transactions *before* I2C bus modelling.
Based-on: <[email protected]>
"hw/ppc: SysBus simplifications" [1]
Based-on: <[email protected]>
"hw/arm/pxa2xx: SysBus/QDev fixes" [2]
v1: https://lore.kernel.org/qemu-devel/[email protected]/
[1] https://lore.kernel.org/qemu-devel/[email protected]/
[2] https://lore.kernel.org/qemu-devel/[email protected]/
Philippe Mathieu-Daudé (12):
hw/i386/amd_iommu: Do not use SysBus API to map local MMIO region
hw/i386/intel_iommu: Do not use SysBus API to map local MMIO region
hw/misc/allwinner-dramc: Move sysbus_mmio_map call from init ->
realize
hw/misc/allwinner-dramc: Do not use SysBus API to map local MMIO
region
hw/pci-host/bonito: Do not use SysBus API to map local MMIO region
hw/acpi: Realize ACPI_GED sysbus device before accessing it
hw/arm/virt: Realize ARM_GICV2M sysbus device before accessing it
hw/isa: Realize ISA BUS sysbus device before accessing it
hw/s390x/css-bridge: Realize sysbus device before accessing it
hw/qdev: Ensure parent device is not realized before adding bus
hw/sysbus: Ensure device is not realized before adding MMIO region
hw/sysbus: Ensure device is realized before mapping it
hw/arm/virt.c | 5 ++---
hw/core/bus.c | 7 +++++++
hw/core/sysbus.c | 13 +++++++++++++
hw/i386/amd_iommu.c | 5 ++---
hw/i386/intel_iommu.c | 5 ++---
hw/i386/microvm.c | 2 +-
hw/isa/isa-bus.c | 11 +++++++++--
hw/loongarch/virt.c | 2 +-
hw/misc/allwinner-r40-dramc.c | 20 +++++++++-----------
hw/pci-host/bonito.c | 32 ++++++++++++++++----------------
hw/s390x/css-bridge.c | 7 +++----
11 files changed, 65 insertions(+), 44 deletions(-)
--
2.41.0
