mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [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®