[PATCH 21.11] crypto/ipsec_mb: fix allocating mb_mgr in session
Ciara Power
ciara.power at intel.com
Wed May 17 17:24:04 CEST 2023
When configuring the session, use the queue pair mb_mgr that was already
allocated if possible. If this doesn't exist, then allocate a new
mb_mgr.
This reduces unnecessary cycles creating an mb_mgr for every session
configured.
Fixes: c75542a ("crypto/ipsec_mb: introduce IPsec_mb framework")
Signed-off-by: Pablo de Lara <pablo.de.lara.guarch at intel.com>
Signed-off-by: Ciara Power <ciara.power at intel.com>
---
Similar fix sent upstream for 23.07, and 22.11 backport:
https://patches.dpdk.org/project/dpdk/patch/20230517151916.1048475-1-ciara.power@intel.com/
The 21.11 fix needed a few extra lines, so kept separate.
---
drivers/crypto/ipsec_mb/ipsec_mb_ops.c | 20 +++++++++++++++-----
1 file changed, 15 insertions(+), 5 deletions(-)
diff --git a/drivers/crypto/ipsec_mb/ipsec_mb_ops.c b/drivers/crypto/ipsec_mb/ipsec_mb_ops.c
index 58ea4ee476..12f0dd25bd 100644
--- a/drivers/crypto/ipsec_mb/ipsec_mb_ops.c
+++ b/drivers/crypto/ipsec_mb/ipsec_mb_ops.c
@@ -319,21 +319,29 @@ ipsec_mb_sym_session_configure(
struct ipsec_mb_dev_private *internals = dev->data->dev_private;
struct ipsec_mb_internals *pmd_data =
&ipsec_mb_pmds[internals->pmd_type];
- IMB_MGR *mb_mgr = alloc_init_mb_mgr();
+ struct ipsec_mb_qp *qp = dev->data->queue_pairs[0];
+ IMB_MGR *mb_mgr;
int ret = 0;
+ if (qp != NULL)
+ mb_mgr = qp->mb_mgr;
+ else
+ mb_mgr = alloc_init_mb_mgr();
+
if (!mb_mgr)
return -ENOMEM;
if (unlikely(sess == NULL)) {
IPSEC_MB_LOG(ERR, "invalid session struct");
- free_mb_mgr(mb_mgr);
+ if (qp == NULL)
+ free_mb_mgr(mb_mgr);
return -EINVAL;
}
if (rte_mempool_get(mempool, &sess_private_data)) {
IPSEC_MB_LOG(ERR, "Couldn't get object from session mempool");
- free_mb_mgr(mb_mgr);
+ if (qp == NULL)
+ free_mb_mgr(mb_mgr);
return -ENOMEM;
}
@@ -343,13 +351,15 @@ ipsec_mb_sym_session_configure(
/* Return session to mempool */
rte_mempool_put(mempool, sess_private_data);
- free_mb_mgr(mb_mgr);
+ if (qp == NULL)
+ free_mb_mgr(mb_mgr);
return ret;
}
set_sym_session_private_data(sess, dev->driver_id, sess_private_data);
- free_mb_mgr(mb_mgr);
+ if (qp == NULL)
+ free_mb_mgr(mb_mgr);
return 0;
}
--
2.25.1
More information about the stable
mailing list