patch 'event/sw: remove unused inflight events count' has been queued to stable release 20.11.4

Xueming Li xuemingl at nvidia.com
Sun Nov 28 15:53:25 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/ae0fe7d799603111cbbf558c2d174d7e99ad1924

Thanks.

Xueming Li <xuemingl at nvidia.com>

---
>From ae0fe7d799603111cbbf558c2d174d7e99ad1924 Mon Sep 17 00:00:00 2001
From: Conor Walsh <conor.walsh at intel.com>
Date: Mon, 15 Nov 2021 17:58:53 +0000
Subject: [PATCH] event/sw: remove unused inflight events count
Cc: Xueming Li <xuemingl at nvidia.com>

[ upstream commit 0f66d415c467daa1a41bb5184d72e61980c8991b ]

Reported by clang 13.
This patch removes the inflights variable from the sw_dump function
within the software section of the event driver as it is an unused but
set variable.

Bugzilla ID: 881
Fixes: c66baa68e453 ("event/sw: add dump function for easier debugging")

Reported-by: Liang Longfeng <longfengx.liang at intel.com>
Signed-off-by: Conor Walsh <conor.walsh at intel.com>
Acked-by: Harry van Haaren <harry.van.haaren at intel.com>
Reviewed-by: David Marchand <david.marchand at redhat.com>
---
 drivers/event/sw/sw_evdev.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/drivers/event/sw/sw_evdev.c b/drivers/event/sw/sw_evdev.c
index 0d8013adf7..54c7abbf93 100644
--- a/drivers/event/sw/sw_evdev.c
+++ b/drivers/event/sw/sw_evdev.c
@@ -712,7 +712,6 @@ sw_dump(struct rte_eventdev *dev, FILE *f)
 			continue;
 		}
 		int affinities_per_port[SW_PORTS_MAX] = {0};
-		uint32_t inflights = 0;
 
 		fprintf(f, "  Queue %d (%s)\n", i, q_type_strings[qid->type]);
 		fprintf(f, "\trx   %"PRIu64"\tdrop %"PRIu64"\ttx   %"PRIu64"\n",
@@ -733,7 +732,6 @@ sw_dump(struct rte_eventdev *dev, FILE *f)
 		for (flow = 0; flow < RTE_DIM(qid->fids); flow++)
 			if (qid->fids[flow].cq != -1) {
 				affinities_per_port[qid->fids[flow].cq]++;
-				inflights += qid->fids[flow].pcount;
 			}
 
 		uint32_t port;
-- 
2.34.0

---
  Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- -	2021-11-28 22:41:04.557584695 +0800
+++ 0021-event-sw-remove-unused-inflight-events-count.patch	2021-11-28 22:41:03.230209276 +0800
@@ -1 +1 @@
-From 0f66d415c467daa1a41bb5184d72e61980c8991b Mon Sep 17 00:00:00 2001
+From ae0fe7d799603111cbbf558c2d174d7e99ad1924 Mon Sep 17 00:00:00 2001
@@ -4,0 +5,3 @@
+Cc: Xueming Li <xuemingl at nvidia.com>
+
+[ upstream commit 0f66d415c467daa1a41bb5184d72e61980c8991b ]
@@ -13 +15,0 @@
-Cc: stable at dpdk.org
@@ -24 +26 @@
-index 070a4802e9..fa72ceb949 100644
+index 0d8013adf7..54c7abbf93 100644
@@ -27 +29 @@
-@@ -711,7 +711,6 @@ sw_dump(struct rte_eventdev *dev, FILE *f)
+@@ -712,7 +712,6 @@ sw_dump(struct rte_eventdev *dev, FILE *f)
@@ -35 +37 @@
-@@ -732,7 +731,6 @@ sw_dump(struct rte_eventdev *dev, FILE *f)
+@@ -733,7 +732,6 @@ sw_dump(struct rte_eventdev *dev, FILE *f)


More information about the stable mailing list