From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from foss.arm.com (foss.arm.com [217.140.101.70]) by lists.ozlabs.org (Postfix) with ESMTP id 878BD1A020A for ; Wed, 18 Mar 2015 01:47:12 +1100 (AEDT) Date: Tue, 17 Mar 2015 14:47:02 +0000 From: Will Deacon To: Kees Cook Subject: Re: [PATCH v4 03/10] arm64: standardize mmap_rnd() usage Message-ID: <20150317144702.GN8399@arm.com> References: <1425503454-7531-1-git-send-email-keescook@chromium.org> <1425503454-7531-4-git-send-email-keescook@chromium.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <1425503454-7531-4-git-send-email-keescook@chromium.org> Cc: "linux-mips@linux-mips.org" , Min-Hua Chen , Heiko Carstens , "linux-kernel@vger.kernel.org" , Arun Chandran , Paul Mackerras , Ismael Ripoll , Yann Droneaud , "linux-s390@vger.kernel.org" , Russell King , Andrey Ryabinin , "x86@kernel.org" , Hector Marco-Gisbert , Ingo Molnar , "David A. Long" , Catalin Marinas , Borislav Petkov , Ben Hutchings , "linux-fsdevel@vger.kernel.org" , Alexander Viro , Michael Holzheu , "linux-arm-kernel@lists.infradead.org" , Jeff Bailey , Paul Burton , Oleg Nesterov , Ralf Baechle , Andy Lutomirski , Vineeth Vijayan , Markos Chandras , Jan-Simon =?iso-8859-1?Q?M=F6ller?= , Martin Schwidefsky , "linux390@de.ibm.com" , "akpm@linux-foundation.org" , "linuxppc-dev@lists.ozlabs.org" , Alex Smith List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Wed, Mar 04, 2015 at 09:10:47PM +0000, Kees Cook wrote: > In preparation for splitting out ET_DYN ASLR, this refactors the use of > mmap_rnd() to be used similarly to arm and x86. This additionally enables > mmap ASLR on legacy mmap layouts, which appeared to be missing on arm64, > and was already supported on arm. Additionally removes a copy/pasted > declaration of an unused function. > > Signed-off-by: Kees Cook > --- > arch/arm64/include/asm/elf.h | 1 - > arch/arm64/mm/mmap.c | 18 +++++++++++------- > 2 files changed, 11 insertions(+), 8 deletions(-) Looks fine to me: Acked-by: Will Deacon Do you want me to pick this up, or are you taking it along with the rest of your series (it doesn't have any obvious dependencies to me)? Will