* [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
* [dpdk-stable] [PATCH v2] examples/qos_sched: fix core limit for lower num of lcore
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 ` Hemant Agrawal
2017-07-19 5:24 ` Thomas Monjalon
2017-07-19 8:13 ` [dpdk-stable] [PATCH v3] " Hemant Agrawal
0 siblings, 2 replies; 6+ messages in thread
From: Hemant Agrawal @ 2017-07-17 8:37 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>
---
v2: fix the compilation error for higher number of cores
examples/qos_sched/main.h | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/examples/qos_sched/main.h b/examples/qos_sched/main.h
index c7490c6..7134e92 100644
--- a/examples/qos_sched/main.h
+++ b/examples/qos_sched/main.h
@@ -69,8 +69,13 @@ extern "C" {
#define BURST_TX_DRAIN_US 100
#ifndef APP_MAX_LCORE
+#if (APP_MAX_LCORE > 64)
#define APP_MAX_LCORE 64
+#else
+#define APP_MAX_LCORE RTE_MAX_LCORE
+#endif
#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
* Re: [dpdk-stable] [PATCH v2] examples/qos_sched: fix core limit for lower num of lcore
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
1 sibling, 1 reply; 6+ messages in thread
From: Thomas Monjalon @ 2017-07-19 5:24 UTC (permalink / raw)
To: Hemant Agrawal; +Cc: stable, slawomirx.mrozowicz, cristian.dumitrescu, dev
17/07/2017 11:37, Hemant Agrawal:
> --- a/examples/qos_sched/main.h
> +++ b/examples/qos_sched/main.h
> @@ -69,8 +69,13 @@ extern "C" {
> #define BURST_TX_DRAIN_US 100
>
> #ifndef APP_MAX_LCORE
> +#if (APP_MAX_LCORE > 64)
> #define APP_MAX_LCORE 64
If APP_MAX_LCORE is not defined, it cannot be > 64, right?
Or what am I missing?
> +#else
> +#define APP_MAX_LCORE RTE_MAX_LCORE
> +#endif
> #endif
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-stable] [PATCH v2] examples/qos_sched: fix core limit for lower num of lcore
2017-07-19 5:24 ` Thomas Monjalon
@ 2017-07-19 7:58 ` Hemant Agrawal
0 siblings, 0 replies; 6+ messages in thread
From: Hemant Agrawal @ 2017-07-19 7:58 UTC (permalink / raw)
To: Thomas Monjalon; +Cc: stable, slawomirx.mrozowicz, cristian.dumitrescu, dev
On 7/19/2017 10:54 AM, Thomas Monjalon wrote:
> 17/07/2017 11:37, Hemant Agrawal:
>> --- a/examples/qos_sched/main.h
>> +++ b/examples/qos_sched/main.h
>> @@ -69,8 +69,13 @@ extern "C" {
>> #define BURST_TX_DRAIN_US 100
>>
>> #ifndef APP_MAX_LCORE
>> +#if (APP_MAX_LCORE > 64)
>> #define APP_MAX_LCORE 64
>
> If APP_MAX_LCORE is not defined, it cannot be > 64, right?
> Or what am I missing?
>
It should be RTE_MAX_LCORE
>> +#else
>> +#define APP_MAX_LCORE RTE_MAX_LCORE
>> +#endif
>> #endif
>
>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* [dpdk-stable] [PATCH v3] examples/qos_sched: fix core limit for lower num of lcore
2017-07-17 8:37 ` [dpdk-stable] [PATCH v2] " Hemant Agrawal
2017-07-19 5:24 ` Thomas Monjalon
@ 2017-07-19 8:13 ` Hemant Agrawal
2017-07-21 14:38 ` [dpdk-stable] [dpdk-dev] " Thomas Monjalon
1 sibling, 1 reply; 6+ messages in thread
From: Hemant Agrawal @ 2017-07-19 8:13 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>
---
v3: fix as per review comment of Thomas
examples/qos_sched/main.h | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/examples/qos_sched/main.h b/examples/qos_sched/main.h
index c7490c6..8d02e1a 100644
--- a/examples/qos_sched/main.h
+++ b/examples/qos_sched/main.h
@@ -69,8 +69,13 @@ extern "C" {
#define BURST_TX_DRAIN_US 100
#ifndef APP_MAX_LCORE
+#if (RTE_MAX_LCORE > 64)
#define APP_MAX_LCORE 64
+#else
+#define APP_MAX_LCORE RTE_MAX_LCORE
+#endif
#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
* Re: [dpdk-stable] [dpdk-dev] [PATCH v3] examples/qos_sched: fix core limit for lower num of lcore
2017-07-19 8:13 ` [dpdk-stable] [PATCH v3] " Hemant Agrawal
@ 2017-07-21 14:38 ` Thomas Monjalon
0 siblings, 0 replies; 6+ messages in thread
From: Thomas Monjalon @ 2017-07-21 14:38 UTC (permalink / raw)
To: Hemant Agrawal; +Cc: dev, slawomirx.mrozowicz, cristian.dumitrescu, stable
19/07/2017 11:13, Hemant Agrawal:
> 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>
> ---
> v3: fix as per review comment of Thomas
Applied, thanks
^ 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).