Re: [PATCH v2] pci: fix handling of PCI bridges with subordinate bus number 0xff

2021-09-28 Thread Bertrand Marquis
Hi, > On 24 Sep 2021, at 10:10, Igor Druzhinin wrote: > > 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

Re: [PATCH v2] pci: fix handling of PCI bridges with subordinate bus number 0xff

2021-09-24 Thread Jan Beulich
On 24.09.2021 11:10, Igor Druzhinin wrote: > 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

[PATCH v2] pci: fix handling of PCI bridges with subordinate bus number 0xff

2021-09-24 Thread Igor Druzhinin
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().