> -----Original Message----- > From: Yigit, Ferruh > Sent: Friday, October 6, 2017 7:25 AM > To: Olivier Matz <olivier.m...@6wind.com>; dev@dpdk.org; Wu, Jingjing > <jingjing...@intel.com>; Xing, Beilei <beilei.x...@intel.com> > Subject: Re: [dpdk-dev] [PATCH 04/10] net/i40e: fix compilation with -Og > > On 9/11/2017 4:13 PM, Olivier Matz wrote: > > The compilation with gcc-6.3.0 and EXTRA_CFLAGS=-Og gives the > > following > > error: > > > > CC i40e_adminq.o > > i40e_adminq.c: In function ‘i40e_clean_arq_element’: > > i40e_adminq.c:1145:56: error: ‘ntu’ may be used uninitialized in > > this function [-Werror=maybe-uninitialized] > > *pending = (ntc > ntu ? hw->aq.arq.count : 0) + (ntu - ntc); > > ~~~~~^~~~~~ > > > > Depending on what is defined, ntu actually be used without being > > initialized. Fix it by initializing it to 0, despite this is probably > > not the proper fix. Moreover, the error is located in base/ directory, > > which contains driver code common to several platforms (not only dpdk). > > This practically seems false positive because driver makefile hard-coded both > defines. > > Also ntu used before this location, not sure warning generated for here. > > Overall I am for fixing the compile warning, but for decision maintainer cc'ed > because this being base code update requires maintainer communicate other > channels to reflect update into base code. > Right, for base code, we are getting them from internal release. In principle, based code would keep unchanged until the base code update.
I will go to report this issue to base code develop team, and then make It be fixed in next base code drop. Is that OK? Thanks Jingjing
