From mboxrd@z Thu Jan 1 00:00:00 1970 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934778AbeAKRdI (ORCPT + 1 other); Thu, 11 Jan 2018 12:33:08 -0500 Received: from usa-sjc-mx-foss1.foss.arm.com ([217.140.101.70]:34506 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932703AbeAKRdH (ORCPT ); Thu, 11 Jan 2018 12:33:07 -0500 Date: Thu, 11 Jan 2018 17:33:10 +0000 From: Will Deacon To: Andrey Konovalov Cc: Andrey Ryabinin , Alexander Potapenko , Dmitry Vyukov , Catalin Marinas , Thomas Gleixner , Ingo Molnar , "H . Peter Anvin" , kasan-dev@googlegroups.com, linux-mm@kvack.org, linux-arm-kernel@lists.infradead.org, x86@kernel.org, linux-kernel@vger.kernel.org, Kostya Serebryany Subject: Re: [PATCH 2/2] kasan: clean up KASAN_SHADOW_SCALE_SHIFT usage Message-ID: <20180111173309.GG13216@arm.com> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Return-Path: On Thu, Jan 11, 2018 at 04:29:09PM +0100, Andrey Konovalov wrote: > Right now the fact that KASAN uses a single shadow byte for 8 bytes of > memory is scattered all over the code. > > This change defines KASAN_SHADOW_SCALE_SHIFT early in asm include files > and makes use of this constant where necessary. > > Signed-off-by: Andrey Konovalov > --- > arch/arm64/include/asm/kasan.h | 3 ++- > arch/arm64/include/asm/memory.h | 3 ++- > arch/arm64/mm/kasan_init.c | 3 ++- > arch/x86/include/asm/kasan.h | 8 ++++++-- > include/linux/kasan.h | 2 -- > 5 files changed, 12 insertions(+), 7 deletions(-) For the arm64 parts: Acked-by: Will Deacon Will From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-oi0-f70.google.com (mail-oi0-f70.google.com [209.85.218.70]) by kanga.kvack.org (Postfix) with ESMTP id 9101B6B026E for ; Thu, 11 Jan 2018 12:33:08 -0500 (EST) Received: by mail-oi0-f70.google.com with SMTP id s136so936034oie.0 for ; Thu, 11 Jan 2018 09:33:08 -0800 (PST) Received: from foss.arm.com (foss.arm.com. [217.140.101.70]) by mx.google.com with ESMTP id c5si5276729oif.347.2018.01.11.09.33.07 for ; Thu, 11 Jan 2018 09:33:07 -0800 (PST) Date: Thu, 11 Jan 2018 17:33:10 +0000 From: Will Deacon Subject: Re: [PATCH 2/2] kasan: clean up KASAN_SHADOW_SCALE_SHIFT usage Message-ID: <20180111173309.GG13216@arm.com> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: owner-linux-mm@kvack.org List-ID: To: Andrey Konovalov Cc: Andrey Ryabinin , Alexander Potapenko , Dmitry Vyukov , Catalin Marinas , Thomas Gleixner , Ingo Molnar , "H . Peter Anvin" , kasan-dev@googlegroups.com, linux-mm@kvack.org, linux-arm-kernel@lists.infradead.org, x86@kernel.org, linux-kernel@vger.kernel.org, Kostya Serebryany On Thu, Jan 11, 2018 at 04:29:09PM +0100, Andrey Konovalov wrote: > Right now the fact that KASAN uses a single shadow byte for 8 bytes of > memory is scattered all over the code. > > This change defines KASAN_SHADOW_SCALE_SHIFT early in asm include files > and makes use of this constant where necessary. > > Signed-off-by: Andrey Konovalov > --- > arch/arm64/include/asm/kasan.h | 3 ++- > arch/arm64/include/asm/memory.h | 3 ++- > arch/arm64/mm/kasan_init.c | 3 ++- > arch/x86/include/asm/kasan.h | 8 ++++++-- > include/linux/kasan.h | 2 -- > 5 files changed, 12 insertions(+), 7 deletions(-) For the arm64 parts: Acked-by: Will Deacon Will -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org From mboxrd@z Thu Jan 1 00:00:00 1970 From: will.deacon@arm.com (Will Deacon) Date: Thu, 11 Jan 2018 17:33:10 +0000 Subject: [PATCH 2/2] kasan: clean up KASAN_SHADOW_SCALE_SHIFT usage In-Reply-To: References: Message-ID: <20180111173309.GG13216@arm.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Thu, Jan 11, 2018 at 04:29:09PM +0100, Andrey Konovalov wrote: > Right now the fact that KASAN uses a single shadow byte for 8 bytes of > memory is scattered all over the code. > > This change defines KASAN_SHADOW_SCALE_SHIFT early in asm include files > and makes use of this constant where necessary. > > Signed-off-by: Andrey Konovalov > --- > arch/arm64/include/asm/kasan.h | 3 ++- > arch/arm64/include/asm/memory.h | 3 ++- > arch/arm64/mm/kasan_init.c | 3 ++- > arch/x86/include/asm/kasan.h | 8 ++++++-- > include/linux/kasan.h | 2 -- > 5 files changed, 12 insertions(+), 7 deletions(-) For the arm64 parts: Acked-by: Will Deacon Will