linux-kernel - Re: [PATCH] xtensa: fix build with KASAN

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] [day] [month] [year] [list]

Date:   Sun, 11 Feb 2018 11:21:24 +0100
From:   Andrey Konovalov <andreyknvl@...gle.com>
To:     Max Filippov <jcmvbkbc@...il.com>
Cc:     linux-xtensa@...ux-xtensa.org, Chris Zankel <chris@...kel.net>,
        Andrey Ryabinin <aryabinin@...tuozzo.com>,
        Dmitry Vyukov <dvyukov@...gle.com>,
        LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] xtensa: fix build with KASAN

On Sun, Feb 11, 2018 at 10:15 AM, Max Filippov <jcmvbkbc@...il.com> wrote:
> The commit 917538e212a2 ("kasan: clean up KASAN_SHADOW_SCALE_SHIFT
> usage") removed KASAN_SHADOW_SCALE_SHIFT definition from
> include/linux/kasan.h and added it to architecture-specific headers,
> except for xtensa. This broke the xtensa build with KASAN enabled.
> Define KASAN_SHADOW_SCALE_SHIFT in arch/xtensa/include/asm/kasan.h
>
> Reported by: kbuild test robot <fengguang.wu@...el.com>
> Fixes: 917538e212a2 ("kasan: clean up KASAN_SHADOW_SCALE_SHIFT usage")
> Signed-off-by: Max Filippov <jcmvbkbc@...il.com>
> ---
>  arch/xtensa/include/asm/kasan.h | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/arch/xtensa/include/asm/kasan.h b/arch/xtensa/include/asm/kasan.h
> index 54be80876e57..216b6f32c375 100644
> --- a/arch/xtensa/include/asm/kasan.h
> +++ b/arch/xtensa/include/asm/kasan.h
> @@ -10,6 +10,8 @@
>  #include <linux/sizes.h>
>  #include <asm/kmem_layout.h>
>
> +#define KASAN_SHADOW_SCALE_SHIFT 3
> +
>  /* Start of area covered by KASAN */
>  #define KASAN_START_VADDR __XTENSA_UL_CONST(0x90000000)
>  /* Start of the shadow map */
> --
> 2.1.4
>

Thanks!

Acked-by: Andrey Konovalov <andreyknvl@...gle.com>