linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] MAINTAINERS: add missing mm_slot.h file THP section
@ 2025-07-22 17:19 Lorenzo Stoakes
  2025-07-22 17:26 ` Zi Yan
                   ` (3 more replies)
  0 siblings, 4 replies; 8+ messages in thread
From: Lorenzo Stoakes @ 2025-07-22 17:19 UTC (permalink / raw)
  To: Andrew Morton
  Cc: linux-mm, linux-kernel, David Hildenbrand, Zi Yan, Baolin Wang,
	Liam R . Howlett, Nico Pache, Ryan Roberts, Dev Jain, Barry Song

This seems to be the most appropriate place for this file.

Signed-off-by: Lorenzo Stoakes <lorenzo.stoakes@oracle.com>
---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 837bc5cd6166..3f83fb317f51 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -15966,6 +15966,7 @@ F:	include/linux/khugepaged.h
 F:	include/trace/events/huge_memory.h
 F:	mm/huge_memory.c
 F:	mm/khugepaged.c
+F:	mm/mm_slot.h
 F:	tools/testing/selftests/mm/khugepaged.c
 F:	tools/testing/selftests/mm/split_huge_page_test.c
 F:	tools/testing/selftests/mm/transhuge-stress.c
-- 
2.50.1


^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [PATCH] MAINTAINERS: add missing mm_slot.h file THP section
  2025-07-22 17:19 [PATCH] MAINTAINERS: add missing mm_slot.h file THP section Lorenzo Stoakes
@ 2025-07-22 17:26 ` Zi Yan
  2025-07-22 17:43 ` Nico Pache
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 8+ messages in thread
From: Zi Yan @ 2025-07-22 17:26 UTC (permalink / raw)
  To: Lorenzo Stoakes
  Cc: Andrew Morton, linux-mm, linux-kernel, David Hildenbrand,
	Baolin Wang, Liam R . Howlett, Nico Pache, Ryan Roberts, Dev Jain,
	Barry Song

On 22 Jul 2025, at 13:19, Lorenzo Stoakes wrote:

> This seems to be the most appropriate place for this file.
>
> Signed-off-by: Lorenzo Stoakes <lorenzo.stoakes@oracle.com>
> ---
>  MAINTAINERS | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 837bc5cd6166..3f83fb317f51 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -15966,6 +15966,7 @@ F:	include/linux/khugepaged.h
>  F:	include/trace/events/huge_memory.h
>  F:	mm/huge_memory.c
>  F:	mm/khugepaged.c
> +F:	mm/mm_slot.h
>  F:	tools/testing/selftests/mm/khugepaged.c
>  F:	tools/testing/selftests/mm/split_huge_page_test.c
>  F:	tools/testing/selftests/mm/transhuge-stress.c
> -- 
> 2.50.1

Acked-by: Zi Yan <ziy@nvidia.com>

Best Regards,
Yan, Zi

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] MAINTAINERS: add missing mm_slot.h file THP section
  2025-07-22 17:19 [PATCH] MAINTAINERS: add missing mm_slot.h file THP section Lorenzo Stoakes
  2025-07-22 17:26 ` Zi Yan
@ 2025-07-22 17:43 ` Nico Pache
  2025-07-22 19:52   ` David Hildenbrand
                     ` (2 more replies)
  2025-07-23  4:53 ` Lorenzo Stoakes
  2025-07-23 15:11 ` Dev Jain
  3 siblings, 3 replies; 8+ messages in thread
From: Nico Pache @ 2025-07-22 17:43 UTC (permalink / raw)
  To: Lorenzo Stoakes
  Cc: Andrew Morton, linux-mm, linux-kernel, David Hildenbrand, Zi Yan,
	Baolin Wang, Liam R . Howlett, Ryan Roberts, Dev Jain, Barry Song

On Tue, Jul 22, 2025 at 11:19 AM Lorenzo Stoakes
<lorenzo.stoakes@oracle.com> wrote:
>
> This seems to be the most appropriate place for this file.
>
> Signed-off-by: Lorenzo Stoakes <lorenzo.stoakes@oracle.com>
> ---
>  MAINTAINERS | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 837bc5cd6166..3f83fb317f51 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -15966,6 +15966,7 @@ F:      include/linux/khugepaged.h
>  F:     include/trace/events/huge_memory.h
>  F:     mm/huge_memory.c
>  F:     mm/khugepaged.c
> +F:     mm/mm_slot.h
>  F:     tools/testing/selftests/mm/khugepaged.c
>  F:     tools/testing/selftests/mm/split_huge_page_test.c
>  F:     tools/testing/selftests/mm/transhuge-stress.c
> --
> 2.50.1
It might also be worth adding it to the KSM section since both of them
utilize it.

