From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from psmtp.com (na3sys010amx110.postini.com [74.125.245.110]) by kanga.kvack.org (Postfix) with SMTP id B41C56B0033 for ; Mon, 17 Jun 2013 08:33:10 -0400 (EDT) Date: Mon, 17 Jun 2013 16:27:46 +0300 From: "Kirill A. Shutemov" Subject: Re: + mm-thp-dont-use-hpage_shift-in-transparent-hugepage-code.patch added to -mm tree Message-ID: <20130617132746.GA30262@shutemov.name> References: <20130513231406.D912031C276@corp2gmr1-1.hot.corp.google.com> MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="BOKacYhQ+x31HxR3" Content-Disposition: inline In-Reply-To: <20130513231406.D912031C276@corp2gmr1-1.hot.corp.google.com> Sender: owner-linux-mm@kvack.org List-ID: To: "Aneesh Kumar K.V" , Andrea Arcangeli , Andrew Morton Cc: linux-kernel@vger.kernel.org, linux-mm@kvack.org --BOKacYhQ+x31HxR3 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Mon, May 13, 2013 at 04:14:06PM -0700, akpm@linux-foundation.org wrote: > From: "Aneesh Kumar K.V" > Subject: mm/THP: don't use HPAGE_SHIFT in transparent hugepage code > > For architectures like powerpc that support multiple explicit hugepage > sizes, HPAGE_SHIFT indicate the default explicit hugepage shift. For THP > to work the hugepage size should be same as PMD_SIZE. So use PMD_SHIFT > directly. So move the define outside CONFIG_TRANSPARENT_HUGEPAGE #ifdef > because we want to use these defines in generic code with if > (pmd_trans_huge()) conditional. I would propose to partly revert the patch with the patch bellow. Rationale: PMD_SHIFT is not defined in some configurations like nommu (allnoconfig on ARM). It blocks valid usecases in common code, like: if (PageTransHuge(page)) do_something_with(HPAGE_PMD_SIZE); And requires ugly ifdefs. I also found BUILD_BUG() useful to trigger bugs earlier for !THP configurations. The original patch was proposed as part of THP enabling on PPC. The patch below requires trivial adjustment for PPC THP patchset. Changes required for V10 is attached. diff --git a/include/linux/huge_mm.h b/include/linux/huge_mm.h index cc276d2..e2dbefb 100644 --- a/include/linux/huge_mm.h +++ b/include/linux/huge_mm.h @@ -58,11 +58,12 @@ extern pmd_t *page_check_address_pmd(struct page *page, #define HPAGE_PMD_ORDER (HPAGE_PMD_SHIFT-PAGE_SHIFT) #define HPAGE_PMD_NR (1<> shift; + max_hpte_count = 1U << (PMD_SHIFT - shift); local_irq_save(flags); for (i = 0; i < max_hpte_count; i++) { diff --git a/arch/powerpc/platforms/pseries/lpar.c b/arch/powerpc/platforms/pseries/lpar.c index f92ff2f..fd0f2f2 100644 --- a/arch/powerpc/platforms/pseries/lpar.c +++ b/arch/powerpc/platforms/pseries/lpar.c @@ -415,7 +415,7 @@ static void pSeries_lpar_hugepage_invalidate(struct mm_struct *mm, unsigned long shift, hidx, vpn = 0, vsid, hash, slot; shift = mmu_psize_defs[psize].shift; - max_hpte_count = HPAGE_PMD_SIZE >> shift; + max_hpte_count = 1U << (PMD_SHIFT - shift); for (i = 0; i < max_hpte_count; i++) { valid = hpte_valid(hpte_slot_array, i); --BOKacYhQ+x31HxR3-- -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org