[dpdk-stable] patch 'crypto/dpaa2_sec: fix stats query without queue pair' has been queued to LTS release 18.11.11
Kevin Traynor
ktraynor at redhat.com
Thu Nov 5 13:40:13 CET 2020
Hi,
FYI, your patch has been queued to LTS release 18.11.11
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/10/20. 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/kevintraynor/dpdk-stable-queue
This queued commit can be viewed at:
https://github.com/kevintraynor/dpdk-stable-queue/commit/303642f06bdd3643a12bd0ffea0c70be21be9c53
Thanks.
Kevin.
---
>From 303642f06bdd3643a12bd0ffea0c70be21be9c53 Mon Sep 17 00:00:00 2001
From: Hemant Agrawal <hemant.agrawal at nxp.com>
Date: Thu, 3 Sep 2020 22:47:44 +0530
Subject: [PATCH] crypto/dpaa2_sec: fix stats query without queue pair
[ upstream commit 356bcac8f11ef916f47be74b295c36755d8165cf ]
dpdk-procinfo calls the crypto stats API, which results segmentation
fault on DPAA2_SEC.
The queue pair array will be NULL, when it is used without
configuring the SEC device.
Fixes: 02f35eee264b ("crypto/dpaa2_sec: support statistics")
Signed-off-by: Hemant Agrawal <hemant.agrawal at nxp.com>
Acked-by: Akhil Goyal <akhil.goyal at nxp.com>
---
drivers/crypto/dpaa2_sec/dpaa2_sec_dpseci.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/crypto/dpaa2_sec/dpaa2_sec_dpseci.c b/drivers/crypto/dpaa2_sec/dpaa2_sec_dpseci.c
index db8a6b0ee3..0db5b4e6d9 100644
--- a/drivers/crypto/dpaa2_sec/dpaa2_sec_dpseci.c
+++ b/drivers/crypto/dpaa2_sec/dpaa2_sec_dpseci.c
@@ -3067,5 +3067,5 @@ void dpaa2_sec_stats_get(struct rte_cryptodev *dev,
}
for (i = 0; i < dev->data->nb_queue_pairs; i++) {
- if (qp[i] == NULL) {
+ if (qp == NULL || qp[i] == NULL) {
DPAA2_SEC_DEBUG("Uninitialised queue pair");
continue;
--
2.26.2
---
Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- - 2020-11-05 12:38:55.801917573 +0000
+++ 0066-crypto-dpaa2_sec-fix-stats-query-without-queue-pair.patch 2020-11-05 12:38:54.278896049 +0000
@@ -1 +1 @@
-From 356bcac8f11ef916f47be74b295c36755d8165cf Mon Sep 17 00:00:00 2001
+From 303642f06bdd3643a12bd0ffea0c70be21be9c53 Mon Sep 17 00:00:00 2001
@@ -5,0 +6,2 @@
+[ upstream commit 356bcac8f11ef916f47be74b295c36755d8165cf ]
+
@@ -12 +13,0 @@
-Cc: stable at dpdk.org
@@ -21 +22 @@
-index 109e61ae16..f480aafd01 100644
+index db8a6b0ee3..0db5b4e6d9 100644
@@ -24 +25 @@
-@@ -3502,5 +3502,5 @@ void dpaa2_sec_stats_get(struct rte_cryptodev *dev,
+@@ -3067,5 +3067,5 @@ void dpaa2_sec_stats_get(struct rte_cryptodev *dev,
More information about the stable
mailing list