Acked-by: Nico Pache <npache@redhat.com>
>


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] MAINTAINERS: add missing mm_slot.h file THP section
  2025-07-22 17:43 ` Nico Pache
@ 2025-07-22 19:52   ` David Hildenbrand
  2025-07-23  3:50   ` Qi Zheng
  2025-07-23  4:48   ` Lorenzo Stoakes
  2 siblings, 0 replies; 8+ messages in thread
From: David Hildenbrand @ 2025-07-22 19:52 UTC (permalink / raw)
  To: Nico Pache, Lorenzo Stoakes
  Cc: Andrew Morton, linux-mm, linux-kernel, Zi Yan, Baolin Wang,
	Liam R . Howlett, Ryan Roberts, Dev Jain, Barry Song

On 22.07.25 19:43, Nico Pache wrote:
> On Tue, Jul 22, 2025 at 11:19 AM Lorenzo Stoakes
> <lorenzo.stoakes@oracle.com> wrote:
>>
>> This seems to be the most appropriate place for this file.
>>
>> Signed-off-by: Lorenzo Stoakes <lorenzo.stoakes@oracle.com>
>> ---
>>   MAINTAINERS | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 837bc5cd6166..3f83fb317f51 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -15966,6 +15966,7 @@ F:      include/linux/khugepaged.h
>>   F:     include/trace/events/huge_memory.h
>>   F:     mm/huge_memory.c
>>   F:     mm/khugepaged.c
>> +F:     mm/mm_slot.h
>>   F:     tools/testing/selftests/mm/khugepaged.c
>>   F:     tools/testing/selftests/mm/split_huge_page_test.c
>>   F:     tools/testing/selftests/mm/transhuge-stress.c
>> --
>> 2.50.1
> It might also be worth adding it to the KSM section since both of them
> utilize it.

Yes, good point

Acked-by: David Hildenbrand <david@redhat.com>

-- 
Cheers,

David / dhildenb


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] MAINTAINERS: add missing mm_slot.h file THP section
  2025-07-22 17:43 ` Nico Pache
  2025-07-22 19:52   ` David Hildenbrand
@ 2025-07-23  3:50   ` Qi Zheng
  2025-07-23  4:48   ` Lorenzo Stoakes
  2 siblings, 0 replies; 8+ messages in thread
From: Qi Zheng @ 2025-07-23  3:50 UTC (permalink / raw)
  To: Nico Pache, Lorenzo Stoakes
  Cc: Andrew Morton, linux-mm, linux-kernel, David Hildenbrand, Zi Yan,
	Baolin Wang, Liam R . Howlett, Ryan Roberts, Dev Jain, Barry Song



On 7/23/25 1:43 AM, Nico Pache wrote:
> On Tue, Jul 22, 2025 at 11:19 AM Lorenzo Stoakes
> <lorenzo.stoakes@oracle.com> wrote:
>>
>> This seems to be the most appropriate place for this file.
>>
>> Signed-off-by: Lorenzo Stoakes <lorenzo.stoakes@oracle.com>
>> ---
>>   MAINTAINERS | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 837bc5cd6166..3f83fb317f51 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -15966,6 +15966,7 @@ F:      include/linux/khugepaged.h
>>   F:     include/trace/events/huge_memory.h
>>   F:     mm/huge_memory.c
>>   F:     mm/khugepaged.c
>> +F:     mm/mm_slot.h
>>   F:     tools/testing/selftests/mm/khugepaged.c
>>   F:     tools/testing/selftests/mm/split_huge_page_test.c
>>   F:     tools/testing/selftests/mm/transhuge-stress.c
>> --
>> 2.50.1
> It might also be worth adding it to the KSM section since both of them
> utilize it.
> 

Agree, mm_slot is originally a common part extracted from THP and KSM,
and it is hoped that it can be reused by other modules in the future.

Acked-by: Qi Zheng <zhengqi.arch@bytedance.com>

Thanks.

> 


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] MAINTAINERS: add missing mm_slot.h file THP section
  2025-07-22 17:43 ` Nico Pache
  2025-07-22 19:52   ` David Hildenbrand
  2025-07-23  3:50   ` Qi Zheng
@ 2025-07-23  4:48   ` Lorenzo Stoakes
  2 siblings, 0 replies; 8+ messages in thread
From: Lorenzo Stoakes @ 2025-07-23  4:48 UTC (permalink / raw)
  To: Nico Pache
  Cc: Andrew Morton, linux-mm, linux-kernel, David Hildenbrand, Zi Yan,
	Baolin Wang, Liam R . Howlett, Ryan Roberts, Dev Jain, Barry Song

