Re: [PATCH] arm64: use is_vmalloc_addr

[Miles Chen]

On Thu, 2017-02-09 at 16:37 +0800, kbuild test robot wrote:
> 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)
>                        ^~~~~~~~~~~~~~~
> 

Sorry for that. I'll send patch v2 without the build warning.

> 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