[PATCH v2 2/4] powerpc/mm: replace hugetlb_cache by PGT_CACHE(PTE_T_ORDER)

[Christophe Leroy]

Instead of opencoding cache handling for the special case
of hugepage tables having a single pte_t element, this
patch makes use of the common pgtable_cache helpers

Signed-off-by: Christophe Leroy <christophe.le...@c-s.fr>
---
 arch/powerpc/include/asm/hugetlb.h |  2 --
 arch/powerpc/mm/hugetlbpage.c      | 26 +++++++-------------------
 2 files changed, 7 insertions(+), 21 deletions(-)

diff --git a/arch/powerpc/include/asm/hugetlb.h 
b/arch/powerpc/include/asm/hugetlb.h
index 2d00cc530083..e13843556414 100644
--- a/arch/powerpc/include/asm/hugetlb.h
+++ b/arch/powerpc/include/asm/hugetlb.h
@@ -6,8 +6,6 @@
 #include <asm/page.h>
 #include <asm-generic/hugetlb.h>
 
-extern struct kmem_cache *hugepte_cache;
-
 #ifdef CONFIG_PPC_BOOK3S_64
 
 #include <asm/book3s/64/hugetlb.h>
diff --git a/arch/powerpc/mm/hugetlbpage.c b/arch/powerpc/mm/hugetlbpage.c
index 7296a42eb62e..53b7a605c3a8 100644
--- a/arch/powerpc/mm/hugetlbpage.c
+++ b/arch/powerpc/mm/hugetlbpage.c
@@ -43,6 +43,8 @@ EXPORT_SYMBOL(HPAGE_SHIFT);
 
 #define hugepd_none(hpd)       (hpd_val(hpd) == 0)
 
+#define PTE_T_ORDER    (__builtin_ffs(sizeof(pte_t)) - 
__builtin_ffs(sizeof(void *)))
+
 pte_t *huge_pte_offset(struct mm_struct *mm, unsigned long addr, unsigned long 
sz)
 {
        /*
@@ -62,7 +64,7 @@ static int __hugepte_alloc(struct mm_struct *mm, hugepd_t 
*hpdp,
        int num_hugepd;
 
        if (pshift >= pdshift) {
-               cachep = hugepte_cache;
+               cachep = PGT_CACHE(PTE_T_ORDER);
                num_hugepd = 1 << (pshift - pdshift);
        } else {
                cachep = PGT_CACHE(pdshift - pshift);
@@ -265,7 +267,7 @@ static void hugepd_free_rcu_callback(struct rcu_head *head)
        unsigned int i;
 
        for (i = 0; i < batch->index; i++)
-               kmem_cache_free(hugepte_cache, batch->ptes[i]);
+               kmem_cache_free(PGT_CACHE(PTE_T_ORDER), batch->ptes[i]);
 
        free_page((unsigned long)batch);
 }
@@ -278,7 +280,7 @@ static void hugepd_free(struct mmu_gather *tlb, void 
*hugepte)
 
        if (atomic_read(&tlb->mm->mm_users) < 2 ||
            mm_is_thread_local(tlb->mm)) {
-               kmem_cache_free(hugepte_cache, hugepte);
+               kmem_cache_free(PGT_CACHE(PTE_T_ORDER), hugepte);
                put_cpu_var(hugepd_freelist_cur);
                return;
        }
@@ -653,7 +655,6 @@ static int __init hugepage_setup_sz(char *str)
 }
 __setup("hugepagesz=", hugepage_setup_sz);
 
-struct kmem_cache *hugepte_cache;
 static int __init hugetlbpage_init(void)
 {
        int psize;
@@ -703,21 +704,8 @@ static int __init hugetlbpage_init(void)
                if (pdshift > shift)
                        pgtable_cache_add(pdshift - shift, NULL);
 #if defined(CONFIG_PPC_FSL_BOOK3E) || defined(CONFIG_PPC_8xx)
-               else if (!hugepte_cache) {
-                       /*
-                        * Create a kmem cache for hugeptes.  The bottom bits in
-                        * the pte have size information encoded in them, so
-                        * align them to allow this
-                        */
-                       hugepte_cache = kmem_cache_create("hugepte-cache",
-                                                         sizeof(pte_t),
-                                                         HUGEPD_SHIFT_MASK + 1,
-                                                         0, NULL);
-                       if (hugepte_cache == NULL)
-                               panic("%s: Unable to create kmem cache "
-                                     "for hugeptes\n", __func__);
-
-               }
+               else
+                       pgtable_cache_add(PTE_T_ORDER, NULL);
 #endif
        }
 
-- 
2.13.3