Re: [PATCH 2/3] i40e: Add XDP_TX support

2016-12-08 20:20 GMT+01:00 kbuild test robot [off-list ref]:

Hi Björn,

[auto build test ERROR on next-20161208]
[cannot apply to jkirsher-next-queue/dev-queue v4.9-rc8 v4.9-rc7 v4.9-rc6 v4.9-rc8]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Bj-rn-T-pel/i40e-Support-for-XDP/20161209-013138
config: sparc64-allmodconfig (attached as .config)
compiler: sparc64-linux-gnu-gcc (Debian 6.1.1-9) 6.1.1 20160705
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=sparc64

All error/warnings (new ones prefixed by >>):

   In file included from include/linux/cache.h:4:0,
                    from include/linux/printk.h:8,
                    from include/linux/kernel.h:13,
                    from include/linux/list.h:8,
                    from include/linux/timer.h:4,
                    from include/linux/workqueue.h:8,
                    from include/linux/bpf.h:11,
                    from drivers/net/ethernet/intel/i40e/i40e_txrx.c:27:
   drivers/net/ethernet/intel/i40e/i40e_txrx.c: In function 'i40e_try_flip_rx_page':

quoted

quoted

drivers/net/ethernet/intel/i40e/i40e_txrx.c:1613:32: error: 'size' undeclared (first use in this function)

     unsigned int truesize = ALIGN(size, L1_CACHE_BYTES);
                                   ^
   include/uapi/linux/kernel.h:10:41: note: in definition of macro '__ALIGN_KERNEL_MASK'
    #define __ALIGN_KERNEL_MASK(x, mask) (((x) + (mask)) & ~(mask))
                                            ^
   include/linux/kernel.h:49:22: note: in expansion of macro '__ALIGN_KERNEL'
    #define ALIGN(x, a)  __ALIGN_KERNEL((x), (a))
                         ^~~~~~~~~~~~~~

quoted

quoted

drivers/net/ethernet/intel/i40e/i40e_txrx.c:1613:26: note: in expansion of macro 'ALIGN'

     unsigned int truesize = ALIGN(size, L1_CACHE_BYTES);
                             ^~~~~
   drivers/net/ethernet/intel/i40e/i40e_txrx.c:1613:32: note: each undeclared identifier is reported only once for each function it appears in
     unsigned int truesize = ALIGN(size, L1_CACHE_BYTES);
                                   ^
   include/uapi/linux/kernel.h:10:41: note: in definition of macro '__ALIGN_KERNEL_MASK'
    #define __ALIGN_KERNEL_MASK(x, mask) (((x) + (mask)) & ~(mask))
                                            ^
   include/linux/kernel.h:49:22: note: in expansion of macro '__ALIGN_KERNEL'
    #define ALIGN(x, a)  __ALIGN_KERNEL((x), (a))
                         ^~~~~~~~~~~~~~

quoted

quoted

drivers/net/ethernet/intel/i40e/i40e_txrx.c:1613:26: note: in expansion of macro 'ALIGN'

     unsigned int truesize = ALIGN(size, L1_CACHE_BYTES);
                             ^~~~~

vim +/size +1613 drivers/net/ethernet/intel/i40e/i40e_txrx.c

  1607   */
  1608  static bool i40e_try_flip_rx_page(struct i40e_rx_buffer *rx_buffer)
  1609  {
  1610  #if (PAGE_SIZE < 8192)
  1611          unsigned int truesize = I40E_RXBUFFER_2048;
  1612  #else

quoted

1613          unsigned int truesize = ALIGN(size, L1_CACHE_BYTES);

  1614          unsigned int last_offset = PAGE_SIZE - I40E_RXBUFFER_2048;
  1615  #endif
  1616

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Ick. I'll respin this.


Björn