[PATCH 08/10] powerpc/hugetlb: Selectively enable ARCH_WANT_GENERAL_HUGETLB

From: Anshuman Khandual
Date: Thu Apr 07 2016 - 01:38:26 EST


This enables ARCH_WANT_GENERAL_HUGETLB config option only for BOOK3S
platforms with 64K page size implementation. Existing arch specific
functions for ARCH_WANT_GENERAL_HUGETLB config like 'huge_pte_alloc'
and 'huge_pte_offset' are no longer required and are removed with
this change.

Signed-off-by: Anshuman Khandual <khandual@xxxxxxxxxxxxxxxxxx>
---
arch/powerpc/Kconfig | 4 +++
arch/powerpc/mm/hugetlbpage.c | 58 -------------------------------------------
2 files changed, 4 insertions(+), 58 deletions(-)

diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig
index 7cd32c0..9b3ce18 100644
--- a/arch/powerpc/Kconfig
+++ b/arch/powerpc/Kconfig
@@ -33,6 +33,10 @@ config HAVE_SETUP_PER_CPU_AREA
config NEED_PER_CPU_EMBED_FIRST_CHUNK
def_bool PPC64

+config ARCH_WANT_GENERAL_HUGETLB
+ depends on HUGETLB_PAGE && PPC_64K_PAGES && PPC_BOOK3S_64
+ def_bool y
+
config NR_IRQS
int "Number of virtual interrupt numbers"
range 32 32768
diff --git a/arch/powerpc/mm/hugetlbpage.c b/arch/powerpc/mm/hugetlbpage.c
index 4f44c62..bd0e584 100644
--- a/arch/powerpc/mm/hugetlbpage.c
+++ b/arch/powerpc/mm/hugetlbpage.c
@@ -59,39 +59,6 @@ pte_t *huge_pte_offset(struct mm_struct *mm, unsigned long addr)
/* Only called for hugetlbfs pages, hence can ignore THP */
return __find_linux_pte_or_hugepte(mm->pgd, addr, NULL, NULL);
}
-#else /* CONFIG_ARCH_WANT_GENERAL_HUGETLB */
-pte_t *huge_pte_offset(struct mm_struct *mm, unsigned long addr)
-{
- pgd_t pgd, *pgdp;
- pud_t pud, *pudp;
- pmd_t pmd, *pmdp;
-
- pgdp = mm->pgd + pgd_index(addr);
- pgd = READ_ONCE(*pgdp);
-
- if (pgd_none(pgd))
- return NULL;
-
- if (pgd_huge(pgd))
- return (pte_t *)pgdp;
-
- pudp = pud_offset(&pgd, addr);
- pud = READ_ONCE(*pudp);
- if (pud_none(pud))
- return NULL;
-
- if (pud_huge(pud))
- return (pte_t *)pudp;
-
- pmdp = pmd_offset(&pud, addr);
- pmd = READ_ONCE(*pmdp);
- if (pmd_none(pmd))
- return NULL;
-
- if (pmd_huge(pmd))
- return (pte_t *)pmdp;
- return NULL;
-}
#endif /* !CONFIG_ARCH_WANT_GENERAL_HUGETLB */

#ifndef CONFIG_ARCH_WANT_GENERAL_HUGETLB
@@ -210,31 +177,6 @@ hugepd_search:

return hugepte_offset(*hpdp, addr, pdshift);
}
-
-#else /* CONFIG_ARCH_WANT_GENERAL_HUGETLB */
-pte_t *huge_pte_alloc(struct mm_struct *mm, unsigned long addr, unsigned long sz)
-{
- pgd_t *pg;
- pud_t *pu;
- pmd_t *pm;
- unsigned pshift = __ffs(sz);
-
- addr &= ~(sz-1);
- pg = pgd_offset(mm, addr);
-
- if (pshift == PGDIR_SHIFT) /* 16GB Huge Page */
- return (pte_t *)pg;
-
- pu = pud_alloc(mm, pg, addr); /* NA, skipped */
- if (pshift == PUD_SHIFT)
- return (pte_t *)pu;
-
- pm = pmd_alloc(mm, pu, addr); /* 16MB Huge Page */
- if (pshift == PMD_SHIFT)
- return (pte_t *)pm;
-
- return NULL;
-}
#endif /* !CONFIG_ARCH_WANT_GENERAL_HUGETLB */
#else

--
2.1.0