Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934913AbcCJDib (ORCPT ); Wed, 9 Mar 2016 22:38:31 -0500 Received: from e23smtp05.au.ibm.com ([202.81.31.147]:55163 "EHLO e23smtp05.au.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753283AbcCJDi1 (ORCPT ); Wed, 9 Mar 2016 22:38:27 -0500 X-IBM-Helo: d23dlp01.au.ibm.com X-IBM-MailFrom: khandual@linux.vnet.ibm.com X-IBM-RcptTo: linux-kernel@vger.kernel.org Message-ID: <56E0EBF7.3040104@linux.vnet.ibm.com> Date: Thu, 10 Mar 2016 09:07:27 +0530 From: Anshuman Khandual User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 MIME-Version: 1.0 To: Dave Hansen , linux-mm@kvack.org, linux-kernel@vger.kernel.org, linuxppc-dev@lists.ozlabs.org CC: hughd@google.com, aneesh.kumar@linux.vnet.ibm.com, kirill@shutemov.name, n-horiguchi@ah.jp.nec.com, mgorman@techsingularity.net, akpm@linux-foundation.org Subject: Re: [RFC 5/9] powerpc/mm: Split huge_pte_offset function for BOOK3S 64K References: <1457525450-4262-1-git-send-email-khandual@linux.vnet.ibm.com> <1457525450-4262-5-git-send-email-khandual@linux.vnet.ibm.com> <56E0AA59.4030905@intel.com> In-Reply-To: <56E0AA59.4030905@intel.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-TM-AS-MML: disable X-Content-Scanned: Fidelis XPS MAILER x-cbid: 16031003-0017-0000-0000-0000030653E6 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1226 Lines: 25 On 03/10/2016 04:27 AM, Dave Hansen wrote: > On 03/09/2016 04:10 AM, Anshuman Khandual wrote: >> > Currently the 'huge_pte_offset' function has only one version for >> > all the configuations and platforms. This change splits the function >> > into two versions, one for 64K page size based BOOK3S implementation >> > and the other one for everything else. This change is also one of the >> > prerequisites towards enabling GENERAL_HUGETLB implementation for >> > BOOK3S 64K based huge pages. > I think there's a bit of background missing here for random folks on > linux-mm to make sense of these patches. > > What is BOOK3S and what does it mean for these patches? Why is its 64K BOOK3S is the server type in powerpc family of processors which can support multiple base page sizes like 64K and 4K. > page size implementation different than all the others? Is there a 4K > page size BOOK3S? It supports huge pages of size 16M as well as 16G and their implementations are different with respect to base page sizes of 64K and 4K. Patches 1, 2 and 3 are generic VM changes and the rest are powerpc specific changes. Should I have split them accordingly and send out differently for generic and powerpc specific reviews ?