Discussion:
[RFC PATCH 0/4] Allow Link state changes for Hot-Plug
Rajat Jain
2013-11-19 22:39:50 UTC
Permalink
Hello,

This patch set enables the use of PCI Express link up and link down events
for Hotplug or Unplug. The requirement of such a feature was originally
discussed here:

http://www.spinics.net/lists/linux-pci/msg05783.html
http://www.spinics.net/lists/hotplug/msg05801.html

Patch [1/4]: makes a function non-static for use by patch 2.
Patch [2/4]: Contains the bulk logic to allow link events to be used
for hotplug and removal.
Patch [3/4]: Makes the pciehp_power_thread() lock free by making it
look at a work info->req instead of slot->state.
Patch [4/4]: Introduce slot->hotplug_lock to serialize the hotplug
operations.

I'd appreciate if you could please review and provide me with any comments.

Thanks,

Rajat

Rajat Jain (4):
pciehp: Make check_link_active() non-static
pciehp: Use link state change notifications for hot-plug and removal
pciehp: Ensure all hotplug events are processed, even very fast ones.
pciehp: Introduce hotplug_lock to serialize HP events on each slot

drivers/pci/hotplug/pciehp.h | 6 ++
drivers/pci/hotplug/pciehp_core.c | 10 +-
drivers/pci/hotplug/pciehp_ctrl.c | 181 ++++++++++++++++++++++++++++++++++---
drivers/pci/hotplug/pciehp_hpc.c | 63 +++++++++----
4 files changed, 229 insertions(+), 31 deletions(-)
--
1.7.9.5
Rajat Jain
2013-11-21 13:16:26 UTC
Permalink
Hello Bjorn / Folks,

Was wondering if you got a chance to look at this patchset?

Thanks & Best Regards,

Rajat Jain
Post by Rajat Jain
Hello,
This patch set enables the use of PCI Express link up and link down events
for Hotplug or Unplug. The requirement of such a feature was originally
http://www.spinics.net/lists/linux-pci/msg05783.html
http://www.spinics.net/lists/hotplug/msg05801.html
Patch [1/4]: makes a function non-static for use by patch 2.
Patch [2/4]: Contains the bulk logic to allow link events to be used
for hotplug and removal.
Patch [3/4]: Makes the pciehp_power_thread() lock free by making it
look at a work info->req instead of slot->state.
Patch [4/4]: Introduce slot->hotplug_lock to serialize the hotplug
operations.
I'd appreciate if you could please review and provide me with any comments.
Thanks,
Rajat
pciehp: Make check_link_active() non-static
pciehp: Use link state change notifications for hot-plug and removal
pciehp: Ensure all hotplug events are processed, even very fast ones.
pciehp: Introduce hotplug_lock to serialize HP events on each slot
drivers/pci/hotplug/pciehp.h | 6 ++
drivers/pci/hotplug/pciehp_core.c | 10 +-
drivers/pci/hotplug/pciehp_ctrl.c | 181 ++++++++++++++++++++++++++++++++++---
drivers/pci/hotplug/pciehp_hpc.c | 63 +++++++++----
4 files changed, 229 insertions(+), 31 deletions(-)
--
1.7.9.5
Bjorn Helgaas
2013-11-22 04:36:51 UTC
Permalink
I haven't had a chance yet, sorry.
Post by Rajat Jain
Hello Bjorn / Folks,
Was wondering if you got a chance to look at this patchset?
Thanks & Best Regards,
Rajat Jain
Post by Rajat Jain
Hello,
This patch set enables the use of PCI Express link up and link down events
for Hotplug or Unplug. The requirement of such a feature was originally
http://www.spinics.net/lists/linux-pci/msg05783.html
http://www.spinics.net/lists/hotplug/msg05801.html
Patch [1/4]: makes a function non-static for use by patch 2.
Patch [2/4]: Contains the bulk logic to allow link events to be used
for hotplug and removal.
Patch [3/4]: Makes the pciehp_power_thread() lock free by making it
look at a work info->req instead of slot->state.
Patch [4/4]: Introduce slot->hotplug_lock to serialize the hotplug
operations.
I'd appreciate if you could please review and provide me with any comments.
Thanks,
Rajat
pciehp: Make check_link_active() non-static
pciehp: Use link state change notifications for hot-plug and removal
pciehp: Ensure all hotplug events are processed, even very fast ones.
pciehp: Introduce hotplug_lock to serialize HP events on each slot
drivers/pci/hotplug/pciehp.h | 6 ++
drivers/pci/hotplug/pciehp_core.c | 10 +-
drivers/pci/hotplug/pciehp_ctrl.c | 181 ++++++++++++++++++++++++++++++++++---
drivers/pci/hotplug/pciehp_hpc.c | 63 +++++++++----
4 files changed, 229 insertions(+), 31 deletions(-)
--
1.7.9.5
--
To unsubscribe from this list: send the line "unsubscribe linux-hotplug" in
the body of a message to ***@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Continue reading on narkive:
Loading...