[PATCH RFC net-next 1/3] dpaa2-eth: skip isolated CPUs when allocating channels
flat view
COOLING10d
From: Josua Mayer <hidden>
Date: 2026-09-30 16:55:48
Also in:
linux-arm-kernel, linux-crypto, lkml, netdev
Subsystem:
dpaa2 ethernet driver, networking drivers, the rest · Maintainers:
Ioana Ciornei, Andrew Lunn, "David S. Miller", Eric Dumazet, Jakub Kicinski, Paolo Abeni, Linus Torvalds
The driver allocates one channel per online CPU and registers it with that CPU's affine DPIO. When isolcpus=managed_irq is used to reserve CPUs for latency-sensitive applications, those CPUs are expected to be free of routine device interrupts. The DPIO driver assigns affine CPUs only from the HK_TYPE_MANAGED_IRQ housekeeping mask, leaving isolated CPUs without an affine DPIO. This defers dpaa2-eth probe indefinitely once channel setup reaches an isolated CPU. Restrict channel setup to the online CPUs in the HK_TYPE_MANAGED_IRQ housekeeping mask, so that isolated CPUs are skipped and don't defer probe. Signed-off-by: Josua Mayer <redacted> --- drivers/net/ethernet/freescale/dpaa2/dpaa2-eth.c | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/drivers/net/ethernet/freescale/dpaa2/dpaa2-eth.c b/drivers/net/ethernet/freescale/dpaa2/dpaa2-eth.c
index 6f1046c9cc515..850834388790a 100644
--- a/drivers/net/ethernet/freescale/dpaa2/dpaa2-eth.c
+++ b/drivers/net/ethernet/freescale/dpaa2/dpaa2-eth.c@@ -15,6 +15,7 @@ #include <linux/bpf_trace.h> #include <linux/fsl/ptp_qoriq.h> #include <linux/ptp_classify.h> +#include <linux/sched/isolation.h> #include <net/pkt_cls.h> #include <net/sock.h> #include <net/tso.h>
@@ -3173,6 +3174,7 @@ static int dpaa2_eth_setup_dpio(struct dpaa2_eth_priv *priv) struct dpaa2_eth_channel *channel; struct dpcon_notification_cfg dpcon_notif_cfg; struct device *dev = priv->net_dev->dev.parent; + const struct cpumask *hk_mask = housekeeping_cpumask(HK_TYPE_MANAGED_IRQ); int i, err; /* We want the ability to spread ingress traffic (RX, TX conf) to as
@@ -3182,10 +3184,11 @@ static int dpaa2_eth_setup_dpio(struct dpaa2_eth_priv *priv) * Allocate one channel per core and register it to the core's * affine DPIO. If not enough channels are available for all cores * or if some cores don't have an affine DPIO, there will be no - * ingress frame processing on those cores. + * ingress frame processing on those cores. Isolated cores + * (isolcpus=managed_irq) have no affine DPIO. */ cpumask_clear(&priv->dpio_cpumask); - for_each_online_cpu(i) { + for_each_cpu_and(i, cpu_online_mask, hk_mask) { /* Try to allocate a channel */ channel = dpaa2_eth_alloc_channel(priv); if (IS_ERR_OR_NULL(channel)) {
@@ -3245,6 +3248,9 @@ static int dpaa2_eth_setup_dpio(struct dpaa2_eth_priv *priv) break; } + if (cpumask_empty(&priv->dpio_cpumask)) + goto err_alloc_ch; + return 0; err_set_cdan:
--
2.51.0