Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932345AbdDQEsM (ORCPT ); Mon, 17 Apr 2017 00:48:12 -0400 Received: from mail-qk0-f195.google.com ([209.85.220.195]:35859 "EHLO mail-qk0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753639AbdDQEsL (ORCPT ); Mon, 17 Apr 2017 00:48:11 -0400 MIME-Version: 1.0 In-Reply-To: References: <1490730347-5165-1-git-send-email-bhsharma@redhat.com> <87shlcq0es.fsf@skywalker.in.ibm.com> <03284709-498c-1e52-40f0-b19d20e56b8e@linux.vnet.ibm.com> From: Bhupesh SHARMA Date: Mon, 17 Apr 2017 10:18:09 +0530 Message-ID: Subject: Re: [PATCH v3] powerpc: mm: support ARCH_MMAP_RND_BITS To: Balbir Singh Cc: "Aneesh Kumar K.V" , Bhupesh Sharma , Kees Cook , kernel-hardening@lists.openwall.com, Alistair Popple , "linux-kernel@vger.kernel.org" , Scott Wood , Paul Mackerras , Alexander Graf , Daniel Cashman , Anatolij Gustschin , "open list:LINUX FOR POWERPC (32-BIT AND 64-BIT)" , Daniel Cashman Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1064 Lines: 30 On Thu, Apr 13, 2017 at 12:39 PM, Balbir Singh wrote: >>> >>> Yes. It was derived from TASK_SIZE : >>> >>> http://lxr.free-electrons.com/source/arch/powerpc/include/asm/processor.h#L105 >>> >> >> That is getting update to 128TB by default and conditionally to 512TB >> > > Since this is compile time, we should probably keep the scope to 128TB > for now and see if we want to change things at run time later, since > the expansion is based on a hint. Suggestions? > I think this makes sense. If the conditional expansion to 512TB is protected by a kconfig symbol, we can use the same to have separate ranges for 128TB and 512TB using the kconfig symbol as the differentiating factor. Also please let me know which branch/tree to use where we are done with the change making the default to 128TB. My v2 was based on git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git (mater branch), where the TASK_SIZE is set to 46-bits inside 'arch/powerpc/include/asm/processor.h'. So that I can spin the v3 accordingly. Thanks, Bhupesh