Since the downstream port is not checked, two slots can be linked to
a single port. However, this can prevent the driver from detecting the
device properly.
It is necessary to ensure that a downstream port is not linked more than
once.
Links: https://lore.kernel.org/qemu-devel/OSZPR01MB6453BC61D2FF4035F18084EF8DDC2@OSZPR01MB6453.jpnprd01.prod.outlook.com
Signed-off-by: Yao Xingtao <yaoxt.fnst@fujitsu.com>
---
hw/pci-bridge/cxl_downstream.c | 6 ++++++
hw/pci-bridge/pcie_root_port.c | 6 ++++++
hw/pci-bridge/xio3130_downstream.c | 6 ++++++
3 files changed, 18 insertions(+)
diff --git a/hw/pci-bridge/cxl_downstream.c b/hw/pci-bridge/cxl_downstream.c
index 742da07a01..fa33cdb36b 100644
--- a/hw/pci-bridge/cxl_downstream.c
+++ b/hw/pci-bridge/cxl_downstream.c
@@ -153,6 +153,12 @@ static void cxl_dsp_realize(PCIDevice *d, Error **errp)
goto err_bridge;
}
+ if (pcie_find_port_by_pn(pci_get_bus(d), p->port) != NULL) {
+ rc = -EBUSY;
+ error_setg(errp, "Can't link port, error %d", rc);
+ goto err_msi;
+ }
+
rc = pcie_cap_init(d, CXL_DOWNSTREAM_PORT_EXP_OFFSET,
PCI_EXP_TYPE_DOWNSTREAM, p->port,
errp);
diff --git a/hw/pci-bridge/pcie_root_port.c b/hw/pci-bridge/pcie_root_port.c
index 09a34786bc..350137bd8c 100644
--- a/hw/pci-bridge/pcie_root_port.c
+++ b/hw/pci-bridge/pcie_root_port.c
@@ -89,6 +89,12 @@ static void rp_realize(PCIDevice *d, Error **errp)
}
}
+ if (pcie_find_port_by_pn(pci_get_bus(d), p->port) != NULL) {
+ rc = -EBUSY;
+ error_setg(errp, "Can't link port, error %d", rc);
+ goto err_int;
+ }
+
rc = pcie_cap_init(d, rpc->exp_offset, PCI_EXP_TYPE_ROOT_PORT,
p->port, errp);
if (rc < 0) {
diff --git a/hw/pci-bridge/xio3130_downstream.c b/hw/pci-bridge/xio3130_downstream.c
index 907d5105b0..af04d2efad 100644
--- a/hw/pci-bridge/xio3130_downstream.c
+++ b/hw/pci-bridge/xio3130_downstream.c
@@ -88,6 +88,12 @@ static void xio3130_downstream_realize(PCIDevice *d, Error **errp)
goto err_msi;
}
+ if (pcie_find_port_by_pn(pci_get_bus(d), p->port) != NULL) {
+ rc = -EBUSY;
+ error_setg(errp, "Can't link port, error %d", rc);
+ goto err_msi;
+ }
+
rc = pcie_cap_init(d, XIO3130_EXP_OFFSET, PCI_EXP_TYPE_DOWNSTREAM,
p->port, errp);
if (rc < 0) {
--
2.37.3