Re: [PATCH 21/21] usb: remove HAS_IOMEM dependency from USB_SUPPORT

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Vegard,

[auto build test WARNING on usb/usb-testing]
[also build test WARNING on v4.5-rc3 next-20160210]
[if your patch is applied to the wrong git tree, please drop us a note to help improving the system]

url:    https://github.com/0day-ci/linux/commits/Vegard-Nossum/usb-add-HAS_IOMEM-dependency-to-USB_ISP116X_HCD/20160210-223436
base:   https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git usb-testing
config: um-allyesconfig (attached as .config)
reproduce:
        # save the attached .config to linux build tree
        make ARCH=um 

All warnings (new ones prefixed by >>):

   In file included from include/linux/linkage.h:4:0,
                    from include/linux/preempt.h:9,
                    from include/linux/spinlock.h:50,
                    from drivers/staging/rtl8723au/include/osdep_service.h:22,
                    from drivers/staging/rtl8723au/core/rtw_cmd.c:17:
   include/asm-generic/fixmap.h: In function 'fix_to_virt':
>> include/asm-generic/fixmap.h:31:19: warning: comparison of unsigned expression >= 0 is always true [-Wtype-limits]
     BUILD_BUG_ON(idx >= __end_of_fixed_addresses);
                      ^
   include/linux/compiler.h:481:19: note: in definition of macro '__compiletime_assert'
      bool __cond = !(condition);    \
                      ^
   include/linux/compiler.h:501:2: note: in expansion of macro '_compiletime_assert'
     _compiletime_assert(condition, msg, __compiletime_assert_, __LINE__)
     ^
   include/linux/bug.h:50:37: note: in expansion of macro 'compiletime_assert'
    #define BUILD_BUG_ON_MSG(cond, msg) compiletime_assert(!(cond), msg)
                                        ^
   include/linux/bug.h:74:2: note: in expansion of macro 'BUILD_BUG_ON_MSG'
     BUILD_BUG_ON_MSG(condition, "BUILD_BUG_ON failed: " #condition)
     ^
>> include/asm-generic/fixmap.h:31:2: note: in expansion of macro 'BUILD_BUG_ON'
     BUILD_BUG_ON(idx >= __end_of_fixed_addresses);
     ^
   In file included from include/net/checksum.h:25:0,
                    from include/linux/skbuff.h:31,
                    from include/linux/if_ether.h:23,
                    from include/uapi/linux/ethtool.h:17,
                    from include/linux/ethtool.h:16,
                    from include/linux/netdevice.h:42,
                    from drivers/staging/rtl8723au/include/osdep_service.h:30,
                    from drivers/staging/rtl8723au/core/rtw_cmd.c:17:
   arch/um/include/asm/uaccess.h: In function '__access_ok':
>> arch/um/include/asm/uaccess.h:18:29: warning: comparison of unsigned expression >= 0 is always true [-Wtype-limits]
       (((unsigned long) (addr) >= FIXADDR_USER_START) && \
                                ^
>> arch/um/include/asm/uaccess.h:48:3: note: in expansion of macro '__access_ok_vsyscall'
      __access_ok_vsyscall(addr, size) ||
      ^

vim +31 include/asm-generic/fixmap.h

d57c33c5 Mark Salter 2014-01-23  15  #ifndef __ASM_GENERIC_FIXMAP_H
d57c33c5 Mark Salter 2014-01-23  16  #define __ASM_GENERIC_FIXMAP_H
d57c33c5 Mark Salter 2014-01-23  17  
d57c33c5 Mark Salter 2014-01-23  18  #include <linux/bug.h>
d57c33c5 Mark Salter 2014-01-23  19  
d57c33c5 Mark Salter 2014-01-23  20  #define __fix_to_virt(x)	(FIXADDR_TOP - ((x) << PAGE_SHIFT))
d57c33c5 Mark Salter 2014-01-23  21  #define __virt_to_fix(x)	((FIXADDR_TOP - ((x)&PAGE_MASK)) >> PAGE_SHIFT)
d57c33c5 Mark Salter 2014-01-23  22  
d57c33c5 Mark Salter 2014-01-23  23  #ifndef __ASSEMBLY__
d57c33c5 Mark Salter 2014-01-23  24  /*
d57c33c5 Mark Salter 2014-01-23  25   * 'index to address' translation. If anyone tries to use the idx
d57c33c5 Mark Salter 2014-01-23  26   * directly without translation, we catch the bug with a NULL-deference
d57c33c5 Mark Salter 2014-01-23  27   * kernel oops. Illegal ranges of incoming indices are caught too.
d57c33c5 Mark Salter 2014-01-23  28   */
d57c33c5 Mark Salter 2014-01-23  29  static __always_inline unsigned long fix_to_virt(const unsigned int idx)
d57c33c5 Mark Salter 2014-01-23  30  {
d57c33c5 Mark Salter 2014-01-23 @31  	BUILD_BUG_ON(idx >= __end_of_fixed_addresses);
d57c33c5 Mark Salter 2014-01-23  32  	return __fix_to_virt(idx);
d57c33c5 Mark Salter 2014-01-23  33  }
d57c33c5 Mark Salter 2014-01-23  34  
d57c33c5 Mark Salter 2014-01-23  35  static inline unsigned long virt_to_fix(const unsigned long vaddr)
d57c33c5 Mark Salter 2014-01-23  36  {
d57c33c5 Mark Salter 2014-01-23  37  	BUG_ON(vaddr >= FIXADDR_TOP || vaddr < FIXADDR_START);
d57c33c5 Mark Salter 2014-01-23  38  	return __virt_to_fix(vaddr);
d57c33c5 Mark Salter 2014-01-23  39  }

:::::: The code at line 31 was first introduced by commit
:::::: d57c33c5daa4efa9e4d303bd0faf868080b532be add generic fixmap.h

:::::: TO: Mark Salter <msalter@xxxxxxxxxx>
:::::: CC: Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx>

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

Attachment: .config.gz
Description: Binary data


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux