Skip to content

Commit

Permalink
test/eal: fix lcore check
Browse files Browse the repository at this point in the history
[ upstream commit 357f915 ]

The expression for checking which lcore is enabled for 0-7
was wrong (missing case for 6).

Link: https://pvs-studio.com/en/blog/posts/cpp/1179/
Fixes: b020903 ("test/eal: check number of cores before running subtests")

Signed-off-by: Stephen Hemminger <[email protected]>
Acked-by: Bruce Richardson <[email protected]>
Acked-by: Aaron Conole <[email protected]>
  • Loading branch information
shemminger authored and steevenlee committed Dec 6, 2024
1 parent d3a7104 commit e5eb4ae
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions app/test/test_eal_flags.c
Original file line number Diff line number Diff line change
Expand Up @@ -677,8 +677,8 @@ test_missing_c_flag(void)

if (rte_lcore_is_enabled(0) && rte_lcore_is_enabled(1) &&
rte_lcore_is_enabled(2) && rte_lcore_is_enabled(3) &&
rte_lcore_is_enabled(3) && rte_lcore_is_enabled(5) &&
rte_lcore_is_enabled(4) && rte_lcore_is_enabled(7) &&
rte_lcore_is_enabled(4) && rte_lcore_is_enabled(5) &&
rte_lcore_is_enabled(6) && rte_lcore_is_enabled(7) &&
launch_proc(argv29) != 0) {
printf("Error - "
"process did not run ok with valid corelist value\n");
Expand Down

0 comments on commit e5eb4ae

Please sign in to comment.