Re: [PATCH 0/4, v3] Physical PCI slot objects

[Alex Chiang <achiang@hp.com>]

* Kristen Carlson Accardi <kristen.c.accardi@intel.com>:
> On Sat, 17 Nov 2007 11:29:54 -0700
> Alex Chiang <achiang@hp.com> wrote:
> 
> > I have done quite a bit more testing, and verified that this
> > series plays nicely with acpiphp during all stages of the series.
> > Notably, you can modprobe/rmmod acpiphp repeatedly no matter
> > where you are in the series, and no matter whether you have
> > CONFIG_ACPI_PCI_SLOT turned on. The correct entries in
> > /sys/bus/pci/slots/ will appear and disappear, and we correctly
> > register/deregister ACPI slots with the pci_hp core. 
> 
> How does this patch play with non-acpi based hotplug such as the pciehp
> driver 

Not well. :(

pciehp: HPC vendor_id 103c device_id 403b ss_vid 0 ss_did 0
pciehp: pci_hp_register failed with error -17
pciehp: pciehp: slot initialization failed
pciehp: HPC vendor_id 111d device_id 801c ss_vid 0 ss_did 0
pciehp: Can't get irq 0 for the hotplug controller
pciehp: HPC vendor_id 111d device_id 801c ss_vid 0 ss_did 0
pciehp: Can't get irq 0 for the hotplug controller
pciehp: HPC vendor_id 103c device_id 403b ss_vid 0 ss_did 0
pciehp: pci_hp_register failed with error -17
pciehp: pciehp: slot initialization failed

I'll take a look at this today. Thanks for pointing it out.

> or the shpchp driver for example?

This one, I'm not sure on, as I don't have any shpc hardware.
I'll do my best by just looking at the code, but maybe if someone
out there has that hardware, they could let me know what breaks?

Thanks.

/ac