|
Michal Suchanek |
46cbce |
From 870e04ae45ea2e569d1ca2780439b16e988da08d Mon Sep 17 00:00:00 2001
|
|
Michal Suchanek |
46cbce |
From: Lijun Pan <lijunp213@gmail.com>
|
|
Michal Suchanek |
46cbce |
Date: Tue, 13 Apr 2021 14:33:39 -0500
|
|
Michal Suchanek |
46cbce |
Subject: [PATCH] ibmvnic: queue reset work in system_long_wq
|
|
Michal Suchanek |
46cbce |
|
|
Michal Suchanek |
46cbce |
References: bsc#1152457 ltc#174432 git-fixes
|
|
Michal Suchanek |
46cbce |
Patch-mainline: v5.13-rc1
|
|
Michal Suchanek |
46cbce |
Git-commit: 870e04ae45ea2e569d1ca2780439b16e988da08d
|
|
Michal Suchanek |
46cbce |
|
|
Michal Suchanek |
46cbce |
The reset process for ibmvnic commonly takes multiple seconds, clearly
|
|
Michal Suchanek |
46cbce |
making it inappropriate for schedule_work/system_wq. The reason to make
|
|
Michal Suchanek |
46cbce |
this change is that ibmvnic's use of the default system-wide workqueue
|
|
Michal Suchanek |
46cbce |
for a relatively long-running work item can negatively affect other
|
|
Michal Suchanek |
46cbce |
workqueue users. So, queue the relatively slow reset job to the
|
|
Michal Suchanek |
46cbce |
system_long_wq.
|
|
Michal Suchanek |
46cbce |
|
|
Michal Suchanek |
46cbce |
Suggested-by: Nathan Lynch <nathanl@linux.ibm.com>
|
|
Michal Suchanek |
46cbce |
Signed-off-by: Lijun Pan <lijunp213@gmail.com>
|
|
Michal Suchanek |
46cbce |
Signed-off-by: David S. Miller <davem@davemloft.net>
|
|
Michal Suchanek |
46cbce |
Acked-by: Michal Suchanek <msuchanek@suse.de>
|
|
Michal Suchanek |
46cbce |
---
|
|
Michal Suchanek |
46cbce |
drivers/net/ethernet/ibm/ibmvnic.c | 7 ++++---
|
|
Michal Suchanek |
46cbce |
1 file changed, 4 insertions(+), 3 deletions(-)
|
|
Michal Suchanek |
46cbce |
|
|
Michal Suchanek |
46cbce |
diff --git a/drivers/net/ethernet/ibm/ibmvnic.c b/drivers/net/ethernet/ibm/ibmvnic.c
|
|
Michal Suchanek |
46cbce |
index 0961d36833d5..b72159ccca3a 100644
|
|
Michal Suchanek |
46cbce |
--- a/drivers/net/ethernet/ibm/ibmvnic.c
|
|
Michal Suchanek |
46cbce |
+++ b/drivers/net/ethernet/ibm/ibmvnic.c
|
|
Michal Suchanek |
46cbce |
@@ -2292,8 +2292,9 @@ static void __ibmvnic_reset(struct work_struct *work)
|
|
Michal Suchanek |
46cbce |
adapter = container_of(work, struct ibmvnic_adapter, ibmvnic_reset);
|
|
Michal Suchanek |
46cbce |
|
|
Michal Suchanek |
46cbce |
if (test_and_set_bit_lock(0, &adapter->resetting)) {
|
|
Michal Suchanek |
46cbce |
- schedule_delayed_work(&adapter->ibmvnic_delayed_reset,
|
|
Michal Suchanek |
46cbce |
- IBMVNIC_RESET_DELAY);
|
|
Michal Suchanek |
46cbce |
+ queue_delayed_work(system_long_wq,
|
|
Michal Suchanek |
46cbce |
+ &adapter->ibmvnic_delayed_reset,
|
|
Michal Suchanek |
46cbce |
+ IBMVNIC_RESET_DELAY);
|
|
Michal Suchanek |
46cbce |
return;
|
|
Michal Suchanek |
46cbce |
}
|
|
Michal Suchanek |
46cbce |
|
|
Michal Suchanek |
46cbce |
@@ -2437,7 +2438,7 @@ static int ibmvnic_reset(struct ibmvnic_adapter *adapter,
|
|
Michal Suchanek |
46cbce |
list_add_tail(&rwi->list, &adapter->rwi_list);
|
|
Michal Suchanek |
46cbce |
netdev_dbg(adapter->netdev, "Scheduling reset (reason %s)\n",
|
|
Michal Suchanek |
46cbce |
reset_reason_to_string(reason));
|
|
Michal Suchanek |
46cbce |
- schedule_work(&adapter->ibmvnic_reset);
|
|
Michal Suchanek |
46cbce |
+ queue_work(system_long_wq, &adapter->ibmvnic_reset);
|
|
Michal Suchanek |
46cbce |
|
|
Michal Suchanek |
46cbce |
ret = 0;
|
|
Michal Suchanek |
46cbce |
err:
|
|
Michal Suchanek |
46cbce |
--
|
|
Michal Suchanek |
46cbce |
2.26.2
|
|
Michal Suchanek |
46cbce |
|