Hotplug a network card would take more than 5 seconds
in qemu + shpchp scene. It’s because 5 seconds
delayed_work in func handle_button_press_event with
case STATIC_STATE. And this will break some
protocols with timeout within 5 seconds.

Signed-off-by: Miaohe Lin <[email protected]>
---
 drivers/pci/hotplug/shpchp_ctrl.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/pci/hotplug/shpchp_ctrl.c 
b/drivers/pci/hotplug/shpchp_ctrl.c
index 078003dcde5b..cbb00acaba0d 100644
--- a/drivers/pci/hotplug/shpchp_ctrl.c
+++ b/drivers/pci/hotplug/shpchp_ctrl.c
@@ -478,7 +478,7 @@ static void handle_button_press_event(struct slot *p_slot)
                p_slot->hpc_ops->green_led_blink(p_slot);
                p_slot->hpc_ops->set_attention_status(p_slot, 0);
 
-               queue_delayed_work(p_slot->wq, &p_slot->work, 5*HZ);
+               queue_delayed_work(p_slot->wq, &p_slot->work, 0);
                break;
        case BLINKINGOFF_STATE:
        case BLINKINGON_STATE:
-- 
2.21.GIT

Reply via email to