[PATCH v3 1/6] PCI: handle PCI->PCIe bridges as well in free_pdev()

Jan Beulich posted 6 patches 1 week, 4 days ago
[PATCH v3 1/6] PCI: handle PCI->PCIe bridges as well in free_pdev()
Posted by Jan Beulich 1 week, 4 days ago
Don't know how I managed to overlook the freeing side when adding the case
to alloc_pdev().

Fixes: cd2b9f0b1986 ("PCI: handle PCI->PCIe bridges as well in alloc_pdev()")
Signed-off-by: Jan Beulich <jbeulich@suse.com>
---
v3: New.
---
Noticed due to the original patch still applying cleanly, just with an
offset of a few dozen lines.

--- a/xen/drivers/passthrough/pci.c
+++ b/xen/drivers/passthrough/pci.c
@@ -436,6 +436,7 @@ static void free_pdev(struct pci_seg *ps
         unsigned long flags;
 
         case DEV_TYPE_PCIe2PCI_BRIDGE:
+        case DEV_TYPE_PCI2PCIe_BRIDGE:
         case DEV_TYPE_LEGACY_PCI_BRIDGE:
             sec_bus = pci_conf_read8(pdev->sbdf, PCI_SECONDARY_BUS);
             sub_bus = pci_conf_read8(pdev->sbdf, PCI_SUBORDINATE_BUS);
Re: [PATCH v3 1/6] PCI: handle PCI->PCIe bridges as well in free_pdev()
Posted by Roger Pau Monné 1 week, 4 days ago
On Thu, Jan 29, 2026 at 02:07:58PM +0100, Jan Beulich wrote:
> Don't know how I managed to overlook the freeing side when adding the case
> to alloc_pdev().
> 
> Fixes: cd2b9f0b1986 ("PCI: handle PCI->PCIe bridges as well in alloc_pdev()")
> Signed-off-by: Jan Beulich <jbeulich@suse.com>

Acked-by: Roger Pau Monné <roger.pau@citrix.com>

Thanks, Roger.