mirror of
https://github.com/torvalds/linux.git
synced 2026-06-01 02:53:36 +02:00
Merge branch 'net-replace-wq-users-and-add-wq_percpu-to-alloc_workqueue-users'
Marco Crivellari says: ==================== net: replace wq users and add WQ_PERCPU to alloc_workqueue() users Below is a summary of a discussion about the Workqueue API and cpu isolation considerations. Details and more information are available here: "workqueue: Always use wq_select_unbound_cpu() for WORK_CPU_UNBOUND." Link: https://lore.kernel.org/20250221112003.1dSuoGyc@linutronix.de === Current situation: problems === Let's consider a nohz_full system with isolated CPUs: wq_unbound_cpumask is set to the housekeeping CPUs, for !WQ_UNBOUND the local CPU is selected. This leads to different scenarios if a work item is scheduled on an isolated CPU where "delay" value is 0 or greater then 0: schedule_delayed_work(, 0); This will be handled by __queue_work() that will queue the work item on the current local (isolated) CPU, while: schedule_delayed_work(, 1); Will move the timer on an housekeeping CPU, and schedule the work there. Currently if a user enqueue a work item using schedule_delayed_work() the used wq is "system_wq" (per-cpu wq) while queue_delayed_work() use WORK_CPU_UNBOUND (used when a cpu is not specified). The same applies to schedule_work() that is using system_wq and queue_work(), that makes use again of WORK_CPU_UNBOUND. This lack of consistentcy cannot be addressed without refactoring the API. === Plan and future plans === This patchset is the first stone on a refactoring needed in order to address the points aforementioned; it will have a positive impact also on the cpu isolation, in the long term, moving away percpu workqueue in favor to an unbound model. These are the main steps: 1) API refactoring (that this patch is introducing) - Make more clear and uniform the system wq names, both per-cpu and unbound. This to avoid any possible confusion on what should be used. - Introduction of WQ_PERCPU: this flag is the complement of WQ_UNBOUND, introduced in this patchset and used on all the callers that are not currently using WQ_UNBOUND. WQ_UNBOUND will be removed in a future release cycle. Most users don't need to be per-cpu, because they don't have locality requirements, because of that, a next future step will be make "unbound" the default behavior. 2) Check who really needs to be per-cpu - Remove the WQ_PERCPU flag when is not strictly required. 3) Add a new API (prefer local cpu) - There are users that don't require a local execution, like mentioned above; despite that, local execution yeld to performance gain. This new API will prefer the local execution, without requiring it. === Introduced Changes by this series === 1) [P 1-2] Replace use of system_wq and system_unbound_wq system_wq is a per-CPU workqueue, but his name is not clear. system_unbound_wq is to be used when locality is not required. Because of that, system_wq has been renamed in system_percpu_wq, and system_unbound_wq has been renamed in system_dfl_wq. 2) [P 3] add WQ_PERCPU to remaining alloc_workqueue() users Every alloc_workqueue() caller should use one among WQ_PERCPU or WQ_UNBOUND. WQ_UNBOUND will be removed in a next release cycle. ==================== Link: https://patch.msgid.link/20250918142427.309519-1-marco.crivellari@suse.com Signed-off-by: Jakub Kicinski <kuba@kernel.org>
This commit is contained in:
commit
c5aaf0225a
|
|
@ -770,7 +770,8 @@ static int hi3110_open(struct net_device *net)
|
|||
goto out_close;
|
||||
}
|
||||
|
||||
priv->wq = alloc_workqueue("hi3110_wq", WQ_FREEZABLE | WQ_MEM_RECLAIM,
|
||||
priv->wq = alloc_workqueue("hi3110_wq",
|
||||
WQ_FREEZABLE | WQ_MEM_RECLAIM | WQ_PERCPU,
|
||||
0);
|
||||
if (!priv->wq) {
|
||||
ret = -ENOMEM;
|
||||
|
|
|
|||
|
|
@ -1378,7 +1378,8 @@ static int mcp251x_can_probe(struct spi_device *spi)
|
|||
if (ret)
|
||||
goto out_clk;
|
||||
|
||||
priv->wq = alloc_workqueue("mcp251x_wq", WQ_FREEZABLE | WQ_MEM_RECLAIM,
|
||||
priv->wq = alloc_workqueue("mcp251x_wq",
|
||||
WQ_FREEZABLE | WQ_MEM_RECLAIM | WQ_PERCPU,
|
||||
0);
|
||||
if (!priv->wq) {
|
||||
ret = -ENOMEM;
|
||||
|
|
|
|||
|
|
@ -472,7 +472,7 @@ int setup_rx_oom_poll_fn(struct net_device *netdev)
|
|||
q_no = lio->linfo.rxpciq[q].s.q_no;
|
||||
wq = &lio->rxq_status_wq[q_no];
|
||||
wq->wq = alloc_workqueue("rxq-oom-status",
|
||||
WQ_MEM_RECLAIM, 0);
|
||||
WQ_MEM_RECLAIM | WQ_PERCPU, 0);
|
||||
if (!wq->wq) {
|
||||
dev_err(&oct->pci_dev->dev, "unable to create cavium rxq oom status wq\n");
|
||||
return -ENOMEM;
|
||||
|
|
|
|||
|
|
@ -526,7 +526,8 @@ static inline int setup_link_status_change_wq(struct net_device *netdev)
|
|||
struct octeon_device *oct = lio->oct_dev;
|
||||
|
||||
lio->link_status_wq.wq = alloc_workqueue("link-status",
|
||||
WQ_MEM_RECLAIM, 0);
|
||||
WQ_MEM_RECLAIM | WQ_PERCPU,
|
||||
0);
|
||||
if (!lio->link_status_wq.wq) {
|
||||
dev_err(&oct->pci_dev->dev, "unable to create cavium link status wq\n");
|
||||
return -1;
|
||||
|
|
@ -659,7 +660,8 @@ static inline int setup_sync_octeon_time_wq(struct net_device *netdev)
|
|||
struct octeon_device *oct = lio->oct_dev;
|
||||
|
||||
lio->sync_octeon_time_wq.wq =
|
||||
alloc_workqueue("update-octeon-time", WQ_MEM_RECLAIM, 0);
|
||||
alloc_workqueue("update-octeon-time",
|
||||
WQ_MEM_RECLAIM | WQ_PERCPU, 0);
|
||||
if (!lio->sync_octeon_time_wq.wq) {
|
||||
dev_err(&oct->pci_dev->dev, "Unable to create wq to update octeon time\n");
|
||||
return -1;
|
||||
|
|
@ -1734,7 +1736,7 @@ static inline int setup_tx_poll_fn(struct net_device *netdev)
|
|||
struct octeon_device *oct = lio->oct_dev;
|
||||
|
||||
lio->txq_status_wq.wq = alloc_workqueue("txq-status",
|
||||
WQ_MEM_RECLAIM, 0);
|
||||
WQ_MEM_RECLAIM | WQ_PERCPU, 0);
|
||||
if (!lio->txq_status_wq.wq) {
|
||||
dev_err(&oct->pci_dev->dev, "unable to create cavium txq status wq\n");
|
||||
return -1;
|
||||
|
|
|
|||
|
|
@ -304,7 +304,8 @@ static int setup_link_status_change_wq(struct net_device *netdev)
|
|||
struct octeon_device *oct = lio->oct_dev;
|
||||
|
||||
lio->link_status_wq.wq = alloc_workqueue("link-status",
|
||||
WQ_MEM_RECLAIM, 0);
|
||||
WQ_MEM_RECLAIM | WQ_PERCPU,
|
||||
0);
|
||||
if (!lio->link_status_wq.wq) {
|
||||
dev_err(&oct->pci_dev->dev, "unable to create cavium link status wq\n");
|
||||
return -1;
|
||||
|
|
|
|||
|
|
@ -132,7 +132,7 @@ int octeon_init_instr_queue(struct octeon_device *oct,
|
|||
oct->fn_list.setup_iq_regs(oct, iq_no);
|
||||
|
||||
oct->check_db_wq[iq_no].wq = alloc_workqueue("check_iq_db",
|
||||
WQ_MEM_RECLAIM,
|
||||
WQ_MEM_RECLAIM | WQ_PERCPU,
|
||||
0);
|
||||
if (!oct->check_db_wq[iq_no].wq) {
|
||||
vfree(iq->request_list);
|
||||
|
|
|
|||
|
|
@ -39,7 +39,8 @@ int octeon_setup_response_list(struct octeon_device *oct)
|
|||
}
|
||||
spin_lock_init(&oct->cmd_resp_wqlock);
|
||||
|
||||
oct->dma_comp_wq.wq = alloc_workqueue("dma-comp", WQ_MEM_RECLAIM, 0);
|
||||
oct->dma_comp_wq.wq = alloc_workqueue("dma-comp",
|
||||
WQ_MEM_RECLAIM | WQ_PERCPU, 0);
|
||||
if (!oct->dma_comp_wq.wq) {
|
||||
dev_err(&oct->pci_dev->dev, "failed to create wq thread\n");
|
||||
return -ENOMEM;
|
||||
|
|
|
|||
|
|
@ -4884,7 +4884,7 @@ static int dpaa2_eth_probe(struct fsl_mc_device *dpni_dev)
|
|||
priv->tx_tstamp_type = HWTSTAMP_TX_OFF;
|
||||
priv->rx_tstamp = false;
|
||||
|
||||
priv->dpaa2_ptp_wq = alloc_workqueue("dpaa2_ptp_wq", 0, 0);
|
||||
priv->dpaa2_ptp_wq = alloc_workqueue("dpaa2_ptp_wq", WQ_PERCPU, 0);
|
||||
if (!priv->dpaa2_ptp_wq) {
|
||||
err = -ENOMEM;
|
||||
goto err_wq_alloc;
|
||||
|
|
|
|||
|
|
@ -12912,7 +12912,8 @@ static int __init hclge_init(void)
|
|||
{
|
||||
pr_debug("%s is initializing\n", HCLGE_NAME);
|
||||
|
||||
hclge_wq = alloc_workqueue("%s", WQ_UNBOUND, 0, HCLGE_NAME);
|
||||
hclge_wq = alloc_workqueue("%s", WQ_UNBOUND, 0,
|
||||
HCLGE_NAME);
|
||||
if (!hclge_wq) {
|
||||
pr_err("%s: failed to create workqueue\n", HCLGE_NAME);
|
||||
return -ENOMEM;
|
||||
|
|
|
|||
|
|
@ -37,7 +37,7 @@ static int __init fm10k_init_module(void)
|
|||
pr_info("%s\n", fm10k_copyright);
|
||||
|
||||
/* create driver workqueue */
|
||||
fm10k_workqueue = alloc_workqueue("%s", WQ_MEM_RECLAIM, 0,
|
||||
fm10k_workqueue = alloc_workqueue("%s", WQ_MEM_RECLAIM | WQ_PERCPU, 0,
|
||||
fm10k_driver_name);
|
||||
if (!fm10k_workqueue)
|
||||
return -ENOMEM;
|
||||
|
|
|
|||
|
|
@ -16617,7 +16617,7 @@ static int __init i40e_init_module(void)
|
|||
* since we need to be able to guarantee forward progress even under
|
||||
* memory pressure.
|
||||
*/
|
||||
i40e_wq = alloc_workqueue("%s", 0, 0, i40e_driver_name);
|
||||
i40e_wq = alloc_workqueue("%s", WQ_PERCPU, 0, i40e_driver_name);
|
||||
if (!i40e_wq) {
|
||||
pr_err("%s: Failed to create workqueue\n", i40e_driver_name);
|
||||
return -ENOMEM;
|
||||
|
|
|
|||
|
|
@ -2005,7 +2005,7 @@ static int cgx_probe(struct pci_dev *pdev, const struct pci_device_id *id)
|
|||
|
||||
/* init wq for processing linkup requests */
|
||||
INIT_WORK(&cgx->cgx_cmd_work, cgx_lmac_linkup_work);
|
||||
cgx->cgx_cmd_workq = alloc_workqueue("cgx_cmd_workq", 0, 0);
|
||||
cgx->cgx_cmd_workq = alloc_workqueue("cgx_cmd_workq", WQ_PERCPU, 0);
|
||||
if (!cgx->cgx_cmd_workq) {
|
||||
dev_err(dev, "alloc workqueue failed for cgx cmd");
|
||||
err = -ENOMEM;
|
||||
|
|
|
|||
|
|
@ -913,7 +913,7 @@ int rvu_mcs_init(struct rvu *rvu)
|
|||
/* Initialize the wq for handling mcs interrupts */
|
||||
INIT_LIST_HEAD(&rvu->mcs_intrq_head);
|
||||
INIT_WORK(&rvu->mcs_intr_work, mcs_intr_handler_task);
|
||||
rvu->mcs_intr_wq = alloc_workqueue("mcs_intr_wq", 0, 0);
|
||||
rvu->mcs_intr_wq = alloc_workqueue("mcs_intr_wq", WQ_PERCPU, 0);
|
||||
if (!rvu->mcs_intr_wq) {
|
||||
dev_err(rvu->dev, "mcs alloc workqueue failed\n");
|
||||
return -ENOMEM;
|
||||
|
|
|
|||
|
|
@ -315,7 +315,7 @@ static int cgx_lmac_event_handler_init(struct rvu *rvu)
|
|||
spin_lock_init(&rvu->cgx_evq_lock);
|
||||
INIT_LIST_HEAD(&rvu->cgx_evq_head);
|
||||
INIT_WORK(&rvu->cgx_evh_work, cgx_evhandler_task);
|
||||
rvu->cgx_evh_wq = alloc_workqueue("rvu_evh_wq", 0, 0);
|
||||
rvu->cgx_evh_wq = alloc_workqueue("rvu_evh_wq", WQ_PERCPU, 0);
|
||||
if (!rvu->cgx_evh_wq) {
|
||||
dev_err(rvu->dev, "alloc workqueue failed");
|
||||
return -ENOMEM;
|
||||
|
|
|
|||
|
|
@ -376,7 +376,7 @@ int rvu_rep_install_mcam_rules(struct rvu *rvu)
|
|||
spin_lock_init(&rvu->rep_evtq_lock);
|
||||
INIT_LIST_HEAD(&rvu->rep_evtq_head);
|
||||
INIT_WORK(&rvu->rep_evt_work, rvu_rep_wq_handler);
|
||||
rvu->rep_evt_wq = alloc_workqueue("rep_evt_wq", 0, 0);
|
||||
rvu->rep_evt_wq = alloc_workqueue("rep_evt_wq", WQ_PERCPU, 0);
|
||||
if (!rvu->rep_evt_wq) {
|
||||
dev_err(rvu->dev, "REP workqueue allocation failed\n");
|
||||
return -ENOMEM;
|
||||
|
|
|
|||
|
|
@ -798,7 +798,8 @@ int cn10k_ipsec_init(struct net_device *netdev)
|
|||
pf->ipsec.sa_size = sa_size;
|
||||
|
||||
INIT_WORK(&pf->ipsec.sa_work, cn10k_ipsec_sa_wq_handler);
|
||||
pf->ipsec.sa_workq = alloc_workqueue("cn10k_ipsec_sa_workq", 0, 0);
|
||||
pf->ipsec.sa_workq = alloc_workqueue("cn10k_ipsec_sa_workq",
|
||||
WQ_PERCPU, 0);
|
||||
if (!pf->ipsec.sa_workq) {
|
||||
netdev_err(pf->netdev, "SA alloc workqueue failed\n");
|
||||
return -ENOMEM;
|
||||
|
|
|
|||
|
|
@ -1500,7 +1500,7 @@ EXPORT_SYMBOL(prestera_device_unregister);
|
|||
|
||||
static int __init prestera_module_init(void)
|
||||
{
|
||||
prestera_wq = alloc_workqueue("prestera", 0, 0);
|
||||
prestera_wq = alloc_workqueue("prestera", WQ_PERCPU, 0);
|
||||
if (!prestera_wq)
|
||||
return -ENOMEM;
|
||||
|
||||
|
|
|
|||
|
|
@ -898,7 +898,7 @@ static int prestera_pci_probe(struct pci_dev *pdev,
|
|||
|
||||
dev_info(fw->dev.dev, "Prestera FW is ready\n");
|
||||
|
||||
fw->wq = alloc_workqueue("prestera_fw_wq", WQ_HIGHPRI, 1);
|
||||
fw->wq = alloc_workqueue("prestera_fw_wq", WQ_HIGHPRI | WQ_PERCPU, 1);
|
||||
if (!fw->wq) {
|
||||
err = -ENOMEM;
|
||||
goto err_wq_alloc;
|
||||
|
|
|
|||
|
|
@ -886,7 +886,7 @@ static int mlxsw_emad_init(struct mlxsw_core *mlxsw_core)
|
|||
if (!(mlxsw_core->bus->features & MLXSW_BUS_F_TXRX))
|
||||
return 0;
|
||||
|
||||
emad_wq = alloc_workqueue("mlxsw_core_emad", 0, 0);
|
||||
emad_wq = alloc_workqueue("mlxsw_core_emad", WQ_PERCPU, 0);
|
||||
if (!emad_wq)
|
||||
return -ENOMEM;
|
||||
mlxsw_core->emad_wq = emad_wq;
|
||||
|
|
@ -3381,7 +3381,7 @@ static int __init mlxsw_core_module_init(void)
|
|||
if (err)
|
||||
return err;
|
||||
|
||||
mlxsw_wq = alloc_workqueue(mlxsw_core_driver_name, 0, 0);
|
||||
mlxsw_wq = alloc_workqueue(mlxsw_core_driver_name, WQ_PERCPU, 0);
|
||||
if (!mlxsw_wq) {
|
||||
err = -ENOMEM;
|
||||
goto err_alloc_workqueue;
|
||||
|
|
|
|||
|
|
@ -797,7 +797,7 @@ static int nfp_pci_probe(struct pci_dev *pdev,
|
|||
pf->pdev = pdev;
|
||||
pf->dev_info = dev_info;
|
||||
|
||||
pf->wq = alloc_workqueue("nfp-%s", 0, 2, pci_name(pdev));
|
||||
pf->wq = alloc_workqueue("nfp-%s", WQ_PERCPU, 2, pci_name(pdev));
|
||||
if (!pf->wq) {
|
||||
err = -ENOMEM;
|
||||
goto err_pci_priv_unset;
|
||||
|
|
|
|||
|
|
@ -1214,7 +1214,8 @@ static int qed_slowpath_wq_start(struct qed_dev *cdev)
|
|||
hwfn = &cdev->hwfns[i];
|
||||
|
||||
hwfn->slowpath_wq = alloc_workqueue("slowpath-%02x:%02x.%02x",
|
||||
0, 0, cdev->pdev->bus->number,
|
||||
WQ_PERCPU, 0,
|
||||
cdev->pdev->bus->number,
|
||||
PCI_SLOT(cdev->pdev->devfn),
|
||||
hwfn->abs_pf_id);
|
||||
|
||||
|
|
|
|||
|
|
@ -1281,7 +1281,7 @@ static int efx_poll(struct napi_struct *napi, int budget)
|
|||
time = jiffies - channel->rfs_last_expiry;
|
||||
/* Would our quota be >= 20? */
|
||||
if (channel->rfs_filter_count * time >= 600 * HZ)
|
||||
mod_delayed_work(system_wq, &channel->filter_work, 0);
|
||||
mod_delayed_work(system_percpu_wq, &channel->filter_work, 0);
|
||||
#endif
|
||||
|
||||
/* There is no race here; although napi_disable() will
|
||||
|
|
|
|||
|
|
@ -1300,7 +1300,7 @@ static int efx_poll(struct napi_struct *napi, int budget)
|
|||
time = jiffies - channel->rfs_last_expiry;
|
||||
/* Would our quota be >= 20? */
|
||||
if (channel->rfs_filter_count * time >= 600 * HZ)
|
||||
mod_delayed_work(system_wq, &channel->filter_work, 0);
|
||||
mod_delayed_work(system_percpu_wq, &channel->filter_work, 0);
|
||||
#endif
|
||||
|
||||
/* There is no race here; although napi_disable() will
|
||||
|
|
|
|||
|
|
@ -1142,7 +1142,7 @@ int w5100_probe(struct device *dev, const struct w5100_ops *ops,
|
|||
if (err < 0)
|
||||
goto err_register;
|
||||
|
||||
priv->xfer_wq = alloc_workqueue("%s", WQ_MEM_RECLAIM, 0,
|
||||
priv->xfer_wq = alloc_workqueue("%s", WQ_MEM_RECLAIM | WQ_PERCPU, 0,
|
||||
netdev_name(ndev));
|
||||
if (!priv->xfer_wq) {
|
||||
err = -ENOMEM;
|
||||
|
|
|
|||
|
|
@ -1364,14 +1364,15 @@ static int fjes_probe(struct platform_device *plat_dev)
|
|||
adapter->force_reset = false;
|
||||
adapter->open_guard = false;
|
||||
|
||||
adapter->txrx_wq = alloc_workqueue(DRV_NAME "/txrx", WQ_MEM_RECLAIM, 0);
|
||||
adapter->txrx_wq = alloc_workqueue(DRV_NAME "/txrx",
|
||||
WQ_MEM_RECLAIM | WQ_PERCPU, 0);
|
||||
if (unlikely(!adapter->txrx_wq)) {
|
||||
err = -ENOMEM;
|
||||
goto err_free_netdev;
|
||||
}
|
||||
|
||||
adapter->control_wq = alloc_workqueue(DRV_NAME "/control",
|
||||
WQ_MEM_RECLAIM, 0);
|
||||
WQ_MEM_RECLAIM | WQ_PERCPU, 0);
|
||||
if (unlikely(!adapter->control_wq)) {
|
||||
err = -ENOMEM;
|
||||
goto err_free_txrx_wq;
|
||||
|
|
|
|||
|
|
@ -369,7 +369,7 @@ static void macvlan_broadcast_enqueue(struct macvlan_port *port,
|
|||
}
|
||||
spin_unlock(&port->bc_queue.lock);
|
||||
|
||||
queue_work(system_unbound_wq, &port->bc_work);
|
||||
queue_work(system_dfl_wq, &port->bc_work);
|
||||
|
||||
if (err)
|
||||
goto free_nskb;
|
||||
|
|
|
|||
|
|
@ -851,7 +851,7 @@ static void nsim_dev_trap_report_work(struct work_struct *work)
|
|||
nsim_dev = nsim_trap_data->nsim_dev;
|
||||
|
||||
if (!devl_trylock(priv_to_devlink(nsim_dev))) {
|
||||
queue_delayed_work(system_unbound_wq,
|
||||
queue_delayed_work(system_dfl_wq,
|
||||
&nsim_dev->trap_data->trap_report_dw, 1);
|
||||
return;
|
||||
}
|
||||
|
|
@ -867,7 +867,7 @@ static void nsim_dev_trap_report_work(struct work_struct *work)
|
|||
cond_resched();
|
||||
}
|
||||
devl_unlock(priv_to_devlink(nsim_dev));
|
||||
queue_delayed_work(system_unbound_wq,
|
||||
queue_delayed_work(system_dfl_wq,
|
||||
&nsim_dev->trap_data->trap_report_dw,
|
||||
msecs_to_jiffies(NSIM_TRAP_REPORT_INTERVAL_MS));
|
||||
}
|
||||
|
|
@ -924,7 +924,7 @@ static int nsim_dev_traps_init(struct devlink *devlink)
|
|||
|
||||
INIT_DELAYED_WORK(&nsim_dev->trap_data->trap_report_dw,
|
||||
nsim_dev_trap_report_work);
|
||||
queue_delayed_work(system_unbound_wq,
|
||||
queue_delayed_work(system_dfl_wq,
|
||||
&nsim_dev->trap_data->trap_report_dw,
|
||||
msecs_to_jiffies(NSIM_TRAP_REPORT_INTERVAL_MS));
|
||||
|
||||
|
|
|
|||
|
|
@ -911,7 +911,7 @@ static void sfp_soft_start_poll(struct sfp *sfp)
|
|||
|
||||
if (sfp->state_soft_mask & (SFP_F_LOS | SFP_F_TX_FAULT) &&
|
||||
!sfp->need_poll)
|
||||
mod_delayed_work(system_wq, &sfp->poll, poll_jiffies);
|
||||
mod_delayed_work(system_percpu_wq, &sfp->poll, poll_jiffies);
|
||||
mutex_unlock(&sfp->st_mutex);
|
||||
}
|
||||
|
||||
|
|
@ -1682,7 +1682,7 @@ static void sfp_hwmon_probe(struct work_struct *work)
|
|||
err = sfp_read(sfp, true, 0, &sfp->diag, sizeof(sfp->diag));
|
||||
if (err < 0) {
|
||||
if (sfp->hwmon_tries--) {
|
||||
mod_delayed_work(system_wq, &sfp->hwmon_probe,
|
||||
mod_delayed_work(system_percpu_wq, &sfp->hwmon_probe,
|
||||
T_PROBE_RETRY_SLOW);
|
||||
} else {
|
||||
dev_warn(sfp->dev, "hwmon probe failed: %pe\n",
|
||||
|
|
@ -1709,7 +1709,7 @@ static void sfp_hwmon_probe(struct work_struct *work)
|
|||
static int sfp_hwmon_insert(struct sfp *sfp)
|
||||
{
|
||||
if (sfp->have_a2 && sfp->id.ext.diagmon & SFP_DIAGMON_DDM) {
|
||||
mod_delayed_work(system_wq, &sfp->hwmon_probe, 1);
|
||||
mod_delayed_work(system_percpu_wq, &sfp->hwmon_probe, 1);
|
||||
sfp->hwmon_tries = R_PROBE_RETRY_SLOW;
|
||||
}
|
||||
|
||||
|
|
@ -2563,7 +2563,7 @@ static void sfp_sm_module(struct sfp *sfp, unsigned int event)
|
|||
/* Force a poll to re-read the hardware signal state after
|
||||
* sfp_sm_mod_probe() changed state_hw_mask.
|
||||
*/
|
||||
mod_delayed_work(system_wq, &sfp->poll, 1);
|
||||
mod_delayed_work(system_percpu_wq, &sfp->poll, 1);
|
||||
|
||||
err = sfp_hwmon_insert(sfp);
|
||||
if (err)
|
||||
|
|
@ -3008,7 +3008,7 @@ static void sfp_poll(struct work_struct *work)
|
|||
// it's unimportant if we race while reading this.
|
||||
if (sfp->state_soft_mask & (SFP_F_LOS | SFP_F_TX_FAULT) ||
|
||||
sfp->need_poll)
|
||||
mod_delayed_work(system_wq, &sfp->poll, poll_jiffies);
|
||||
mod_delayed_work(system_percpu_wq, &sfp->poll, poll_jiffies);
|
||||
}
|
||||
|
||||
static struct sfp *sfp_alloc(struct device *dev)
|
||||
|
|
@ -3178,7 +3178,7 @@ static int sfp_probe(struct platform_device *pdev)
|
|||
}
|
||||
|
||||
if (sfp->need_poll)
|
||||
mod_delayed_work(system_wq, &sfp->poll, poll_jiffies);
|
||||
mod_delayed_work(system_percpu_wq, &sfp->poll, poll_jiffies);
|
||||
|
||||
/* We could have an issue in cases no Tx disable pin is available or
|
||||
* wired as modules using a laser as their light source will continue to
|
||||
|
|
|
|||
|
|
@ -333,7 +333,8 @@ static int wg_newlink(struct net_device *dev,
|
|||
goto err_free_peer_hashtable;
|
||||
|
||||
wg->handshake_receive_wq = alloc_workqueue("wg-kex-%s",
|
||||
WQ_CPU_INTENSIVE | WQ_FREEZABLE, 0, dev->name);
|
||||
WQ_CPU_INTENSIVE | WQ_FREEZABLE | WQ_PERCPU, 0,
|
||||
dev->name);
|
||||
if (!wg->handshake_receive_wq)
|
||||
goto err_free_index_hashtable;
|
||||
|
||||
|
|
@ -343,7 +344,8 @@ static int wg_newlink(struct net_device *dev,
|
|||
goto err_destroy_handshake_receive;
|
||||
|
||||
wg->packet_crypt_wq = alloc_workqueue("wg-crypt-%s",
|
||||
WQ_CPU_INTENSIVE | WQ_MEM_RECLAIM, 0, dev->name);
|
||||
WQ_CPU_INTENSIVE | WQ_MEM_RECLAIM | WQ_PERCPU, 0,
|
||||
dev->name);
|
||||
if (!wg->packet_crypt_wq)
|
||||
goto err_destroy_handshake_send;
|
||||
|
||||
|
|
|
|||
|
|
@ -1085,7 +1085,8 @@ static void t7xx_dpmaif_bat_release_work(struct work_struct *work)
|
|||
int t7xx_dpmaif_bat_rel_wq_alloc(struct dpmaif_ctrl *dpmaif_ctrl)
|
||||
{
|
||||
dpmaif_ctrl->bat_release_wq = alloc_workqueue("dpmaif_bat_release_work_queue",
|
||||
WQ_MEM_RECLAIM, 1);
|
||||
WQ_MEM_RECLAIM | WQ_PERCPU,
|
||||
1);
|
||||
if (!dpmaif_ctrl->bat_release_wq)
|
||||
return -ENOMEM;
|
||||
|
||||
|
|
|
|||
|
|
@ -509,7 +509,7 @@ static int __init wwan_hwsim_init(void)
|
|||
if (wwan_hwsim_devsnum < 0 || wwan_hwsim_devsnum > 128)
|
||||
return -EINVAL;
|
||||
|
||||
wwan_wq = alloc_workqueue("wwan_wq", 0, 0);
|
||||
wwan_wq = alloc_workqueue("wwan_wq", WQ_PERCPU, 0);
|
||||
if (!wwan_wq)
|
||||
return -ENOMEM;
|
||||
|
||||
|
|
|
|||
|
|
@ -134,7 +134,7 @@ static void ccm_rx_timer_start(struct br_cfm_peer_mep *peer_mep)
|
|||
* of the configured CC 'expected_interval'
|
||||
* in order to detect CCM defect after 3.25 interval.
|
||||
*/
|
||||
queue_delayed_work(system_wq, &peer_mep->ccm_rx_dwork,
|
||||
queue_delayed_work(system_percpu_wq, &peer_mep->ccm_rx_dwork,
|
||||
usecs_to_jiffies(interval_us / 4));
|
||||
}
|
||||
|
||||
|
|
@ -285,7 +285,7 @@ static void ccm_tx_work_expired(struct work_struct *work)
|
|||
ccm_frame_tx(skb);
|
||||
|
||||
interval_us = interval_to_us(mep->cc_config.exp_interval);
|
||||
queue_delayed_work(system_wq, &mep->ccm_tx_dwork,
|
||||
queue_delayed_work(system_percpu_wq, &mep->ccm_tx_dwork,
|
||||
usecs_to_jiffies(interval_us));
|
||||
}
|
||||
|
||||
|
|
@ -809,7 +809,7 @@ int br_cfm_cc_ccm_tx(struct net_bridge *br, const u32 instance,
|
|||
* to send first frame immediately
|
||||
*/
|
||||
mep->ccm_tx_end = jiffies + usecs_to_jiffies(tx_info->period * 1000000);
|
||||
queue_delayed_work(system_wq, &mep->ccm_tx_dwork, 0);
|
||||
queue_delayed_work(system_percpu_wq, &mep->ccm_tx_dwork, 0);
|
||||
|
||||
save:
|
||||
mep->cc_ccm_tx_info = *tx_info;
|
||||
|
|
|
|||
|
|
@ -341,7 +341,7 @@ static void br_mrp_test_work_expired(struct work_struct *work)
|
|||
out:
|
||||
rcu_read_unlock();
|
||||
|
||||
queue_delayed_work(system_wq, &mrp->test_work,
|
||||
queue_delayed_work(system_percpu_wq, &mrp->test_work,
|
||||
usecs_to_jiffies(mrp->test_interval));
|
||||
}
|
||||
|
||||
|
|
@ -418,7 +418,7 @@ static void br_mrp_in_test_work_expired(struct work_struct *work)
|
|||
out:
|
||||
rcu_read_unlock();
|
||||
|
||||
queue_delayed_work(system_wq, &mrp->in_test_work,
|
||||
queue_delayed_work(system_percpu_wq, &mrp->in_test_work,
|
||||
usecs_to_jiffies(mrp->in_test_interval));
|
||||
}
|
||||
|
||||
|
|
@ -725,7 +725,7 @@ int br_mrp_start_test(struct net_bridge *br,
|
|||
mrp->test_max_miss = test->max_miss;
|
||||
mrp->test_monitor = test->monitor;
|
||||
mrp->test_count_miss = 0;
|
||||
queue_delayed_work(system_wq, &mrp->test_work,
|
||||
queue_delayed_work(system_percpu_wq, &mrp->test_work,
|
||||
usecs_to_jiffies(test->interval));
|
||||
|
||||
return 0;
|
||||
|
|
@ -865,7 +865,7 @@ int br_mrp_start_in_test(struct net_bridge *br,
|
|||
mrp->in_test_end = jiffies + usecs_to_jiffies(in_test->period);
|
||||
mrp->in_test_max_miss = in_test->max_miss;
|
||||
mrp->in_test_count_miss = 0;
|
||||
queue_delayed_work(system_wq, &mrp->in_test_work,
|
||||
queue_delayed_work(system_percpu_wq, &mrp->in_test_work,
|
||||
usecs_to_jiffies(in_test->interval));
|
||||
|
||||
return 0;
|
||||
|
|
|
|||
|
|
@ -252,7 +252,8 @@ int __init ceph_msgr_init(void)
|
|||
* The number of active work items is limited by the number of
|
||||
* connections, so leave @max_active at default.
|
||||
*/
|
||||
ceph_msgr_wq = alloc_workqueue("ceph-msgr", WQ_MEM_RECLAIM, 0);
|
||||
ceph_msgr_wq = alloc_workqueue("ceph-msgr",
|
||||
WQ_MEM_RECLAIM | WQ_PERCPU, 0);
|
||||
if (ceph_msgr_wq)
|
||||
return 0;
|
||||
|
||||
|
|
|
|||
|
|
@ -314,7 +314,7 @@ static void __schedule_delayed(struct ceph_mon_client *monc)
|
|||
delay = CEPH_MONC_PING_INTERVAL;
|
||||
|
||||
dout("__schedule_delayed after %lu\n", delay);
|
||||
mod_delayed_work(system_wq, &monc->delayed_work,
|
||||
mod_delayed_work(system_percpu_wq, &monc->delayed_work,
|
||||
round_jiffies_relative(delay));
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -157,9 +157,9 @@ static void linkwatch_schedule_work(int urgent)
|
|||
* override the existing timer.
|
||||
*/
|
||||
if (test_bit(LW_URGENT, &linkwatch_flags))
|
||||
mod_delayed_work(system_unbound_wq, &linkwatch_work, 0);
|
||||
mod_delayed_work(system_dfl_wq, &linkwatch_work, 0);
|
||||
else
|
||||
queue_delayed_work(system_unbound_wq, &linkwatch_work, delay);
|
||||
queue_delayed_work(system_dfl_wq, &linkwatch_work, delay);
|
||||
}
|
||||
|
||||
|
||||
|
|
|
|||
|
|
@ -876,7 +876,7 @@ void sk_psock_drop(struct sock *sk, struct sk_psock *psock)
|
|||
sk_psock_stop(psock);
|
||||
|
||||
INIT_RCU_WORK(&psock->rwork, sk_psock_destroy);
|
||||
queue_rcu_work(system_wq, &psock->rwork);
|
||||
queue_rcu_work(system_percpu_wq, &psock->rwork);
|
||||
}
|
||||
EXPORT_SYMBOL_GPL(sk_psock_drop);
|
||||
|
||||
|
|
|
|||
|
|
@ -348,7 +348,7 @@ static struct pernet_operations diag_net_ops = {
|
|||
|
||||
static int __init sock_diag_init(void)
|
||||
{
|
||||
broadcast_wq = alloc_workqueue("sock_diag_events", 0, 0);
|
||||
broadcast_wq = alloc_workqueue("sock_diag_events", WQ_PERCPU, 0);
|
||||
BUG_ON(!broadcast_wq);
|
||||
return register_pernet_subsys(&diag_net_ops);
|
||||
}
|
||||
|
|
|
|||
|
|
@ -320,7 +320,7 @@ static void devlink_release(struct work_struct *work)
|
|||
void devlink_put(struct devlink *devlink)
|
||||
{
|
||||
if (refcount_dec_and_test(&devlink->refcount))
|
||||
queue_rcu_work(system_wq, &devlink->rwork);
|
||||
queue_rcu_work(system_percpu_wq, &devlink->rwork);
|
||||
}
|
||||
|
||||
struct devlink *devlinks_xa_find_get(struct net *net, unsigned long *indexp)
|
||||
|
|
|
|||
|
|
@ -183,7 +183,7 @@ static void fqdir_work_fn(struct work_struct *work)
|
|||
rhashtable_free_and_destroy(&fqdir->rhashtable, inet_frags_free_cb, NULL);
|
||||
|
||||
if (llist_add(&fqdir->free_list, &fqdir_free_list))
|
||||
queue_delayed_work(system_wq, &fqdir_free_work, HZ);
|
||||
queue_delayed_work(system_percpu_wq, &fqdir_free_work, HZ);
|
||||
}
|
||||
|
||||
int fqdir_init(struct fqdir **fqdirp, struct inet_frags *f, struct net *net)
|
||||
|
|
|
|||
|
|
@ -301,7 +301,7 @@ void nf_conntrack_ecache_work(struct net *net, enum nf_ct_ecache_state state)
|
|||
net->ct.ecache_dwork_pending = true;
|
||||
} else if (state == NFCT_ECACHE_DESTROY_SENT) {
|
||||
if (!hlist_nulls_empty(&cnet->ecache.dying_list))
|
||||
mod_delayed_work(system_wq, &cnet->ecache.dwork, 0);
|
||||
mod_delayed_work(system_percpu_wq, &cnet->ecache.dwork, 0);
|
||||
else
|
||||
net->ct.ecache_dwork_pending = false;
|
||||
}
|
||||
|
|
|
|||
|
|
@ -75,7 +75,7 @@ static int dp_device_event(struct notifier_block *unused, unsigned long event,
|
|||
|
||||
/* schedule vport destroy, dev_put and genl notification */
|
||||
ovs_net = net_generic(dev_net(dev), ovs_net_id);
|
||||
queue_work(system_wq, &ovs_net->dp_notify_work);
|
||||
queue_work(system_percpu_wq, &ovs_net->dp_notify_work);
|
||||
}
|
||||
|
||||
return NOTIFY_DONE;
|
||||
|
|
|
|||
|
|
@ -672,7 +672,8 @@ struct rds_ib_mr_pool *rds_ib_create_mr_pool(struct rds_ib_device *rds_ibdev,
|
|||
|
||||
int rds_ib_mr_init(void)
|
||||
{
|
||||
rds_ib_mr_wq = alloc_workqueue("rds_mr_flushd", WQ_MEM_RECLAIM, 0);
|
||||
rds_ib_mr_wq = alloc_workqueue("rds_mr_flushd",
|
||||
WQ_MEM_RECLAIM | WQ_PERCPU, 0);
|
||||
if (!rds_ib_mr_wq)
|
||||
return -ENOMEM;
|
||||
return 0;
|
||||
|
|
|
|||
|
|
@ -159,7 +159,7 @@ static void rfkill_schedule_global_op(enum rfkill_sched_op op)
|
|||
rfkill_op_pending = true;
|
||||
if (op == RFKILL_GLOBAL_OP_EPO && !rfkill_is_epo_lock_active()) {
|
||||
/* bypass the limiter for EPO */
|
||||
mod_delayed_work(system_wq, &rfkill_op_work, 0);
|
||||
mod_delayed_work(system_percpu_wq, &rfkill_op_work, 0);
|
||||
rfkill_last_scheduled = jiffies;
|
||||
} else
|
||||
rfkill_schedule_ratelimited();
|
||||
|
|
|
|||
|
|
@ -630,7 +630,7 @@ static int __init rxperf_init(void)
|
|||
|
||||
pr_info("Server registering\n");
|
||||
|
||||
rxperf_workqueue = alloc_workqueue("rxperf", 0, 0);
|
||||
rxperf_workqueue = alloc_workqueue("rxperf", WQ_PERCPU, 0);
|
||||
if (!rxperf_workqueue)
|
||||
goto error_workqueue;
|
||||
|
||||
|
|
|
|||
|
|
@ -3535,15 +3535,15 @@ static int __init smc_init(void)
|
|||
|
||||
rc = -ENOMEM;
|
||||
|
||||
smc_tcp_ls_wq = alloc_workqueue("smc_tcp_ls_wq", 0, 0);
|
||||
smc_tcp_ls_wq = alloc_workqueue("smc_tcp_ls_wq", WQ_PERCPU, 0);
|
||||
if (!smc_tcp_ls_wq)
|
||||
goto out_pnet;
|
||||
|
||||
smc_hs_wq = alloc_workqueue("smc_hs_wq", 0, 0);
|
||||
smc_hs_wq = alloc_workqueue("smc_hs_wq", WQ_PERCPU, 0);
|
||||
if (!smc_hs_wq)
|
||||
goto out_alloc_tcp_ls_wq;
|
||||
|
||||
smc_close_wq = alloc_workqueue("smc_close_wq", 0, 0);
|
||||
smc_close_wq = alloc_workqueue("smc_close_wq", WQ_PERCPU, 0);
|
||||
if (!smc_close_wq)
|
||||
goto out_alloc_hs_wq;
|
||||
|
||||
|
|
|
|||
|
|
@ -85,7 +85,7 @@ static void smc_lgr_schedule_free_work(struct smc_link_group *lgr)
|
|||
* otherwise there is a risk of out-of-sync link groups.
|
||||
*/
|
||||
if (!lgr->freeing) {
|
||||
mod_delayed_work(system_wq, &lgr->free_work,
|
||||
mod_delayed_work(system_percpu_wq, &lgr->free_work,
|
||||
(!lgr->is_smcd && lgr->role == SMC_CLNT) ?
|
||||
SMC_LGR_FREE_DELAY_CLNT :
|
||||
SMC_LGR_FREE_DELAY_SERV);
|
||||
|
|
@ -896,7 +896,7 @@ static int smc_lgr_create(struct smc_sock *smc, struct smc_init_info *ini)
|
|||
rc = SMC_CLC_DECL_MEM;
|
||||
goto ism_put_vlan;
|
||||
}
|
||||
lgr->tx_wq = alloc_workqueue("smc_tx_wq-%*phN", 0, 0,
|
||||
lgr->tx_wq = alloc_workqueue("smc_tx_wq-%*phN", WQ_PERCPU, 0,
|
||||
SMC_LGR_ID_SIZE, &lgr->id);
|
||||
if (!lgr->tx_wq) {
|
||||
rc = -ENOMEM;
|
||||
|
|
|
|||
|
|
@ -1412,7 +1412,7 @@ int __init tls_device_init(void)
|
|||
if (!dummy_page)
|
||||
return -ENOMEM;
|
||||
|
||||
destruct_wq = alloc_workqueue("ktls_device_destruct", 0, 0);
|
||||
destruct_wq = alloc_workqueue("ktls_device_destruct", WQ_PERCPU, 0);
|
||||
if (!destruct_wq) {
|
||||
err = -ENOMEM;
|
||||
goto err_free_dummy;
|
||||
|
|
|
|||
|
|
@ -592,7 +592,7 @@ static DECLARE_WORK(unix_gc_work, __unix_gc);
|
|||
void unix_gc(void)
|
||||
{
|
||||
WRITE_ONCE(gc_in_progress, true);
|
||||
queue_work(system_unbound_wq, &unix_gc_work);
|
||||
queue_work(system_dfl_wq, &unix_gc_work);
|
||||
}
|
||||
|
||||
#define UNIX_INFLIGHT_TRIGGER_GC 16000
|
||||
|
|
|
|||
|
|
@ -1649,7 +1649,7 @@ static int vsock_connect(struct socket *sock, struct sockaddr *addr,
|
|||
* reschedule it, then ungrab the socket refcount to
|
||||
* keep it balanced.
|
||||
*/
|
||||
if (mod_delayed_work(system_wq, &vsk->connect_work,
|
||||
if (mod_delayed_work(system_percpu_wq, &vsk->connect_work,
|
||||
timeout))
|
||||
sock_put(sk);
|
||||
|
||||
|
|
|
|||
|
|
@ -926,7 +926,7 @@ static int __init virtio_vsock_init(void)
|
|||
{
|
||||
int ret;
|
||||
|
||||
virtio_vsock_workqueue = alloc_workqueue("virtio_vsock", 0, 0);
|
||||
virtio_vsock_workqueue = alloc_workqueue("virtio_vsock", WQ_PERCPU, 0);
|
||||
if (!virtio_vsock_workqueue)
|
||||
return -ENOMEM;
|
||||
|
||||
|
|
|
|||
|
|
@ -139,7 +139,7 @@ static int __init vsock_loopback_init(void)
|
|||
struct vsock_loopback *vsock = &the_vsock_loopback;
|
||||
int ret;
|
||||
|
||||
vsock->workqueue = alloc_workqueue("vsock-loopback", 0, 0);
|
||||
vsock->workqueue = alloc_workqueue("vsock-loopback", WQ_PERCPU, 0);
|
||||
if (!vsock->workqueue)
|
||||
return -ENOMEM;
|
||||
|
||||
|
|
|
|||
Loading…
Reference in New Issue
Block a user