Inter-revision diff: patch 25

Comparing v1 (message) to v6 (message)

--- v1
+++ v6
@@ -2,17 +2,18 @@
 ptdesc equivalents, convert various page table functions to use ptdescs.
 
 Some of the functions use the *get*page*() helper functions. Convert
-these to use ptdesc_alloc() and ptdesc_address() instead to help
+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>
+Acked-by: Mike Rapoport (IBM) <rppt@kernel.org>
 ---
- arch/mips/include/asm/pgalloc.h | 31 +++++++++++++++++--------------
- arch/mips/mm/pgtable.c          |  7 ++++---
- 2 files changed, 21 insertions(+), 17 deletions(-)
+ arch/mips/include/asm/pgalloc.h | 32 ++++++++++++++++++--------------
+ arch/mips/mm/pgtable.c          |  8 +++++---
+ 2 files changed, 23 insertions(+), 17 deletions(-)
 
 diff --git a/arch/mips/include/asm/pgalloc.h b/arch/mips/include/asm/pgalloc.h
-index f72e737dda21..7f7cc3140b27 100644
+index f72e737dda21..40e40a7eb94a 100644
 --- a/arch/mips/include/asm/pgalloc.h
 +++ b/arch/mips/include/asm/pgalloc.h
 @@ -51,13 +51,13 @@ extern pgd_t *pgd_alloc(struct mm_struct *mm);
@@ -20,7 +21,7 @@
  static inline void pgd_free(struct mm_struct *mm, pgd_t *pgd)
  {
 -	free_pages((unsigned long)pgd, PGD_TABLE_ORDER);
-+	ptdesc_free(virt_to_ptdesc(pgd));
++	pagetable_free(virt_to_ptdesc(pgd));
  }
  
 -#define __pte_free_tlb(tlb,pte,address)			\
@@ -29,7 +30,7 @@
 -	tlb_remove_page((tlb), pte);			\
 +#define __pte_free_tlb(tlb, pte, address)			\
 +do {								\
-+	ptdesc_pte_dtor(page_ptdesc(pte));			\
++	pagetable_pte_dtor(page_ptdesc(pte));			\
 +	tlb_remove_page_ptdesc((tlb), page_ptdesc(pte));	\
  } while (0)
  
@@ -43,60 +44,62 @@
  
 -	pg = alloc_pages(GFP_KERNEL_ACCOUNT, PMD_TABLE_ORDER);
 -	if (!pg)
-+	ptdesc = ptdesc_alloc(GFP_KERNEL_ACCOUNT, PMD_TABLE_ORDER);
++	ptdesc = pagetable_alloc(GFP_KERNEL_ACCOUNT, PMD_TABLE_ORDER);
 +	if (!ptdesc)
  		return NULL;
  
 -	if (!pgtable_pmd_page_ctor(pg)) {
 -		__free_pages(pg, PMD_TABLE_ORDER);
-+	if (!ptdesc_pmd_ctor(ptdesc)) {
-+		ptdesc_free(ptdesc);
++	if (!pagetable_pmd_ctor(ptdesc)) {
++		pagetable_free(ptdesc);
  		return NULL;
  	}
  
 -	pmd = (pmd_t *)page_address(pg);
-+	pmd = (pmd_t *)ptdesc_address(ptdesc);
++	pmd = ptdesc_address(ptdesc);
  	pmd_init(pmd);
  	return pmd;
  }
-@@ -90,10 +90,13 @@ static inline pmd_t *pmd_alloc_one(struct mm_struct *mm, unsigned long address)
+@@ -90,10 +90,14 @@ static inline pmd_t *pmd_alloc_one(struct mm_struct *mm, unsigned long address)
  static inline pud_t *pud_alloc_one(struct mm_struct *mm, unsigned long address)
  {
  	pud_t *pud;
-+	struct ptdesc *ptdesc = ptdesc_alloc(GFP_KERNEL, PUD_TABLE_ORDER);
++	struct ptdesc *ptdesc = pagetable_alloc(GFP_KERNEL & ~__GFP_HIGHMEM,
++			PUD_TABLE_ORDER);
  
 -	pud = (pud_t *) __get_free_pages(GFP_KERNEL, PUD_TABLE_ORDER);
 -	if (pud)
 -		pud_init(pud);
 +	if (!ptdesc)
 +		return NULL;
-+	pud = (pud_t *)ptdesc_address(ptdesc);
++	pud = ptdesc_address(ptdesc);
 +
 +	pud_init(pud);
  	return pud;
  }
  
 diff --git a/arch/mips/mm/pgtable.c b/arch/mips/mm/pgtable.c
-index b13314be5d0e..d626db9ac224 100644
+index b13314be5d0e..1506e458040d 100644
 --- a/arch/mips/mm/pgtable.c
 +++ b/arch/mips/mm/pgtable.c
-@@ -10,10 +10,11 @@
+@@ -10,10 +10,12 @@
  
  pgd_t *pgd_alloc(struct mm_struct *mm)
  {
 -	pgd_t *ret, *init;
 +	pgd_t *init, *ret = NULL;
-+	struct ptdesc *ptdesc = ptdesc_alloc(GFP_KERNEL, PGD_TABLE_ORDER);
++	struct ptdesc *ptdesc = pagetable_alloc(GFP_KERNEL & ~__GFP_HIGHMEM,
++			PGD_TABLE_ORDER);
  
 -	ret = (pgd_t *) __get_free_pages(GFP_KERNEL, PGD_TABLE_ORDER);
 -	if (ret) {
 +	if (ptdesc) {
-+		ret = (pgd_t *) ptdesc_address(ptdesc);
++		ret = ptdesc_address(ptdesc);
  		init = pgd_offset(&init_mm, 0UL);
  		pgd_init(ret);
  		memcpy(ret + USER_PTRS_PER_PGD, init + USER_PTRS_PER_PGD,
 -- 
-2.39.2
+2.40.1
 
 
 _______________________________________________
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help