Re: [PATCH] mm: kmemleak: Avoid using __va() on addresses that don't have a lowmem mapping

[Catalin Marinas]

On Wed, Aug 17, 2016 at 01:15:53AM +0800, kbuild test robot wrote:
> [auto build test ERROR on mmotm/master]
> [also build test ERROR on v4.8-rc2 next-20160816]
> [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/Catalin-Marinas/mm-kmemleak-Avoid-using-__va-on-addresses-that-don-t-have-a-lowmem-mapping/20160816-232733
> base:   git://git.cmpxchg.org/linux-mmotm.git master
> config: tile-tilegx_defconfig (attached as .config)
> compiler: tilegx-linux-gcc (GCC) 4.6.2
> 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=tile 
> 
> All error/warnings (new ones prefixed by >>):
> 
>    In file included from include/linux/kmemleak.h:24:0,
>    from include/linux/slab.h:117,
>    from arch/tile/include/asm/pgtable.h:27,
>    from mm/init-mm.c:9:
>    include/linux/mm.h: In function 'is_vmalloc_addr':
>    include/linux/mm.h:486:17: error: 'VMALLOC_START' undeclared (first use in this function)
>    include/linux/mm.h:486:17: note: each undeclared identifier is reported only once for each function it appears in
>    include/linux/mm.h:486:41: error: 'VMALLOC_END' undeclared (first use in this function)
>    include/linux/mm.h: In function 'maybe_mkwrite':
>    include/linux/mm.h:624:3: error: implicit declaration of function 'pte_mkwrite'
>    include/linux/mm.h:624:7: error: incompatible types when assigning to type 'pte_t' from type 'int'
>    In file included from include/linux/kmemleak.h:24:0,
>    from include/linux/slab.h:117,
>    from arch/tile/include/asm/pgtable.h:27,
>    from mm/init-mm.c:9:

It looks like some architectures don't really like including linux/mm.h
from linux/kmemleak.h. I'll change the patch to avoid this include and
explicitly declare high_memory in kmemleak.h

-- 
Catalin