* [PATCH 1/2] MAINTAINERS: add include/linux/hugetlb_cgroup.h to HUGETLB SUBSYSTEM
@ 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
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Enze Li @ 2026-10-01 12:37 UTC (permalink / raw)
To: muchun.song, osalvador, david; +Cc: linux-mm, linux-kernel, Enze Li
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.
Signed-off-by: Enze Li <lienze@kylinos.cn>
---
MAINTAINERS | 1 +
1 file changed, 1 insertion(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 596efa354ea2..6c210e90722f 100644
--- 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
F: include/linux/vmemmap-optimization.h
F: include/trace/events/hugetlbfs.h
F: mm/hugetlb.c
--
2.25.1
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH 2/2] mm/hugetlb_cgroup: add trailing #endif comment for include guard
2026-10-01 12:37 [PATCH 1/2] MAINTAINERS: add include/linux/hugetlb_cgroup.h to HUGETLB SUBSYSTEM Enze Li
@ 2026-10-01 12:37 ` Enze Li
2026-10-01 13:18 ` David Hildenbrand (Arm)
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)
2 siblings, 1 reply; 5+ messages in thread
From: Enze Li @ 2026-10-01 12:37 UTC (permalink / raw)
To: muchun.song, osalvador, david; +Cc: linux-mm, linux-kernel, Enze Li
The final #endif that closes the _LINUX_HUGETLB_CGROUP_H include guard
is left without a comment. Add the macro name to the comment for
consistency and easier navigation.
Signed-off-by: Enze Li <lienze@kylinos.cn>
---
include/linux/hugetlb_cgroup.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/linux/hugetlb_cgroup.h b/include/linux/hugetlb_cgroup.h
index 16d72c8c71f6..b4e0958eabf3 100644
--- a/include/linux/hugetlb_cgroup.h
+++ b/include/linux/hugetlb_cgroup.h
@@ -268,4 +268,4 @@ static inline void hugetlb_cgroup_migrate(struct folio *old_folio,
}
#endif /* CONFIG_CGROUP_HUGETLB */
-#endif
+#endif /* _LINUX_HUGETLB_CGROUP_H */
--
2.25.1
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/2] MAINTAINERS: add include/linux/hugetlb_cgroup.h to HUGETLB SUBSYSTEM
2026-10-01 12:37 [PATCH 1/2] MAINTAINERS: add include/linux/hugetlb_cgroup.h to HUGETLB SUBSYSTEM 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 12:50 ` Muchun Song
2026-10-01 13:18 ` David Hildenbrand (Arm)
2 siblings, 0 replies; 5+ messages in thread
From: Muchun Song @ 2026-10-01 12:50 UTC (permalink / raw)
To: Enze Li; +Cc: osalvador, david, linux-mm, linux-kernel
> On Oct 1, 2026, at 20:37, Enze Li <lienze@kylinos.cn> 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.
>
> Signed-off-by: Enze Li <lienze@kylinos.cn>
Acked-by: Muchun Song <muchun.song@linux.dev>
Thanks
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/2] MAINTAINERS: add include/linux/hugetlb_cgroup.h to HUGETLB SUBSYSTEM
2026-10-01 12:37 [PATCH 1/2] MAINTAINERS: add include/linux/hugetlb_cgroup.h to HUGETLB SUBSYSTEM 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 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)
2 siblings, 0 replies; 5+ messages in thread
From: David Hildenbrand (Arm) @ 2026-10-01 13:18 UTC (permalink / raw)
To: Enze Li, muchun.song, osalvador; +Cc: linux-mm, linux-kernel
On 10/1/26 14:37, 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.
>
> Signed-off-by: Enze Li <lienze@kylinos.cn>
> ---
> MAINTAINERS | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 596efa354ea2..6c210e90722f 100644
> --- 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
> F: include/linux/vmemmap-optimization.h
> F: include/trace/events/hugetlbfs.h
> F: mm/hugetlb.c
Great, thanks!
Acked-by: David Hildenbrand (Arm) <david@kernel.org>
--
Cheers,
David
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 2/2] mm/hugetlb_cgroup: add trailing #endif comment for include guard
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)
0 siblings, 0 replies; 5+ messages in thread
From: David Hildenbrand (Arm) @ 2026-10-01 13:18 UTC (permalink / raw)
To: Enze Li, muchun.song, osalvador; +Cc: linux-mm, linux-kernel
On 10/1/26 14:37, Enze Li wrote:
> The final #endif that closes the _LINUX_HUGETLB_CGROUP_H include guard
> is left without a comment. Add the macro name to the comment for
> consistency and easier navigation.
>
> Signed-off-by: Enze Li <lienze@kylinos.cn>
> ---
> include/linux/hugetlb_cgroup.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/linux/hugetlb_cgroup.h b/include/linux/hugetlb_cgroup.h
> index 16d72c8c71f6..b4e0958eabf3 100644
> --- a/include/linux/hugetlb_cgroup.h
> +++ b/include/linux/hugetlb_cgroup.h
> @@ -268,4 +268,4 @@ static inline void hugetlb_cgroup_migrate(struct folio *old_folio,
> }
>
> #endif /* CONFIG_CGROUP_HUGETLB */
> -#endif
> +#endif /* _LINUX_HUGETLB_CGROUP_H */
Acked-by: David Hildenbrand (Arm) <david@kernel.org>
--
Cheers,
David
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2026-10-01 13:18 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-10-01 12:37 [PATCH 1/2] MAINTAINERS: add include/linux/hugetlb_cgroup.h to HUGETLB SUBSYSTEM 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-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)
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®