All of lore.kernel.org
 help / color / mirror / Atom feed
From: Kees Cook <kees@kernel.org>
To: Lorenzo Stoakes <lorenzo.stoakes@oracle.com>
Cc: Andrew Morton <akpm@linux-foundation.org>,
	Ingo Molnar <mingo@redhat.com>,
	Peter Zijlstra <peterz@infradead.org>,
	Juri Lelli <juri.lelli@redhat.com>,
	Vincent Guittot <vincent.guittot@linaro.org>,
	Dietmar Eggemann <dietmar.eggemann@arm.com>,
	Steven Rostedt <rostedt@goodmis.org>,
	Ben Segall <bsegall@google.com>, Mel Gorman <mgorman@suse.de>,
	Valentin Schneider <vschneid@redhat.com>,
	David Hildenbrand <david@redhat.com>,
	"Liam R . Howlett" <Liam.Howlett@oracle.com>,
	Vlastimil Babka <vbabka@suse.cz>, Mike Rapoport <rppt@kernel.org>,
	Suren Baghdasaryan <surenb@google.com>,
	Michal Hocko <mhocko@suse.com>,
	linux-kernel@vger.kernel.org, linux-mm@kvack.org,
	Christian Brauner <christian@brauner.io>
Subject: Re: [RFC PATCH] MAINTAINERS: add kernel/fork.c to relevant sections
Date: Thu, 8 May 2025 09:11:35 -0700	[thread overview]
Message-ID: <202505080911.FB6E102@keescook> (raw)
In-Reply-To: <f0bdf898-42bd-48ae-89a2-8f93b49d071f@lucifer.local>

On Thu, May 08, 2025 at 04:33:33PM +0100, Lorenzo Stoakes wrote:
> +cc Christian
> 
> And Murphy's law dictates that I forget to add the pidfd bit. Fix-patch
> included...
> 
> Adding pidfd as there's a fair chunk of pidfd code that lives in
> kernel/fork.c. Obviously let me know Christian if you feel this makes
> sense.
> 
> Thanks! And apologies for noise...
> 
> On Thu, May 08, 2025 at 04:28:25PM +0100, Lorenzo Stoakes wrote:
> > Currently kernel/fork.c both contains absolutely key logic relating to a
> > number of kernel subsystems and also has absolutely no assignment in
> > MAINTAINERS.
> >
> > Correct this by placing this file in relevant sections - mm core, exec,
> > scheduler and pidfd so people know who to contact when making changes here.
> >
> > scripts/get_maintainers.pl can perfectly well handle a file being in
> > multiple sections, so this functions correctly.
> >
> > Intent is that we keep putting changes to kernel/fork.c through Andrew's
> > tree.
> >
> > Signed-off-by: Lorenzo Stoakes <lorenzo.stoakes@oracle.com>
> > ---
> >  MAINTAINERS | 3 +++
> >  1 file changed, 3 insertions(+)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index ccc45b0ba843..55332d5bc499 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -8830,6 +8830,7 @@ F:	include/linux/elf.h
> >  F:	include/uapi/linux/auxvec.h
> >  F:	include/uapi/linux/binfmts.h
> >  F:	include/uapi/linux/elf.h
> > +F:	kernel/fork.c
> >  F:	mm/vma_exec.c
> >  F:	tools/testing/selftests/exec/
> >  N:	asm/elf.h
> > @@ -15525,6 +15526,7 @@ F:	include/linux/mm.h
> >  F:	include/linux/mm_*.h
> >  F:	include/linux/mmdebug.h
> >  F:	include/linux/pagewalk.h
> > +F:	kernel/fork.c
> >  F:	mm/Kconfig
> >  F:	mm/debug.c
> >  F:	mm/init-mm.c
> > @@ -21742,6 +21744,7 @@ F:	include/linux/preempt.h
> >  F:	include/linux/sched.h
> >  F:	include/linux/wait.h
> >  F:	include/uapi/linux/sched.h
> > +F:	kernel/fork.c
> >  F:	kernel/sched/
> >
> >  SCHEDULER - SCHED_EXT
> > --
> > 2.49.0
> >
> 
> ----8<----
> From 6ab14cfd38a34fc097207dd6b898004a87b9cfa2 Mon Sep 17 00:00:00 2001
> From: Lorenzo Stoakes <lorenzo.stoakes@oracle.com>
> Date: Thu, 8 May 2025 16:30:58 +0100
> Subject: [PATCH] add missing pidfd entry
> 
> Signed-off-by: Lorenzo Stoakes <lorenzo.stoakes@oracle.com>

Yes, please. :) Thanks!

Reviewed-by: Kees Cook <kees@kernel.org>

-- 
Kees Cook


  reply	other threads:[~2025-05-08 16:11 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-05-08 15:28 [RFC PATCH] MAINTAINERS: add kernel/fork.c to relevant sections Lorenzo Stoakes
2025-05-08 15:33 ` Lorenzo Stoakes
2025-05-08 16:11   ` Kees Cook [this message]
2025-05-08 15:38 ` David Hildenbrand
2025-05-12  7:41 ` Vlastimil Babka
2025-05-12  7:55 ` Michal Hocko
2025-05-12 13:04 ` Peter Zijlstra
2025-05-12 13:16   ` Lorenzo Stoakes

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=202505080911.FB6E102@keescook \
    --to=kees@kernel.org \
    --cc=Liam.Howlett@oracle.com \
    --cc=akpm@linux-foundation.org \
    --cc=bsegall@google.com \
    --cc=christian@brauner.io \
    --cc=david@redhat.com \
    --cc=dietmar.eggemann@arm.com \
    --cc=juri.lelli@redhat.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=lorenzo.stoakes@oracle.com \
    --cc=mgorman@suse.de \
    --cc=mhocko@suse.com \
    --cc=mingo@redhat.com \
    --cc=peterz@infradead.org \
    --cc=rostedt@goodmis.org \
    --cc=rppt@kernel.org \
    --cc=surenb@google.com \
    --cc=vbabka@suse.cz \
    --cc=vincent.guittot@linaro.org \
    --cc=vschneid@redhat.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.