Skip to contentSkip to conversation

Allow Link state changes for Hot-Plug

RFC PATCH Superseded Rajat Jain 2 participants 3 messages Nov 19, 2013 – Nov 22, 2013 PCI

v1 of 3 Superseded Latest is v3 →

lore
Share & tools
lore

Rajat Jain

author
#0 lore raw

cover Allow Link state changes for Hot-Plug

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(-)
Signature · 1 line
1.7.9.5
author
#1 lore raw

Hello Bjorn / Folks,

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

Thanks & Best Regards,

Rajat Jain

Quoted 39 lines · 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 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

maintainer
#2 lore raw

I haven't had a chance yet, sorry.

Quoted 48 lines · Rajat Jain

Hello Bjorn / Folks,

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

Thanks & Best Regards,

Rajat Jain

Quoted 39 lines · 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 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