patches for DPDK stable branches
 help / color / mirror / Atom feed
* [dpdk-stable] [PATCH] examples/qos_sched: fix core limit for lower num of lcore
@ 2017-07-17  5:49 Hemant Agrawal
  2017-07-17  8:37 ` [dpdk-stable] [PATCH v2] " Hemant Agrawal
  0 siblings, 1 reply; 6+ messages in thread
From: Hemant Agrawal @ 2017-07-17  5:49 UTC (permalink / raw)
  To: slawomirx.mrozowicz; +Cc: cristian.dumitrescu, dev, stable

APP_MAX_LCORES is hardcoded as 64.
This will cause build err when RTE_MAX_LCORE is less then 64.

"args.c:127:22: error: iteration 8 invokes undefined behavior
	[-Werror=aggressive-loop-optimizations]
   if (cfg->lcore_role[i] == ROLE_RTE)
       ~~~~~~~~~~~~~~~^~~
args.c:126:2: note: within this loop
  for (i = 0; i < APP_MAX_LCORE; i++) {"

Fixes: d52b5e735aa3 ("examples/qos_sched: fix lcore limit")
Cc: stable@dpdk.org

Signed-off-by: Hemant Agrawal <hemant.agrawal@nxp.com>
---
 examples/qos_sched/main.h | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/examples/qos_sched/main.h b/examples/qos_sched/main.h
index c7490c6..33655c4 100644
--- a/examples/qos_sched/main.h
+++ b/examples/qos_sched/main.h
@@ -69,8 +69,12 @@ extern "C" {
 #define BURST_TX_DRAIN_US 100
 
 #ifndef APP_MAX_LCORE
+#define APP_MAX_LCORE RTE_MAX_LCORE
+#endif
+#if (APP_MAX_LCORE > 64)
 #define APP_MAX_LCORE 64
 #endif
+
 #define MAX_DATA_STREAMS (APP_MAX_LCORE/2)
 #define MAX_SCHED_SUBPORTS		8
 #define MAX_SCHED_PIPES		4096
-- 
2.7.4

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2017-07-21 14:38 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-07-17  5:49 [dpdk-stable] [PATCH] examples/qos_sched: fix core limit for lower num of lcore Hemant Agrawal
2017-07-17  8:37 ` [dpdk-stable] [PATCH v2] " Hemant Agrawal
2017-07-19  5:24   ` Thomas Monjalon
2017-07-19  7:58     ` Hemant Agrawal
2017-07-19  8:13   ` [dpdk-stable] [PATCH v3] " Hemant Agrawal
2017-07-21 14:38     ` [dpdk-stable] [dpdk-dev] " Thomas Monjalon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).