[dpdk-dev] app/testpmd: fix compilation when bitrate lib is disabled

Message ID 1493794929-105798-1-git-send-email-pablo.de.lara.guarch@intel.com (mailing list archive)
State Accepted, archived
Headers

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK

Commit Message

De Lara Guarch, Pablo May 3, 2017, 7:02 a.m. UTC
  Fixes: e25e6c70fb56 ("app/testpmd: add --bitrate-stats option")

Signed-off-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
---
 app/test-pmd/testpmd.c | 2 ++
 1 file changed, 2 insertions(+)
  

Comments

Remy Horton May 3, 2017, 7:57 a.m. UTC | #1
On 03/05/2017 08:02, Pablo de Lara wrote:
> Fixes: e25e6c70fb56 ("app/testpmd: add --bitrate-stats option")
>
> Signed-off-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
> ---
>  app/test-pmd/testpmd.c | 2 ++
>  1 file changed, 2 insertions(+)

Acked-by: Remy Horton <remy.horton@intel.com>
  
Jingjing Wu May 6, 2017, 1 a.m. UTC | #2
> -----Original Message-----
> From: De Lara Guarch, Pablo
> Sent: Wednesday, May 3, 2017 3:02 PM
> To: Wu, Jingjing <jingjing.wu@intel.com>; Horton, Remy
> <remy.horton@intel.com>
> Cc: dev@dpdk.org; De Lara Guarch, Pablo <pablo.de.lara.guarch@intel.com>
> Subject: [PATCH] app/testpmd: fix compilation when bitrate lib is disabled
> 
> Fixes: e25e6c70fb56 ("app/testpmd: add --bitrate-stats option")
> 
> Signed-off-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
Acked-by: Jingjing Wu <jingjing.wu@intel.com>
  
Thomas Monjalon May 6, 2017, 7:18 a.m. UTC | #3
06/05/2017 03:00, Wu, Jingjing:
> From: De Lara Guarch, Pablo
> > 
> > Fixes: e25e6c70fb56 ("app/testpmd: add --bitrate-stats option")
> > 
> > Signed-off-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
> Acked-by: Jingjing Wu <jingjing.wu@intel.com>

Applied, thanks
  

Patch

diff --git a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c
index dfe6442..776b8dd 100644
--- a/app/test-pmd/testpmd.c
+++ b/app/test-pmd/testpmd.c
@@ -2239,7 +2239,9 @@  main(int argc, char** argv)
 			  "core mask supplied in the command parameters\n");
 
 	/* Bitrate stats disabled by default */
+#ifdef RTE_LIBRTE_BITRATE
 	bitrate_enabled = 0;
+#endif
 
 	argc -= diag;
 	argv += diag;