netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: longli@linuxonhyperv.com
To: "K. Y. Srinivasan" <kys@microsoft.com>,
	Haiyang Zhang <haiyangz@microsoft.com>,
	Stephen Hemminger <sthemmin@microsoft.com>,
	Wei Liu <wei.liu@kernel.org>, Dexuan Cui <decui@microsoft.com>,
	"David S. Miller" <davem@davemloft.net>,
	Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>,
	Jason Gunthorpe <jgg@ziepe.ca>, Leon Romanovsky <leon@kernel.org>
Cc: linux-hyperv@vger.kernel.org, netdev@vger.kernel.org,
	linux-kernel@vger.kernel.org, linux-rdma@vger.kernel.org,
	Long Li <longli@microsoft.com>
Subject: [PATCH 03/12] net: mana: Handle vport sharing between devices
Date: Tue, 17 May 2022 02:04:27 -0700	[thread overview]
Message-ID: <1652778276-2986-4-git-send-email-longli@linuxonhyperv.com> (raw)
In-Reply-To: <1652778276-2986-1-git-send-email-longli@linuxonhyperv.com>

From: Long Li <longli@microsoft.com>

For outgoing packets, the PF requires the VF to configure the vport with
corresponding protection domain and doorbell ID for the kernel or user
context. The vport can't be shared between different contexts.

Implement the logic to exclusively take over the vport by either the
Ethernet device or RDMA device.

Signed-off-by: Long Li <longli@microsoft.com>
---
 drivers/net/ethernet/microsoft/mana/mana.h    |  4 ++++
 drivers/net/ethernet/microsoft/mana/mana_en.c | 19 +++++++++++++++++--
 2 files changed, 21 insertions(+), 2 deletions(-)

