Thread (16 messages) 16 messages, 6 authors, 2025-09-05

Re: [PATCH v3 5/7] powerpc: Stop calling page_address() in free_pages()

From: Christophe Leroy <hidden>
Date: 2025-09-04 07:21:14
Also in: linux-arm-kernel, linux-block, linux-efi, linux-mm, linux-riscv, lkml, virtualization


Le 03/09/2025 à 20:59, Vishal Moola (Oracle) a écrit :
free_pages() should be used when we only have a virtual address. We
should call __free_pages() directly on our page instead.

Signed-off-by: Vishal Moola (Oracle) <redacted>
Reviewed-by: Ritesh Harjani (IBM) <ritesh.list@gmail.com>
Reviewed-by: Christophe Leroy <redacted>
quoted hunk ↗ jump to hunk
---
  arch/powerpc/mm/book3s64/radix_pgtable.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/powerpc/mm/book3s64/radix_pgtable.c b/arch/powerpc/mm/book3s64/radix_pgtable.c
index be523e5fe9c5..73977dbabcf2 100644
--- a/arch/powerpc/mm/book3s64/radix_pgtable.c
+++ b/arch/powerpc/mm/book3s64/radix_pgtable.c
@@ -780,7 +780,7 @@ static void __meminit free_vmemmap_pages(struct page *page,
  		while (nr_pages--)
  			free_reserved_page(page++);
  	} else
-		free_pages((unsigned long)page_address(page), order);
+		__free_pages(page, order);
  }
  
  static void __meminit remove_pte_table(pte_t *pte_start, unsigned long addr,
  
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help