patch 'event/cnxk: fix variables casting' has been queued to stable release 21.11.1

Kevin Traynor ktraynor at redhat.com
Tue Mar 1 11:41:39 CET 2022


Hi,

FYI, your patch has been queued to stable release 21.11.1

Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
It will be pushed if I get no objections before 03/06/22. 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

This queued commit can be viewed at:
https://github.com/kevintraynor/dpdk-stable/commit/52d824d106ee34fc52ba603c06b4c69b3a7911a8

Thanks.

Kevin

---
>From 52d824d106ee34fc52ba603c06b4c69b3a7911a8 Mon Sep 17 00:00:00 2001
From: Gowrishankar Muthukrishnan <gmuthukrishn at marvell.com>
Date: Sat, 12 Feb 2022 18:09:46 +0530
Subject: [PATCH] event/cnxk: fix variables casting

[ upstream commit 894851527eb08c153b13e7385d02d138b57a6c43 ]

Fix incompatible casting on variables reported by coverity scan.

Coverity issue: 370582 370584
Fixes: 6223ede2036 ("event/cnxk: add event port link and unlink")

Signed-off-by: Gowrishankar Muthukrishnan <gmuthukrishn at marvell.com>
---
 drivers/event/cnxk/cn10k_eventdev.c |  4 ++--
 drivers/event/cnxk/cn9k_eventdev.c  | 11 ++++-------
 2 files changed, 6 insertions(+), 9 deletions(-)

diff --git a/drivers/event/cnxk/cn10k_eventdev.c b/drivers/event/cnxk/cn10k_eventdev.c
index c5a8c1ae8f..4d878fc2b7 100644
--- a/drivers/event/cnxk/cn10k_eventdev.c
+++ b/drivers/event/cnxk/cn10k_eventdev.c
@@ -112,8 +112,8 @@ cn10k_sso_hws_release(void *arg, void *hws)
 	struct cnxk_sso_evdev *dev = arg;
 	struct cn10k_sso_hws *ws = hws;
-	int i;
+	uint16_t i;
 
 	for (i = 0; i < dev->nb_event_queues; i++)
-		roc_sso_hws_unlink(&dev->sso, ws->hws_id, (uint16_t *)&i, 1);
+		roc_sso_hws_unlink(&dev->sso, ws->hws_id, &i, 1);
 	memset(ws, 0, sizeof(*ws));
 }
diff --git a/drivers/event/cnxk/cn9k_eventdev.c b/drivers/event/cnxk/cn9k_eventdev.c
index b68ce6c0a4..d2ec4aedd7 100644
--- a/drivers/event/cnxk/cn9k_eventdev.c
+++ b/drivers/event/cnxk/cn9k_eventdev.c
@@ -110,5 +110,5 @@ cn9k_sso_hws_release(void *arg, void *hws)
 	struct cn9k_sso_hws_dual *dws;
 	struct cn9k_sso_hws *ws;
-	int i;
+	uint16_t i;
 
 	if (dev->dual_ws) {
@@ -116,9 +116,7 @@ cn9k_sso_hws_release(void *arg, void *hws)
 		for (i = 0; i < dev->nb_event_queues; i++) {
 			roc_sso_hws_unlink(&dev->sso,
-					   CN9K_DUAL_WS_PAIR_ID(dws->hws_id, 0),
-					   (uint16_t *)&i, 1);
+					   CN9K_DUAL_WS_PAIR_ID(dws->hws_id, 0), &i, 1);
 			roc_sso_hws_unlink(&dev->sso,
-					   CN9K_DUAL_WS_PAIR_ID(dws->hws_id, 1),
-					   (uint16_t *)&i, 1);
+					   CN9K_DUAL_WS_PAIR_ID(dws->hws_id, 1), &i, 1);
 		}
 		memset(dws, 0, sizeof(*dws));
@@ -126,6 +124,5 @@ cn9k_sso_hws_release(void *arg, void *hws)
 		ws = hws;
 		for (i = 0; i < dev->nb_event_queues; i++)
-			roc_sso_hws_unlink(&dev->sso, ws->hws_id,
-					   (uint16_t *)&i, 1);
+			roc_sso_hws_unlink(&dev->sso, ws->hws_id, &i, 1);
 		memset(ws, 0, sizeof(*ws));
 	}
-- 
2.34.1

---
  Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- -	2022-03-01 10:41:02.006497226 +0000
+++ 0023-event-cnxk-fix-variables-casting.patch	2022-03-01 10:41:01.272244091 +0000
@@ -1 +1 @@
-From 894851527eb08c153b13e7385d02d138b57a6c43 Mon Sep 17 00:00:00 2001
+From 52d824d106ee34fc52ba603c06b4c69b3a7911a8 Mon Sep 17 00:00:00 2001
@@ -5,0 +6,2 @@
+[ upstream commit 894851527eb08c153b13e7385d02d138b57a6c43 ]
+
@@ -10 +11,0 @@
-Cc: stable at dpdk.org
@@ -19 +20 @@
-index 26d65e3568..249688b6d7 100644
+index c5a8c1ae8f..4d878fc2b7 100644
@@ -22 +23 @@
-@@ -102,8 +102,8 @@ cn10k_sso_hws_release(void *arg, void *hws)
+@@ -112,8 +112,8 @@ cn10k_sso_hws_release(void *arg, void *hws)
@@ -34 +35 @@
-index 6d3d03c97c..ccd98c27fb 100644
+index b68ce6c0a4..d2ec4aedd7 100644
@@ -37 +38 @@
-@@ -99,5 +99,5 @@ cn9k_sso_hws_release(void *arg, void *hws)
+@@ -110,5 +110,5 @@ cn9k_sso_hws_release(void *arg, void *hws)
@@ -44 +45 @@
-@@ -105,9 +105,7 @@ cn9k_sso_hws_release(void *arg, void *hws)
+@@ -116,9 +116,7 @@ cn9k_sso_hws_release(void *arg, void *hws)
@@ -56 +57 @@
-@@ -115,6 +113,5 @@ cn9k_sso_hws_release(void *arg, void *hws)
+@@ -126,6 +124,5 @@ cn9k_sso_hws_release(void *arg, void *hws)



More information about the stable mailing list