From nobody Wed Jun 10 09:22:25 2026 Received: from linux.microsoft.com (linux.microsoft.com [13.77.154.182]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 100C73CF050; Fri, 15 May 2026 04:05:18 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=13.77.154.182 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778817921; cv=none; b=Mkchnq6rP9Vu9Y4YwX8RaUXnrmHMNLvTG30EhHZCBN/jiYsNDrdExal0KEW+dZcrt8NayVAhFkdv49JE/Y4YnYw3Tqj9xFPB5O6+qlOV9pF20Da30ilUUvobwD8TNZ+ioGfU6DEakJTQvpY0qXFNJlmUmuLO0tpMD9LsXnbHObk= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778817921; c=relaxed/simple; bh=+gAHt/EzgjiXAygIy6TTEcoxjLlJiMCcEoEdfSdo1Mc=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=bNtxW290WHiCEd6805Rq7hQmIa2ZxVX2OuzWOjZCI2ucYmtXPpTLkxdDP1M6GasCfC3BgQe/tsplUWriJxLP3oQTNSEcKTfNxDDRIIP27CiUDnWBvkERmrI7+MNaXh7rHMp/H/aaA/TO6FB8CKYXWyGsYAg7BAEc0+QJs6JG4jE= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=microsoft.com; spf=pass smtp.mailfrom=linux.microsoft.com; arc=none smtp.client-ip=13.77.154.182 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=microsoft.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linux.microsoft.com Received: by linux.microsoft.com (Postfix, from userid 1202) id 4BE8C20B7167; Thu, 14 May 2026 21:05:15 -0700 (PDT) DKIM-Filter: OpenDKIM Filter v2.11.0 linux.microsoft.com 4BE8C20B7167 From: Long Li To: Long Li , Konstantin Taranov , Jakub Kicinski , "David S . Miller" , Paolo Abeni , Eric Dumazet , Andrew Lunn , Jason Gunthorpe , Leon Romanovsky , Haiyang Zhang , "K . Y . Srinivasan" , Wei Liu , Dexuan Cui , shradhagupta@linux.microsoft.com Cc: Simon Horman , netdev@vger.kernel.org, linux-rdma@vger.kernel.org, linux-hyperv@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH net-next v10 1/6] net: mana: Create separate EQs for each vPort Date: Thu, 14 May 2026 21:05:03 -0700 Message-ID: <20260515040508.491748-2-longli@microsoft.com> X-Mailer: git-send-email 2.43.7 In-Reply-To: <20260515040508.491748-1-longli@microsoft.com> References: <20260515040508.491748-1-longli@microsoft.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" To prepare for assigning vPorts to dedicated MSI-X vectors, remove EQ sharing among the vPorts and create dedicated EQs for each vPort. Move the EQ definition from struct mana_context to struct mana_port_context and update related support functions. Export mana_create_eq() and mana_destroy_eq() for use by the MANA RDMA driver. RSS QPs now take a vport reference via pd->vport_use_count to ensure EQs outlive all QP consumers. The vport must already be configured by a raw QP before an RSS QP can be created. EQs are only destroyed when the last QP (raw or RSS) on the PD releases its reference. Reject cross-port PD sharing for raw QPs. Since EQs and vport configuration are per-port, a PD is bound to the port used by its first raw QP. Subsequent raw QPs on the same PD must use the same port or the creation fails with -EINVAL. Serialize mana_set_channels() against RDMA vport configuration to prevent num_queues from changing while RDMA holds EQs sized to the current value. When the port is down, apc->vport_mutex is held for the entire operation since mana_detach()/mana_attach() do not take vport_mutex in that case. When the port is up, Ethernet owns the vport exclusively so no additional locking is needed. Signed-off-by: Long Li --- drivers/infiniband/hw/mana/main.c | 40 ++++-- drivers/infiniband/hw/mana/mana_ib.h | 8 ++ drivers/infiniband/hw/mana/qp.c | 37 +++++- drivers/net/ethernet/microsoft/mana/mana_en.c | 117 +++++++++++------- .../ethernet/microsoft/mana/mana_ethtool.c | 23 +++- include/net/mana/mana.h | 15 ++- 6 files changed, 174 insertions(+), 66 deletions(-) diff --git a/drivers/infiniband/hw/mana/main.c b/drivers/infiniband/hw/mana= /main.c index ac5e75dd3494..f8a9013f0ca3 100644 --- a/drivers/infiniband/hw/mana/main.c +++ b/drivers/infiniband/hw/mana/main.c @@ -20,8 +20,10 @@ void mana_ib_uncfg_vport(struct mana_ib_dev *dev, struct= mana_ib_pd *pd, pd->vport_use_count--; WARN_ON(pd->vport_use_count < 0); =20 - if (!pd->vport_use_count) + if (!pd->vport_use_count) { + mana_destroy_eq(mpc); mana_uncfg_vport(mpc); + } =20 mutex_unlock(&pd->vport_mutex); } @@ -40,13 +42,27 @@ int mana_ib_cfg_vport(struct mana_ib_dev *dev, u32 port= , struct mana_ib_pd *pd, =20 pd->vport_use_count++; if (pd->vport_use_count > 1) { + /* Reject cross-port PD sharing. EQs and vport config + * are per-port, so the PD must stay bound to the port + * that was configured on the first raw QP creation. + */ + if (pd->vport_port !=3D port) { + pd->vport_use_count--; + mutex_unlock(&pd->vport_mutex); + ibdev_dbg(&dev->ib_dev, + "PD already bound to port %u\n", + pd->vport_port); + return -EINVAL; + } ibdev_dbg(&dev->ib_dev, "Skip as this PD is already configured vport\n"); mutex_unlock(&pd->vport_mutex); return 0; } =20 - err =3D mana_cfg_vport(mpc, pd->pdn, doorbell_id); + pd->vport_port =3D port; + + err =3D mana_cfg_vport(mpc, pd->pdn, doorbell_id, true); if (err) { pd->vport_use_count--; mutex_unlock(&pd->vport_mutex); @@ -55,15 +71,23 @@ int mana_ib_cfg_vport(struct mana_ib_dev *dev, u32 port= , struct mana_ib_pd *pd, return err; } =20 - mutex_unlock(&pd->vport_mutex); =20 - pd->tx_shortform_allowed =3D mpc->tx_shortform_allowed; - pd->tx_vp_offset =3D mpc->tx_vp_offset; + err =3D mana_create_eq(mpc); + if (err) { + mana_uncfg_vport(mpc); + pd->vport_use_count--; + } else { + pd->tx_shortform_allowed =3D mpc->tx_shortform_allowed; + pd->tx_vp_offset =3D mpc->tx_vp_offset; + } + + mutex_unlock(&pd->vport_mutex); =20 - ibdev_dbg(&dev->ib_dev, "vport handle %llx pdid %x doorbell_id %x\n", - mpc->port_handle, pd->pdn, doorbell_id); + if (!err) + ibdev_dbg(&dev->ib_dev, "vport handle %llx pdid %x doorbell_id %x\n", + mpc->port_handle, pd->pdn, doorbell_id); =20 - return 0; + return err; } =20 int mana_ib_alloc_pd(struct ib_pd *ibpd, struct ib_udata *udata) diff --git a/drivers/infiniband/hw/mana/mana_ib.h b/drivers/infiniband/hw/m= ana/mana_ib.h index c9c94e86a72b..916f03fb96dd 100644 --- a/drivers/infiniband/hw/mana/mana_ib.h +++ b/drivers/infiniband/hw/mana/mana_ib.h @@ -102,6 +102,14 @@ struct mana_ib_pd { struct mutex vport_mutex; int vport_use_count; =20 + /* Port bound to this PD for raw QP usage. A PD can only be + * associated with a single physical port because per-port EQs + * and vport configuration are tied to the PD's refcount. + * Set on the first raw QP creation; subsequent QPs on the + * same PD must use the same port or get -EINVAL. + */ + u32 vport_port; + bool tx_shortform_allowed; u32 tx_vp_offset; }; diff --git a/drivers/infiniband/hw/mana/qp.c b/drivers/infiniband/hw/mana/q= p.c index 0fbcf449c134..108ec4c5ce51 100644 --- a/drivers/infiniband/hw/mana/qp.c +++ b/drivers/infiniband/hw/mana/qp.c @@ -79,6 +79,7 @@ static int mana_ib_create_qp_rss(struct ib_qp *ibqp, stru= ct ib_pd *pd, struct ib_qp_init_attr *attr, struct ib_udata *udata) { + struct mana_ib_pd *mana_pd =3D container_of(pd, struct mana_ib_pd, ibpd); struct mana_ib_qp *qp =3D container_of(ibqp, struct mana_ib_qp, ibqp); struct mana_ib_dev *mdev =3D container_of(pd->device, struct mana_ib_dev, ib_dev); @@ -155,6 +156,18 @@ static int mana_ib_create_qp_rss(struct ib_qp *ibqp, s= truct ib_pd *pd, =20 qp->port =3D port; =20 + /* Take a reference on the vport to ensure EQs outlive this QP. + * The vport must already be configured by a raw QP. + */ + mutex_lock(&mana_pd->vport_mutex); + if (!mana_pd->vport_use_count) { + mutex_unlock(&mana_pd->vport_mutex); + ret =3D -EINVAL; + goto fail; + } + mana_pd->vport_use_count++; + mutex_unlock(&mana_pd->vport_mutex); + for (i =3D 0; i < ind_tbl_size; i++) { struct mana_obj_spec wq_spec =3D {}; struct mana_obj_spec cq_spec =3D {}; @@ -171,13 +184,13 @@ static int mana_ib_create_qp_rss(struct ib_qp *ibqp, = struct ib_pd *pd, cq_spec.gdma_region =3D cq->queue.gdma_region; cq_spec.queue_size =3D cq->cqe * COMP_ENTRY_SIZE; cq_spec.modr_ctx_id =3D 0; - eq =3D &mpc->ac->eqs[cq->comp_vector]; + eq =3D &mpc->eqs[cq->comp_vector % mpc->num_queues]; cq_spec.attached_eq =3D eq->eq->id; =20 ret =3D mana_create_wq_obj(mpc, mpc->port_handle, GDMA_RQ, &wq_spec, &cq_spec, &wq->rx_object); if (ret) - goto fail; + goto free_vport; =20 /* The GDMA regions are now owned by the WQ object */ wq->queue.gdma_region =3D GDMA_INVALID_DMA_REGION; @@ -199,7 +212,7 @@ static int mana_ib_create_qp_rss(struct ib_qp *ibqp, st= ruct ib_pd *pd, ret =3D mana_ib_install_cq_cb(mdev, cq); if (ret) { mana_destroy_wq_obj(mpc, GDMA_RQ, wq->rx_object); - goto fail; + goto free_vport; } } resp.num_entries =3D i; @@ -210,7 +223,7 @@ static int mana_ib_create_qp_rss(struct ib_qp *ibqp, st= ruct ib_pd *pd, ucmd.rx_hash_key_len, ucmd.rx_hash_key); if (ret) - goto fail; + goto free_vport; =20 ret =3D ib_copy_to_udata(udata, &resp, sizeof(resp)); if (ret) { @@ -226,7 +239,7 @@ static int mana_ib_create_qp_rss(struct ib_qp *ibqp, st= ruct ib_pd *pd, =20 err_disable_vport_rx: mana_disable_vport_rx(mpc); -fail: +free_vport: while (i-- > 0) { ibwq =3D ind_tbl->ind_tbl[i]; ibcq =3D ibwq->cq; @@ -237,6 +250,9 @@ static int mana_ib_create_qp_rss(struct ib_qp *ibqp, st= ruct ib_pd *pd, mana_destroy_wq_obj(mpc, GDMA_RQ, wq->rx_object); } =20 + mana_ib_uncfg_vport(mdev, mana_pd, port); + +fail: kfree(mana_ind_table); =20 return ret; @@ -321,7 +337,11 @@ static int mana_ib_create_qp_raw(struct ib_qp *ibqp, s= truct ib_pd *ibpd, cq_spec.queue_size =3D send_cq->cqe * COMP_ENTRY_SIZE; cq_spec.modr_ctx_id =3D 0; eq_vec =3D send_cq->comp_vector; - eq =3D &mpc->ac->eqs[eq_vec]; + if (!mpc->eqs) { + err =3D -EINVAL; + goto err_destroy_queue; + } + eq =3D &mpc->eqs[eq_vec % mpc->num_queues]; cq_spec.attached_eq =3D eq->eq->id; =20 err =3D mana_create_wq_obj(mpc, mpc->port_handle, GDMA_SQ, &wq_spec, @@ -785,14 +805,17 @@ static int mana_ib_destroy_qp_rss(struct mana_ib_qp *= qp, { struct mana_ib_dev *mdev =3D container_of(qp->ibqp.device, struct mana_ib_dev, ib_dev); + struct ib_pd *ibpd =3D qp->ibqp.pd; struct mana_port_context *mpc; struct net_device *ndev; + struct mana_ib_pd *pd; struct mana_ib_wq *wq; struct ib_wq *ibwq; int i; =20 ndev =3D mana_ib_get_netdev(qp->ibqp.device, qp->port); mpc =3D netdev_priv(ndev); + pd =3D container_of(ibpd, struct mana_ib_pd, ibpd); =20 /* Disable vPort RX steering before destroying RX WQ objects. * Otherwise firmware still routes traffic to the destroyed queues, @@ -817,6 +840,8 @@ static int mana_ib_destroy_qp_rss(struct mana_ib_qp *qp, mana_destroy_wq_obj(mpc, GDMA_RQ, wq->rx_object); } =20 + mana_ib_uncfg_vport(mdev, pd, qp->port); + return 0; } =20 diff --git a/drivers/net/ethernet/microsoft/mana/mana_en.c b/drivers/net/et= hernet/microsoft/mana/mana_en.c index b2faa7cf398f..18f8f653da3d 100644 --- a/drivers/net/ethernet/microsoft/mana/mana_en.c +++ b/drivers/net/ethernet/microsoft/mana/mana_en.c @@ -1295,7 +1295,7 @@ void mana_uncfg_vport(struct mana_port_context *apc) EXPORT_SYMBOL_NS(mana_uncfg_vport, "NET_MANA"); =20 int mana_cfg_vport(struct mana_port_context *apc, u32 protection_dom_id, - u32 doorbell_pg_id) + u32 doorbell_pg_id, bool check_channel_changing) { struct mana_config_vport_resp resp =3D {}; struct mana_config_vport_req req =3D {}; @@ -1320,7 +1320,8 @@ int mana_cfg_vport(struct mana_port_context *apc, u32= protection_dom_id, * Ethernet usage on the same port. */ mutex_lock(&apc->vport_mutex); - if (apc->vport_use_count > 0) { + if (apc->vport_use_count > 0 || + (check_channel_changing && apc->channel_changing)) { mutex_unlock(&apc->vport_mutex); return -EBUSY; } @@ -1615,78 +1616,84 @@ void mana_destroy_wq_obj(struct mana_port_context *= apc, u32 wq_type, } EXPORT_SYMBOL_NS(mana_destroy_wq_obj, "NET_MANA"); =20 -static void mana_destroy_eq(struct mana_context *ac) +void mana_destroy_eq(struct mana_port_context *apc) { + struct mana_context *ac =3D apc->ac; struct gdma_context *gc =3D ac->gdma_dev->gdma_context; struct gdma_queue *eq; int i; =20 - if (!ac->eqs) + if (!apc->eqs) return; =20 - debugfs_remove_recursive(ac->mana_eqs_debugfs); - ac->mana_eqs_debugfs =3D NULL; + debugfs_remove_recursive(apc->mana_eqs_debugfs); + apc->mana_eqs_debugfs =3D NULL; =20 - for (i =3D 0; i < gc->max_num_queues; i++) { - eq =3D ac->eqs[i].eq; + for (i =3D 0; i < apc->num_queues; i++) { + eq =3D apc->eqs[i].eq; if (!eq) continue; =20 mana_gd_destroy_queue(gc, eq); } =20 - kfree(ac->eqs); - ac->eqs =3D NULL; + kfree(apc->eqs); + apc->eqs =3D NULL; } +EXPORT_SYMBOL_NS(mana_destroy_eq, "NET_MANA"); =20 -static void mana_create_eq_debugfs(struct mana_context *ac, int i) +static void mana_create_eq_debugfs(struct mana_port_context *apc, int i) { - struct mana_eq eq =3D ac->eqs[i]; + struct mana_eq eq =3D apc->eqs[i]; char eqnum[32]; =20 sprintf(eqnum, "eq%d", i); - eq.mana_eq_debugfs =3D debugfs_create_dir(eqnum, ac->mana_eqs_debugfs); + eq.mana_eq_debugfs =3D debugfs_create_dir(eqnum, apc->mana_eqs_debugfs); debugfs_create_u32("head", 0400, eq.mana_eq_debugfs, &eq.eq->head); debugfs_create_u32("tail", 0400, eq.mana_eq_debugfs, &eq.eq->tail); debugfs_create_file("eq_dump", 0400, eq.mana_eq_debugfs, eq.eq, &mana_dbg= _q_fops); } =20 -static int mana_create_eq(struct mana_context *ac) +int mana_create_eq(struct mana_port_context *apc) { - struct gdma_dev *gd =3D ac->gdma_dev; + struct gdma_dev *gd =3D apc->ac->gdma_dev; struct gdma_context *gc =3D gd->gdma_context; struct gdma_queue_spec spec =3D {}; int err; int i; =20 - ac->eqs =3D kzalloc_objs(struct mana_eq, gc->max_num_queues); - if (!ac->eqs) + if (WARN_ON(apc->eqs)) + return -EEXIST; + apc->eqs =3D kzalloc_objs(struct mana_eq, apc->num_queues); + if (!apc->eqs) return -ENOMEM; =20 spec.type =3D GDMA_EQ; spec.monitor_avl_buf =3D false; spec.queue_size =3D EQ_SIZE; spec.eq.callback =3D NULL; - spec.eq.context =3D ac->eqs; + spec.eq.context =3D apc->eqs; spec.eq.log2_throttle_limit =3D LOG2_EQ_THROTTLE; =20 - ac->mana_eqs_debugfs =3D debugfs_create_dir("EQs", gc->mana_pci_debugfs); + apc->mana_eqs_debugfs =3D + debugfs_create_dir("EQs", apc->mana_port_debugfs); =20 - for (i =3D 0; i < gc->max_num_queues; i++) { + for (i =3D 0; i < apc->num_queues; i++) { spec.eq.msix_index =3D (i + 1) % gc->num_msix_usable; - err =3D mana_gd_create_mana_eq(gd, &spec, &ac->eqs[i].eq); + err =3D mana_gd_create_mana_eq(gd, &spec, &apc->eqs[i].eq); if (err) { dev_err(gc->dev, "Failed to create EQ %d : %d\n", i, err); goto out; } - mana_create_eq_debugfs(ac, i); + mana_create_eq_debugfs(apc, i); } =20 return 0; out: - mana_destroy_eq(ac); + mana_destroy_eq(apc); return err; } +EXPORT_SYMBOL_NS(mana_create_eq, "NET_MANA"); =20 static int mana_fence_rq(struct mana_port_context *apc, struct mana_rxq *r= xq) { @@ -2451,7 +2458,7 @@ static int mana_create_txq(struct mana_port_context *= apc, spec.monitor_avl_buf =3D false; spec.queue_size =3D cq_size; spec.cq.callback =3D mana_schedule_napi; - spec.cq.parent_eq =3D ac->eqs[i].eq; + spec.cq.parent_eq =3D apc->eqs[i].eq; spec.cq.context =3D cq; err =3D mana_gd_create_mana_wq_cq(gd, &spec, &cq->gdma_cq); if (err) @@ -2844,13 +2851,12 @@ static void mana_create_rxq_debugfs(struct mana_por= t_context *apc, int idx) static int mana_add_rx_queues(struct mana_port_context *apc, struct net_device *ndev) { - struct mana_context *ac =3D apc->ac; struct mana_rxq *rxq; int err =3D 0; int i; =20 for (i =3D 0; i < apc->num_queues; i++) { - rxq =3D mana_create_rxq(apc, i, &ac->eqs[i], ndev); + rxq =3D mana_create_rxq(apc, i, &apc->eqs[i], ndev); if (!rxq) { err =3D -ENOMEM; netdev_err(ndev, "Failed to create rxq %d : %d\n", i, err); @@ -2869,9 +2875,8 @@ static int mana_add_rx_queues(struct mana_port_contex= t *apc, return err; } =20 -static void mana_destroy_vport(struct mana_port_context *apc) +static void mana_destroy_rxqs(struct mana_port_context *apc) { - struct gdma_dev *gd =3D apc->ac->gdma_dev; struct mana_rxq *rxq; u32 rxq_idx; =20 @@ -2883,8 +2888,12 @@ static void mana_destroy_vport(struct mana_port_cont= ext *apc) mana_destroy_rxq(apc, rxq, true); apc->rxqs[rxq_idx] =3D NULL; } +} + +static void mana_destroy_vport(struct mana_port_context *apc) +{ + struct gdma_dev *gd =3D apc->ac->gdma_dev; =20 - mana_destroy_txq(apc); mana_uncfg_vport(apc); =20 if (gd->gdma_context->is_pf && !apc->ac->bm_hostmode) @@ -2905,11 +2914,7 @@ static int mana_create_vport(struct mana_port_contex= t *apc, return err; } =20 - err =3D mana_cfg_vport(apc, gd->pdid, gd->doorbell); - if (err) - return err; - - return mana_create_txq(apc, net); + return mana_cfg_vport(apc, gd->pdid, gd->doorbell, false); } =20 static int mana_rss_table_alloc(struct mana_port_context *apc) @@ -3195,21 +3200,36 @@ int mana_alloc_queues(struct net_device *ndev) =20 err =3D mana_create_vport(apc, ndev); if (err) { - netdev_err(ndev, "Failed to create vPort %u : %d\n", apc->port_idx, err); + netdev_err(ndev, "Failed to create vPort %u : %d\n", + apc->port_idx, err); return err; } =20 + err =3D mana_create_eq(apc); + if (err) { + netdev_err(ndev, "Failed to create EQ on vPort %u: %d\n", + apc->port_idx, err); + goto destroy_vport; + } + + err =3D mana_create_txq(apc, ndev); + if (err) { + netdev_err(ndev, "Failed to create TXQ on vPort %u: %d\n", + apc->port_idx, err); + goto destroy_eq; + } + err =3D netif_set_real_num_tx_queues(ndev, apc->num_queues); if (err) { netdev_err(ndev, "netif_set_real_num_tx_queues () failed for ndev with num_queues %u = : %d\n", apc->num_queues, err); - goto destroy_vport; + goto destroy_txq; } =20 err =3D mana_add_rx_queues(apc, ndev); if (err) - goto destroy_vport; + goto destroy_rxq; =20 apc->rss_state =3D apc->num_queues > 1 ? TRI_STATE_TRUE : TRI_STATE_FALSE; =20 @@ -3218,7 +3238,7 @@ int mana_alloc_queues(struct net_device *ndev) netdev_err(ndev, "netif_set_real_num_rx_queues () failed for ndev with num_queues %u = : %d\n", apc->num_queues, err); - goto destroy_vport; + goto destroy_rxq; } =20 mana_rss_table_init(apc); @@ -3226,19 +3246,25 @@ int mana_alloc_queues(struct net_device *ndev) err =3D mana_config_rss(apc, TRI_STATE_TRUE, true, true); if (err) { netdev_err(ndev, "Failed to configure RSS table: %d\n", err); - goto destroy_vport; + goto destroy_rxq; } =20 if (gd->gdma_context->is_pf && !apc->ac->bm_hostmode) { err =3D mana_pf_register_filter(apc); if (err) - goto destroy_vport; + goto destroy_rxq; } =20 mana_chn_setxdp(apc, mana_xdp_get(apc)); =20 return 0; =20 +destroy_rxq: + mana_destroy_rxqs(apc); +destroy_txq: + mana_destroy_txq(apc); +destroy_eq: + mana_destroy_eq(apc); destroy_vport: mana_destroy_vport(apc); return err; @@ -3343,6 +3369,9 @@ static int mana_dealloc_queues(struct net_device *nde= v) mana_fence_rqs(apc); =20 /* Even in err case, still need to cleanup the vPort */ + mana_destroy_rxqs(apc); + mana_destroy_txq(apc); + mana_destroy_eq(apc); mana_destroy_vport(apc); =20 return 0; @@ -3663,12 +3692,6 @@ int mana_probe(struct gdma_dev *gd, bool resuming) =20 INIT_DELAYED_WORK(&ac->gf_stats_work, mana_gf_stats_work_handler); =20 - err =3D mana_create_eq(ac); - if (err) { - dev_err(dev, "Failed to create EQs: %d\n", err); - goto out; - } - err =3D mana_query_device_cfg(ac, MANA_MAJOR_VERSION, MANA_MINOR_VERSION, MANA_MICRO_VERSION, &num_ports, &bm_hostmode); if (err) @@ -3808,8 +3831,6 @@ void mana_remove(struct gdma_dev *gd, bool suspending) free_netdev(ndev); } =20 - mana_destroy_eq(ac); - if (ac->per_port_queue_reset_wq) { destroy_workqueue(ac->per_port_queue_reset_wq); ac->per_port_queue_reset_wq =3D NULL; diff --git a/drivers/net/ethernet/microsoft/mana/mana_ethtool.c b/drivers/n= et/ethernet/microsoft/mana/mana_ethtool.c index 04350973e19e..4633acc976f0 100644 --- a/drivers/net/ethernet/microsoft/mana/mana_ethtool.c +++ b/drivers/net/ethernet/microsoft/mana/mana_ethtool.c @@ -454,6 +454,11 @@ static int mana_set_coalesce(struct net_device *ndev, return err; } =20 +/* mana_set_channels - change the number of queues on a port + * + * Returns -EBUSY if RDMA holds the vport with EQs sized to the + * current num_queues. + */ static int mana_set_channels(struct net_device *ndev, struct ethtool_channels *channels) { @@ -462,10 +467,22 @@ static int mana_set_channels(struct net_device *ndev, unsigned int old_count =3D apc->num_queues; int err; =20 + /* Set channel_changing to block RDMA from grabbing the vport + * during the detach/attach window. mana_cfg_vport() checks + * this flag under vport_mutex and returns -EBUSY if set. + */ + mutex_lock(&apc->vport_mutex); + if (!apc->port_is_up && apc->vport_use_count) { + mutex_unlock(&apc->vport_mutex); + return -EBUSY; + } + apc->channel_changing =3D true; + mutex_unlock(&apc->vport_mutex); + err =3D mana_pre_alloc_rxbufs(apc, ndev->mtu, new_count); if (err) { netdev_err(ndev, "Insufficient memory for new allocations"); - return err; + goto clear_flag; } =20 err =3D mana_detach(ndev, false); @@ -483,6 +500,10 @@ static int mana_set_channels(struct net_device *ndev, =20 out: mana_pre_dealloc_rxbufs(apc); +clear_flag: + mutex_lock(&apc->vport_mutex); + apc->channel_changing =3D false; + mutex_unlock(&apc->vport_mutex); return err; } =20 diff --git a/include/net/mana/mana.h b/include/net/mana/mana.h index aa90a858c8e3..e2384c5ab1df 100644 --- a/include/net/mana/mana.h +++ b/include/net/mana/mana.h @@ -480,8 +480,6 @@ struct mana_context { u8 bm_hostmode; =20 struct mana_ethtool_hc_stats hc_stats; - struct mana_eq *eqs; - struct dentry *mana_eqs_debugfs; struct workqueue_struct *per_port_queue_reset_wq; /* Workqueue for querying hardware stats */ struct delayed_work gf_stats_work; @@ -501,6 +499,9 @@ struct mana_port_context { =20 u8 mac_addr[ETH_ALEN]; =20 + struct mana_eq *eqs; + struct dentry *mana_eqs_debugfs; + enum TRI_STATE rss_state; =20 mana_handle_t default_rxobj; @@ -547,6 +548,12 @@ struct mana_port_context { struct mutex vport_mutex; int vport_use_count; =20 + /* Set by mana_set_channels() under vport_mutex to block RDMA + * from grabbing the vport during the detach/attach window. + * Checked by mana_cfg_vport() when called from the RDMA path. + */ + bool channel_changing; + /* Net shaper handle*/ struct net_shaper_handle handle; =20 @@ -1032,8 +1039,10 @@ void mana_destroy_wq_obj(struct mana_port_context *a= pc, u32 wq_type, mana_handle_t wq_obj); =20 int mana_cfg_vport(struct mana_port_context *apc, u32 protection_dom_id, - u32 doorbell_pg_id); + u32 doorbell_pg_id, bool check_channel_changing); void mana_uncfg_vport(struct mana_port_context *apc); +int mana_create_eq(struct mana_port_context *apc); +void mana_destroy_eq(struct mana_port_context *apc); =20 struct net_device *mana_get_primary_netdev(struct mana_context *ac, u32 port_index, --=20 2.43.0 From nobody Wed Jun 10 09:22:25 2026 Received: from linux.microsoft.com (linux.microsoft.com [13.77.154.182]) by smtp.subspace.kernel.org (Postfix) with ESMTP id A701B3D0C13; Fri, 15 May 2026 04:05:19 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=13.77.154.182 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778817921; cv=none; b=pq76QpPxmEeE6QDPmfse1sn2Uq5tIEsFo33fUre2RhHZDYLpuge3RDTjbjgV28NxcIeSrakKSpJyDDdoSzRELGMeWSj15jgoN9N2vVeWWZZ8UaFqCO9CiSssvMQ71Czl9gR47G6cNOaim0bTQUGsSzR+CPLRVtLOESaTRRNPZCE= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778817921; c=relaxed/simple; bh=1khhet+8q7b0+F2/Xe+XXlz9HLHhe9WOmPtCJ+sV5kc=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=oImP48h2UUPk88fPFM2O1tWmsBEw8HT13t+inJQumSCd6j4mYgqq/ZKRJsRs9LbmKhzv2kvWmQeWKjGTt6aLSMpnueQ+kmer0xE1lhCbZWhr5DBBvKVFxJMq0P5Sxx9NRi/A4Os3oziByoRP2MyhbhtyHKKL9helvu1zvd7DrQs= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=microsoft.com; spf=pass smtp.mailfrom=linux.microsoft.com; arc=none smtp.client-ip=13.77.154.182 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=microsoft.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linux.microsoft.com Received: by linux.microsoft.com (Postfix, from userid 1202) id 0264520B7168; Thu, 14 May 2026 21:05:16 -0700 (PDT) DKIM-Filter: OpenDKIM Filter v2.11.0 linux.microsoft.com 0264520B7168 From: Long Li To: Long Li , Konstantin Taranov , Jakub Kicinski , "David S . Miller" , Paolo Abeni , Eric Dumazet , Andrew Lunn , Jason Gunthorpe , Leon Romanovsky , Haiyang Zhang , "K . Y . Srinivasan" , Wei Liu , Dexuan Cui , shradhagupta@linux.microsoft.com Cc: Simon Horman , netdev@vger.kernel.org, linux-rdma@vger.kernel.org, linux-hyperv@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH net-next v10 2/6] net: mana: Query device capabilities and configure MSI-X sharing for EQs Date: Thu, 14 May 2026 21:05:04 -0700 Message-ID: <20260515040508.491748-3-longli@microsoft.com> X-Mailer: git-send-email 2.43.7 In-Reply-To: <20260515040508.491748-1-longli@microsoft.com> References: <20260515040508.491748-1-longli@microsoft.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" When querying the device, adjust the max number of queues to allow dedicated MSI-X vectors for each vPort. The per-vPort queue count is clamped towards MANA_DEF_NUM_QUEUES but will not exceed the hardware maximum reported by the device. MSI-X sharing among vPorts is enabled when there are not enough MSI-X vectors for dedicated allocation, or when the platform does not support dynamic MSI-X allocation (in which case all vectors are pre-allocated at probe time and sharing is always used). The msi_sharing flag is reset at the top of mana_gd_query_max_resources() so it is recomputed from current hardware state on each probe or resume cycle. A device reporting zero ports now results in a fatal probe error since the per-vPort MSI-X math requires at least one port. Rename mana_query_device_cfg() to mana_gd_query_device_cfg() as it is used at GDMA device probe time for querying device capabilities. Signed-off-by: Long Li --- .../net/ethernet/microsoft/mana/gdma_main.c | 69 ++++++++++++++++++- drivers/net/ethernet/microsoft/mana/mana_en.c | 40 +++++++---- include/net/mana/gdma.h | 13 +++- 3 files changed, 103 insertions(+), 19 deletions(-) diff --git a/drivers/net/ethernet/microsoft/mana/gdma_main.c b/drivers/net/= ethernet/microsoft/mana/gdma_main.c index 3bc3fff55999..13feea93d3c2 100644 --- a/drivers/net/ethernet/microsoft/mana/gdma_main.c +++ b/drivers/net/ethernet/microsoft/mana/gdma_main.c @@ -179,8 +179,21 @@ static int mana_gd_query_max_resources(struct pci_dev = *pdev) struct gdma_context *gc =3D pci_get_drvdata(pdev); struct gdma_query_max_resources_resp resp =3D {}; struct gdma_general_req req =3D {}; + unsigned int max_num_queues; + u8 bm_hostmode; + u16 num_ports; int err; =20 + /* Reset msi_sharing so it is recomputed from current hardware + * state. On resume, num_online_cpus() or num_msix_usable may + * have changed, making dedicated MSI-X feasible where it was + * not before. Only reset on platforms that support dynamic + * MSI-X allocation; on non-dyn platforms msi_sharing is + * unconditionally true (set in mana_gd_setup_hwc_irqs). + */ + if (pci_msix_can_alloc_dyn(to_pci_dev(gc->dev))) + gc->msi_sharing =3D false; + mana_gd_init_req_hdr(&req.hdr, GDMA_QUERY_MAX_RESOURCES, sizeof(req), sizeof(resp)); =20 @@ -227,6 +240,43 @@ static int mana_gd_query_max_resources(struct pci_dev = *pdev) if (gc->max_num_queues =3D=3D 0) return -ENOSPC; =20 + err =3D mana_gd_query_device_cfg(gc, MANA_MAJOR_VERSION, + MANA_MINOR_VERSION, + MANA_MICRO_VERSION, + &num_ports, &bm_hostmode); + if (err) + return err; + + if (!num_ports) + return -EINVAL; + + /* + * Adjust the per-vPort max queue count to allow dedicated + * MSIx for each vPort. Clamp to no less than MANA_DEF_NUM_QUEUES. + */ + max_num_queues =3D (gc->num_msix_usable - 1) / num_ports; + max_num_queues =3D rounddown_pow_of_two(max(max_num_queues, 1U)); + if (max_num_queues < MANA_DEF_NUM_QUEUES) + max_num_queues =3D MANA_DEF_NUM_QUEUES; + + /* + * Use dedicated MSIx for EQs whenever possible, use MSIx sharing for + * Ethernet EQs when (max_num_queues * num_ports > num_msix_usable - 1). + */ + max_num_queues =3D min(gc->max_num_queues, max_num_queues); + if (max_num_queues * num_ports > gc->num_msix_usable - 1) + gc->msi_sharing =3D true; + + /* If MSI is shared, use max allowed value */ + if (gc->msi_sharing) + gc->max_num_queues_vport =3D min(gc->num_msix_usable - 1, + gc->max_num_queues); + else + gc->max_num_queues_vport =3D max_num_queues; + + dev_info(gc->dev, "MSI sharing mode %d max queues %d\n", + gc->msi_sharing, gc->max_num_queues); + return 0; } =20 @@ -1889,6 +1939,7 @@ static int mana_gd_setup_hwc_irqs(struct pci_dev *pde= v) /* Need 1 interrupt for HWC */ max_irqs =3D min(num_online_cpus(), MANA_MAX_NUM_QUEUES) + 1; min_irqs =3D 2; + gc->msi_sharing =3D true; } =20 nvec =3D pci_alloc_irq_vectors(pdev, min_irqs, max_irqs, PCI_IRQ_MSIX); @@ -1967,6 +2018,8 @@ static void mana_gd_remove_irqs(struct pci_dev *pdev) =20 pci_free_irq_vectors(pdev); =20 + bitmap_free(gc->msi_bitmap); + gc->msi_bitmap =3D NULL; gc->max_num_msix =3D 0; gc->num_msix_usable =3D 0; } @@ -2001,6 +2054,10 @@ static int mana_gd_setup(struct pci_dev *pdev) if (err) goto destroy_hwc; =20 + err =3D mana_gd_detect_devices(pdev); + if (err) + goto destroy_hwc; + err =3D mana_gd_query_max_resources(pdev); if (err) goto destroy_hwc; @@ -2011,9 +2068,15 @@ static int mana_gd_setup(struct pci_dev *pdev) goto destroy_hwc; } =20 - err =3D mana_gd_detect_devices(pdev); - if (err) - goto destroy_hwc; + if (!gc->msi_sharing) { + gc->msi_bitmap =3D bitmap_zalloc(gc->num_msix_usable, GFP_KERNEL); + if (!gc->msi_bitmap) { + err =3D -ENOMEM; + goto destroy_hwc; + } + /* Set bit for HWC */ + set_bit(0, gc->msi_bitmap); + } =20 dev_dbg(&pdev->dev, "mana gdma setup successful\n"); return 0; diff --git a/drivers/net/ethernet/microsoft/mana/mana_en.c b/drivers/net/et= hernet/microsoft/mana/mana_en.c index 18f8f653da3d..d23b856f48f6 100644 --- a/drivers/net/ethernet/microsoft/mana/mana_en.c +++ b/drivers/net/ethernet/microsoft/mana/mana_en.c @@ -1007,10 +1007,9 @@ static int mana_init_port_context(struct mana_port_c= ontext *apc) return !apc->rxqs ? -ENOMEM : 0; } =20 -static int mana_send_request(struct mana_context *ac, void *in_buf, - u32 in_len, void *out_buf, u32 out_len) +static int gdma_mana_send_request(struct gdma_context *gc, void *in_buf, + u32 in_len, void *out_buf, u32 out_len) { - struct gdma_context *gc =3D ac->gdma_dev->gdma_context; struct gdma_resp_hdr *resp =3D out_buf; struct gdma_req_hdr *req =3D in_buf; struct device *dev =3D gc->dev; @@ -1044,6 +1043,14 @@ static int mana_send_request(struct mana_context *ac= , void *in_buf, return 0; } =20 +static int mana_send_request(struct mana_context *ac, void *in_buf, + u32 in_len, void *out_buf, u32 out_len) +{ + struct gdma_context *gc =3D ac->gdma_dev->gdma_context; + + return gdma_mana_send_request(gc, in_buf, in_len, out_buf, out_len); +} + static int mana_verify_resp_hdr(const struct gdma_resp_hdr *resp_hdr, const enum mana_command_code expected_code, const u32 min_size) @@ -1177,11 +1184,10 @@ static void mana_pf_deregister_filter(struct mana_p= ort_context *apc) err, resp.hdr.status); } =20 -static int mana_query_device_cfg(struct mana_context *ac, u32 proto_major_= ver, - u32 proto_minor_ver, u32 proto_micro_ver, - u16 *max_num_vports, u8 *bm_hostmode) +int mana_gd_query_device_cfg(struct gdma_context *gc, u32 proto_major_ver, + u32 proto_minor_ver, u32 proto_micro_ver, + u16 *max_num_vports, u8 *bm_hostmode) { - struct gdma_context *gc =3D ac->gdma_dev->gdma_context; struct mana_query_device_cfg_resp resp =3D {}; struct mana_query_device_cfg_req req =3D {}; struct device *dev =3D gc->dev; @@ -1196,7 +1202,8 @@ static int mana_query_device_cfg(struct mana_context = *ac, u32 proto_major_ver, req.proto_minor_ver =3D proto_minor_ver; req.proto_micro_ver =3D proto_micro_ver; =20 - err =3D mana_send_request(ac, &req, sizeof(req), &resp, sizeof(resp)); + err =3D gdma_mana_send_request(gc, &req, sizeof(req), + &resp, sizeof(resp)); if (err) { dev_err(dev, "Failed to query config: %d", err); return err; @@ -1230,8 +1237,6 @@ static int mana_query_device_cfg(struct mana_context = *ac, u32 proto_major_ver, else *bm_hostmode =3D 0; =20 - debugfs_create_u16("adapter-MTU", 0400, gc->mana_pci_debugfs, &gc->adapte= r_mtu); - return 0; } =20 @@ -3417,7 +3422,7 @@ static int mana_probe_port(struct mana_context *ac, i= nt port_idx, int err; =20 ndev =3D alloc_etherdev_mq(sizeof(struct mana_port_context), - gc->max_num_queues); + gc->max_num_queues_vport); if (!ndev) return -ENOMEM; =20 @@ -3426,9 +3431,9 @@ static int mana_probe_port(struct mana_context *ac, i= nt port_idx, apc =3D netdev_priv(ndev); apc->ac =3D ac; apc->ndev =3D ndev; - apc->max_queues =3D gc->max_num_queues; + apc->max_queues =3D gc->max_num_queues_vport; /* Use MANA_DEF_NUM_QUEUES as default, still honoring the HW limit */ - apc->num_queues =3D min(gc->max_num_queues, MANA_DEF_NUM_QUEUES); + apc->num_queues =3D min(gc->max_num_queues_vport, MANA_DEF_NUM_QUEUES); apc->tx_queue_size =3D DEF_TX_BUFFERS_PER_QUEUE; apc->rx_queue_size =3D DEF_RX_BUFFERS_PER_QUEUE; apc->port_handle =3D INVALID_MANA_HANDLE; @@ -3692,13 +3697,18 @@ int mana_probe(struct gdma_dev *gd, bool resuming) =20 INIT_DELAYED_WORK(&ac->gf_stats_work, mana_gf_stats_work_handler); =20 - err =3D mana_query_device_cfg(ac, MANA_MAJOR_VERSION, MANA_MINOR_VERSION, - MANA_MICRO_VERSION, &num_ports, &bm_hostmode); + err =3D mana_gd_query_device_cfg(gc, MANA_MAJOR_VERSION, + MANA_MINOR_VERSION, + MANA_MICRO_VERSION, + &num_ports, &bm_hostmode); if (err) goto out; =20 ac->bm_hostmode =3D bm_hostmode; =20 + debugfs_create_u16("adapter-MTU", 0400, + gc->mana_pci_debugfs, &gc->adapter_mtu); + if (!resuming) { ac->num_ports =3D num_ports; } else { diff --git a/include/net/mana/gdma.h b/include/net/mana/gdma.h index 6d836060976a..9c05b1e15c3e 100644 --- a/include/net/mana/gdma.h +++ b/include/net/mana/gdma.h @@ -399,8 +399,10 @@ struct gdma_context { struct device *dev; struct dentry *mana_pci_debugfs; =20 - /* Per-vPort max number of queues */ + /* Hardware max number of queues */ unsigned int max_num_queues; + /* Per-vPort max number of queues */ + unsigned int max_num_queues_vport; unsigned int max_num_msix; unsigned int num_msix_usable; struct xarray irq_contexts; @@ -446,6 +448,12 @@ struct gdma_context { struct workqueue_struct *service_wq; =20 unsigned long flags; + + /* Indicate if this device is sharing MSI for EQs on MANA */ + bool msi_sharing; + + /* Bitmap tracks where MSI is allocated when it is not shared for EQs */ + unsigned long *msi_bitmap; }; =20 static inline bool mana_gd_is_mana(struct gdma_dev *gd) @@ -1018,4 +1026,7 @@ int mana_gd_resume(struct pci_dev *pdev); =20 bool mana_need_log(struct gdma_context *gc, int err); =20 +int mana_gd_query_device_cfg(struct gdma_context *gc, u32 proto_major_ver, + u32 proto_minor_ver, u32 proto_micro_ver, + u16 *max_num_vports, u8 *bm_hostmode); #endif /* _GDMA_H */ --=20 2.43.0 From nobody Wed Jun 10 09:22:25 2026 Received: from linux.microsoft.com (linux.microsoft.com [13.77.154.182]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 4FED33D1CB0; Fri, 15 May 2026 04:05:20 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=13.77.154.182 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778817922; cv=none; b=JqOe7LumzrdfLsD2Wpj5i8g0QRhNYUab+0bNwbNZp8c3jyx2WIjpksqKi91Pjp4fkTu9EEhzPvHVBdnxjsIxRwR5wdVLwisuKcxtxIowdr4brGUA7XIAV9s8Za5rkzOk0e3k1qC2iUTgP0DCpzv00oj4dzVo3wugTTnLDMQzYf4= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778817922; c=relaxed/simple; bh=HuvgilKnIavRKaqQTYbQFxfCOb5hTV1o89jgvKwMSNQ=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=iZrcPDXwCiwYhMRivKITQCHU7p6LNhW7VYIAdikhv5+6m/eGnqyn61Z73zkQYOavLN2RXrVl08fB0keowWz2NKZuS4rNcRahanYWgON39tlqZpQqecpp0BsMIIrkiw0JB1CHgp171E+KxfUDdBxt+hFc7mLlSLCwk6HB7W1jGVI= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=microsoft.com; spf=pass smtp.mailfrom=linux.microsoft.com; arc=none smtp.client-ip=13.77.154.182 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=microsoft.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linux.microsoft.com Received: by linux.microsoft.com (Postfix, from userid 1202) id 99ADD20B7169; Thu, 14 May 2026 21:05:16 -0700 (PDT) DKIM-Filter: OpenDKIM Filter v2.11.0 linux.microsoft.com 99ADD20B7169 From: Long Li To: Long Li , Konstantin Taranov , Jakub Kicinski , "David S . Miller" , Paolo Abeni , Eric Dumazet , Andrew Lunn , Jason Gunthorpe , Leon Romanovsky , Haiyang Zhang , "K . Y . Srinivasan" , Wei Liu , Dexuan Cui , shradhagupta@linux.microsoft.com Cc: Simon Horman , netdev@vger.kernel.org, linux-rdma@vger.kernel.org, linux-hyperv@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH net-next v10 3/6] net: mana: Introduce GIC context with refcounting for interrupt management Date: Thu, 14 May 2026 21:05:05 -0700 Message-ID: <20260515040508.491748-4-longli@microsoft.com> X-Mailer: git-send-email 2.43.7 In-Reply-To: <20260515040508.491748-1-longli@microsoft.com> References: <20260515040508.491748-1-longli@microsoft.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" To allow Ethernet EQs to use dedicated or shared MSI-X vectors and RDMA EQs to share the same MSI-X, introduce a GIC (GDMA IRQ Context) with reference counting. This allows the driver to create an interrupt context on an assigned or unassigned MSI-X vector and share it across multiple EQ consumers. Signed-off-by: Long Li --- .../net/ethernet/microsoft/mana/gdma_main.c | 159 ++++++++++++++++++ include/net/mana/gdma.h | 12 ++ 2 files changed, 171 insertions(+) diff --git a/drivers/net/ethernet/microsoft/mana/gdma_main.c b/drivers/net/= ethernet/microsoft/mana/gdma_main.c index 13feea93d3c2..ffe293b561ff 100644 --- a/drivers/net/ethernet/microsoft/mana/gdma_main.c +++ b/drivers/net/ethernet/microsoft/mana/gdma_main.c @@ -1658,6 +1658,164 @@ static irqreturn_t mana_gd_intr(int irq, void *arg) return IRQ_HANDLED; } =20 +void mana_gd_put_gic(struct gdma_context *gc, bool use_msi_bitmap, int msi) +{ + struct pci_dev *dev =3D to_pci_dev(gc->dev); + struct gdma_irq_context *gic; + struct msi_map irq_map; + int irq; + + mutex_lock(&gc->gic_mutex); + + gic =3D xa_load(&gc->irq_contexts, msi); + if (WARN_ON(!gic)) { + mutex_unlock(&gc->gic_mutex); + return; + } + + if (use_msi_bitmap) + gic->bitmap_refs--; + + if (use_msi_bitmap && gic->bitmap_refs =3D=3D 0) + clear_bit(msi, gc->msi_bitmap); + + if (!refcount_dec_and_test(&gic->refcount)) + goto out; + + irq =3D gic->irq; + + irq_update_affinity_hint(irq, NULL); + free_irq(irq, gic); + + if (gic->dyn_msix) { + irq_map.virq =3D irq; + irq_map.index =3D msi; + pci_msix_free_irq(dev, irq_map); + } + + xa_erase(&gc->irq_contexts, msi); + kfree(gic); + +out: + mutex_unlock(&gc->gic_mutex); +} +EXPORT_SYMBOL_NS(mana_gd_put_gic, "NET_MANA"); + +/* + * Get a GIC (GDMA IRQ Context) on a MSI vector + * a MSI can be shared between different EQs, this function supports setti= ng + * up separate MSIs using a bitmap, or directly using the MSI index + * + * @use_msi_bitmap: + * True if MSI is assigned by this function on available slots from bitmap. + * False if MSI is passed from *msi_requested + */ +struct gdma_irq_context *mana_gd_get_gic(struct gdma_context *gc, + bool use_msi_bitmap, + int *msi_requested) +{ + struct pci_dev *dev =3D to_pci_dev(gc->dev); + struct gdma_irq_context *gic; + struct msi_map irq_map =3D { }; + int irq; + int msi; + int err; + + mutex_lock(&gc->gic_mutex); + + if (use_msi_bitmap) { + msi =3D find_first_zero_bit(gc->msi_bitmap, gc->num_msix_usable); + if (msi >=3D gc->num_msix_usable) { + dev_err(gc->dev, "No free MSI vectors available\n"); + gic =3D NULL; + goto out; + } + *msi_requested =3D msi; + } else { + msi =3D *msi_requested; + } + + gic =3D xa_load(&gc->irq_contexts, msi); + if (gic) { + refcount_inc(&gic->refcount); + if (use_msi_bitmap) { + gic->bitmap_refs++; + set_bit(msi, gc->msi_bitmap); + } + goto out; + } + + irq =3D pci_irq_vector(dev, msi); + if (irq =3D=3D -EINVAL) { + irq_map =3D pci_msix_alloc_irq_at(dev, msi, NULL); + if (!irq_map.virq) { + err =3D irq_map.index; + dev_err(gc->dev, + "Failed to alloc irq_map msi %d err %d\n", + msi, err); + gic =3D NULL; + goto out; + } + irq =3D irq_map.virq; + msi =3D irq_map.index; + } + + gic =3D kzalloc(sizeof(*gic), GFP_KERNEL); + if (!gic) { + if (irq_map.virq) + pci_msix_free_irq(dev, irq_map); + goto out; + } + + gic->handler =3D mana_gd_process_eq_events; + gic->msi =3D msi; + gic->irq =3D irq; + INIT_LIST_HEAD(&gic->eq_list); + spin_lock_init(&gic->lock); + + if (!gic->msi) + snprintf(gic->name, MANA_IRQ_NAME_SZ, "mana_hwc@pci:%s", + pci_name(dev)); + else + snprintf(gic->name, MANA_IRQ_NAME_SZ, "mana_msi%d@pci:%s", + gic->msi, pci_name(dev)); + + err =3D request_irq(irq, mana_gd_intr, 0, gic->name, gic); + if (err) { + dev_err(gc->dev, "Failed to request irq %d %s\n", + irq, gic->name); + kfree(gic); + gic =3D NULL; + if (irq_map.virq) + pci_msix_free_irq(dev, irq_map); + goto out; + } + + gic->dyn_msix =3D !!irq_map.virq; + refcount_set(&gic->refcount, 1); + gic->bitmap_refs =3D use_msi_bitmap ? 1 : 0; + + err =3D xa_err(xa_store(&gc->irq_contexts, msi, gic, GFP_KERNEL)); + if (err) { + dev_err(gc->dev, "Failed to store irq context for msi %d: %d\n", + msi, err); + free_irq(irq, gic); + kfree(gic); + gic =3D NULL; + if (irq_map.virq) + pci_msix_free_irq(dev, irq_map); + goto out; + } + + if (use_msi_bitmap) + set_bit(msi, gc->msi_bitmap); + +out: + mutex_unlock(&gc->gic_mutex); + return gic; +} +EXPORT_SYMBOL_NS(mana_gd_get_gic, "NET_MANA"); + int mana_gd_alloc_res_map(u32 res_avail, struct gdma_resource *r) { r->map =3D bitmap_zalloc(res_avail, GFP_KERNEL); @@ -2147,6 +2305,7 @@ static int mana_gd_probe(struct pci_dev *pdev, const = struct pci_device_id *ent) goto release_region; =20 mutex_init(&gc->eq_test_event_mutex); + mutex_init(&gc->gic_mutex); pci_set_drvdata(pdev, gc); gc->bar0_pa =3D pci_resource_start(pdev, 0); gc->bar0_size =3D pci_resource_len(pdev, 0); diff --git a/include/net/mana/gdma.h b/include/net/mana/gdma.h index 9c05b1e15c3e..fbe3c1427b45 100644 --- a/include/net/mana/gdma.h +++ b/include/net/mana/gdma.h @@ -388,6 +388,11 @@ struct gdma_irq_context { spinlock_t lock; struct list_head eq_list; char name[MANA_IRQ_NAME_SZ]; + unsigned int msi; + unsigned int irq; + refcount_t refcount; + unsigned int bitmap_refs; + bool dyn_msix; }; =20 enum gdma_context_flags { @@ -449,6 +454,9 @@ struct gdma_context { =20 unsigned long flags; =20 + /* Protect access to GIC context */ + struct mutex gic_mutex; + /* Indicate if this device is sharing MSI for EQs on MANA */ bool msi_sharing; =20 @@ -1026,6 +1034,10 @@ int mana_gd_resume(struct pci_dev *pdev); =20 bool mana_need_log(struct gdma_context *gc, int err); =20 +struct gdma_irq_context *mana_gd_get_gic(struct gdma_context *gc, + bool use_msi_bitmap, + int *msi_requested); +void mana_gd_put_gic(struct gdma_context *gc, bool use_msi_bitmap, int msi= ); int mana_gd_query_device_cfg(struct gdma_context *gc, u32 proto_major_ver, u32 proto_minor_ver, u32 proto_micro_ver, u16 *max_num_vports, u8 *bm_hostmode); --=20 2.43.0 From nobody Wed Jun 10 09:22:25 2026 Received: from linux.microsoft.com (linux.microsoft.com [13.77.154.182]) by smtp.subspace.kernel.org (Postfix) with ESMTP id C4DDC2EB10; Fri, 15 May 2026 04:05:20 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=13.77.154.182 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778817922; cv=none; b=CJwguc/ncqPwQ2ZvQoM69WoJl674NtTws9HRMBfLmCqnzQn/e5FVdO4KaNwq4ajAZTmjMOknhSCFkRqAsoaJusOI9iZ6XlvnDr+xIYSgSUb6O4s3PU2e/sX/f+tKPpo6jEa14plw/8l8iL0nbs+Atvt5VvvCslS/6HJmzfv8700= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778817922; c=relaxed/simple; bh=y4xq+CC7S/OtB9Fe0He393yZzo1C+PMsiCju/ZieNSU=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=h6Ob1IJjpCGunguiJCd8QapKtbVWO+BbWcXpSYLMuWxRVYHHPUmYMzf7EmNK7Mh+H/jO+HO6UV66ctmBd4cPl1e6uQ1sK3bHqahpOG90CZo8ZxzW3i6xq6WonfQPiynk0k4NCuRyyOceakMEQLv3zAVgMPICY92b3K7HlwuP4yA= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=microsoft.com; spf=pass smtp.mailfrom=linux.microsoft.com; arc=none smtp.client-ip=13.77.154.182 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=microsoft.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linux.microsoft.com Received: by linux.microsoft.com (Postfix, from userid 1202) id 21B6420B716A; Thu, 14 May 2026 21:05:17 -0700 (PDT) DKIM-Filter: OpenDKIM Filter v2.11.0 linux.microsoft.com 21B6420B716A From: Long Li To: Long Li , Konstantin Taranov , Jakub Kicinski , "David S . Miller" , Paolo Abeni , Eric Dumazet , Andrew Lunn , Jason Gunthorpe , Leon Romanovsky , Haiyang Zhang , "K . Y . Srinivasan" , Wei Liu , Dexuan Cui , shradhagupta@linux.microsoft.com Cc: Simon Horman , netdev@vger.kernel.org, linux-rdma@vger.kernel.org, linux-hyperv@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH net-next v10 4/6] net: mana: Use GIC functions to allocate global EQs Date: Thu, 14 May 2026 21:05:06 -0700 Message-ID: <20260515040508.491748-5-longli@microsoft.com> X-Mailer: git-send-email 2.43.7 In-Reply-To: <20260515040508.491748-1-longli@microsoft.com> References: <20260515040508.491748-1-longli@microsoft.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Replace the GDMA global interrupt setup code with the new GIC allocation and release functions for managing interrupt contexts. This changes the per-queue interrupt names in /proc/interrupts from mana_q0, mana_q1, ... to mana_msi1, mana_msi2, ... to reflect the MSI-X index rather than a zero-based queue number. The HWC interrupt name (mana_hwc) is unchanged. Signed-off-by: Long Li --- .../net/ethernet/microsoft/mana/gdma_main.c | 96 +++---------------- 1 file changed, 13 insertions(+), 83 deletions(-) diff --git a/drivers/net/ethernet/microsoft/mana/gdma_main.c b/drivers/net/= ethernet/microsoft/mana/gdma_main.c index ffe293b561ff..265cf32a980f 100644 --- a/drivers/net/ethernet/microsoft/mana/gdma_main.c +++ b/drivers/net/ethernet/microsoft/mana/gdma_main.c @@ -1918,7 +1918,7 @@ static int mana_gd_setup_dyn_irqs(struct pci_dev *pde= v, int nvec) struct gdma_context *gc =3D pci_get_drvdata(pdev); struct gdma_irq_context *gic; bool skip_first_cpu =3D false; - int *irqs, irq, err, i; + int *irqs, err, i; =20 irqs =3D kmalloc_objs(int, nvec); if (!irqs) @@ -1931,30 +1931,13 @@ static int mana_gd_setup_dyn_irqs(struct pci_dev *p= dev, int nvec) * further used in irq_setup() */ for (i =3D 1; i <=3D nvec; i++) { - gic =3D kzalloc_obj(*gic); + gic =3D mana_gd_get_gic(gc, false, &i); if (!gic) { err =3D -ENOMEM; goto free_irq; } - gic->handler =3D mana_gd_process_eq_events; - INIT_LIST_HEAD(&gic->eq_list); - spin_lock_init(&gic->lock); - - snprintf(gic->name, MANA_IRQ_NAME_SZ, "mana_q%d@pci:%s", - i - 1, pci_name(pdev)); - - /* one pci vector is already allocated for HWC */ - irqs[i - 1] =3D pci_irq_vector(pdev, i); - if (irqs[i - 1] < 0) { - err =3D irqs[i - 1]; - goto free_current_gic; - } - - err =3D request_irq(irqs[i - 1], mana_gd_intr, 0, gic->name, gic); - if (err) - goto free_current_gic; =20 - xa_store(&gc->irq_contexts, i, gic, GFP_KERNEL); + irqs[i - 1] =3D gic->irq; } =20 /* @@ -1976,20 +1959,9 @@ static int mana_gd_setup_dyn_irqs(struct pci_dev *pd= ev, int nvec) kfree(irqs); return 0; =20 -free_current_gic: - kfree(gic); free_irq: - for (i -=3D 1; i > 0; i--) { - irq =3D pci_irq_vector(pdev, i); - gic =3D xa_load(&gc->irq_contexts, i); - if (WARN_ON(!gic)) - continue; - - irq_update_affinity_hint(irq, NULL); - free_irq(irq, gic); - xa_erase(&gc->irq_contexts, i); - kfree(gic); - } + for (i -=3D 1; i > 0; i--) + mana_gd_put_gic(gc, false, i); kfree(irqs); return err; } @@ -1998,7 +1970,7 @@ static int mana_gd_setup_irqs(struct pci_dev *pdev, i= nt nvec) { struct gdma_context *gc =3D pci_get_drvdata(pdev); struct gdma_irq_context *gic; - int *irqs, *start_irqs, irq; + int *irqs, *start_irqs; unsigned int cpu; int err, i; =20 @@ -2009,34 +1981,13 @@ static int mana_gd_setup_irqs(struct pci_dev *pdev,= int nvec) start_irqs =3D irqs; =20 for (i =3D 0; i < nvec; i++) { - gic =3D kzalloc_obj(*gic); + gic =3D mana_gd_get_gic(gc, false, &i); if (!gic) { err =3D -ENOMEM; goto free_irq; } =20 - gic->handler =3D mana_gd_process_eq_events; - INIT_LIST_HEAD(&gic->eq_list); - spin_lock_init(&gic->lock); - - if (!i) - snprintf(gic->name, MANA_IRQ_NAME_SZ, "mana_hwc@pci:%s", - pci_name(pdev)); - else - snprintf(gic->name, MANA_IRQ_NAME_SZ, "mana_q%d@pci:%s", - i - 1, pci_name(pdev)); - - irqs[i] =3D pci_irq_vector(pdev, i); - if (irqs[i] < 0) { - err =3D irqs[i]; - goto free_current_gic; - } - - err =3D request_irq(irqs[i], mana_gd_intr, 0, gic->name, gic); - if (err) - goto free_current_gic; - - xa_store(&gc->irq_contexts, i, gic, GFP_KERNEL); + irqs[i] =3D gic->irq; } =20 /* If number of IRQ is one extra than number of online CPUs, @@ -2065,20 +2016,9 @@ static int mana_gd_setup_irqs(struct pci_dev *pdev, = int nvec) kfree(start_irqs); return 0; =20 -free_current_gic: - kfree(gic); free_irq: - for (i -=3D 1; i >=3D 0; i--) { - irq =3D pci_irq_vector(pdev, i); - gic =3D xa_load(&gc->irq_contexts, i); - if (WARN_ON(!gic)) - continue; - - irq_update_affinity_hint(irq, NULL); - free_irq(irq, gic); - xa_erase(&gc->irq_contexts, i); - kfree(gic); - } + for (i -=3D 1; i >=3D 0; i--) + mana_gd_put_gic(gc, false, i); =20 kfree(start_irqs); return err; @@ -2152,26 +2092,16 @@ static int mana_gd_setup_remaining_irqs(struct pci_= dev *pdev) static void mana_gd_remove_irqs(struct pci_dev *pdev) { struct gdma_context *gc =3D pci_get_drvdata(pdev); - struct gdma_irq_context *gic; - int irq, i; + int i; =20 if (gc->max_num_msix < 1) return; =20 for (i =3D 0; i < gc->max_num_msix; i++) { - irq =3D pci_irq_vector(pdev, i); - if (irq < 0) - continue; - - gic =3D xa_load(&gc->irq_contexts, i); - if (WARN_ON(!gic)) + if (!xa_load(&gc->irq_contexts, i)) continue; =20 - /* Need to clear the hint before free_irq */ - irq_update_affinity_hint(irq, NULL); - free_irq(irq, gic); - xa_erase(&gc->irq_contexts, i); - kfree(gic); + mana_gd_put_gic(gc, false, i); } =20 pci_free_irq_vectors(pdev); --=20 2.43.0 From nobody Wed Jun 10 09:22:25 2026 Received: from linux.microsoft.com (linux.microsoft.com [13.77.154.182]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 5477B3D1CD0; Fri, 15 May 2026 04:05:21 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=13.77.154.182 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778817923; cv=none; b=tYQGXNsODkikGxJN/R+cSvHKE2Y7masjD6MQ0fkrtPZB/sBzigO+qvDwNu7dCeiZ2oC29h4tIqOIUUTWAPJZxoHbFxAfVBKxkeGduPj2eoBrRuLO78SUx7xy5Lhei9Uj08WAKEa68nWSZWro8/2CN1b2Hbzs13m5E5kZIMJrwGA= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778817923; c=relaxed/simple; bh=nP3mrHN36auxoSmfxMpZZ0zy2j5CyUh05c+rAkpxxrQ=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=igFURtpV+hD6EHKcGVazmsX9a+U9nPH1dTrJtlIMIh1Zv8dVvVBpicI+QU6yz/tHlsD5kCIRkXgkI8el+4DPdEoMdWx5ZN2mqnYvmRwDC13iNjIbQoIYejJ233WKJPyRcEmVdsDzLKArdHJbk3h5A1pKmbT94zLHg+Ls2IW3HdQ= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=microsoft.com; spf=pass smtp.mailfrom=linux.microsoft.com; arc=none smtp.client-ip=13.77.154.182 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=microsoft.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linux.microsoft.com Received: by linux.microsoft.com (Postfix, from userid 1202) id 9BC6720B716B; Thu, 14 May 2026 21:05:17 -0700 (PDT) DKIM-Filter: OpenDKIM Filter v2.11.0 linux.microsoft.com 9BC6720B716B From: Long Li To: Long Li , Konstantin Taranov , Jakub Kicinski , "David S . Miller" , Paolo Abeni , Eric Dumazet , Andrew Lunn , Jason Gunthorpe , Leon Romanovsky , Haiyang Zhang , "K . Y . Srinivasan" , Wei Liu , Dexuan Cui , shradhagupta@linux.microsoft.com Cc: Simon Horman , netdev@vger.kernel.org, linux-rdma@vger.kernel.org, linux-hyperv@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH net-next v10 5/6] net: mana: Allocate interrupt context for each EQ when creating vPort Date: Thu, 14 May 2026 21:05:07 -0700 Message-ID: <20260515040508.491748-6-longli@microsoft.com> X-Mailer: git-send-email 2.43.7 In-Reply-To: <20260515040508.491748-1-longli@microsoft.com> References: <20260515040508.491748-1-longli@microsoft.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Use GIC functions to create a dedicated interrupt context or acquire a shared interrupt context for each EQ when setting up a vPort. The caller now owns the GIC reference across the EQ create/destroy lifecycle: mana_create_eq() calls mana_gd_get_gic() before creating each EQ and mana_destroy_eq() calls mana_gd_put_gic() after destroying it. The msix_index invalidation is moved from mana_gd_deregister_irq() to the mana_gd_create_eq() error path so that mana_destroy_eq() can read the index before teardown. Signed-off-by: Long Li --- .../net/ethernet/microsoft/mana/gdma_main.c | 2 +- drivers/net/ethernet/microsoft/mana/mana_en.c | 18 +++++++++++++++++- include/net/mana/gdma.h | 1 + 3 files changed, 19 insertions(+), 2 deletions(-) diff --git a/drivers/net/ethernet/microsoft/mana/gdma_main.c b/drivers/net/= ethernet/microsoft/mana/gdma_main.c index 265cf32a980f..6ab7e441bd9f 100644 --- a/drivers/net/ethernet/microsoft/mana/gdma_main.c +++ b/drivers/net/ethernet/microsoft/mana/gdma_main.c @@ -897,7 +897,6 @@ static void mana_gd_deregister_irq(struct gdma_queue *q= ueue) } spin_unlock_irqrestore(&gic->lock, flags); =20 - queue->eq.msix_index =3D INVALID_PCI_MSIX_INDEX; synchronize_rcu(); } =20 @@ -1012,6 +1011,7 @@ static int mana_gd_create_eq(struct gdma_dev *gd, out: dev_err(dev, "Failed to create EQ: %d\n", err); mana_gd_destroy_eq(gc, false, queue); + queue->eq.msix_index =3D INVALID_PCI_MSIX_INDEX; return err; } =20 diff --git a/drivers/net/ethernet/microsoft/mana/mana_en.c b/drivers/net/et= hernet/microsoft/mana/mana_en.c index d23b856f48f6..8020089de50f 100644 --- a/drivers/net/ethernet/microsoft/mana/mana_en.c +++ b/drivers/net/ethernet/microsoft/mana/mana_en.c @@ -1626,6 +1626,7 @@ void mana_destroy_eq(struct mana_port_context *apc) struct mana_context *ac =3D apc->ac; struct gdma_context *gc =3D ac->gdma_dev->gdma_context; struct gdma_queue *eq; + unsigned int msi; int i; =20 if (!apc->eqs) @@ -1639,7 +1640,9 @@ void mana_destroy_eq(struct mana_port_context *apc) if (!eq) continue; =20 + msi =3D eq->eq.msix_index; mana_gd_destroy_queue(gc, eq); + mana_gd_put_gic(gc, !gc->msi_sharing, msi); } =20 kfree(apc->eqs); @@ -1656,6 +1659,7 @@ static void mana_create_eq_debugfs(struct mana_port_c= ontext *apc, int i) eq.mana_eq_debugfs =3D debugfs_create_dir(eqnum, apc->mana_eqs_debugfs); debugfs_create_u32("head", 0400, eq.mana_eq_debugfs, &eq.eq->head); debugfs_create_u32("tail", 0400, eq.mana_eq_debugfs, &eq.eq->tail); + debugfs_create_u32("irq", 0400, eq.mana_eq_debugfs, &eq.eq->eq.irq); debugfs_create_file("eq_dump", 0400, eq.mana_eq_debugfs, eq.eq, &mana_dbg= _q_fops); } =20 @@ -1664,7 +1668,9 @@ int mana_create_eq(struct mana_port_context *apc) struct gdma_dev *gd =3D apc->ac->gdma_dev; struct gdma_context *gc =3D gd->gdma_context; struct gdma_queue_spec spec =3D {}; + struct gdma_irq_context *gic; int err; + int msi; int i; =20 if (WARN_ON(apc->eqs)) @@ -1684,12 +1690,22 @@ int mana_create_eq(struct mana_port_context *apc) debugfs_create_dir("EQs", apc->mana_port_debugfs); =20 for (i =3D 0; i < apc->num_queues; i++) { - spec.eq.msix_index =3D (i + 1) % gc->num_msix_usable; + msi =3D (i + 1) % gc->num_msix_usable; + + gic =3D mana_gd_get_gic(gc, !gc->msi_sharing, &msi); + if (!gic) { + err =3D -ENOMEM; + goto out; + } + spec.eq.msix_index =3D msi; + err =3D mana_gd_create_mana_eq(gd, &spec, &apc->eqs[i].eq); if (err) { dev_err(gc->dev, "Failed to create EQ %d : %d\n", i, err); + mana_gd_put_gic(gc, !gc->msi_sharing, msi); goto out; } + apc->eqs[i].eq->eq.irq =3D gic->irq; mana_create_eq_debugfs(apc, i); } =20 diff --git a/include/net/mana/gdma.h b/include/net/mana/gdma.h index fbe3c1427b45..6c138cc77407 100644 --- a/include/net/mana/gdma.h +++ b/include/net/mana/gdma.h @@ -342,6 +342,7 @@ struct gdma_queue { void *context; =20 unsigned int msix_index; + unsigned int irq; =20 u32 log2_throttle_limit; } eq; --=20 2.43.0 From nobody Wed Jun 10 09:22:25 2026 Received: from linux.microsoft.com (linux.microsoft.com [13.77.154.182]) by smtp.subspace.kernel.org (Postfix) with ESMTP id CE20D3C09F4; Fri, 15 May 2026 04:05:21 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=13.77.154.182 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778817923; cv=none; b=KvyeEFenSQnqh0X2Pftku0gIKljVy1Xi3IwDupoxFpZjUoJ0obkvE759h1v5/r6lOqhX6NQLVXtMTY/bbAY5FBKHdM2unXbMNrVO44mJljYUCKvcArd4taCSZGLucXECPAcPFOTPkiG1t/lNRl6P7N96eIQ6Gna6e+lCZOYdctg= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778817923; c=relaxed/simple; bh=ww3iDhHqj2sZVRmFfdi5sIDnONKXle9ygwVgSzDwxfg=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=WwuH3n+1+fn69STOD2EPrq7KTFSeudZ9cznrvnUbmVnbqyXjsW6Vwgm/X4ZZfvkUaKbzlYs5XWiJs1wqrTvkDspVflxI9YObK+WGebMCtKcTmHxSp77vAQm4Dy7rFP7mXFtud++GX8R0aQUl0fkFtVhAmX6+Ye/KfM45YahJ9/I= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=microsoft.com; spf=pass smtp.mailfrom=linux.microsoft.com; arc=none smtp.client-ip=13.77.154.182 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=microsoft.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linux.microsoft.com Received: by linux.microsoft.com (Postfix, from userid 1202) id 2967820B716C; Thu, 14 May 2026 21:05:18 -0700 (PDT) DKIM-Filter: OpenDKIM Filter v2.11.0 linux.microsoft.com 2967820B716C From: Long Li To: Long Li , Konstantin Taranov , Jakub Kicinski , "David S . Miller" , Paolo Abeni , Eric Dumazet , Andrew Lunn , Jason Gunthorpe , Leon Romanovsky , Haiyang Zhang , "K . Y . Srinivasan" , Wei Liu , Dexuan Cui , shradhagupta@linux.microsoft.com Cc: Simon Horman , netdev@vger.kernel.org, linux-rdma@vger.kernel.org, linux-hyperv@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH net-next v10 6/6] RDMA/mana_ib: Allocate interrupt contexts on EQs Date: Thu, 14 May 2026 21:05:08 -0700 Message-ID: <20260515040508.491748-7-longli@microsoft.com> X-Mailer: git-send-email 2.43.7 In-Reply-To: <20260515040508.491748-1-longli@microsoft.com> References: <20260515040508.491748-1-longli@microsoft.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Use the GIC functions to allocate interrupt contexts for RDMA EQs. These interrupt contexts may be shared with Ethernet EQs when MSI-X vectors are limited. The driver now supports allocating dedicated MSI-X for each EQ. Indicate this capability through driver capability bits. The RDMA EQs pass use_msi_bitmap=3Dfalse to share MSI-X vectors with Ethernet, while the capability flag advertises that the driver supports per-vPort EQ separation when hardware has sufficient vectors. Populate eq.irq on all RDMA EQs for consistency with the Ethernet path. Also relocate the GDMA_DRV_CAP_FLAG_1_HW_VPORT_LINK_AWARE define to its numeric BIT(6) position among the other capability flags. Signed-off-by: Long Li --- drivers/infiniband/hw/mana/main.c | 43 +++++++++++++++++++++++++------ include/net/mana/gdma.h | 7 +++-- 2 files changed, 40 insertions(+), 10 deletions(-) diff --git a/drivers/infiniband/hw/mana/main.c b/drivers/infiniband/hw/mana= /main.c index f8a9013f0ca3..df465a9078c1 100644 --- a/drivers/infiniband/hw/mana/main.c +++ b/drivers/infiniband/hw/mana/main.c @@ -764,7 +764,8 @@ int mana_ib_create_eqs(struct mana_ib_dev *mdev) { struct gdma_context *gc =3D mdev_to_gc(mdev); struct gdma_queue_spec spec =3D {}; - int err, i; + struct gdma_irq_context *gic; + int err, i, msi; =20 spec.type =3D GDMA_EQ; spec.monitor_avl_buf =3D false; @@ -772,11 +773,19 @@ int mana_ib_create_eqs(struct mana_ib_dev *mdev) spec.eq.callback =3D mana_ib_event_handler; spec.eq.context =3D mdev; spec.eq.log2_throttle_limit =3D LOG2_EQ_THROTTLE; - spec.eq.msix_index =3D 0; + + msi =3D 0; + gic =3D mana_gd_get_gic(gc, false, &msi); + if (!gic) + return -ENOMEM; + spec.eq.msix_index =3D msi; =20 err =3D mana_gd_create_mana_eq(mdev->gdma_dev, &spec, &mdev->fatal_err_eq= ); - if (err) + if (err) { + mana_gd_put_gic(gc, false, 0); return err; + } + mdev->fatal_err_eq->eq.irq =3D gic->irq; =20 mdev->eqs =3D kzalloc_objs(struct gdma_queue *, mdev->ib_dev.num_comp_vectors); @@ -786,32 +795,50 @@ int mana_ib_create_eqs(struct mana_ib_dev *mdev) } spec.eq.callback =3D NULL; for (i =3D 0; i < mdev->ib_dev.num_comp_vectors; i++) { - spec.eq.msix_index =3D (i + 1) % gc->num_msix_usable; + msi =3D (i + 1) % gc->num_msix_usable; + + gic =3D mana_gd_get_gic(gc, false, &msi); + if (!gic) { + err =3D -ENOMEM; + goto destroy_eqs; + } + spec.eq.msix_index =3D msi; + err =3D mana_gd_create_mana_eq(mdev->gdma_dev, &spec, &mdev->eqs[i]); - if (err) + if (err) { + mana_gd_put_gic(gc, false, msi); goto destroy_eqs; + } + mdev->eqs[i]->eq.irq =3D gic->irq; } =20 return 0; =20 destroy_eqs: - while (i-- > 0) + while (i-- > 0) { mana_gd_destroy_queue(gc, mdev->eqs[i]); + mana_gd_put_gic(gc, false, (i + 1) % gc->num_msix_usable); + } kfree(mdev->eqs); destroy_fatal_eq: mana_gd_destroy_queue(gc, mdev->fatal_err_eq); + mana_gd_put_gic(gc, false, 0); return err; } =20 void mana_ib_destroy_eqs(struct mana_ib_dev *mdev) { struct gdma_context *gc =3D mdev_to_gc(mdev); - int i; + int i, msi; =20 mana_gd_destroy_queue(gc, mdev->fatal_err_eq); + mana_gd_put_gic(gc, false, 0); =20 - for (i =3D 0; i < mdev->ib_dev.num_comp_vectors; i++) + for (i =3D 0; i < mdev->ib_dev.num_comp_vectors; i++) { mana_gd_destroy_queue(gc, mdev->eqs[i]); + msi =3D (i + 1) % gc->num_msix_usable; + mana_gd_put_gic(gc, false, msi); + } =20 kfree(mdev->eqs); } diff --git a/include/net/mana/gdma.h b/include/net/mana/gdma.h index 6c138cc77407..d84e474309a3 100644 --- a/include/net/mana/gdma.h +++ b/include/net/mana/gdma.h @@ -615,6 +615,7 @@ enum { #define GDMA_DRV_CAP_FLAG_1_HWC_TIMEOUT_RECONFIG BIT(3) #define GDMA_DRV_CAP_FLAG_1_GDMA_PAGES_4MB_1GB_2GB BIT(4) #define GDMA_DRV_CAP_FLAG_1_VARIABLE_INDIRECTION_TABLE_SUPPORT BIT(5) +#define GDMA_DRV_CAP_FLAG_1_HW_VPORT_LINK_AWARE BIT(6) =20 /* Driver can handle holes (zeros) in the device list */ #define GDMA_DRV_CAP_FLAG_1_DEV_LIST_HOLES_SUP BIT(11) @@ -631,7 +632,8 @@ enum { /* Driver detects stalled send queues and recovers them */ #define GDMA_DRV_CAP_FLAG_1_HANDLE_STALL_SQ_RECOVERY BIT(18) =20 -#define GDMA_DRV_CAP_FLAG_1_HW_VPORT_LINK_AWARE BIT(6) +/* Driver supports separate EQ/MSIs for each vPort */ +#define GDMA_DRV_CAP_FLAG_1_EQ_MSI_UNSHARE_MULTI_VPORT BIT(19) =20 /* Driver supports linearizing the skb when num_sge exceeds hardware limit= */ #define GDMA_DRV_CAP_FLAG_1_SKB_LINEARIZE BIT(20) @@ -659,7 +661,8 @@ enum { GDMA_DRV_CAP_FLAG_1_SKB_LINEARIZE | \ GDMA_DRV_CAP_FLAG_1_PROBE_RECOVERY | \ GDMA_DRV_CAP_FLAG_1_HANDLE_STALL_SQ_RECOVERY | \ - GDMA_DRV_CAP_FLAG_1_HWC_TIMEOUT_RECOVERY) + GDMA_DRV_CAP_FLAG_1_HWC_TIMEOUT_RECOVERY | \ + GDMA_DRV_CAP_FLAG_1_EQ_MSI_UNSHARE_MULTI_VPORT) =20 #define GDMA_DRV_CAP_FLAGS2 0 =20 --=20 2.43.0