Re: [PATCH v3 00/14] PCI: let the core manage slot names

[Alex Chiang]

Hi Taku-san,

* Taku Izumi <izumi.taku@xxxxxxxxxxxxxx>:
>> This series applies against linux-next. I've tested this by loading
>> pci_slot and various hotplug drivers to ensure that the hotplug drivers
>> can successfully override the pci_slot names.
>
>  It seems that there are some bugs. Pciehp and shpchp modules can not
>  be loaded when the debug option is specified. Segmentation fault occurs
>  with stack dump.
>
>  pciehp_probe() or shpchp_probe()
>    -> init_slots()
>     -> get_power_status()
>      -> slot_name() : This is called when the debug option is specified.
>       -> hotplug_slot_name()
>        -> pci_slot_name(slot->pci_slot)
>                         ~~~~~~~~~~~~~~
>                        Not initialized yet.

Thank you for pointing this out. I've fixed this (and the other
drivers) by make sure that the slot names are initialized before
we attempt to use them.

Trying to fix rpaphp was too much work for very little gain, so I
deleted that patch from my patch series. But the changes to the
hotplug core still make sense, so it's ok to have just one
oddball left.

I'll wait until Kenji-san sends out his final review comments
before sending out v4.

Thanks.

/ac

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