Thu, Aug 11, 2016 at 05:23:00PM CEST, had...@dev.mellanox.co.il wrote: >On Thu, Aug 11, 2016 at 12:58 AM, kbuild test robot <l...@intel.com> wrote: >> Hi Hadar, >> >> [auto build test ERROR on net-next/master] >> >> url: >> https://github.com/0day-ci/linux/commits/Hadar-Hen-Zion/flow_dissector-Get-vlan-info-from-skb-vlan_tci-instead-of-skb-data/20160811-042500 >> config: cris-etrax-100lx_v2_defconfig (attached as .config) >> compiler: cris-linux-gcc (GCC) 4.6.3 >> reproduce: >> wget >> https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross >> -O ~/bin/make.cross >> chmod +x ~/bin/make.cross >> # save the attached .config to linux build tree >> make.cross ARCH=cris >> >> All errors (new ones prefixed by >>): >> >> In function 'flow_keys_hash_length.isra.6', >> inlined from '__flow_hash_from_keys' at >> net/core/flow_dissector.c:599:26, >> inlined from 'flow_hash_from_keys' at net/core/flow_dissector.c:610:2: >>>> net/core/flow_dissector.c:512:2: error: call to '__compiletime_assert_512' >>>> declared with attribute error: BUILD_BUG_ON failed: (sizeof(*flow) - >>>> FLOW_KEYS_HASH_OFFSET) % sizeof(u32) > >[...] > > >I'm working on a fix, will send it soon. > >I'll be happy to get your review and comments on my flow_dissector and >flower patches.
This patch looks good to me. I'll ack the fixed version. Thanks.
