From: Joe Perches <joe@perches.com>
To: Enze Li <lienze@kylinos.cn>,
muchun.song@linux.dev, osalvador@suse.de, david@kernel.org
Cc: linux-mm@kvack.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/2] MAINTAINERS: add include/linux/hugetlb_cgroup.h to HUGETLB SUBSYSTEM
Date: Fri, 02 Oct 2026 10:37:25 -0700 [thread overview]
Message-ID: <bfefed1916a675a32ddcbebcf5a23b91fc750bcd.camel@perches.com> (raw)
In-Reply-To: <20261001123747.634845-1-lienze@kylinos.cn>
On Thu, 2026-10-01 at 20:37 +0800, Enze Li wrote:
> include/linux/hugetlb_cgroup.h is the header file paired with
> mm/hugetlb_cgroup.c, but it is not listed among the HUGETLB SUBSYSTEM
> F: entries. As a result get_maintainer.pl does not attribute the header
> to the hugetlb maintainers explicitly. Add the F: entry so the file is
> covered.
This misses another file: include/linux/hugetlb_inline.h
The current HUGETLB block is:
HUGETLB SUBSYSTEM
M: Muchun Song <muchun.song@linux.dev>
M: Oscar Salvador <osalvador@suse.de>
R: David Hildenbrand <david@kernel.org>
L: linux-mm@kvack.org
S: Maintained
F: Documentation/ABI/testing/sysfs-kernel-mm-hugepages
F: Documentation/admin-guide/mm/hugetlbpage.rst
F: Documentation/mm/hugetlbfs_reserv.rst
F: Documentation/mm/vmemmap_dedup.rst
F: fs/hugetlbfs/
F: include/linux/hugetlb.h
F: include/trace/events/hugetlbfs.h
F: mm/hugetlb.c
F: mm/hugetlb_cgroup.c
F: mm/hugetlb_cma.c
F: mm/hugetlb_cma.h
F: mm/hugetlb_sysctl.c
F: mm/hugetlb_sysfs.c
F: mm/hugetlb_vmemmap.c
F: mm/hugetlb_vmemmap.h
F: tools/testing/selftests/cgroup/test_hugetlb_memcg.c
And your suggestion is:
> diff --git a/MAINTAINERS b/MAINTAINERS
[]
> @@ -12097,6 +12097,7 @@ F: Documentation/mm/hugetlbfs_reserv.rst
> F: Documentation/mm/vmemmap_dedup.rst
> F: fs/hugetlbfs/
> F: include/linux/hugetlb.h
> +F: include/linux/hugetlb_cgroup.h
but:
$ git ls-files include/linux/hugetlb*
include/linux/hugetlb.h
include/linux/hugetlb_cgroup.h
include/linux/hugetlb_inline.h
so maybe:
F: include/linux/hugetlb*
or maybe:
X: arch/**/*hugetlb*
F: **/*hugetlb*
and remove a lot of other entries
$ git ls-files -- '**/*hugetlb*'
Documentation/admin-guide/cgroup-v1/hugetlb.rst
Documentation/admin-guide/mm/hugetlbpage.rst
Documentation/arch/arm64/hugetlbpage.rst
Documentation/mm/hugetlbfs_reserv.rst
Documentation/translations/zh_CN/arch/arm64/hugetlbpage.rst
Documentation/translations/zh_CN/mm/hugetlbfs_reserv.rst
Documentation/translations/zh_TW/arch/arm64/hugetlbpage.rst
arch/arm/include/asm/hugetlb-3level.h
arch/arm/include/asm/hugetlb.h
arch/arm64/include/asm/hugetlb.h
arch/arm64/mm/hugetlbpage.c
arch/loongarch/include/asm/hugetlb.h
arch/loongarch/mm/hugetlbpage.c
arch/mips/include/asm/hugetlb.h
arch/mips/mm/hugetlbpage.c
arch/parisc/include/asm/hugetlb.h
arch/parisc/mm/hugetlbpage.c
arch/powerpc/include/asm/book3s/64/hugetlb.h
arch/powerpc/include/asm/hugetlb.h
arch/powerpc/include/asm/nohash/32/hugetlb-8xx.h
arch/powerpc/include/asm/nohash/hugetlb-e500.h
arch/powerpc/mm/book3s64/hugetlbpage.c
arch/powerpc/mm/book3s64/radix_hugetlbpage.c
arch/powerpc/mm/hugetlbpage.c
arch/powerpc/mm/nohash/e500_hugetlbpage.c
arch/riscv/include/asm/hugetlb.h
arch/riscv/mm/hugetlbpage.c
arch/s390/include/asm/hugetlb.h
arch/s390/mm/hugetlbpage.c
arch/sh/include/asm/hugetlb.h
arch/sh/mm/hugetlbpage.c
arch/sparc/include/asm/hugetlb.h
arch/sparc/mm/hugetlbpage.c
arch/x86/include/asm/hugetlb.h
arch/x86/mm/hugetlbpage.c
fs/hugetlbfs/Makefile
fs/hugetlbfs/inode.c
include/asm-generic/hugetlb.h
include/linux/hugetlb.h
include/linux/hugetlb_cgroup.h
include/linux/hugetlb_inline.h
include/trace/events/hugetlbfs.h
include/uapi/asm-generic/hugetlb_encode.h
mm/hugetlb.c
mm/hugetlb_cgroup.c
mm/hugetlb_cma.c
mm/hugetlb_cma.h
mm/hugetlb_internal.h
mm/hugetlb_sysctl.c
mm/hugetlb_sysfs.c
mm/hugetlb_vmemmap.c
mm/hugetlb_vmemmap.h
tools/include/asm-generic/hugetlb_encode.h
tools/testing/selftests/arm64/mte/check_hugetlb_options.c
tools/testing/selftests/cgroup/test_hugetlb_memcg.c
tools/testing/selftests/memfd/run_hugetlbfs_test.sh
tools/testing/selftests/mm/charge_reserved_hugetlb.sh
tools/testing/selftests/mm/hugetlb-madvise.c
tools/testing/selftests/mm/hugetlb-mmap.c
tools/testing/selftests/mm/hugetlb-mremap.c
tools/testing/selftests/mm/hugetlb-read-hwpoison.c
tools/testing/selftests/mm/hugetlb-shm.c
tools/testing/selftests/mm/hugetlb-soft-offline.c
tools/testing/selftests/mm/hugetlb_dio.c
tools/testing/selftests/mm/hugetlb_fault_after_madv.c
tools/testing/selftests/mm/hugetlb_madv_vs_map.c
tools/testing/selftests/mm/hugetlb_reparenting_test.sh
tools/testing/selftests/mm/ksft_hugetlb.sh
tools/testing/selftests/mm/write_hugetlb_memory.sh
tools/testing/selftests/mm/write_to_hugetlbfs.c
tools/testing/selftests/powerpc/mm/hugetlb_vs_thp_test.c
next prev parent reply other threads:[~2026-10-02 17:37 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-01 12:37 Enze Li
2026-10-01 12:37 ` [PATCH 2/2] mm/hugetlb_cgroup: add trailing #endif comment for include guard Enze Li
2026-10-01 13:18 ` David Hildenbrand (Arm)
2026-10-02 2:54 ` Muchun Song
2026-10-02 18:27 ` SJ Park
2026-10-01 12:50 ` [PATCH 1/2] MAINTAINERS: add include/linux/hugetlb_cgroup.h to HUGETLB SUBSYSTEM Muchun Song
2026-10-01 13:18 ` David Hildenbrand (Arm)
2026-10-02 17:37 ` Joe Perches [this message]
2026-10-02 18:48 ` David Hildenbrand (Arm)
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=bfefed1916a675a32ddcbebcf5a23b91fc750bcd.camel@perches.com \
--to=joe@perches.com \
--cc=david@kernel.org \
--cc=lienze@kylinos.cn \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=muchun.song@linux.dev \
--cc=osalvador@suse.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®