--- v8
+++ v1
@@ -1,143 +1,210 @@
In order to split struct ptdesc from struct page, convert various
functions to use ptdescs.
-Some of the functions use the *get*page*() helper functions. Convert
-these to use pagetable_alloc() and ptdesc_address() instead to help
-standardize page tables further.
-
Signed-off-by: Vishal Moola (Oracle) <vishal.moola@gmail.com>
---
- arch/x86/mm/pgtable.c | 47 ++++++++++++++++++++++++++-----------------
- 1 file changed, 28 insertions(+), 19 deletions(-)
-
-diff --git a/arch/x86/mm/pgtable.c b/arch/x86/mm/pgtable.c
-index 15a8009a4480..d3a93e8766ee 100644
---- a/arch/x86/mm/pgtable.c
-+++ b/arch/x86/mm/pgtable.c
-@@ -52,7 +52,7 @@ early_param("userpte", setup_userpte);
-
- void ___pte_free_tlb(struct mmu_gather *tlb, struct page *pte)
- {
-- pgtable_pte_page_dtor(pte);
-+ pagetable_pte_dtor(page_ptdesc(pte));
- paravirt_release_pte(page_to_pfn(pte));
- paravirt_tlb_remove_table(tlb, pte);
- }
-@@ -60,7 +60,7 @@ void ___pte_free_tlb(struct mmu_gather *tlb, struct page *pte)
- #if CONFIG_PGTABLE_LEVELS > 2
- void ___pmd_free_tlb(struct mmu_gather *tlb, pmd_t *pmd)
+ arch/powerpc/mm/book3s64/mmu_context.c | 10 +++---
+ arch/powerpc/mm/book3s64/pgtable.c | 32 +++++++++---------
+ arch/powerpc/mm/pgtable-frag.c | 46 +++++++++++++-------------
+ 3 files changed, 44 insertions(+), 44 deletions(-)
+
+diff --git a/arch/powerpc/mm/book3s64/mmu_context.c b/arch/powerpc/mm/book3s64/mmu_context.c
+index c766e4c26e42..b22ad2839897 100644
+--- a/arch/powerpc/mm/book3s64/mmu_context.c
++++ b/arch/powerpc/mm/book3s64/mmu_context.c
+@@ -246,15 +246,15 @@ static void destroy_contexts(mm_context_t *ctx)
+ static void pmd_frag_destroy(void *pmd_frag)
+ {
+ int count;
+- struct page *page;
++ struct ptdesc *ptdesc;
+
+- page = virt_to_page(pmd_frag);
++ ptdesc = virt_to_ptdesc(pmd_frag);
+ /* drop all the pending references */
+ count = ((unsigned long)pmd_frag & ~PAGE_MASK) >> PMD_FRAG_SIZE_SHIFT;
+ /* We allow PTE_FRAG_NR fragments from a PTE page */
+- if (atomic_sub_and_test(PMD_FRAG_NR - count, &page->pt_frag_refcount)) {
+- pgtable_pmd_page_dtor(page);
+- __free_page(page);
++ if (atomic_sub_and_test(PMD_FRAG_NR - count, &ptdesc->pt_frag_refcount)) {
++ ptdesc_pmd_dtor(ptdesc);
++ ptdesc_free(ptdesc);
+ }
+ }
+
+diff --git a/arch/powerpc/mm/book3s64/pgtable.c b/arch/powerpc/mm/book3s64/pgtable.c
+index 85c84e89e3ea..7693be80c0f9 100644
+--- a/arch/powerpc/mm/book3s64/pgtable.c
++++ b/arch/powerpc/mm/book3s64/pgtable.c
+@@ -306,22 +306,22 @@ static pmd_t *get_pmd_from_cache(struct mm_struct *mm)
+ static pmd_t *__alloc_for_pmdcache(struct mm_struct *mm)
+ {
+ void *ret = NULL;
+- struct page *page;
++ struct ptdesc *ptdesc;
+ gfp_t gfp = GFP_KERNEL_ACCOUNT | __GFP_ZERO;
+
+ if (mm == &init_mm)
+ gfp &= ~__GFP_ACCOUNT;
+- page = alloc_page(gfp);
+- if (!page)
++ ptdesc = ptdesc_alloc(gfp);
++ if (!ptdesc)
+ return NULL;
+- if (!pgtable_pmd_page_ctor(page)) {
+- __free_pages(page, 0);
++ if (!ptdesc_pmd_ctor(ptdesc)) {
++ ptdesc_free(ptdesc);
+ return NULL;
+ }
+
+- atomic_set(&page->pt_frag_refcount, 1);
++ atomic_set(&ptdesc->pt_frag_refcount, 1);
+
+- ret = page_address(page);
++ ret = ptdesc_address(ptdesc);
+ /*
+ * if we support only one fragment just return the
+ * allocated page.
+@@ -331,12 +331,12 @@ static pmd_t *__alloc_for_pmdcache(struct mm_struct *mm)
+
+ spin_lock(&mm->page_table_lock);
+ /*
+- * If we find pgtable_page set, we return
++ * If we find ptdesc_page set, we return
+ * the allocated page with single fragment
+ * count.
+ */
+ if (likely(!mm->context.pmd_frag)) {
+- atomic_set(&page->pt_frag_refcount, PMD_FRAG_NR);
++ atomic_set(&ptdesc->pt_frag_refcount, PMD_FRAG_NR);
+ mm->context.pmd_frag = ret + PMD_FRAG_SIZE;
+ }
+ spin_unlock(&mm->page_table_lock);
+@@ -357,15 +357,15 @@ pmd_t *pmd_fragment_alloc(struct mm_struct *mm, unsigned long vmaddr)
+
+ void pmd_fragment_free(unsigned long *pmd)
{
- struct page *page = virt_to_page(pmd);
+ struct ptdesc *ptdesc = virt_to_ptdesc(pmd);
- paravirt_release_pmd(__pa(pmd) >> PAGE_SHIFT);
- /*
- * NOTE! For PAE, any changes to the top page-directory-pointer-table
-@@ -69,8 +69,8 @@ void ___pmd_free_tlb(struct mmu_gather *tlb, pmd_t *pmd)
- #ifdef CONFIG_X86_PAE
- tlb->need_flush_all = 1;
- #endif
-- pgtable_pmd_page_dtor(page);
-- paravirt_tlb_remove_table(tlb, page);
-+ pagetable_pmd_dtor(ptdesc);
-+ paravirt_tlb_remove_table(tlb, ptdesc_page(ptdesc));
- }
-
- #if CONFIG_PGTABLE_LEVELS > 3
-@@ -92,16 +92,16 @@ void ___p4d_free_tlb(struct mmu_gather *tlb, p4d_t *p4d)
-
- static inline void pgd_list_add(pgd_t *pgd)
- {
-- struct page *page = virt_to_page(pgd);
-+ struct ptdesc *ptdesc = virt_to_ptdesc(pgd);
-
-- list_add(&page->lru, &pgd_list);
-+ list_add(&ptdesc->pt_list, &pgd_list);
- }
-
- static inline void pgd_list_del(pgd_t *pgd)
- {
-- struct page *page = virt_to_page(pgd);
-+ struct ptdesc *ptdesc = virt_to_ptdesc(pgd);
-
-- list_del(&page->lru);
-+ list_del(&ptdesc->pt_list);
- }
-
- #define UNSHARED_PTRS_PER_PGD \
-@@ -112,12 +112,12 @@ static inline void pgd_list_del(pgd_t *pgd)
-
- static void pgd_set_mm(pgd_t *pgd, struct mm_struct *mm)
- {
-- virt_to_page(pgd)->pt_mm = mm;
-+ virt_to_ptdesc(pgd)->pt_mm = mm;
- }
-
- struct mm_struct *pgd_page_get_mm(struct page *page)
- {
-- return page->pt_mm;
-+ return page_ptdesc(page)->pt_mm;
- }
-
- static void pgd_ctor(struct mm_struct *mm, pgd_t *pgd)
-@@ -213,11 +213,14 @@ void pud_populate(struct mm_struct *mm, pud_t *pudp, pmd_t *pmd)
- static void free_pmds(struct mm_struct *mm, pmd_t *pmds[], int count)
- {
- int i;
-+ struct ptdesc *ptdesc;
-
- for (i = 0; i < count; i++)
- if (pmds[i]) {
-- pgtable_pmd_page_dtor(virt_to_page(pmds[i]));
-- free_page((unsigned long)pmds[i]);
-+ ptdesc = virt_to_ptdesc(pmds[i]);
-+
-+ pagetable_pmd_dtor(ptdesc);
-+ pagetable_free(ptdesc);
- mm_dec_nr_pmds(mm);
+
+- if (PageReserved(page))
+- return free_reserved_page(page);
++ if (ptdesc_is_reserved(ptdesc))
++ return free_reserved_ptdesc(ptdesc);
+
+- BUG_ON(atomic_read(&page->pt_frag_refcount) <= 0);
+- if (atomic_dec_and_test(&page->pt_frag_refcount)) {
+- pgtable_pmd_page_dtor(page);
+- __free_page(page);
++ BUG_ON(atomic_read(&ptdesc->pt_frag_refcount) <= 0);
++ if (atomic_dec_and_test(&ptdesc->pt_frag_refcount)) {
++ ptdesc_pmd_dtor(ptdesc);
++ ptdesc_free(ptdesc);
+ }
+ }
+
+diff --git a/arch/powerpc/mm/pgtable-frag.c b/arch/powerpc/mm/pgtable-frag.c
+index 20652daa1d7e..cf08831fa7c3 100644
+--- a/arch/powerpc/mm/pgtable-frag.c
++++ b/arch/powerpc/mm/pgtable-frag.c
+@@ -18,15 +18,15 @@
+ void pte_frag_destroy(void *pte_frag)
+ {
+ int count;
+- struct page *page;
++ struct ptdesc *ptdesc;
+
+- page = virt_to_page(pte_frag);
++ ptdesc = virt_to_ptdesc(pte_frag);
+ /* drop all the pending references */
+ count = ((unsigned long)pte_frag & ~PAGE_MASK) >> PTE_FRAG_SIZE_SHIFT;
+ /* We allow PTE_FRAG_NR fragments from a PTE page */
+- if (atomic_sub_and_test(PTE_FRAG_NR - count, &page->pt_frag_refcount)) {
+- pgtable_pte_page_dtor(page);
+- __free_page(page);
++ if (atomic_sub_and_test(PTE_FRAG_NR - count, &ptdesc->pt_frag_refcount)) {
++ ptdesc_pte_dtor(ptdesc);
++ ptdesc_free(ptdesc);
+ }
+ }
+
+@@ -55,25 +55,25 @@ static pte_t *get_pte_from_cache(struct mm_struct *mm)
+ static pte_t *__alloc_for_ptecache(struct mm_struct *mm, int kernel)
+ {
+ void *ret = NULL;
+- struct page *page;
++ struct ptdesc *ptdesc;
+
+ if (!kernel) {
+- page = alloc_page(PGALLOC_GFP | __GFP_ACCOUNT);
+- if (!page)
++ ptdesc = ptdesc_alloc(PGALLOC_GFP | __GFP_ACCOUNT);
++ if (!ptdesc)
+ return NULL;
+- if (!pgtable_pte_page_ctor(page)) {
+- __free_page(page);
++ if (!ptdesc_pte_ctor(ptdesc)) {
++ ptdesc_free(ptdesc);
+ return NULL;
}
- }
-@@ -230,18 +233,24 @@ static int preallocate_pmds(struct mm_struct *mm, pmd_t *pmds[], int count)
-
- if (mm == &init_mm)
- gfp &= ~__GFP_ACCOUNT;
-+ gfp &= ~__GFP_HIGHMEM;
-
- for (i = 0; i < count; i++) {
-- pmd_t *pmd = (pmd_t *)__get_free_page(gfp);
-- if (!pmd)
-+ pmd_t *pmd = NULL;
-+ struct ptdesc *ptdesc = pagetable_alloc(gfp, 0);
-+
+ } else {
+- page = alloc_page(PGALLOC_GFP);
+- if (!page)
++ ptdesc = ptdesc_alloc(PGALLOC_GFP);
+ if (!ptdesc)
- failed = true;
-- if (pmd && !pgtable_pmd_page_ctor(virt_to_page(pmd))) {
-- free_page((unsigned long)pmd);
-- pmd = NULL;
-+ if (ptdesc && !pagetable_pmd_ctor(ptdesc)) {
-+ pagetable_free(ptdesc);
-+ ptdesc = NULL;
- failed = true;
- }
-- if (pmd)
-+ if (ptdesc) {
- mm_inc_nr_pmds(mm);
-+ pmd = ptdesc_address(ptdesc);
-+ }
-+
- pmds[i] = pmd;
- }
-
-@@ -830,7 +839,7 @@ int pud_free_pmd_page(pud_t *pud, unsigned long addr)
-
- free_page((unsigned long)pmd_sv);
-
-- pgtable_pmd_page_dtor(virt_to_page(pmd));
-+ pagetable_pmd_dtor(virt_to_ptdesc(pmd));
- free_page((unsigned long)pmd);
-
- return 1;
+ return NULL;
+ }
+
+- atomic_set(&page->pt_frag_refcount, 1);
++ atomic_set(&ptdesc->pt_frag_refcount, 1);
+
+- ret = page_address(page);
++ ret = ptdesc_address(ptdesc);
+ /*
+ * if we support only one fragment just return the
+ * allocated page.
+@@ -82,12 +82,12 @@ static pte_t *__alloc_for_ptecache(struct mm_struct *mm, int kernel)
+ return ret;
+ spin_lock(&mm->page_table_lock);
+ /*
+- * If we find pgtable_page set, we return
++ * If we find ptdesc_page set, we return
+ * the allocated page with single fragment
+ * count.
+ */
+ if (likely(!pte_frag_get(&mm->context))) {
+- atomic_set(&page->pt_frag_refcount, PTE_FRAG_NR);
++ atomic_set(&ptdesc->pt_frag_refcount, PTE_FRAG_NR);
+ pte_frag_set(&mm->context, ret + PTE_FRAG_SIZE);
+ }
+ spin_unlock(&mm->page_table_lock);
+@@ -108,15 +108,15 @@ pte_t *pte_fragment_alloc(struct mm_struct *mm, int kernel)
+
+ void pte_fragment_free(unsigned long *table, int kernel)
+ {
+- struct page *page = virt_to_page(table);
++ struct ptdesc *ptdesc = virt_to_ptdesc(table);
+
+- if (PageReserved(page))
+- return free_reserved_page(page);
++ if (ptdesc_is_reserved(ptdesc))
++ return free_reserved_ptdesc(ptdesc);
+
+- BUG_ON(atomic_read(&page->pt_frag_refcount) <= 0);
+- if (atomic_dec_and_test(&page->pt_frag_refcount)) {
++ BUG_ON(atomic_read(&ptdesc->pt_frag_refcount) <= 0);
++ if (atomic_dec_and_test(&ptdesc->pt_frag_refcount)) {
+ if (!kernel)
+- pgtable_pte_page_dtor(page);
+- __free_page(page);
++ ptdesc_pte_dtor(ptdesc);
++ ptdesc_free(ptdesc);
+ }
+ }
--
-2.40.1
+2.39.2
_______________________________________________