linux-kernel - Re: [PATCH v3 0/2] mm/hugetlb: refactor sysfs/sysctl interfaces

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <20251104150618.0606afefb2a9e226aaa6ce77@linux-foundation.org>
Date: Tue, 4 Nov 2025 15:06:18 -0800
From: Andrew Morton <akpm@...ux-foundation.org>
To: Hui Zhu <hui.zhu@...ux.dev>
Cc: Muchun Song <muchun.song@...ux.dev>, Oscar Salvador <osalvador@...e.de>,
 David Hildenbrand <david@...hat.com>, linux-kernel@...r.kernel.org,
 linux-mm@...ck.org, Hui Zhu <zhuhui@...inos.cn>
Subject: Re: [PATCH v3 0/2] mm/hugetlb: refactor sysfs/sysctl interfaces

On Tue,  4 Nov 2025 16:37:41 +0800 Hui Zhu <hui.zhu@...ux.dev> wrote:

> From: Hui Zhu <zhuhui@...inos.cn>
> 
> The hugetlb.c file has grown significantly and become difficult to
> maintain. This patch series extracts the sysfs and sysctl interface
> code into separate dedicated files to improve code organization.
> 
> The refactoring includes:
> - Patch 1: Extract sysfs interface into mm/hugetlb_sysfs.c
> - Patch 2: Extract sysctl interface into mm/hugetlb_sysctl.c
> 
> No functional changes are introduced in this series. The code is moved
> as-is, with only minor formatting adjustments for code style
> consistency. This should make future maintenance and enhancements to
> the hugetlb subsystem easier.
> 
> Testing: The patch series has been compile-tested and maintains the
> same functionality as the original code.

Thanks.

>  MAINTAINERS           |   2 +
>  mm/Makefile           |   2 +-
>  mm/hugetlb.c          | 852 +-----------------------------------------
>  mm/hugetlb_internal.h | 113 ++++++
>  mm/hugetlb_sysctl.c   | 133 +++++++
>  mm/hugetlb_sysfs.c    | 629 +++++++++++++++++++++++++++++++
>  6 files changed, 887 insertions(+), 844 deletions(-)
>  create mode 100644 mm/hugetlb_internal.h
>  create mode 100644 mm/hugetlb_sysctl.c
>  create mode 100644 mm/hugetlb_sysfs.c

Seems a reasonable thing to do, although those who work on this code
should have the final call.

This amount of code motion might cause some (easily resolved) merge
issues during the rest of the -rc window - we'll see how it goes.