Skip to content
Permalink
Browse files

app/testpmd: fix latency stats deinit on signal

[ upstream commit 8b36297d15f90265b50fae211157dff573513359 ]

On receiving signal, testpmd showing warning as
"LATENCY_STATS: failed to remove Rx/Tx callback"
because rte_latencystats_uninit is called without
checking if latencystats is enabled or not.

After this fix, rte_latencystats_uninit will be
called only if latencystats is enabled.

Fixes: 62d3216 ("app/testpmd: add latency statistics calculation")

Signed-off-by: Amit Gupta <agupta3@marvell.com>
Acked-by: Jerin Jacob <jerinj@marvell.com>
Tested-by: Jerin Jacob <jerinj@marvell.com>
  • Loading branch information...
Amit Gupta authored and kevintraynor committed Aug 7, 2019
1 parent c11aabb commit 62aa226ee8102543cbd8c6d354924aaf52aacfdb
Showing with 2 additions and 1 deletion.
  1. +2 −1 app/test-pmd/testpmd.c
@@ -3078,7 +3078,8 @@ signal_handler(int signum)
rte_pdump_uninit();
#endif
#ifdef RTE_LIBRTE_LATENCY_STATS
rte_latencystats_uninit();
if (latencystats_enabled != 0)
rte_latencystats_uninit();
#endif
force_quit();
/* Set flag to indicate the force termination. */

0 comments on commit 62aa226

Please sign in to comment.
You can’t perform that action at this time.