From: Markus Armbruster <armbru@redhat.com>
To: qemu-devel@nongnu.org
Cc: pbonzini@redhat.com, berrange@redhat.com, ehabkost@redhat.com
Subject: [PATCH v2 00/24] Fixes around device realization
Date: Thu, 28 May 2020 13:04:20 +0200 [thread overview]
Message-ID: <20200528110444.20456-1-armbru@redhat.com> (raw)
This fixes a bunch of bugs I ran into while reworking how qdevs plug
into buses.
I instrumented the code a bit to flush out instances of bug patterns.
I posted these hacks separately as '[PATCH not-for-merge 0/5]
Instrumentation for "Fixes around device realization"'. PATCH 2/5
since became "[PATCH 0/2] qom: Make "info qom-tree" show children
sorted". It should be applied first.
v2:
* Rebased
* PATCH 01: Also fix MMIO addresses, with Alistair's help
* PATCH 04+05: Replaced by better patches from Cédric
* PATCH 01-03+06+08-11+18: Commit messages improved [Peter, Paolo]
* PATCH 08+09+18: Avoid qdev_init_nofail() [Peter]
* PATCH 22: Assertion simplified
Based-on: Message-Id: <20200527084754.7531-1-armbru@redhat.com>
Cédric Le Goater (2):
arm/aspeed: Compute the number of CPUs from the SoC definition
arm/aspeed: Rework NIC attachment
Markus Armbruster (22):
arm/stm32f405: Fix realization of "stm32f2xx-adc" devices
display/xlnx_dp: Fix to realize "i2c-ddc" and "aux-to-i2c-bridge"
sd/pxa2xx_mmci: Fix to realize "pxa2xx-mmci" device
armv7m: Delete unused "ARM,bitband-memory" devices
auxbus: Fix aux-to-i2c-bridge to be a subtype of aux-slave
mac_via: Fix to realize "mos6522-q800-via*" devices
macio: Fix to realize "mos6522-cuda" and "mos6522-pmu" devices
macio: Delete unused "macio-gpio" devices
pnv/phb4: Delete unused "pnv-phb4-pec-stack" devices
MAINTAINERS: Make section PowerNV cover pci-host/pnv* as well
ppc4xx: Drop redundant device realization
macio: Put "macio-nvram" device on the macio bus
macio: Fix macio-bus to be a subtype of System bus
ppc/pnv: Put "*-pnv-chip" and "pnv-xive" on the main system bus
pnv/psi: Correct the pnv-psi* devices not to be sysbus devices
display/sm501 display/ati: Fix to realize "i2c-ddc"
riscv: Fix to put "riscv.hart_array" devices on sysbus
riscv: Fix type of SiFive[EU]SocState, member parent_obj
sparc/leon3: Fix to put grlib,* devices on sysbus
qdev: Assert devices are plugged into a bus that can take them
sd: Hide the qdev-but-not-quite thing created by sd_init()
qdev: Assert onboard devices all get realized properly
include/hw/arm/aspeed.h | 6 ++++++
include/hw/arm/aspeed_soc.h | 1 -
include/hw/ppc/pnv_psi.h | 2 +-
include/hw/riscv/sifive_e.h | 2 +-
include/hw/riscv/sifive_u.h | 2 +-
hw/arm/armv7m.c | 6 ++++--
hw/arm/aspeed.c | 42 ++++++++++++++++++++++++++++++++-----
hw/arm/aspeed_ast2600.c | 23 +++++++-------------
hw/arm/aspeed_soc.c | 12 ++---------
hw/arm/stm32f405_soc.c | 23 +++++++++++---------
hw/core/qdev.c | 19 +++++++++++++++++
hw/display/ati.c | 2 ++
hw/display/sm501.c | 2 ++
hw/display/xlnx_dp.c | 4 ++++
hw/misc/auxbus.c | 2 +-
hw/misc/mac_via.c | 5 +++++
hw/misc/macio/cuda.c | 10 ++++-----
hw/misc/macio/macio.c | 7 +++++--
hw/misc/macio/pmu.c | 10 ++++-----
hw/pci-host/pnv_phb4_pec.c | 3 +++
hw/ppc/pnv.c | 6 +++---
hw/ppc/pnv_psi.c | 2 +-
hw/ppc/ppc440_uc.c | 2 --
hw/riscv/sifive_e.c | 5 ++---
hw/riscv/sifive_u.c | 14 ++++++-------
hw/riscv/spike.c | 12 +++++------
hw/riscv/virt.c | 4 ++--
hw/sd/pxa2xx_mmci.c | 1 +
hw/sd/sd.c | 40 ++++++++++++++++++++++++-----------
hw/sparc/leon3.c | 4 ++--
MAINTAINERS | 2 ++
31 files changed, 177 insertions(+), 98 deletions(-)
--
2.21.3
next reply other threads:[~2020-05-28 11:06 UTC|newest]
Thread overview: 52+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-05-28 11:04 Markus Armbruster [this message]
2020-05-28 11:04 ` [PATCH v2 01/24] arm/stm32f405: Fix realization of "stm32f2xx-adc" devices Markus Armbruster
2020-05-28 11:35 ` Philippe Mathieu-Daudé
2020-05-28 11:04 ` [PATCH v2 02/24] display/xlnx_dp: Fix to realize "i2c-ddc" and "aux-to-i2c-bridge" Markus Armbruster
2020-06-08 14:16 ` Philippe Mathieu-Daudé
2020-06-09 5:38 ` Markus Armbruster
2020-06-09 7:42 ` Philippe Mathieu-Daudé
2020-06-09 9:35 ` Markus Armbruster
2020-05-28 11:04 ` [PATCH v2 03/24] sd/pxa2xx_mmci: Fix to realize "pxa2xx-mmci" device Markus Armbruster
2020-06-08 14:04 ` Philippe Mathieu-Daudé
2020-05-28 11:04 ` [PATCH v2 04/24] arm/aspeed: Compute the number of CPUs from the SoC definition Markus Armbruster
2020-05-28 11:04 ` [PATCH v2 05/24] arm/aspeed: Rework NIC attachment Markus Armbruster
2020-05-28 11:04 ` [PATCH v2 06/24] armv7m: Delete unused "ARM,bitband-memory" devices Markus Armbruster
2020-06-08 14:02 ` Philippe Mathieu-Daudé
2020-06-08 14:23 ` [PATCH v2 06/24] armv7m: Delete unused "ARM, bitband-memory" devices Peter Maydell
2020-05-28 11:04 ` [PATCH v2 07/24] auxbus: Fix aux-to-i2c-bridge to be a subtype of aux-slave Markus Armbruster
2020-05-28 11:04 ` [PATCH v2 08/24] mac_via: Fix to realize "mos6522-q800-via*" devices Markus Armbruster
2020-05-28 11:04 ` [PATCH v2 09/24] macio: Fix to realize "mos6522-cuda" and "mos6522-pmu" devices Markus Armbruster
2020-06-08 14:12 ` Philippe Mathieu-Daudé
2020-06-08 14:25 ` Peter Maydell
2020-06-09 7:05 ` Markus Armbruster
2020-05-28 11:04 ` [PATCH v2 10/24] macio: Delete unused "macio-gpio" devices Markus Armbruster
2020-06-08 11:54 ` Mark Cave-Ayland
2020-06-09 5:42 ` Markus Armbruster
2020-05-28 11:04 ` [PATCH v2 11/24] pnv/phb4: Delete unused "pnv-phb4-pec-stack" devices Markus Armbruster
2020-05-28 11:04 ` [PATCH v2 12/24] MAINTAINERS: Make section PowerNV cover pci-host/pnv* as well Markus Armbruster
2020-05-28 11:04 ` [PATCH v2 13/24] ppc4xx: Drop redundant device realization Markus Armbruster
2020-05-28 11:04 ` [PATCH v2 14/24] macio: Put "macio-nvram" device on the macio bus Markus Armbruster
2020-06-08 14:04 ` Philippe Mathieu-Daudé
2020-05-28 11:04 ` [PATCH v2 15/24] macio: Fix macio-bus to be a subtype of System bus Markus Armbruster
2020-05-28 11:04 ` [PATCH v2 16/24] ppc/pnv: Put "*-pnv-chip" and "pnv-xive" on the main system bus Markus Armbruster
2020-05-28 11:04 ` [PATCH v2 17/24] pnv/psi: Correct the pnv-psi* devices not to be sysbus devices Markus Armbruster
2020-05-28 11:04 ` [PATCH v2 18/24] display/sm501 display/ati: Fix to realize "i2c-ddc" Markus Armbruster
2020-05-28 11:08 ` Aleksandar Markovic
2020-06-08 14:07 ` Philippe Mathieu-Daudé
2020-05-28 11:04 ` [PATCH v2 19/24] riscv: Fix to put "riscv.hart_array" devices on sysbus Markus Armbruster
2020-05-28 11:04 ` Markus Armbruster
2020-05-28 11:04 ` [PATCH v2 20/24] riscv: Fix type of SiFive[EU]SocState, member parent_obj Markus Armbruster
2020-05-28 11:04 ` Markus Armbruster
2020-05-28 11:04 ` [PATCH v2 21/24] sparc/leon3: Fix to put grlib,* devices on sysbus Markus Armbruster
2020-06-09 5:15 ` Philippe Mathieu-Daudé
2020-06-09 7:20 ` Philippe Mathieu-Daudé
2020-05-28 11:04 ` [PATCH v2 22/24] qdev: Assert devices are plugged into a bus that can take them Markus Armbruster
2020-05-28 11:04 ` [PATCH v2 23/24] sd: Hide the qdev-but-not-quite thing created by sd_init() Markus Armbruster
2020-06-08 14:24 ` Philippe Mathieu-Daudé
2020-06-09 6:39 ` Markus Armbruster
2020-06-09 7:38 ` Philippe Mathieu-Daudé
2020-05-28 11:04 ` [PATCH v2 24/24] qdev: Assert onboard devices all get realized properly Markus Armbruster
2020-06-05 15:01 ` [PATCH v2 00/24] Fixes around device realization Markus Armbruster
2020-06-05 20:30 ` Paolo Bonzini
2020-06-08 11:08 ` Markus Armbruster
2020-06-08 11:44 ` Mark Cave-Ayland
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200528110444.20456-1-armbru@redhat.com \
--to=armbru@redhat.com \
--cc=berrange@redhat.com \
--cc=ehabkost@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.