From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v3] examples/qos_sched: fix core limit for lower num of lcore Date: Fri, 21 Jul 2017 17:38:21 +0300 Message-ID: <11060553.V3FFf1BMrP@xps> References: <1500280631-6634-1-git-send-email-hemant.agrawal@nxp.com> <1500451990-31199-1-git-send-email-hemant.agrawal@nxp.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org, slawomirx.mrozowicz@intel.com, cristian.dumitrescu@intel.com, stable@dpdk.org To: Hemant Agrawal Return-path: In-Reply-To: <1500451990-31199-1-git-send-email-hemant.agrawal@nxp.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 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 > --- > v3: fix as per review comment of Thomas Applied, thanks