[v2,4/7] app/flow-perf: fix report total stats for masked ports

Message ID 20210310134858.8104-5-wisamm@nvidia.com (mailing list archive)
State New, archived
Series [v2,1/7] app/flow-perf: start using more generic wrapper for cycles |


Context Check Description
ci/checkpatch success coding style OK

Commit Message

Wisam Monther March 10, 2021, 1:48 p.m. UTC
  Take into consideration that the user may call portmask for
any run, thus the app should always check if port is needed
to collect and report or not.

Fixes: 070316d01d3e ("app/flow-perf: add multi-core rule insertion and deletion")
Fixes: d8099d7ecbd0 ("app/flow-perf: split dump functions")
Cc: dongzhou@nvidia.com
Cc: stable@dpdk.org

Signed-off-by: Wisam Jaddo <wisamm@nvidia.com>
 app/test-flow-perf/main.c | 3 +++
 1 file changed, 3 insertions(+)


diff --git a/app/test-flow-perf/main.c b/app/test-flow-perf/main.c
index 01607881df..e32714131c 100644
--- a/app/test-flow-perf/main.c
+++ b/app/test-flow-perf/main.c
@@ -1437,6 +1437,9 @@  run_rte_flow_handler_cores(void *data __rte_unused)
+		/* If port outside portmask */
+		if (!((ports_mask >> port) & 0x1))
+			continue;
 		if (has_meter())
 				port, &mc_pool.meters_record);