Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932882AbdC2RpT (ORCPT ); Wed, 29 Mar 2017 13:45:19 -0400 Received: from mga09.intel.com ([134.134.136.24]:13399 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932172AbdC2RpR (ORCPT ); Wed, 29 Mar 2017 13:45:17 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.36,242,1486454400"; d="scan'208";a="839747277" Date: Wed, 29 Mar 2017 10:45:14 -0700 From: Andi Kleen To: Michal Hocko Cc: Matthew Wilcox , akpm@linux-foundation.org, mtk.manpages@gmail.com, linux-mm@kvack.org, linux-kernel@vger.kernel.org, Davidlohr Bueso , khandual@linux.vnet.ibm.com Subject: Re: [PATCH] mm,hugetlb: compute page_size_log properly Message-ID: <20170329174514.GB4543@tassilo.jf.intel.com> References: <1488992761-9464-1-git-send-email-dave@stgolabs.net> <20170328165343.GB27446@linux-80c1.suse> <20170328165513.GC27446@linux-80c1.suse> <20170328175408.GD7838@bombadil.infradead.org> <20170329080625.GC27994@dhcp22.suse.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170329080625.GC27994@dhcp22.suse.cz> User-Agent: Mutt/1.8.0 (2017-02-23) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 984 Lines: 21 On Wed, Mar 29, 2017 at 10:06:25AM +0200, Michal Hocko wrote: > On Tue 28-03-17 10:54:08, Matthew Wilcox wrote: > > On Tue, Mar 28, 2017 at 09:55:13AM -0700, Davidlohr Bueso wrote: > > > Do we have any consensus here? Keeping SHM_HUGE_* is currently > > > winning 2-1. If there are in fact users out there computing the > > > value manually, then I am ok with keeping it and properly exporting > > > it. Michal? > > > > Well, let's see what it looks like to do that. I went down the rabbit > > hole trying to understand why some of the SHM_ flags had the same value > > as each other until I realised some of them were internal flags, some > > were flags to shmat() and others were flags to shmget(). Hopefully I > > disambiguated them nicely in this patch. I also added 8MB and 16GB sizes. > > Any more architectures with a pet favourite huge/giant page size we > > should add convenience defines for? > > Do we actually have any users? Yes this feature is widely used. -Andi