On Tue, Jul 22, 2025 at 11:43:04AM -0600, Nico Pache wrote:
> On Tue, Jul 22, 2025 at 11:19 AM Lorenzo Stoakes
> <lorenzo.stoakes@oracle.com> wrote:
> >
> > This seems to be the most appropriate place for this file.
> >
> > Signed-off-by: Lorenzo Stoakes <lorenzo.stoakes@oracle.com>
> > ---
> >  MAINTAINERS | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 837bc5cd6166..3f83fb317f51 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -15966,6 +15966,7 @@ F:      include/linux/khugepaged.h
> >  F:     include/trace/events/huge_memory.h
> >  F:     mm/huge_memory.c
> >  F:     mm/khugepaged.c
> > +F:     mm/mm_slot.h
> >  F:     tools/testing/selftests/mm/khugepaged.c
> >  F:     tools/testing/selftests/mm/split_huge_page_test.c
> >  F:     tools/testing/selftests/mm/transhuge-stress.c
> > --
> > 2.50.1
> It might also be worth adding it to the KSM section since both of them
> utilize it.

Good idea, will send a fix-patch to do so!

>
> Acked-by: Nico Pache <npache@redhat.com>

Thanks!

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] MAINTAINERS: add missing mm_slot.h file THP section
  2025-07-22 17:19 [PATCH] MAINTAINERS: add missing mm_slot.h file THP section Lorenzo Stoakes
  2025-07-22 17:26 ` Zi Yan
  2025-07-22 17:43 ` Nico Pache
@ 2025-07-23  4:53 ` Lorenzo Stoakes
  2025-07-23 15:11 ` Dev Jain
  3 siblings, 0 replies; 8+ messages in thread
From: Lorenzo Stoakes @ 2025-07-23  4:53 UTC (permalink / raw)
  To: Andrew Morton
  Cc: linux-mm, linux-kernel, David Hildenbrand, Zi Yan, Baolin Wang,
	Liam R . Howlett, Nico Pache, Ryan Roberts, Dev Jain, Barry Song

Hi Andrew,

This file also contains shared logic with KSM, so we ought to put the file
in the KSM section also. I include a fix-patch to do this.

Thanks, Lorenzo

----8<----
From 5ce64eec3514cc7e30afcd9529f73486008f6770 Mon Sep 17 00:00:00 2001
From: Lorenzo Stoakes <lorenzo.stoakes@oracle.com>
Date: Wed, 23 Jul 2025 05:50:08 +0100
Subject: [PATCH] MAINTAINERS: also add mm_slot.h to KSM section

The mm_slot implementation is in fact common between THP and KSM so ought
to be included in both sections.

Signed-off-by: Lorenzo Stoakes <lorenzo.stoakes@oracle.com>
---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 3f83fb317f51..a77c6677bdbd 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -15821,6 +15821,7 @@ F:	Documentation/mm/ksm.rst
 F:	include/linux/ksm.h
 F:	include/trace/events/ksm.h
 F:	mm/ksm.c
+F:	mm/mm_slot.h

 MEMORY MANAGEMENT - MEMORY POLICY AND MIGRATION
 M:	Andrew Morton <akpm@linux-foundation.org>
--
2.50.1

^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [PATCH] MAINTAINERS: add missing mm_slot.h file THP section
  2025-07-22 17:19 [PATCH] MAINTAINERS: add missing mm_slot.h file THP section Lorenzo Stoakes
                   ` (2 preceding siblings ...)
  2025-07-23  4:53 ` Lorenzo Stoakes
@ 2025-07-23 15:11 ` Dev Jain
  3 siblings, 0 replies; 8+ messages in thread
From: Dev Jain @ 2025-07-23 15:11 UTC (permalink / raw)
  To: Lorenzo Stoakes, Andrew Morton
  Cc: linux-mm, linux-kernel, David Hildenbrand, Zi Yan, Baolin Wang,
	Liam R . Howlett, Nico Pache, Ryan Roberts, Barry Song


On 22/07/25 10:49 pm, Lorenzo Stoakes wrote:
> This seems to be the most appropriate place for this file.
>
> Signed-off-by: Lorenzo Stoakes <lorenzo.stoakes@oracle.com>
> ---
>   

Acked-by: Dev Jain <dev.jain@arm.com>


^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2025-07-23 15:12 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-07-22 17:19 [PATCH] MAINTAINERS: add missing mm_slot.h file THP section Lorenzo Stoakes
2025-07-22 17:26 ` Zi Yan
2025-07-22 17:43 ` Nico Pache
2025-07-22 19:52   ` David Hildenbrand
2025-07-23  3:50   ` Qi Zheng
2025-07-23  4:48   ` Lorenzo Stoakes
2025-07-23  4:53 ` Lorenzo Stoakes
2025-07-23 15:11 ` Dev Jain

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).