Re: [PATCH 1/2] pci: release that leaf bridge' resource that is notbig -v11

[Kenji Kaneshige]

Yinghai Lu wrote:
> > Bad news...
> >
> > My system doesn't boot (hangup) with your latest set of patches.
> > Fusion MPT SAS driver initialization failed on some devices.
> > Please see below
> >
> > ...
> > Fusion MPT SAS Host driver 3.04.12
> > mptsas 0000:09:00.0: PCI INT A -> GSI 18 (level, low) -> IRQ 18
> > mptsas 0000:09:00.0: BAR 1: can't reserve [mem 0x00510000-0x00513fff 64bit]
> > mptbase: ioc0: ERROR - pci_request_selected_regions() with MEM failed
> > mptsas 0000:0a:00.0: PCI INT A -> GSI 19 (level, low) -> IRQ 19
> > mptsas 0000:0a:00.0: BAR 1: can't reserve [mem 0x00610000-0x00613fff 64bit]
> > mptbase: ioc1: ERROR - pci_request_selected_regions() with MEM failed
> > ...
> >
> >
> > This problem disappear when I revert the patch 6/9, and my system
> > can boot. But I found igb driver initialization failed on some
> > devices even in this case. Please see below
> >
> > ...
> > igb 0000:08:00.0: BAR 0: can't reserve [mem 0x00100000-0x0011ffff]
> > igb 0000:08:00.0: PCI INT A disabled
> > igb: probe of 0000:08:00.0 failed with error -16
> > igb 0000:08:00.1: PCI INT B -> GSI 18 (level, low) -> IRQ 18
> > igb 0000:08:00.1: BAR 0: can't reserve [mem 0x00140000-0x0015ffff]
> > igb 0000:08:00.1: PCI INT B disabled
> > igb: probe of 0000:08:00.1 failed with error -16
> > ...
>
> that looks werid, without patch 6/9 should only have pci bridge res shrink problem with your last test?

I don't know why. I might have overlooked it at the previous
test.

> maybe we need keep that feature default to be disabled.

Will do it next week.

Thanks,
Kenji Kaneshige


> please try
>
> ---
>  drivers/pci/setup-bus.c |    3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
>
> Index: linux-2.6/drivers/pci/setup-bus.c
> ===================================================================
> --- linux-2.6.orig/drivers/pci/setup-bus.c
> +++ linux-2.6/drivers/pci/setup-bus.c
> @@ -918,8 +918,7 @@ static void pci_bus_dump_resources(struc
>   * second try will clear small leaf bridge res
>   * third try will clear related bridge: some aggressive
>   */
> -/* assume we only have 4 level bridges, so only try 5 times */
> -int pci_try_num = 5;
> +int pci_try_num = 1;
>  void __init
>  pci_assign_unassigned_resources(void)
>  {


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/