[PATCH 0/2] change "a mm_struct" to "an mm_struct" in comment

Joey Pabalinas posted 2 patches 4 weeks ago
arch/hexagon/include/asm/mmu.h | 2 +-
kernel/fork.c                  | 6 +++---
2 files changed, 4 insertions(+), 4 deletions(-)
[PATCH 0/2] change "a mm_struct" to "an mm_struct" in comment
Posted by Joey Pabalinas 4 weeks ago
The correct english is "an mm_struct" which is used everywhere else
in the kernel.

Joey Pabalinas (2):
  Hexagon: change "a mm_struct" to "an mm_struct" in comment
  fork: change "a mm_struct" to "an mm_struct" in comment

 arch/hexagon/include/asm/mmu.h | 2 +-
 kernel/fork.c                  | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

-- 
Cheers,
Joey Pabalinas
Re: [PATCH 0/2] change "a mm_struct" to "an mm_struct" in comment
Posted by Lorenzo Stoakes 4 weeks ago
On Wed, Sep 03, 2025 at 08:23:24PM -1000, Joey Pabalinas wrote:
> The correct english is "an mm_struct" which is used everywhere else
> in the kernel.

Hi Joey, this doesn't really warrant a patch, it's just not important and there
isn't great RoI on the churcn here. It is also very silly to make this two
patches, it should only be one.

Also please do run scripts/get_maintainers.pl when you're doing this, Kees is on
the list but he is joined by many esteemed kernel colleagues ;))

Ingo Molnar <mingo@redhat.com> (maintainer:SCHEDULER)
Peter Zijlstra <peterz@infradead.org> (maintainer:SCHEDULER)
Juri Lelli <juri.lelli@redhat.com> (maintainer:SCHEDULER)
Vincent Guittot <vincent.guittot@linaro.org> (maintainer:SCHEDULER)
Dietmar Eggemann <dietmar.eggemann@arm.com> (reviewer:SCHEDULER)
Steven Rostedt <rostedt@goodmis.org> (reviewer:SCHEDULER)
Ben Segall <bsegall@google.com> (reviewer:SCHEDULER)
Mel Gorman <mgorman@suse.de> (reviewer:SCHEDULER)
Valentin Schneider <vschneid@redhat.com> (reviewer:SCHEDULER)
Kees Cook <kees@kernel.org> (maintainer:EXEC & BINFMT API, ELF)
Andrew Morton <akpm@linux-foundation.org> (maintainer:MEMORY MANAGEMENT - CORE)
David Hildenbrand <david@redhat.com> (maintainer:MEMORY MANAGEMENT - CORE)
Lorenzo Stoakes <lorenzo.stoakes@oracle.com> (reviewer:MEMORY MANAGEMENT - CORE)
"Liam R. Howlett" <Liam.Howlett@oracle.com> (reviewer:MEMORY MANAGEMENT - CORE)
Vlastimil Babka <vbabka@suse.cz> (reviewer:MEMORY MANAGEMENT - CORE)
Mike Rapoport <rppt@kernel.org> (reviewer:MEMORY MANAGEMENT - CORE)
Suren Baghdasaryan <surenb@google.com> (reviewer:MEMORY MANAGEMENT - CORE)
Michal Hocko <mhocko@suse.com> (reviewer:MEMORY MANAGEMENT - CORE)
Brian Cain <bcain@kernel.org> (maintainer:QUALCOMM HEXAGON ARCHITECTURE)
linux-kernel@vger.kernel.org (open list:SCHEDULER)
linux-mm@kvack.org (open list:EXEC & BINFMT API, ELF)
linux-hexagon@vger.kernel.org (open list:QUALCOMM HEXAGON ARCHITECTURE)
EXEC & BINFMT API, ELF status: Supported
QUALCOMM HEXAGON ARCHITECTURE status: Supported

>
> Joey Pabalinas (2):
>   Hexagon: change "a mm_struct" to "an mm_struct" in comment
>   fork: change "a mm_struct" to "an mm_struct" in comment
>
>  arch/hexagon/include/asm/mmu.h | 2 +-
>  kernel/fork.c                  | 6 +++---
>  2 files changed, 4 insertions(+), 4 deletions(-)
>
> --
> Cheers,
> Joey Pabalinas
>
>
>

So yeah I don't think we should take this, sorry! :(

Cheers, Lorenzo
Re: [PATCH 0/2] change "a mm_struct" to "an mm_struct" in comment
Posted by Lorenzo Stoakes 4 weeks ago
On Thu, Sep 04, 2025 at 11:46:52AM +0100, Lorenzo Stoakes wrote:
> isn't great RoI on the churcn here. It is also very silly to make this two

Oops typo - should be "isn't great RoI on the _churn_ here" :)