Re: [PATCH] arm64: use is_vmalloc_addr

[kbuild test robot]

Hi Miles,

[auto build test WARNING on arm64/for-next/core]
[also build test WARNING on v4.10-rc7 next-20170208]
[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/Miles-Chen/arm64-use-is_vmalloc_addr/20170209-103100
base:   https://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git 
for-next/core
config: arm64-defconfig (attached as .config)
compiler: aarch64-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=arm64 

All warnings (new ones prefixed by >>):

   arch/arm64/mm/ioremap.c: In function '__iounmap':
>> arch/arm64/mm/ioremap.c:91:22: warning: passing argument 1 of 
>> 'is_vmalloc_addr' makes pointer from integer without a cast 
>> [-Wint-conversion]
     if (is_vmalloc_addr(addr))
                         ^~~~
   In file included from arch/arm64/mm/ioremap.c:24:0:
   include/linux/mm.h:472:20: note: expected 'const void *' but argument is of 
type 'long unsigned int'
    static inline bool is_vmalloc_addr(const void *x)
                       ^~~~~~~~~~~~~~~

vim +/is_vmalloc_addr +91 arch/arm64/mm/ioremap.c

    75  
    76  void __iomem *__ioremap(phys_addr_t phys_addr, size_t size, pgprot_t 
prot)
    77  {
    78          return __ioremap_caller(phys_addr, size, prot,
    79                                  __builtin_return_address(0));
    80  }
    81  EXPORT_SYMBOL(__ioremap);
    82  
    83  void __iounmap(volatile void __iomem *io_addr)
    84  {
    85          unsigned long addr = (unsigned long)io_addr & PAGE_MASK;
    86  
    87          /*
    88           * We could get an address outside vmalloc range in case
    89           * of ioremap_cache() reusing a RAM mapping.
    90           */
  > 91          if (is_vmalloc_addr(addr))
    92                  vunmap((void *)addr);
    93  }
    94  EXPORT_SYMBOL(__iounmap);
    95  
    96  void __iomem *ioremap_cache(phys_addr_t phys_addr, size_t size)
    97  {
    98          /* For normal memory we already have a cacheable mapping. */
    99          if (pfn_valid(__phys_to_pfn(phys_addr)))

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

.config.gz
Description: application/gzip