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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Tue, 16 Aug 2016 18:31:44 +0100
From:	Catalin Marinas <catalin.marinas@....com>
To:	kbuild test robot <lkp@...el.com>
Cc:	kbuild-all@...org, Andrew Morton <akpm@...ux-foundation.org>,
	linux-kernel@...r.kernel.org, linux-mm@...ck.org,
	Vignesh R <vigneshr@...com>
Subject: Re: [PATCH] mm: kmemleak: Avoid using __va() on addresses that don't
 have a lowmem mapping

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