patch 'common/mlx5: create wrapped MR' has been queued to stable release 20.11.4

Xueming Li xuemingl at nvidia.com
Sun Nov 28 15:53:06 CET 2021


Hi,

FYI, your patch has been queued to stable release 20.11.4

Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
It will be pushed if I get no objections before 11/30/21. So please
shout if anyone has objections.

Also note that after the patch there's a diff of the upstream commit vs the
patch applied to the branch. This will indicate if there was any rebasing
needed to apply to the stable branch. If there were code changes for rebasing
(ie: not only metadata diffs), please double check that the rebase was
correctly done.

Queued patches are on a temporary branch at:
https://github.com/steevenlee/dpdk

This queued commit can be viewed at:
https://github.com/steevenlee/dpdk/commit/6108eff3be195b61ffd38e8467839c21e3968804

Thanks.

Xueming Li <xuemingl at nvidia.com>

---
>From 6108eff3be195b61ffd38e8467839c21e3968804 Mon Sep 17 00:00:00 2001
From: Matan Azrad <matan at nvidia.com>
Date: Tue, 9 Nov 2021 14:36:09 +0200
Subject: [PATCH] common/mlx5: create wrapped MR
Cc: Xueming Li <xuemingl at nvidia.com>

[ upstream commit 76b5bdf828cfcc3c145960c93dbab322ffe6dd79 ]

The mlx5 PMD uses the kernel mlx5 driver to map physical memory to the
HW.

Using the Verbs API ibv_reg_mr, a mkey can be created for that.
In this case, the mkey is signed on the user ID of the kernel driver.

Using the DevX API, a mkey also can be created, but it should point an
umem object (represents the specific buffer mapping) created by the
kernel. In this case, the mkey is signed on the user ID of the process
DevX context.

In FW DevX control commands which get mkey as a parameter, there is
a security check on the user ID and Verbs mkeys are rejected.

Unfortunately, also when using DevX mkey, there is an error in the FW
command on umem validation because the umem is not designed to be used
for any mkey parameters.

As a workaround to the kernel driver/FW issue, it is needed to use a
wrapped MR, which is an indirect mkey(created by the DevX API) pointing to
direct mkey created by the kernel for any DevX command uses an MR.

Add an API to create and destroy this wrapped MR.

Fixes: 5382d28c2110 ("net/mlx5: accelerate DV flow counter transactions")
Fixes: 9d39e57f21ac ("vdpa/mlx5: support live migration")

Signed-off-by: Michael Baum <michaelba at nvidia.com>
Signed-off-by: Matan Azrad <matan at nvidia.com>
---
 drivers/common/mlx5/linux/mlx5_common_os.c | 56 ++++++++++++++++++++++
 drivers/common/mlx5/mlx5_common.h          | 18 +++++++
 drivers/common/mlx5/version.map            |  3 ++
 3 files changed, 77 insertions(+)

diff --git a/drivers/common/mlx5/linux/mlx5_common_os.c b/drivers/common/mlx5/linux/mlx5_common_os.c
index 5cf9576921..96e036fc66 100644
--- a/drivers/common/mlx5/linux/mlx5_common_os.c
+++ b/drivers/common/mlx5/linux/mlx5_common_os.c
@@ -423,3 +423,59 @@ glue_error:
 	mlx5_glue = NULL;
 }
 
