Re: [PATCH] PCI: Check bridge secondary/subordinate bus register

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Sep 10, 2012 at 6:19 PM, Yinghai Lu <yinghai@xxxxxxxxxx> wrote:
> Intel DP43BF requires "pci=assign-busses" to discover some devices.
>
> It turns out BIOS set the bridge 00:1e.0 bus register wrong.
>
> Try to check that wrong range and reject so next pass could only assign
> bus for that bridge.
>
>         https://bugzilla.kernel.org/show_bug.cgi?id=18412

Applied to pci/yinghai-misc, thanks!

> Reported-by: VL <vl.homutov@xxxxxxxxx>
> Tested-by: VL <vl.homutov@xxxxxxxxx>
> Signed-off-by: Yinghai Lu <yinghai@xxxxxxxxxx>
> Cc: stable@xxxxxxxxxxxxxxx
>
> diff --git a/drivers/pci/probe.c b/drivers/pci/probe.c
> index 3cdba8b..a8f1bf6 100644
> --- a/drivers/pci/probe.c
> +++ b/drivers/pci/probe.c
> @@ -729,7 +729,8 @@ int __devinit pci_scan_bridge(struct pci_bus *bus, struct pci_dev *dev, int max,
>
>         /* Check if setup is sensible at all */
>         if (!pass &&
> -           (primary != bus->number || secondary <= bus->number)) {
> +           (primary != bus->number || secondary <= bus->number ||
> +            secondary > subordinate)) {
>                 dev_dbg(&dev->dev, "bus configuration invalid, reconfiguring\n");
>                 broken = 1;
>         }
--
To unsubscribe from this list: send the line "unsubscribe linux-pci" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux