login
Header Space

 
 

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

Score:
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Gary Hade <garyhade@...>, Matthew Wilcox <matthew@...>, <gregkh@...>, <achiang@...>, <kristen.c.accardi@...>, <lenb@...>, <rick.jones2@...>, <linux-kernel@...>, <linux-pci@...>, <kaneshige.kenji@...>, <pcihpd-discuss@...>, <linux-acpi@...>
Date: Saturday, November 17, 2007 - 2:29 pm

Hi all,

This is v3 of the pci_slot patch series.

The major change is making the ACPI-PCI slot driver a Kconfig
option, as per the recommendations of others (Gary, Kenji-san).
In the process of doing so, it made sense to collapse the former
3/5 and 4/5 patches into a single 3/4 patch. There really wasn't
a reason to introduce a pci_slot patch, and then immediately
follow it with another patch modifying its interface; logically,
the changes should have been in the same patch.

Combining the patches also has the nice side benefit of keeping
the tree fully buildable and bisectable at all stages of series.

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. 

Of course, if you *do* have the ACPI-PCI slot driver configured,
the slots/ entries in sysfs will be persistent. What you will see
is the hotplug attributes appear/disappear, depending on whether
you have acpiphp loaded or not.

Thanks for your consideration and all the feedback comments.
They're appreciated.

/ac

v2 -> v3:
  Patch 1/4 - no change
  Patch 2/4 - incorporate Eike's comments around snprintf
  Patch 3/4 - Separated slot creation and slot hotplug ability
              into two interfaces. Fixed bugs in pci_destroy_slot(), 
              and now properly calling from pci_hp_deregister.
  Patch 4/4 - Add Kconfig option to driver, allowing users to
              [de]config this driver. If configured, take slightly 
              different code paths in pci_hp_register and pci_hp_deregister.

v1 -> v2:
  Patch 1/5 - reworked to fix stupid compile bug
  Patch 2/5 - incorporate Eike, Linas, and Willy's comments
  Patch 3/5 - no change
  Patch 4/5 - was acpi-pci-slot-driver patch, now modifies
  	      pci_add_hotplug(). I changed the ordering on this so
  	      the tree doesn't break at this point in the series
  Patch 5/5 - now is acpi-pci-slot-driver patch, cleaned up
  	      implementation so our slot detection is a little
  	      better
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 0/4, v3] Physical PCI slot objects, Alex Chiang, (Sat Nov 17, 2:29 pm)
Re: [PATCH 0/4, v3] Physical PCI slot objects, Gary Hade, (Mon Nov 19, 7:32 pm)
Re: [PATCH 0/4, v3] Physical PCI slot objects, Alex Chiang, (Mon Nov 26, 6:22 pm)
Re: [PATCH 0/4, v3] Physical PCI slot objects, Kenji Kaneshige, (Thu Nov 29, 3:47 am)
Re: [PATCH 0/4, v3] Physical PCI slot objects, Kristen Carlson Accardi, (Tue Mar 11, 3:15 pm)
Re: [PATCH 0/4, v3] Physical PCI slot objects, Kenji Kaneshige, (Wed Mar 12, 8:25 am)
Re: [PATCH 0/4, v3] Physical PCI slot objects, Alex Chiang, (Thu Nov 29, 9:51 pm)
Re: [PATCH 0/4, v3] Physical PCI slot objects, Kenji Kaneshige, (Sun Dec 2, 11:30 pm)
Re: [PATCH 0/4, v3] Physical PCI slot objects, Alex Chiang, (Mon Dec 3, 6:43 pm)
Re: [PATCH 0/4, v3] Physical PCI slot objects, Kenji Kaneshige, (Tue Dec 4, 8:57 am)
Re: [PATCH 0/4, v3] Physical PCI slot objects, Alex Chiang, (Mon Dec 10, 7:02 pm)
Re: [PATCH 0/4, v3] Physical PCI slot objects, Gary Hade, (Mon Nov 26, 11:04 pm)
Re: [PATCH 0/4, v3] Physical PCI slot objects, Kristen Carlson Accardi, (Tue Nov 27, 3:11 pm)
Re: [PATCH 0/4, v3] Physical PCI slot objects, Gary Hade, (Wed Nov 28, 5:31 pm)
Re: [PATCH 0/4, v3] Physical PCI slot objects, Alex Chiang, (Thu Nov 29, 9:19 pm)
Re: [PATCH 0/4, v3] Physical PCI slot objects, Gary Hade, (Fri Nov 30, 3:10 pm)
Re: [PATCH 0/4, v3] Physical PCI slot objects, Kristen Carlson Accardi, (Wed Nov 28, 8:02 pm)
Re: [PATCH 0/4, v3] Physical PCI slot objects, Gary Hade, (Wed Nov 28, 9:09 pm)
[PATCH 3/4, v5] PCI, PCI Hotplug: Introduce pci_slot, Alex Chiang, (Mon Nov 26, 6:26 pm)
Re: [PATCH 0/4, v3] Physical PCI slot objects, Matthew Garrett, (Mon Nov 19, 10:04 pm)
Re: [PATCH 0/4, v3] Physical PCI slot objects, Gary Hade, (Tue Nov 20, 3:53 pm)
Re: [PATCH 0/4, v3] Physical PCI slot objects, Kenji Kaneshige, (Mon Nov 19, 9:33 pm)
Re: [PATCH 0/4, v3] Physical PCI slot objects, Kristen Carlson Accardi, (Mon Nov 19, 1:43 pm)
Re: [PATCH 0/4, v3] Physical PCI slot objects, Alex Chiang, (Mon Nov 19, 6:02 pm)
Re: [PATCH 0/4, v3] Physical PCI slot objects, Alex Chiang, (Mon Nov 19, 1:57 pm)
[PATCH 3/4, v3] PCI, PCI Hotplug: Introduce pci_slot, Alex Chiang, (Sat Nov 17, 2:37 pm)
[PATCH 3/4, v4] PCI, PCI Hotplug: Introduce pci_slot, Alex Chiang, (Mon Nov 19, 6:03 pm)
speck-geostationary