+
+/*
+ * Create direct mkey using the kernel ibv_reg_mr API and wrap it with a new
+ * indirect mkey created by the DevX API.
+ * This mkey should be used for DevX commands requesting mkey as a parameter.
+ */
+int
+mlx5_os_wrapped_mkey_create(void *ctx, void *pd, uint32_t pdn, void *addr,
+			    size_t length, struct mlx5_pmd_wrapped_mr *pmd_mr)
+{
+	struct mlx5_klm klm = {
+		.byte_count = length,
+		.address = (uintptr_t)addr,
+	};
+	struct mlx5_devx_mkey_attr mkey_attr = {
+		.pd = pdn,
+		.klm_array = &klm,
+		.klm_num = 1,
+	};
+	struct mlx5_devx_obj *mkey;
+	struct ibv_mr *ibv_mr = mlx5_glue->reg_mr(pd, addr, length,
+						  IBV_ACCESS_LOCAL_WRITE |
+						  (haswell_broadwell_cpu ? 0 :
+						  IBV_ACCESS_RELAXED_ORDERING));
+
+	if (!ibv_mr) {
+		rte_errno = errno;
+		return -rte_errno;
+	}
+	klm.mkey = ibv_mr->lkey;
+	mkey_attr.addr = (uintptr_t)addr;
+	mkey_attr.size = length;
+	mkey = mlx5_devx_cmd_mkey_create(ctx, &mkey_attr);
+	if (!mkey) {
+		claim_zero(mlx5_glue->dereg_mr(ibv_mr));
+		return -rte_errno;
+	}
+	pmd_mr->addr = addr;
+	pmd_mr->len = length;
+	pmd_mr->obj = (void *)ibv_mr;
+	pmd_mr->imkey = mkey;
+	pmd_mr->lkey = mkey->id;
+	return 0;
+}
+
+void
+mlx5_os_wrapped_mkey_destroy(struct mlx5_pmd_wrapped_mr *pmd_mr)
+{
+	if (!pmd_mr)
+		return;
+	if (pmd_mr->imkey)
+		claim_zero(mlx5_devx_cmd_destroy(pmd_mr->imkey));
+	if (pmd_mr->obj)
+		claim_zero(mlx5_glue->dereg_mr(pmd_mr->obj));
+	memset(pmd_mr, 0, sizeof(*pmd_mr));
+}
diff --git a/drivers/common/mlx5/mlx5_common.h b/drivers/common/mlx5/mlx5_common.h
index 4c75addd08..86d690af09 100644
--- a/drivers/common/mlx5/mlx5_common.h
+++ b/drivers/common/mlx5/mlx5_common.h
@@ -270,4 +270,22 @@ extern uint8_t haswell_broadwell_cpu;
 __rte_internal
 void mlx5_common_init(void);
 
+/* mlx5 PMD wrapped MR struct. */
+struct mlx5_pmd_wrapped_mr {
+	uint32_t	     lkey;
+	void		     *addr;
+	size_t		     len;
+	void		     *obj; /* verbs mr object or devx umem object. */
+	void		     *imkey; /* DevX indirect mkey object. */
+};
+
+__rte_internal
+int
+mlx5_os_wrapped_mkey_create(void *ctx, void *pd, uint32_t pdn, void *addr,
+			    size_t length, struct mlx5_pmd_wrapped_mr *pmd_mr);
+
+__rte_internal
+void
+mlx5_os_wrapped_mkey_destroy(struct mlx5_pmd_wrapped_mr *pmd_mr);
+
 #endif /* RTE_PMD_MLX5_COMMON_H_ */
