test: l2fwd: fix odp_unlikely() conditions

Instead of checking for `odp_unlikely(pkts) == 0`, check for `odp_unlikely(pkts == 0)` in run functions after extra feature processing. Signed-off-by: Tuomas Taipale <[email protected]> Reviewed-by: Janne Peltonen <[email protected]>
MatiasElo · Jul 29, 2024 · d926872 · d926872
1 parent a52858a
commit d926872
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/test/performance/odp_l2fwd.c b/test/performance/odp_l2fwd.c
@@ -773,7 +773,7 @@ static int run_worker_sched_mode_vector(void *arg)
 
 			pkts = process_extra_features(appl_args, pkt_tbl, pkts, stats);
 
-			if (odp_unlikely(pkts) == 0) {
+			if (odp_unlikely(pkts == 0)) {
 				if (pkt_vec != ODP_PACKET_VECTOR_INVALID)
 					odp_packet_vector_free(pkt_vec);
 				continue;
@@ -928,7 +928,7 @@ static int run_worker_sched_mode(void *arg)
 
 		pkts = process_extra_features(appl_args, pkt_tbl, pkts, stats);
 
-		if (odp_unlikely(pkts) == 0)
+		if (odp_unlikely(pkts == 0))
 			continue;
 
 		/* packets from the same queue are from the same interface */
@@ -1043,7 +1043,7 @@ static int run_worker_plain_queue_mode(void *arg)
 
 		pkts = process_extra_features(appl_args, pkt_tbl, pkts, stats);
 
-		if (odp_unlikely(pkts) == 0)
+		if (odp_unlikely(pkts == 0))
 			continue;
 
 		fill_eth_addrs(pkt_tbl, pkts, dst_idx);
@@ -1137,7 +1137,7 @@ static int run_worker_direct_mode(void *arg)
 
 		pkts = process_extra_features(appl_args, pkt_tbl, pkts, stats);
 
-		if (odp_unlikely(pkts) == 0)
+		if (odp_unlikely(pkts == 0))
 			continue;
 
 		fill_eth_addrs(pkt_tbl, pkts, dst_idx);