Re: [PATCH 2/2] eisa, PCI: init eisa early before pnp step in

From: Bjorn Helgaas
Date: Mon Apr 01 2013 - 14:15:36 EST


On Fri, Mar 29, 2013 at 6:18 PM, Yinghai Lu <yinghai@xxxxxxxxxx> wrote:
> On Fri, Mar 29, 2013 at 3:22 PM, Bjorn Helgaas <bhelgaas@xxxxxxxxxx> wrote:
>> [+cc Rafael, just FYI since it involves PNP resources]
>>
>> On Wed, Mar 27, 2013 at 10:28 PM, Yinghai Lu <yinghai@xxxxxxxxxx> wrote:
>>> Mathhew reported kernels fail the pci_eisa probe and are later successful
>>> with the virtual_eisa_root_init force probe without slot0.
>>>
>>> The reason for that is: pnp probing is early than pci_eisa_init get called
>>> as pci_eisa_init is called via pci_driver.
>>>
>>> pnp 00:0f has 0xc80 - 0xc84 reserved.
>>> [ 9.700409] pnp 00:0f: [io 0x0c80-0x0c84]
>>>
>>> so eisa_probe will fail from pci_eisa_init
>>> ==>eisa_root_register
>>> ==>eisa_probe path.
>>> as force_probe is not set in pci_eisa_root, it will bail early when
>>> slot0 is not probed and initialized.
>>>
>>> Try to use subsys_initcall_sync instead, and will keep following sequence:
>>> pci_subsys_init
>>> pci_eisa_init_early
>>> pnpacpi_init/isapnp_init
>>
>> Is this a regression? This must have worked at one time, but it seems
>> like we've had pnpacpi_init/isapnp_init/pnpbios_init before PCI
>> drivers for quite a while.
>
> Yes.

Do you know when the regression occurred? If you do, I'll add that
info to the "stable" tag.

Bjorn
--
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/