diff --git a/drivers/common/mlx5/version.map b/drivers/common/mlx5/version.map
index 02aedbc431..983714a82c 100644
--- a/drivers/common/mlx5/version.map
+++ b/drivers/common/mlx5/version.map
@@ -89,6 +89,9 @@ INTERNAL {
 	mlx5_nl_vlan_vmwa_create;
 	mlx5_nl_vlan_vmwa_delete;
 
+	mlx5_os_wrapped_mkey_create;
+	mlx5_os_wrapped_mkey_destroy;
+
 	mlx5_release_dbr;
 
 	mlx5_translate_port_name;
-- 
2.34.0

---
  Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- -	2021-11-28 22:41:03.683867000 +0800
+++ 0002-common-mlx5-create-wrapped-MR.patch	2021-11-28 22:41:03.183543455 +0800
@@ -1 +1 @@
-From 76b5bdf828cfcc3c145960c93dbab322ffe6dd79 Mon Sep 17 00:00:00 2001
+From 6108eff3be195b61ffd38e8467839c21e3968804 Mon Sep 17 00:00:00 2001
@@ -4,0 +5,3 @@
+Cc: Xueming Li <xuemingl at nvidia.com>
+
+[ upstream commit 76b5bdf828cfcc3c145960c93dbab322ffe6dd79 ]
@@ -32 +34,0 @@
-Cc: stable at dpdk.org
@@ -37,5 +39,4 @@
- drivers/common/mlx5/linux/mlx5_common_os.c   | 56 ++++++++++++++++++++
- drivers/common/mlx5/mlx5_common.h            | 18 +++++++
- drivers/common/mlx5/version.map              |  3 ++
- drivers/common/mlx5/windows/mlx5_common_os.c | 39 ++++++++++++++
- 4 files changed, 116 insertions(+)
+ drivers/common/mlx5/linux/mlx5_common_os.c | 56 ++++++++++++++++++++++
+ drivers/common/mlx5/mlx5_common.h          | 18 +++++++
+ drivers/common/mlx5/version.map            |  3 ++
+ 3 files changed, 77 insertions(+)
@@ -44 +45 @@
-index b516564b79..0d3e24e04e 100644
+index 5cf9576921..96e036fc66 100644
@@ -47,3 +48,2 @@
-@@ -744,3 +744,59 @@ mlx5_get_device_guid(const struct rte_pci_addr *dev, uint8_t *guid, size_t len)
- 	fclose(id_file);
- 	return ret;
+@@ -423,3 +423,59 @@ glue_error:
+ 	mlx5_glue = NULL;
@@ -50,0 +51 @@
+ 
@@ -108 +109 @@
-index 661d3ab235..e8809844af 100644
+index 4c75addd08..86d690af09 100644
@@ -111,3 +112,3 @@
-@@ -509,4 +509,22 @@ mlx5_devx_uar_release(struct mlx5_uar *uar);
- int mlx5_os_open_device(struct mlx5_common_device *cdev, uint32_t classes);
- int mlx5_os_pd_create(struct mlx5_common_device *cdev);
+@@ -270,4 +270,22 @@ extern uint8_t haswell_broadwell_cpu;
+ __rte_internal
+ void mlx5_common_init(void);
@@ -135 +136 @@
-index 2335edf39d..8a62dc2782 100644
+index 02aedbc431..983714a82c 100644
@@ -138,3 +139,3 @@
-@@ -134,6 +134,9 @@ INTERNAL {
- 	mlx5_os_umem_dereg;
- 	mlx5_os_umem_reg;
+@@ -89,6 +89,9 @@ INTERNAL {
+ 	mlx5_nl_vlan_vmwa_create;
+ 	mlx5_nl_vlan_vmwa_delete;
@@ -145 +146 @@
- 	mlx5_realloc;
+ 	mlx5_release_dbr;
@@ -147,48 +148 @@
- 	mlx5_translate_port_name; # WINDOWS_NO_EXPORT
-diff --git a/drivers/common/mlx5/windows/mlx5_common_os.c b/drivers/common/mlx5/windows/mlx5_common_os.c
-index ea478d7395..5fb45d12ea 100644
---- a/drivers/common/mlx5/windows/mlx5_common_os.c
-+++ b/drivers/common/mlx5/windows/mlx5_common_os.c
-@@ -390,3 +390,42 @@ mlx5_os_set_reg_mr_cb(mlx5_reg_mr_t *reg_mr_cb, mlx5_dereg_mr_t *dereg_mr_cb)
- 	*reg_mr_cb = mlx5_os_reg_mr;
- 	*dereg_mr_cb = mlx5_os_dereg_mr;
- }
-+
-+/*
-+ * In Windows, no need to wrap the MR, no known issue for it in kernel.
-+ * Use the regular function to create direct MR.
-+ */
-+int
-+mlx5_os_wrapped_mkey_create(void *ctx, void *pd, uint32_t pdn, void *addr,
-+			    size_t length, struct mlx5_pmd_wrapped_mr *wpmd_mr)
-+{
-+	struct mlx5_pmd_mr pmd_mr = {0};
-+	int ret = mlx5_os_reg_mr(pd, addr, length, &pmd_mr);
-+
-+	(void)pdn;
-+	(void)ctx;
-+	if (ret != 0)
-+		return -1;
-+	wpmd_mr->addr = addr;
-+	wpmd_mr->len = length;
-+	wpmd_mr->obj = pmd_mr.obj;
-+	wpmd_mr->imkey = pmd_mr.mkey;
-+	wpmd_mr->lkey = pmd_mr.mkey->id;
-+	return 0;
-+}
-+
-+void
-+mlx5_os_wrapped_mkey_destroy(struct mlx5_pmd_wrapped_mr *wpmd_mr)
-+{
-+	struct mlx5_pmd_mr pmd_mr;
-+
-+	if (!wpmd_mr)
-+		return;
-+	pmd_mr.addr = wpmd_mr->addr;
-+	pmd_mr.len = wpmd_mr->len;
-+	pmd_mr.obj = wpmd_mr->obj;
-+	pmd_mr.mkey = wpmd_mr->imkey;
-+	pmd_mr.lkey = wpmd_mr->lkey;
-+	mlx5_os_dereg_mr(&pmd_mr);
-+	memset(wpmd_mr, 0, sizeof(*wpmd_mr));
-+}
+ 	mlx5_translate_port_name;


More information about the stable mailing list