linux-kernel - Re: [PATCH V2] mm: Define default value for FIRST_USER

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:   Wed, 5 May 2021 12:42:17 +0530
From:   Anshuman Khandual <anshuman.khandual@....com>
To:     Stafford Horne <shorne@...il.com>
Cc:     linux-mm@...ck.org, akpm@...ux-foundation.org,
        Richard Henderson <rth@...ddle.net>,
        Vineet Gupta <vgupta@...opsys.com>,
        Catalin Marinas <catalin.marinas@....com>,
        Will Deacon <will@...nel.org>, Guo Ren <guoren@...nel.org>,
        Brian Cain <bcain@...eaurora.org>,
        Geert Uytterhoeven <geert@...ux-m68k.org>,
        Michal Simek <monstr@...str.eu>,
        Thomas Bogendoerfer <tsbogend@...ha.franken.de>,
        Ley Foon Tan <ley.foon.tan@...el.com>,
        Jonas Bonn <jonas@...thpole.se>,
        Stefan Kristiansson <stefan.kristiansson@...nalahti.fi>,
        "James E.J. Bottomley" <James.Bottomley@...senpartnership.com>,
        Michael Ellerman <mpe@...erman.id.au>,
        Christophe Leroy <christophe.leroy@...roup.eu>,
        Paul Walmsley <paul.walmsley@...ive.com>,
        Palmer Dabbelt <palmer@...belt.com>,
        Heiko Carstens <hca@...ux.ibm.com>,
        Yoshinori Sato <ysato@...rs.sourceforge.jp>,
        "David S. Miller" <davem@...emloft.net>,
        Jeff Dike <jdike@...toit.com>,
        Thomas Gleixner <tglx@...utronix.de>,
        Chris Zankel <chris@...kel.net>, linux-arch@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH V2] mm: Define default value for FIRST_USER_ADDRESS



On 5/5/21 11:57 AM, Stafford Horne wrote:
> On Fri, Apr 16, 2021 at 10:47:34AM +0530, Anshuman Khandual wrote:
>> Currently most platforms define FIRST_USER_ADDRESS as 0UL duplication the
>> same code all over. Instead just define a generic default value (i.e 0UL)
>> for FIRST_USER_ADDRESS and let the platforms override when required. This
>> makes it much cleaner with reduced code.
>>
>> The default FIRST_USER_ADDRESS here would be skipped in <linux/pgtable.h>
>> when the given platform overrides its value via <asm/pgtable.h>.
>>
>> Cc: Richard Henderson <rth@...ddle.net>
>> Cc: Vineet Gupta <vgupta@...opsys.com>
>> Cc: Catalin Marinas <catalin.marinas@....com>
>> Cc: Will Deacon <will@...nel.org>
>> Cc: Guo Ren <guoren@...nel.org>
>> Cc: Brian Cain <bcain@...eaurora.org>
>> Cc: Geert Uytterhoeven <geert@...ux-m68k.org>
>> Cc: Michal Simek <monstr@...str.eu>
>> Cc: Thomas Bogendoerfer <tsbogend@...ha.franken.de>
>> Cc: Ley Foon Tan <ley.foon.tan@...el.com>
>> Cc: Jonas Bonn <jonas@...thpole.se>
>> Cc: Stefan Kristiansson <stefan.kristiansson@...nalahti.fi>
>> Cc: Stafford Horne <shorne@...il.com>
>> Cc: "James E.J. Bottomley" <James.Bottomley@...senPartnership.com>
>> Cc: Michael Ellerman <mpe@...erman.id.au>
>> Cc: Christophe Leroy <christophe.leroy@...roup.eu>
>> Cc: Paul Walmsley <paul.walmsley@...ive.com>
>> Cc: Palmer Dabbelt <palmer@...belt.com>
>> Cc: Heiko Carstens <hca@...ux.ibm.com>
>> Cc: Yoshinori Sato <ysato@...rs.sourceforge.jp>
>> Cc: "David S. Miller" <davem@...emloft.net>
>> Cc: Jeff Dike <jdike@...toit.com>
>> Cc: Thomas Gleixner <tglx@...utronix.de>
>> Cc: Chris Zankel <chris@...kel.net>
>> Cc: Andrew Morton <akpm@...ux-foundation.org>
>> Cc: linux-arch@...r.kernel.org
>> Cc: linux-mm@...ck.org
>> Cc: linux-kernel@...r.kernel.org
>> Signed-off-by: Anshuman Khandual <anshuman.khandual@....com>
>> ---
>> This applies on v5.12-rc7 and has been boot tested on arm64 platform.
>> But has been cross compiled on multiple other platforms.
>>
>> Changes in V2:
>>
>> - Dropped ARCH_HAS_FIRST_USER_ADDRESS construct
>>
>> Changes in V1:
>>
>> https://patchwork.kernel.org/project/linux-mm/patch/1618368899-20311-1-git-send-email-anshuman.khandual@arm.com/
>>
>>  arch/alpha/include/asm/pgtable.h             | 1 -
>>  arch/arc/include/asm/pgtable.h               | 6 ------
>>  arch/arm64/include/asm/pgtable.h             | 2 --
>>  arch/csky/include/asm/pgtable.h              | 1 -
>>  arch/hexagon/include/asm/pgtable.h           | 3 ---
>>  arch/ia64/include/asm/pgtable.h              | 1 -
>>  arch/m68k/include/asm/pgtable_mm.h           | 1 -
>>  arch/microblaze/include/asm/pgtable.h        | 2 --
>>  arch/mips/include/asm/pgtable-32.h           | 1 -
>>  arch/mips/include/asm/pgtable-64.h           | 1 -
>>  arch/nios2/include/asm/pgtable.h             | 2 --
>>  arch/openrisc/include/asm/pgtable.h          | 1 -
> 
> Acked-by: Stafford Horne <shorne@...il.com>
> 
>>  arch/parisc/include/asm/pgtable.h            | 2 --
>>  arch/powerpc/include/asm/book3s/pgtable.h    | 1 -
>>  arch/powerpc/include/asm/nohash/32/pgtable.h | 1 -
>>  arch/powerpc/include/asm/nohash/64/pgtable.h | 2 --
>>  arch/riscv/include/asm/pgtable.h             | 2 --
>>  arch/s390/include/asm/pgtable.h              | 2 --
>>  arch/sh/include/asm/pgtable.h                | 2 --
>>  arch/sparc/include/asm/pgtable_32.h          | 1 -
>>  arch/sparc/include/asm/pgtable_64.h          | 3 ---
>>  arch/um/include/asm/pgtable-2level.h         | 1 -
>>  arch/um/include/asm/pgtable-3level.h         | 1 -
>>  arch/x86/include/asm/pgtable_types.h         | 2 --
>>  arch/xtensa/include/asm/pgtable.h            | 1 -
>>  include/linux/pgtable.h                      | 9 +++++++++
>>  26 files changed, 9 insertions(+), 43 deletions(-)
> 
> This all looks fine to me, will this be merged via the arm tree?  I guess you
> have a means for that.

I am hoping that this should go via the MM tree instead.