From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Liang, Ma" Subject: Re: [PATCH] event/opdl: fix ICC fails at compile time Date: Thu, 25 Jan 2018 11:03:40 +0000 Message-ID: <20180125110340.GC9466@sivswdev01.ir.intel.com> References: <20180125070350.14735-1-zhiyong.yang@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: dev@dpdk.org, peter.mccarthy@intel.com, ferruh.yigit@intel.com To: Zhiyong Yang Return-path: Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by dpdk.org (Postfix) with ESMTP id D54A31B172 for ; Thu, 25 Jan 2018 12:03:43 +0100 (CET) Content-Disposition: inline In-Reply-To: <20180125070350.14735-1-zhiyong.yang@intel.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" On 25 Jan 15:03, Zhiyong Yang wrote: > ICC reports the issue at compile time as follows. > error #592: variable "i" is used before its value is set > RTE_SET_USED(i); > > The patch is to fix it. GCC and CLANG has been tested as well. > > Fixes: d548ef513cd7 ("event/opdl: add unit tests") > Cc: liang.j.ma@intel.com > Cc: peter.mccarthy@intel.com > Cc: ferruh.yigit@intel.com > > Signed-off-by: Zhiyong Yang > --- > drivers/event/opdl/opdl_test.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/drivers/event/opdl/opdl_test.c b/drivers/event/opdl/opdl_test.c > index 13d4f6b45..39c6cf8cf 100644 > --- a/drivers/event/opdl/opdl_test.c > +++ b/drivers/event/opdl/opdl_test.c > @@ -562,7 +562,6 @@ single_link_w_stats(struct test *t) > uint32_t deq_pkts; > struct rte_mbuf *mbufs[3]; > RTE_SET_USED(mbufs); > - RTE_SET_USED(i); > > /* Create instance with 3 ports */ > if (init(t, 2, tx_port + 1) < 0 || > @@ -702,10 +701,8 @@ single_link(struct test *t) > /* const uint8_t w3_port = 3; */ > const uint8_t tx_port = 2; > int err; > - int i; > struct rte_mbuf *mbufs[3]; > RTE_SET_USED(mbufs); > - RTE_SET_USED(i); > > /* Create instance with 5 ports */ > if (init(t, 2, tx_port+1) < 0 || > -- > 2.13.3 > Many thanks Zhi Yong. Acked-by: Liang Ma