File 61532102-PCI-bridge-with-subord-bus-0xFF.patch of Package xen.30827

Overview Repositories Revisions Requests Users Attributes Meta

File 61532102-PCI-bridge-with-subord-bus-0xFF.patch of Package xen.30827

# Commit 9c3b9800e2019c93ab22da69e4a0b22d6fb059ec
# Date 2021-09-28 16:04:50 +0200
# Author Igor Druzhinin <igor.druzhinin@citrix.com>
# Committer Jan Beulich <jbeulich@suse.com>
pci: fix handling of PCI bridges with subordinate bus number 0xff

Bus number 0xff is valid according to the PCI spec. Using u8 typed sub_bus
and assigning 0xff to it will result in the following loop getting stuck.

for ( ; sec_bus <= sub_bus; sec_bus++ ) {...}

Just change its type to unsigned int similarly to what is already done in
dmar_scope_add_buses().

Signed-off-by: Igor Druzhinin <igor.druzhinin@citrix.com>
Reviewed-by: Jan Beulich <jbeulich@suse.com>
Reviewed-by: Bertrand Marquis <bertrand.marquis@arm.com>

--- a/xen/drivers/passthrough/pci.c
+++ b/xen/drivers/passthrough/pci.c
@@ -365,8 +365,7 @@ static struct pci_dev *alloc_pdev(struct
     switch ( pdev->type = pdev_type(pseg->nr, bus, devfn) )
     {
         int pos;
-        u16 cap;
-        u8 sec_bus, sub_bus;
+        unsigned int cap, sec_bus, sub_bus;
 
         case DEV_TYPE_PCIe2PCI_BRIDGE:
         case DEV_TYPE_LEGACY_PCI_BRIDGE:
@@ -436,7 +435,7 @@ static void free_pdev(struct pci_seg *ps
     /* update bus2bridge */
     switch ( pdev->type )
     {
-        u8 dev, func, sec_bus, sub_bus;
+        unsigned int dev, func, sec_bus, sub_bus;
 
         case DEV_TYPE_PCIe2PCI_BRIDGE:
         case DEV_TYPE_LEGACY_PCI_BRIDGE:

Places

File 61532102-PCI-bridge-with-subord-bus-0xFF.patch of Package xen.30827

Places