Hi Martin,

I love your patch! Yet something to improve:

[auto build test ERROR on bpf-next/master]

url:    
https://github.com/0day-ci/linux/commits/Martin-KaFai-Lau/bpf-Introduce-bpf-sk-local-storage/20190426-192951
base:   https://git.kernel.org/pub/scm/linux/kernel/git/bpf/bpf-next.git master
config: arm-allmodconfig (attached as .config)
compiler: arm-linux-gnueabi-gcc (Debian 7.2.0-11) 7.2.0
reproduce:
        wget 
https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O 
~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        GCC_VERSION=7.2.0 make.cross ARCH=arm 

If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <l...@intel.com>

All errors (new ones prefixed by >>):

   In file included from include/linux/spinlock_types.h:13:0,
                    from net//core/bpf_sk_storage.c:3:
>> arch/arm/include/asm/spinlock_types.h:13:3: error: unknown type name 'u32'
      u32 slock;
      ^~~
>> arch/arm/include/asm/spinlock_types.h:16:4: error: unknown type name 'u16'
       u16 next;
       ^~~
   arch/arm/include/asm/spinlock_types.h:17:4: error: unknown type name 'u16'
       u16 owner;
       ^~~
   arch/arm/include/asm/spinlock_types.h:29:2: error: unknown type name 'u32'
     u32 lock;
     ^~~

vim +/u32 +13 arch/arm/include/asm/spinlock_types.h

546c2896 arch/arm/include/asm/spinlock_types.h Will Deacon     2012-07-06  10  
fb1c8f93 include/asm-arm/spinlock_types.h      Ingo Molnar     2005-09-10  11  
typedef struct {
546c2896 arch/arm/include/asm/spinlock_types.h Will Deacon     2012-07-06  12   
union {
546c2896 arch/arm/include/asm/spinlock_types.h Will Deacon     2012-07-06 @13   
        u32 slock;
546c2896 arch/arm/include/asm/spinlock_types.h Will Deacon     2012-07-06  14   
        struct __raw_tickets {
546c2896 arch/arm/include/asm/spinlock_types.h Will Deacon     2012-07-06  15  
#ifdef __ARMEB__
546c2896 arch/arm/include/asm/spinlock_types.h Will Deacon     2012-07-06 @16   
                u16 next;
546c2896 arch/arm/include/asm/spinlock_types.h Will Deacon     2012-07-06  17   
                u16 owner;
546c2896 arch/arm/include/asm/spinlock_types.h Will Deacon     2012-07-06  18  
#else
546c2896 arch/arm/include/asm/spinlock_types.h Will Deacon     2012-07-06  19   
                u16 owner;
546c2896 arch/arm/include/asm/spinlock_types.h Will Deacon     2012-07-06  20   
                u16 next;
546c2896 arch/arm/include/asm/spinlock_types.h Will Deacon     2012-07-06  21  
#endif
546c2896 arch/arm/include/asm/spinlock_types.h Will Deacon     2012-07-06  22   
        } tickets;
546c2896 arch/arm/include/asm/spinlock_types.h Will Deacon     2012-07-06  23   
};
445c8951 arch/arm/include/asm/spinlock_types.h Thomas Gleixner 2009-12-02  24  
} arch_spinlock_t;
fb1c8f93 include/asm-arm/spinlock_types.h      Ingo Molnar     2005-09-10  25  

:::::: The code at line 13 was first introduced by commit
:::::: 546c2896a42202dbc7d02f7c6ec9948ac1bf511b ARM: 7446/1: spinlock: use 
ticket algorithm for ARMv6+ locking implementation

:::::: TO: Will Deacon <will.dea...@arm.com>
:::::: CC: Russell King <rmk+ker...@arm.linux.org.uk>

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

Attachment: .config.gz
Description: application/gzip

Reply via email to