From: Jag Raman <jag.raman@oracle.com>
To: "Philippe Mathieu-Daudé" <philmd@linaro.org>
Cc: "qemu-devel@nongnu.org" <qemu-devel@nongnu.org>,
"Jamin Lin" <jamin_lin@aspeedtech.com>,
"Peter Maydell" <peter.maydell@linaro.org>,
"Troy Lee" <leetroy@gmail.com>,
"Mark Cave-Ayland" <mark.cave-ayland@ilande.co.uk>,
"Glenn Miles" <milesg@linux.ibm.com>,
"Elena Ufimtseva" <elena.ufimtseva@oracle.com>,
"Cédric Le Goater" <clg@kaod.org>,
"Paul Burton" <paulburton@kernel.org>,
"Joel Stanley" <joel@jms.id.au>,
"Hervé Poussineau" <hpoussin@reactos.org>,
"Aurelien Jarno" <aurelien@aurel32.net>,
"BALATON Zoltan" <balaton@eik.bme.hu>,
"Aditya Gupta" <adityag@linux.ibm.com>,
"Michael S. Tsirkin" <mst@redhat.com>,
"Aleksandar Rikalo" <arikalo@gmail.com>,
"Bernhard Beschow" <shentey@gmail.com>,
"qemu-arm@nongnu.org" <qemu-arm@nongnu.org>,
"Andrew Jeffery" <andrew@codeconstruct.com.au>,
"Steven Lee" <steven_lee@aspeedtech.com>,
"Nicholas Piggin" <npiggin@gmail.com>,
"Marcel Apfelbaum" <marcel.apfelbaum@gmail.com>,
"Andrey Smirnov" <andrew.smirnov@gmail.com>,
"qemu-ppc@nongnu.org" <qemu-ppc@nongnu.org>
Subject: Re: [PATCH] hw/pci-host: Set DEVICE_CATEGORY_BRIDGE once in parent class_init()
Date: Mon, 27 Oct 2025 15:15:57 +0000 [thread overview]
Message-ID: <F7CDF3CF-9E6C-462F-9BC6-0AAB5C02EC70@oracle.com> (raw)
In-Reply-To: <20251027112306.57634-1-philmd@linaro.org>
> On Oct 27, 2025, at 7:23 AM, Philippe Mathieu-Daudé <philmd@linaro.org> wrote:
>
> No need to set DEVICE_CATEGORY_BRIDGE for each device
> implementing TYPE_PCI_HOST_BRIDGE: set it once in the
> parent.
>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---
> hw/pci-host/articia.c | 1 -
> hw/pci-host/aspeed_pcie.c | 1 -
> hw/pci-host/designware.c | 3 ---
> hw/pci-host/gpex.c | 2 --
> hw/pci-host/grackle.c | 1 -
> hw/pci-host/gt64120.c | 1 -
> hw/pci-host/mv64361.c | 1 -
> hw/pci-host/pnv_phb.c | 1 -
> hw/pci-host/ppce500.c | 1 -
> hw/pci-host/q35.c | 1 -
> hw/pci-host/raven.c | 1 -
> hw/pci-host/remote.c | 1 -
> hw/pci-host/sabre.c | 1 -
> hw/pci-host/uninorth.c | 4 ----
> hw/pci-host/xilinx-pcie.c | 2 --
> hw/pci/pci_host.c | 1 +
> 16 files changed, 1 insertion(+), 22 deletions(-)
>
> diff --git a/hw/pci-host/articia.c b/hw/pci-host/articia.c
> index cc65aac2a87..47f223879f0 100644
> --- a/hw/pci-host/articia.c
> +++ b/hw/pci-host/articia.c
> @@ -200,7 +200,6 @@ static void articia_class_init(ObjectClass *klass, const void *data)
> DeviceClass *dc = DEVICE_CLASS(klass);
>
> dc->realize = articia_realize;
> - set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories);
> }
>
> /* TYPE_ARTICIA_PCI_HOST */
> diff --git a/hw/pci-host/aspeed_pcie.c b/hw/pci-host/aspeed_pcie.c
> index f7593444fc4..2499d3fe680 100644
> --- a/hw/pci-host/aspeed_pcie.c
> +++ b/hw/pci-host/aspeed_pcie.c
> @@ -312,7 +312,6 @@ static void aspeed_pcie_rc_class_init(ObjectClass *klass, const void *data)
> dc->desc = "ASPEED PCIe RC";
> dc->realize = aspeed_pcie_rc_realize;
> dc->fw_name = "pci";
> - set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories);
>
> hc->root_bus_path = aspeed_pcie_rc_root_bus_path;
> device_class_set_props(dc, aspeed_pcie_rc_props);
> diff --git a/hw/pci-host/designware.c b/hw/pci-host/designware.c
> index f6e49ce9b8d..b0b51ad9517 100644
> --- a/hw/pci-host/designware.c
> +++ b/hw/pci-host/designware.c
> @@ -593,8 +593,6 @@ static void designware_pcie_root_class_init(ObjectClass *klass,
> PCIDeviceClass *k = PCI_DEVICE_CLASS(klass);
> DeviceClass *dc = DEVICE_CLASS(klass);
>
> - set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories);
> -
> k->vendor_id = PCI_VENDOR_ID_SYNOPSYS;
> k->device_id = 0xABCD;
> k->revision = 0;
> @@ -736,7 +734,6 @@ static void designware_pcie_host_class_init(ObjectClass *klass,
>
> hc->root_bus_path = designware_pcie_host_root_bus_path;
> dc->realize = designware_pcie_host_realize;
> - set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories);
> dc->fw_name = "pci";
> dc->vmsd = &vmstate_designware_pcie_host;
> }
> diff --git a/hw/pci-host/gpex.c b/hw/pci-host/gpex.c
> index b806a2286f7..5f809028be2 100644
> --- a/hw/pci-host/gpex.c
> +++ b/hw/pci-host/gpex.c
> @@ -200,7 +200,6 @@ static void gpex_host_class_init(ObjectClass *klass, const void *data)
> hc->root_bus_path = gpex_host_root_bus_path;
> dc->realize = gpex_host_realize;
> dc->unrealize = gpex_host_unrealize;
> - set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories);
> dc->fw_name = "pci";
> device_class_set_props(dc, gpex_host_properties);
> }
> @@ -242,7 +241,6 @@ static void gpex_root_class_init(ObjectClass *klass, const void *data)
> PCIDeviceClass *k = PCI_DEVICE_CLASS(klass);
> DeviceClass *dc = DEVICE_CLASS(klass);
>
> - set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories);
> dc->desc = "QEMU generic PCIe host bridge";
> dc->vmsd = &vmstate_gpex_root;
> k->vendor_id = PCI_VENDOR_ID_REDHAT;
> diff --git a/hw/pci-host/grackle.c b/hw/pci-host/grackle.c
> index f9da5a908c1..fa872c17e3b 100644
> --- a/hw/pci-host/grackle.c
> +++ b/hw/pci-host/grackle.c
> @@ -140,7 +140,6 @@ static void grackle_class_init(ObjectClass *klass, const void *data)
>
> dc->realize = grackle_realize;
> device_class_set_props(dc, grackle_properties);
> - set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories);
> dc->fw_name = "pci";
> sbc->explicit_ofw_unit_address = grackle_ofw_unit_address;
> }
> diff --git a/hw/pci-host/gt64120.c b/hw/pci-host/gt64120.c
> index b1d96f62fe9..bffa1165580 100644
> --- a/hw/pci-host/gt64120.c
> +++ b/hw/pci-host/gt64120.c
> @@ -1298,7 +1298,6 @@ static void gt64120_class_init(ObjectClass *klass, const void *data)
> {
> DeviceClass *dc = DEVICE_CLASS(klass);
>
> - set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories);
> device_class_set_props(dc, gt64120_properties);
> dc->realize = gt64120_realize;
> device_class_set_legacy_reset(dc, gt64120_reset);
> diff --git a/hw/pci-host/mv64361.c b/hw/pci-host/mv64361.c
> index e05b6770102..e5e6614b139 100644
> --- a/hw/pci-host/mv64361.c
> +++ b/hw/pci-host/mv64361.c
> @@ -108,7 +108,6 @@ static void mv64361_pcihost_class_init(ObjectClass *klass, const void *data)
>
> dc->realize = mv64361_pcihost_realize;
> device_class_set_props(dc, mv64361_pcihost_props);
> - set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories);
> }
>
> static const TypeInfo mv64361_pcihost_info = {
> diff --git a/hw/pci-host/pnv_phb.c b/hw/pci-host/pnv_phb.c
> index 4b0ced79b08..e07cf8a0d0f 100644
> --- a/hw/pci-host/pnv_phb.c
> +++ b/hw/pci-host/pnv_phb.c
> @@ -202,7 +202,6 @@ static void pnv_phb_class_init(ObjectClass *klass, const void *data)
> hc->root_bus_path = pnv_phb_root_bus_path;
> dc->realize = pnv_phb_realize;
> device_class_set_props(dc, pnv_phb_properties);
> - set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories);
> dc->user_creatable = true;
> }
>
> diff --git a/hw/pci-host/ppce500.c b/hw/pci-host/ppce500.c
> index 975d191ccb8..edadc35384c 100644
> --- a/hw/pci-host/ppce500.c
> +++ b/hw/pci-host/ppce500.c
> @@ -516,7 +516,6 @@ static void e500_pcihost_class_init(ObjectClass *klass, const void *data)
> DeviceClass *dc = DEVICE_CLASS(klass);
>
> dc->realize = e500_pcihost_realize;
> - set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories);
> device_class_set_props(dc, pcihost_properties);
> dc->vmsd = &vmstate_ppce500_pci;
> }
> diff --git a/hw/pci-host/q35.c b/hw/pci-host/q35.c
> index 1951ae440cc..c134679fa0c 100644
> --- a/hw/pci-host/q35.c
> +++ b/hw/pci-host/q35.c
> @@ -194,7 +194,6 @@ static void q35_host_class_init(ObjectClass *klass, const void *data)
> device_class_set_props(dc, q35_host_props);
> /* Reason: needs to be wired up by pc_q35_init */
> dc->user_creatable = false;
> - set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories);
> dc->fw_name = "pci";
> }
>
> diff --git a/hw/pci-host/raven.c b/hw/pci-host/raven.c
> index eacffc86d84..661a4223d69 100644
> --- a/hw/pci-host/raven.c
> +++ b/hw/pci-host/raven.c
> @@ -365,7 +365,6 @@ static void raven_pcihost_class_init(ObjectClass *klass, const void *data)
> {
> DeviceClass *dc = DEVICE_CLASS(klass);
>
> - set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories);
> dc->realize = raven_pcihost_realizefn;
> dc->fw_name = "pci";
> }
> diff --git a/hw/pci-host/remote.c b/hw/pci-host/remote.c
> index e6d2af4502a..959beb5c301 100644
> --- a/hw/pci-host/remote.c
> +++ b/hw/pci-host/remote.c
> @@ -55,7 +55,6 @@ static void remote_pcihost_class_init(ObjectClass *klass, const void *data)
> dc->realize = remote_pcihost_realize;
>
> dc->user_creatable = false;
> - set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories);
Thanks for the patch, Philippe! The change to remote.c looks good.
--
Jag
> dc->fw_name = "pci";
> }
>
> diff --git a/hw/pci-host/sabre.c b/hw/pci-host/sabre.c
> index 538624c5079..f95e5db583a 100644
> --- a/hw/pci-host/sabre.c
> +++ b/hw/pci-host/sabre.c
> @@ -505,7 +505,6 @@ static void sabre_class_init(ObjectClass *klass, const void *data)
> dc->realize = sabre_realize;
> device_class_set_legacy_reset(dc, sabre_reset);
> device_class_set_props(dc, sabre_properties);
> - set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories);
> dc->fw_name = "pci";
> sbc->explicit_ofw_unit_address = sabre_ofw_unit_address;
> }
> diff --git a/hw/pci-host/uninorth.c b/hw/pci-host/uninorth.c
> index 194037d6e75..d337a2fd62c 100644
> --- a/hw/pci-host/uninorth.c
> +++ b/hw/pci-host/uninorth.c
> @@ -435,7 +435,6 @@ static void pci_unin_main_class_init(ObjectClass *klass, const void *data)
>
> dc->realize = pci_unin_main_realize;
> device_class_set_props(dc, pci_unin_main_pci_host_props);
> - set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories);
> dc->fw_name = "pci";
> sbc->explicit_ofw_unit_address = pci_unin_main_ofw_unit_address;
> }
> @@ -453,7 +452,6 @@ static void pci_u3_agp_class_init(ObjectClass *klass, const void *data)
> DeviceClass *dc = DEVICE_CLASS(klass);
>
> dc->realize = pci_u3_agp_realize;
> - set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories);
> }
>
> static const TypeInfo pci_u3_agp_info = {
> @@ -469,7 +467,6 @@ static void pci_unin_agp_class_init(ObjectClass *klass, const void *data)
> DeviceClass *dc = DEVICE_CLASS(klass);
>
> dc->realize = pci_unin_agp_realize;
> - set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories);
> }
>
> static const TypeInfo pci_unin_agp_info = {
> @@ -485,7 +482,6 @@ static void pci_unin_internal_class_init(ObjectClass *klass, const void *data)
> DeviceClass *dc = DEVICE_CLASS(klass);
>
> dc->realize = pci_unin_internal_realize;
> - set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories);
> }
>
> static const TypeInfo pci_unin_internal_info = {
> diff --git a/hw/pci-host/xilinx-pcie.c b/hw/pci-host/xilinx-pcie.c
> index c71492de9e7..1540533223c 100644
> --- a/hw/pci-host/xilinx-pcie.c
> +++ b/hw/pci-host/xilinx-pcie.c
> @@ -172,7 +172,6 @@ static void xilinx_pcie_host_class_init(ObjectClass *klass, const void *data)
>
> hc->root_bus_path = xilinx_pcie_host_root_bus_path;
> dc->realize = xilinx_pcie_host_realize;
> - set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories);
> dc->fw_name = "pci";
> device_class_set_props(dc, xilinx_pcie_host_props);
> }
> @@ -291,7 +290,6 @@ static void xilinx_pcie_root_class_init(ObjectClass *klass, const void *data)
> PCIDeviceClass *k = PCI_DEVICE_CLASS(klass);
> DeviceClass *dc = DEVICE_CLASS(klass);
>
> - set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories);
> dc->desc = "Xilinx AXI-PCIe Host Bridge";
> k->vendor_id = PCI_VENDOR_ID_XILINX;
> k->device_id = 0x7021;
> diff --git a/hw/pci/pci_host.c b/hw/pci/pci_host.c
> index 7179d99178b..b5c624e12e8 100644
> --- a/hw/pci/pci_host.c
> +++ b/hw/pci/pci_host.c
> @@ -245,6 +245,7 @@ static void pci_host_class_init(ObjectClass *klass, const void *data)
> DeviceClass *dc = DEVICE_CLASS(klass);
> device_class_set_props(dc, pci_host_properties_common);
> dc->vmsd = &vmstate_pcihost;
> + set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories);
> }
>
> static const TypeInfo pci_host_type_info = {
> --
> 2.51.0
>
next prev parent reply other threads:[~2025-10-27 15:20 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-10-27 11:23 [PATCH] hw/pci-host: Set DEVICE_CATEGORY_BRIDGE once in parent class_init() Philippe Mathieu-Daudé
2025-10-27 15:15 ` Jag Raman [this message]
2025-10-28 7:16 ` Marc-André Lureau
2025-10-28 7:24 ` Philippe Mathieu-Daudé
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=F7CDF3CF-9E6C-462F-9BC6-0AAB5C02EC70@oracle.com \
--to=jag.raman@oracle.com \
--cc=adityag@linux.ibm.com \
--cc=andrew.smirnov@gmail.com \
--cc=andrew@codeconstruct.com.au \
--cc=arikalo@gmail.com \
--cc=aurelien@aurel32.net \
--cc=balaton@eik.bme.hu \
--cc=clg@kaod.org \
--cc=elena.ufimtseva@oracle.com \
--cc=hpoussin@reactos.org \
--cc=jamin_lin@aspeedtech.com \
--cc=joel@jms.id.au \
--cc=leetroy@gmail.com \
--cc=marcel.apfelbaum@gmail.com \
--cc=mark.cave-ayland@ilande.co.uk \
--cc=milesg@linux.ibm.com \
--cc=mst@redhat.com \
--cc=npiggin@gmail.com \
--cc=paulburton@kernel.org \
--cc=peter.maydell@linaro.org \
--cc=philmd@linaro.org \
--cc=qemu-arm@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=qemu-ppc@nongnu.org \
--cc=shentey@gmail.com \
--cc=steven_lee@aspeedtech.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).