From mboxrd@z Thu Jan 1 00:00:00 1970 From: a.ryabinin@samsung.com (Andrey Ryabinin) Date: Mon, 18 May 2015 10:43:29 +0300 Subject: [PATCH v2 1/5] kasan, x86: move KASAN_SHADOW_OFFSET to the arch Kconfig In-Reply-To: <1431775656.2341.10.camel@x220> References: <1431698344-28054-1-git-send-email-a.ryabinin@samsung.com> <1431698344-28054-2-git-send-email-a.ryabinin@samsung.com> <1431775656.2341.10.camel@x220> Message-ID: <55599821.40409@samsung.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 05/16/2015 02:27 PM, Paul Bolle wrote: > On Fri, 2015-05-15 at 16:59 +0300, Andrey Ryabinin wrote: >> --- a/arch/x86/Kconfig >> +++ b/arch/x86/Kconfig > >> +config KASAN_SHADOW_OFFSET >> + hex >> + default 0xdffffc0000000000 > > This sets CONFIG_KASAN_SHADOW_OFFSET for _all_ X86 configurations, > doesn't it? > Right. >> --- a/lib/Kconfig.kasan >> +++ b/lib/Kconfig.kasan > >> -config KASAN_SHADOW_OFFSET >> - hex >> - default 0xdffffc0000000000 if X86_64 > > While here it used to depend, at least, on HAVE_ARCH_KASAN. But grepping > the tree shows the two places where CONFIG_KASAN_SHADOW_OFFSET is > currently used are guarded by #ifdef CONFIG_KASAN. So perhaps the > default line should actually read > default 0xdffffc0000000000 if KASAN > > after the move. Would that work? > Yes, but I would rather add "depends on KASAN". > > Paul Bolle > > From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pa0-f51.google.com (mail-pa0-f51.google.com [209.85.220.51]) by kanga.kvack.org (Postfix) with ESMTP id C77446B0085 for ; Mon, 18 May 2015 03:43:36 -0400 (EDT) Received: by pacwv17 with SMTP id wv17so140961682pac.2 for ; Mon, 18 May 2015 00:43:36 -0700 (PDT) Received: from mailout1.w1.samsung.com (mailout1.w1.samsung.com. [210.118.77.11]) by mx.google.com with ESMTPS id hx10si14734514pbc.131.2015.05.18.00.43.35 for (version=TLSv1 cipher=RC4-SHA bits=128/128); Mon, 18 May 2015 00:43:35 -0700 (PDT) Received: from eucpsbgm1.samsung.com (unknown [203.254.199.244]) by mailout1.w1.samsung.com (Oracle Communications Messaging Server 7.0.5.31.0 64bit (built May 5 2014)) with ESMTP id <0NOJ007AKC4J4S60@mailout1.w1.samsung.com> for linux-mm@kvack.org; Mon, 18 May 2015 08:43:31 +0100 (BST) Message-id: <55599821.40409@samsung.com> Date: Mon, 18 May 2015 10:43:29 +0300 From: Andrey Ryabinin MIME-version: 1.0 Subject: Re: [PATCH v2 1/5] kasan, x86: move KASAN_SHADOW_OFFSET to the arch Kconfig References: <1431698344-28054-1-git-send-email-a.ryabinin@samsung.com> <1431698344-28054-2-git-send-email-a.ryabinin@samsung.com> <1431775656.2341.10.camel@x220> In-reply-to: <1431775656.2341.10.camel@x220> Content-type: text/plain; charset=utf-8 Content-transfer-encoding: 7bit Sender: owner-linux-mm@kvack.org List-ID: To: Paul Bolle Cc: linux-kernel@vger.kernel.org, Dmitry Vyukov , Alexander Potapenko , David Keitel , Arnd Bergmann , Andrew Morton , Catalin Marinas , Will Deacon , linux-arm-kernel@lists.infradead.org, linux-mm@kvack.org, Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , "maintainer:X86 ARCHITECTURE..." On 05/16/2015 02:27 PM, Paul Bolle wrote: > On Fri, 2015-05-15 at 16:59 +0300, Andrey Ryabinin wrote: >> --- a/arch/x86/Kconfig >> +++ b/arch/x86/Kconfig > >> +config KASAN_SHADOW_OFFSET >> + hex >> + default 0xdffffc0000000000 > > This sets CONFIG_KASAN_SHADOW_OFFSET for _all_ X86 configurations, > doesn't it? > Right. >> --- a/lib/Kconfig.kasan >> +++ b/lib/Kconfig.kasan > >> -config KASAN_SHADOW_OFFSET >> - hex >> - default 0xdffffc0000000000 if X86_64 > > While here it used to depend, at least, on HAVE_ARCH_KASAN. But grepping > the tree shows the two places where CONFIG_KASAN_SHADOW_OFFSET is > currently used are guarded by #ifdef CONFIG_KASAN. So perhaps the > default line should actually read > default 0xdffffc0000000000 if KASAN > > after the move. Would that work? > Yes, but I would rather add "depends on KASAN". > > Paul Bolle > > -- 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 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752474AbbERHnm (ORCPT ); Mon, 18 May 2015 03:43:42 -0400 Received: from mailout1.w1.samsung.com ([210.118.77.11]:24797 "EHLO mailout1.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750895AbbERHnd (ORCPT ); Mon, 18 May 2015 03:43:33 -0400 X-AuditID: cbfec7f4-f79c56d0000012ee-8f-55599823d294 Message-id: <55599821.40409@samsung.com> Date: Mon, 18 May 2015 10:43:29 +0300 From: Andrey Ryabinin User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 MIME-version: 1.0 To: Paul Bolle Cc: linux-kernel@vger.kernel.org, Dmitry Vyukov , Alexander Potapenko , David Keitel , Arnd Bergmann , Andrew Morton , Catalin Marinas , Will Deacon , linux-arm-kernel@lists.infradead.org, linux-mm@kvack.org, Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , "maintainer:X86 ARCHITECTURE..." Subject: Re: [PATCH v2 1/5] kasan, x86: move KASAN_SHADOW_OFFSET to the arch Kconfig References: <1431698344-28054-1-git-send-email-a.ryabinin@samsung.com> <1431698344-28054-2-git-send-email-a.ryabinin@samsung.com> <1431775656.2341.10.camel@x220> In-reply-to: <1431775656.2341.10.camel@x220> Content-type: text/plain; charset=utf-8 Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFlrOIsWRmVeSWpSXmKPExsVy+t/xK7rKMyJDDS5NkLeYs34Nm8XfScfY Ld4v62G02H3pGbPFhIdt7BbtH/cyW0zbKG6x6fE1VovLu+awWdxb85/V4tKBBUwWi26dYbbY vGkqs8XLjydYLH5seMzqwO+xZt4aRo/fvyYxelzu62XyWLCp1GPTqk42j02fJrF7vDt3jt3j xIzfLB6bl9R7vN93lc1j4/YnTB6fN8l5nGj5whrAG8Vlk5Kak1mWWqRvl8CVsffYDOaCHvaK Y6uPsjQwnmHtYuTkkBAwkVh3/QaULSZx4d56ti5GLg4hgaWMEgcvfWaFcL4zSmx8cQXI4eDg FdCQWHRFH6SBRUBVovnyKUYQm01AT+LfrO1sICWiAhES3ScqQcK8AoISPybfYwGxRQSUJVo3 32MHGcksMJFFYvvtTiaQhLBAmMT+z7MZ4RZ/mXAarINTQEei4fYfsKHMAuoSU6bkgoSZBeQl Nq95yzyBUWAWkh2zEKpmIalawMi8ilE0tTS5oDgpPddQrzgxt7g0L10vOT93EyMk+r7sYFx8 zOoQowAHoxIPL0dEZKgQa2JZcWXuIUYJDmYlEd5fzUAh3pTEyqrUovz4otKc1OJDjNIcLEri vHN3vQ8REkhPLEnNTk0tSC2CyTJxcEo1MM6f+lP9KItdYhir2/nJpspNnGU13zv6zn55vv8d i5bjHbnLC9WUn4faWAnN6g79mLjuUpHfzKsi56dovYkOby6bYqWv8Xbjd42VjxdOY2TLD7dz vb3obXkfYxr71cxz+ddnaj00Wye7oWMi7/1r1jv2WK+UF85UOj9/i3bPnK6Qb/sk+61rNiux FGckGmoxFxUnAgDb0elBugIAAA== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 05/16/2015 02:27 PM, Paul Bolle wrote: > On Fri, 2015-05-15 at 16:59 +0300, Andrey Ryabinin wrote: >> --- a/arch/x86/Kconfig >> +++ b/arch/x86/Kconfig > >> +config KASAN_SHADOW_OFFSET >> + hex >> + default 0xdffffc0000000000 > > This sets CONFIG_KASAN_SHADOW_OFFSET for _all_ X86 configurations, > doesn't it? > Right. >> --- a/lib/Kconfig.kasan >> +++ b/lib/Kconfig.kasan > >> -config KASAN_SHADOW_OFFSET >> - hex >> - default 0xdffffc0000000000 if X86_64 > > While here it used to depend, at least, on HAVE_ARCH_KASAN. But grepping > the tree shows the two places where CONFIG_KASAN_SHADOW_OFFSET is > currently used are guarded by #ifdef CONFIG_KASAN. So perhaps the > default line should actually read > default 0xdffffc0000000000 if KASAN > > after the move. Would that work? > Yes, but I would rather add "depends on KASAN". > > Paul Bolle > >