diff --git a/drivers/net/ethernet/microsoft/mana/mana.h b/drivers/net/ethernet/microsoft/mana/mana.h
index 51bff91b63ee..26f14fcb6a61 100644
--- a/drivers/net/ethernet/microsoft/mana/mana.h
+++ b/drivers/net/ethernet/microsoft/mana/mana.h
@@ -375,6 +375,7 @@ struct mana_port_context {
 	unsigned int num_queues;
 
 	mana_handle_t port_handle;
+	atomic_t port_use_count;
 
 	u16 port_idx;
 
@@ -567,4 +568,7 @@ struct mana_adev {
 	struct gdma_dev *mdev;
 };
 
+int mana_cfg_vport(struct mana_port_context *apc, u32 protection_dom_id,
+		   u32 doorbell_pg_id);
+void mana_uncfg_vport(struct mana_port_context *apc);
 #endif /* _MANA_H */
diff --git a/drivers/net/ethernet/microsoft/mana/mana_en.c b/drivers/net/ethernet/microsoft/mana/mana_en.c
index c706bf943e49..4f7a50ace9f6 100644
--- a/drivers/net/ethernet/microsoft/mana/mana_en.c
+++ b/drivers/net/ethernet/microsoft/mana/mana_en.c
@@ -530,13 +530,25 @@ static int mana_query_vport_cfg(struct mana_port_context *apc, u32 vport_index,
 	return 0;
 }
 
-static int mana_cfg_vport(struct mana_port_context *apc, u32 protection_dom_id,
-			  u32 doorbell_pg_id)
+void mana_uncfg_vport(struct mana_port_context *apc)
+{
+	atomic_dec(&apc->port_use_count);
+}
+EXPORT_SYMBOL_GPL(mana_uncfg_vport);
+
+int mana_cfg_vport(struct mana_port_context *apc, u32 protection_dom_id,
+		   u32 doorbell_pg_id)
 {
 	struct mana_config_vport_resp resp = {};
 	struct mana_config_vport_req req = {};
 	int err;
 
+	/* Ethernet driver and IB driver can't take the port at the same time */
+	if (atomic_inc_return(&apc->port_use_count) != 1) {
+		atomic_dec(&apc->port_use_count);
+		return -ENODEV;
+	}
+
 	mana_gd_init_req_hdr(&req.hdr, MANA_CONFIG_VPORT_TX,
 			     sizeof(req), sizeof(resp));
 	req.vport = apc->port_handle;
@@ -566,6 +578,7 @@ static int mana_cfg_vport(struct mana_port_context *apc, u32 protection_dom_id,
 out:
 	return err;
 }
+EXPORT_SYMBOL_GPL(mana_cfg_vport);
 
 static int mana_cfg_vport_steering(struct mana_port_context *apc,
 				   enum TRI_STATE rx,
@@ -1678,6 +1691,8 @@ static void mana_destroy_vport(struct mana_port_context *apc)
 	}
 
 	mana_destroy_txq(apc);
+
+	mana_uncfg_vport(apc);
 }
 
 static int mana_create_vport(struct mana_port_context *apc,
-- 
2.17.1


  parent reply	other threads:[~2022-05-17  9:05 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-05-17  9:04 [PATCH 00/12] Introduce Microsoft Azure Network Adapter (MANA) RDMA driver longli
2022-05-17  9:04 ` [PATCH 01/12] net: mana: Add support for auxiliary device longli
2022-05-19 18:58   ` kernel test robot
2022-05-17  9:04 ` [PATCH 02/12] net: mana: Record the physical address for doorbell page region longli
2022-05-17  9:04 ` longli [this message]
2022-05-17 15:19   ` [PATCH 03/12] net: mana: Handle vport sharing between devices Stephen Hemminger
2022-05-17 23:39     ` Long Li
2022-05-17  9:04 ` [PATCH 04/12] net: mana: Add functions for allocating doorbell page from GDMA longli
2022-05-17  9:04 ` [PATCH 05/12] net: mana: Set the DMA device max page size longli
2022-05-17 14:59   ` Jason Gunthorpe
2022-05-17 19:32     ` Long Li
2022-05-17 19:35       ` Jason Gunthorpe
2022-05-17 20:04         ` Long Li
2022-05-18  0:03           ` Jason Gunthorpe
2022-05-18  5:59             ` [EXTERNAL] " Ajay Sharma
2022-05-18 16:05               ` Jason Gunthorpe
2022-05-18 21:05                 ` Ajay Sharma
2022-05-18 23:11                   ` Jason Gunthorpe
2022-05-19  0:37                     ` Long Li
2022-05-17  9:04 ` [PATCH 06/12] net: mana: Define data structures for protection domain and memory registration longli
2022-05-17  9:04 ` [PATCH 07/12] net: mana: Export Work Queue functions for use by RDMA driver longli
2022-05-17  9:04 ` [PATCH 08/12] net: mana: Record port number in netdev longli
2022-05-17  9:04 ` [PATCH 09/12] net: mana: Move header files to a common location longli
2022-05-17 15:03   ` Jason Gunthorpe
2022-05-17 19:34     ` Long Li
2022-05-17  9:04 ` [PATCH 10/12] net: mana: Define max values for SGL entries longli
2022-05-17  9:04 ` [PATCH 11/12] net: mana: Define and process GDMA response code GDMA_STATUS_MORE_ENTRIES longli
2022-05-17  9:04 ` [PATCH 12/12] RDMA/mana_ib: Add a driver for Microsoft Azure Network Adapter longli
2022-05-17 15:24   ` Jason Gunthorpe
2022-05-19  5:57     ` Long Li
2022-05-19 12:49       ` Jason Gunthorpe
2022-05-19 17:50         ` Long Li
2022-05-17 23:40   ` kernel test robot
2022-05-18  1:14   ` kernel test robot
2022-05-19 21:42   ` kernel test robot

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1652778276-2986-4-git-send-email-longli@linuxonhyperv.com \
    --to=longli@linuxonhyperv.com \
    --cc=davem@davemloft.net \
    --cc=decui@microsoft.com \
    --cc=haiyangz@microsoft.com \
    --cc=jgg@ziepe.ca \
    --cc=kuba@kernel.org \
    --cc=kys@microsoft.com \
    --cc=leon@kernel.org \
    --cc=linux-hyperv@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-rdma@vger.kernel.org \
    --cc=longli@microsoft.com \
    --cc=netdev@vger.kernel.org \
    --cc=pabeni@redhat.com \
    --cc=sthemmin@microsoft.com \
    --cc=wei.liu@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).