On Mar 9, 2015 8:01 PM, "Mcnamara, John" <john.mcnamara at intel.com> wrote: > > > -----Original Message----- > > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Vladislav Zolotarov > > Sent: Monday, March 9, 2015 5:14 PM > > To: Ananyev, Konstantin > > Cc: dev at dpdk.org > > Subject: Re: [dpdk-dev] [PATCH v5 1/3] ixgbe: Cleanups > > > > On Mar 9, 2015 6:39 PM, "Ananyev, Konstantin" > > <konstantin.ananyev at intel.com> > > wrote: > > > > > If I remember things correctly, it considered result at right side > > > > > of > > '=' operator as unsigned int, > > > > > and then complained that we assign it to smaller size (unsigned > > short) operand. > > > > > > > > If that's the case - that's a clear compiler bug. > > > > > > Might be, though if we still have to support it, there is no much > > > choice > > I am afraid. > > > > Well to begin with could anybody who has this icc thing (preferably the > > latest version) post the compilation errors u are talking about > > Hi, > > For what it is worth there aren't any warnings with ICC 13 and the 1/3 patch (+ the previous patchset): > > $ make T=x86_64-native-linuxapp-icc install CC=icc > Build complete > > $ git log --pretty=format:"%h - %an: %s" -4 > b5d06e4 - Vladislav Zolotarov: ixgbe: Cleanups > 3cd0367 - Vladislav Zolotarov: ixgbe: Unify the rx_pkt_bulk callback ... > 10ed30e - Vladislav Zolotarov: ixgbe: Bug fix: Properly configure Rx ... > 2a5bc6a - Vladislav Zolotarov: ixgbe: Use the rte_le_to_cpu_xx()/rte_... > > $ icc --version > icc (ICC) 13.1.1 20130313
That's worth a lot to me!.. :D > > John. > -- > >
