From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754649AbbCQOrQ (ORCPT ); Tue, 17 Mar 2015 10:47:16 -0400 Received: from foss.arm.com ([217.140.101.70]:59162 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752854AbbCQOrL (ORCPT ); Tue, 17 Mar 2015 10:47:11 -0400 Date: Tue, 17 Mar 2015 14:47:02 +0000 From: Will Deacon To: Kees Cook Cc: "akpm@linux-foundation.org" , Ingo Molnar , Michael Ellerman , Hector Marco-Gisbert , Ismael Ripoll , Russell King , Catalin Marinas , Ralf Baechle , Benjamin Herrenschmidt , Paul Mackerras , Martin Schwidefsky , Heiko Carstens , "linux390@de.ibm.com" , "x86@kernel.org" , Alexander Viro , Andy Lutomirski , Oleg Nesterov , "David A. Long" , Andrey Ryabinin , Arun Chandran , Yann Droneaud , Min-Hua Chen , Paul Burton , Alex Smith , Markos Chandras , Vineeth Vijayan , Jeff Bailey , Michael Holzheu , Ben Hutchings , Borislav Petkov , Jan-Simon =?iso-8859-1?Q?M=F6ller?= , "linux-kernel@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-mips@linux-mips.org" , "linuxppc-dev@lists.ozlabs.org" , "linux-s390@vger.kernel.org" , "linux-fsdevel@vger.kernel.org" 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 Content-Disposition: inline In-Reply-To: <1425503454-7531-4-git-send-email-keescook@chromium.org> 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 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