[PATCH 0/3] drop hugetlb_free_pgd_range()

Anthony Yznaga posted 3 patches 2 months, 3 weeks ago
arch/sparc/include/asm/hugetlb.h |   5 --
arch/sparc/mm/hugetlbpage.c      | 119 -------------------------------
include/asm-generic/hugetlb.h    |   9 ---
include/linux/hugetlb.h          |   7 --
mm/memory.c                      |  42 +++++------
5 files changed, 18 insertions(+), 164 deletions(-)
[PATCH 0/3] drop hugetlb_free_pgd_range()
Posted by Anthony Yznaga 2 months, 3 weeks ago
For all architectures that support hugetlb except for sparc,
hugetlb_free_pgd_range() just calls free_pgd_range(). It turns out
the sparc implementation is essentially identical to free_pgd_range()
and can be removed. Remove it and update free_pgtables() to treat
hugetlb VMAs the same as others.

Anthony Yznaga (3):
  sparc64: remove hugetlb_free_pgd_range()
  mm: remove call to hugetlb_free_pgd_range()
  mm: drop hugetlb_free_pgd_range()

 arch/sparc/include/asm/hugetlb.h |   5 --
 arch/sparc/mm/hugetlbpage.c      | 119 -------------------------------
 include/asm-generic/hugetlb.h    |   9 ---
 include/linux/hugetlb.h          |   7 --
 mm/memory.c                      |  42 +++++------
 5 files changed, 18 insertions(+), 164 deletions(-)

-- 
2.47.1
Re: [PATCH 0/3] drop hugetlb_free_pgd_range()
Posted by John Paul Adrian Glaubitz 2 months, 1 week ago
Hi Anthony,

On Tue, 2025-07-15 at 18:26 -0700, Anthony Yznaga wrote:
> For all architectures that support hugetlb except for sparc,
> hugetlb_free_pgd_range() just calls free_pgd_range(). It turns out
> the sparc implementation is essentially identical to free_pgd_range()
> and can be removed. Remove it and update free_pgtables() to treat
> hugetlb VMAs the same as others.
> 
> Anthony Yznaga (3):
>   sparc64: remove hugetlb_free_pgd_range()
>   mm: remove call to hugetlb_free_pgd_range()
>   mm: drop hugetlb_free_pgd_range()
> 
>  arch/sparc/include/asm/hugetlb.h |   5 --
>  arch/sparc/mm/hugetlbpage.c      | 119 -------------------------------
>  include/asm-generic/hugetlb.h    |   9 ---
>  include/linux/hugetlb.h          |   7 --
>  mm/memory.c                      |  42 +++++------
>  5 files changed, 18 insertions(+), 164 deletions(-)

I have applied this series against v6.16-rc7 and booted the kernel inside a
SPARC LDOM on Solaris 11.4 without any problems.

Tested-by: John Paul Adrian Glaubitz <glaubitz@physik.fu-berlin.de>

Adrian

-- 
 .''`.  John Paul Adrian Glaubitz
: :' :  Debian Developer
`. `'   Physicist
  `-    GPG: 62FF 8A75 84E0 2956 9546  0006 7426 3B37 F5B5 F913
Re: [PATCH 0/3] drop hugetlb_free_pgd_range()
Posted by Oscar Salvador 2 months, 3 weeks ago
On Tue, Jul 15, 2025 at 06:26:08PM -0700, Anthony Yznaga wrote:
> For all architectures that support hugetlb except for sparc,
> hugetlb_free_pgd_range() just calls free_pgd_range(). It turns out
> the sparc implementation is essentially identical to free_pgd_range()
> and can be removed. Remove it and update free_pgtables() to treat
> hugetlb VMAs the same as others.
> 
> Anthony Yznaga (3):
>   sparc64: remove hugetlb_free_pgd_range()
>   mm: remove call to hugetlb_free_pgd_range()
>   mm: drop hugetlb_free_pgd_range()

Acked-by: Oscar Salvador <osalvador@sude.de>

Thanks!

 

-- 
Oscar Salvador
SUSE Labs
Re: [PATCH 0/3] drop hugetlb_free_pgd_range()
Posted by Oscar Salvador 2 months, 3 weeks ago
On Wed, Jul 16, 2025 at 10:05:53AM +0200, Oscar Salvador wrote:
> Acked-by: Oscar Salvador <osalvador@sude.de>

Fat fingers, sorry:

Acked-by: Oscar Salvador <osalvador@suse.de>

 

-- 
Oscar Salvador
SUSE Labs
Re: [PATCH 0/3] drop hugetlb_free_pgd_range()
Posted by Mike Rapoport 2 months, 3 weeks ago
On Tue, Jul 15, 2025 at 06:26:08PM -0700, Anthony Yznaga wrote:
> For all architectures that support hugetlb except for sparc,
> hugetlb_free_pgd_range() just calls free_pgd_range(). It turns out
> the sparc implementation is essentially identical to free_pgd_range()
> and can be removed. Remove it and update free_pgtables() to treat
> hugetlb VMAs the same as others.
> 
> Anthony Yznaga (3):
>   sparc64: remove hugetlb_free_pgd_range()
>   mm: remove call to hugetlb_free_pgd_range()
>   mm: drop hugetlb_free_pgd_range()

Acked-by: Mike Rapoport (Microsoft) <rppt@kernel.org>

>  arch/sparc/include/asm/hugetlb.h |   5 --
>  arch/sparc/mm/hugetlbpage.c      | 119 -------------------------------
>  include/asm-generic/hugetlb.h    |   9 ---
>  include/linux/hugetlb.h          |   7 --
>  mm/memory.c                      |  42 +++++------
>  5 files changed, 18 insertions(+), 164 deletions(-)
> 
> -- 
> 2.47.1
> 

-- 
Sincerely yours,
Mike.