linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] MAINTAINERS: add missing files to slab section
@ 2025-07-22 17:59 Lorenzo Stoakes
  2025-07-23 10:07 ` Vlastimil Babka
  0 siblings, 1 reply; 3+ messages in thread
From: Lorenzo Stoakes @ 2025-07-22 17:59 UTC (permalink / raw)
  To: Vlastimil Babka
  Cc: linux-mm, linux-kernel, Christoph Lameter, David Rientjes,
	Roman Gushchin, Harry Yoo

The failslab implementation implements fault injection for slab allocations
so seems best suited to the slab section.

The mempool implementation uses slab in the backend, so that seems best
suited here also.

Signed-off-by: Lorenzo Stoakes <lorenzo.stoakes@oracle.com>
---
NOTE:

To try to reduce conflicts with the latest batch of MAINTAINERS changes, I
think this is best merged through the slab tree.

 MAINTAINERS | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index e55accbef..2d5c844ab 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -15738,7 +15738,6 @@ F:	include/linux/memfd.h
 F:	include/linux/memory_hotplug.h
 F:	include/linux/memory-tiers.h
 F:	include/linux/mempolicy.h
-F:	include/linux/mempool.h
 F:	include/linux/memremap.h
 F:	include/linux/mmzone.h
 F:	include/linux/mmu_notifier.h
@@ -22831,7 +22830,10 @@ S:	Maintained
 T:	git git://git.kernel.org/pub/scm/linux/kernel/git/vbabka/slab.git
 F:	Documentation/admin-guide/mm/slab.rst
 F:	Documentation/mm/slab.rst
+F:	include/linux/mempool.h
 F:	include/linux/slab.h
+F:	mm/failslab.c
+F:	mm/mempool.c
 F:	mm/slab.h
 F:	mm/slab_common.c
 F:	mm/slub.c
--
2.50.1


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

* Re: [PATCH] MAINTAINERS: add missing files to slab section
  2025-07-22 17:59 [PATCH] MAINTAINERS: add missing files to slab section Lorenzo Stoakes
@ 2025-07-23 10:07 ` Vlastimil Babka
  2025-07-23 10:16   ` Lorenzo Stoakes
  0 siblings, 1 reply; 3+ messages in thread
From: Vlastimil Babka @ 2025-07-23 10:07 UTC (permalink / raw)
  To: Lorenzo Stoakes
  Cc: linux-mm, linux-kernel, Christoph Lameter, David Rientjes,
	Roman Gushchin, Harry Yoo

On 7/22/25 19:59, Lorenzo Stoakes wrote:
> The failslab implementation implements fault injection for slab allocations
> so seems best suited to the slab section.
> 
> The mempool implementation uses slab in the backend, so that seems best
> suited here also.

It has other backends too but perhaps slab is the most complex and David was
involved so seems acceptable.

> 
> Signed-off-by: Lorenzo Stoakes <lorenzo.stoakes@oracle.com>
> ---
> NOTE:
> 
> To try to reduce conflicts with the latest batch of MAINTAINERS changes, I
> think this is best merged through the slab tree.
> 
>  MAINTAINERS | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index e55accbef..2d5c844ab 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -15738,7 +15738,6 @@ F:	include/linux/memfd.h
>  F:	include/linux/memory_hotplug.h
>  F:	include/linux/memory-tiers.h
>  F:	include/linux/mempolicy.h
> -F:	include/linux/mempool.h
>  F:	include/linux/memremap.h
>  F:	include/linux/mmzone.h
>  F:	include/linux/mmu_notifier.h

I'll leave out this hunk as your other patch moves the header entry from MM
to MM-MISC (maybe it could be removing it there while at it?).

> @@ -22831,7 +22830,10 @@ S:	Maintained
>  T:	git git://git.kernel.org/pub/scm/linux/kernel/git/vbabka/slab.git
>  F:	Documentation/admin-guide/mm/slab.rst
>  F:	Documentation/mm/slab.rst
> +F:	include/linux/mempool.h
>  F:	include/linux/slab.h
> +F:	mm/failslab.c
> +F:	mm/mempool.c
>  F:	mm/slab.h
>  F:	mm/slab_common.c
>  F:	mm/slub.c
> --
> 2.50.1



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

* Re: [PATCH] MAINTAINERS: add missing files to slab section
  2025-07-23 10:07 ` Vlastimil Babka
@ 2025-07-23 10:16   ` Lorenzo Stoakes
  0 siblings, 0 replies; 3+ messages in thread
From: Lorenzo Stoakes @ 2025-07-23 10:16 UTC (permalink / raw)
  To: Vlastimil Babka
  Cc: linux-mm, linux-kernel, Christoph Lameter, David Rientjes,
	Roman Gushchin, Harry Yoo

On Wed, Jul 23, 2025 at 12:07:39PM +0200, Vlastimil Babka wrote:
> On 7/22/25 19:59, Lorenzo Stoakes wrote:
> > The failslab implementation implements fault injection for slab allocations
> > so seems best suited to the slab section.
> >
> > The mempool implementation uses slab in the backend, so that seems best
> > suited here also.
>
> It has other backends too but perhaps slab is the most complex and David was
> involved so seems acceptable.

Thanks

>
> >
> > Signed-off-by: Lorenzo Stoakes <lorenzo.stoakes@oracle.com>
> > ---
> > NOTE:
> >
> > To try to reduce conflicts with the latest batch of MAINTAINERS changes, I
> > think this is best merged through the slab tree.
> >
> >  MAINTAINERS | 4 +++-
> >  1 file changed, 3 insertions(+), 1 deletion(-)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index e55accbef..2d5c844ab 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -15738,7 +15738,6 @@ F:	include/linux/memfd.h
> >  F:	include/linux/memory_hotplug.h
> >  F:	include/linux/memory-tiers.h
> >  F:	include/linux/mempolicy.h
> > -F:	include/linux/mempool.h
> >  F:	include/linux/memremap.h
> >  F:	include/linux/mmzone.h
> >  F:	include/linux/mmu_notifier.h
>
> I'll leave out this hunk as your other patch moves the header entry from MM
> to MM-MISC (maybe it could be removing it there while at it?).

OK let's just fix this up later potentially, I don't want to create more
confusion at this stage :P

I guess with this being potentially used elsewhere, this actually makes sense.

>
> > @@ -22831,7 +22830,10 @@ S:	Maintained
> >  T:	git git://git.kernel.org/pub/scm/linux/kernel/git/vbabka/slab.git
> >  F:	Documentation/admin-guide/mm/slab.rst
> >  F:	Documentation/mm/slab.rst
> > +F:	include/linux/mempool.h
> >  F:	include/linux/slab.h
> > +F:	mm/failslab.c
> > +F:	mm/mempool.c
> >  F:	mm/slab.h
> >  F:	mm/slab_common.c
> >  F:	mm/slub.c
> > --
> > 2.50.1
>


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

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

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-07-22 17:59 [PATCH] MAINTAINERS: add missing files to slab section Lorenzo Stoakes
2025-07-23 10:07 ` Vlastimil Babka
2025-07-23 10:16   ` Lorenzo Stoakes

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox