Re: [PATCH] mm: constify oom_control, scan_control, and alloc_context nodemask

[Lorenzo Stoakes]

On Mon, Jun 08, 2026 at 09:44:42PM -0400, Zi Yan wrote:
> On 8 Jun 2026, at 20:29, Gregory Price wrote:
>
> > The nodemasks in these structures may come from a variety of sources,
> > including tasks and cpusets - and should never be modified by any code
> > when being passed around inside another context.
> >
> > Signed-off-by: Gregory Price <gourry@xxxxxxxxxx>
> > ---
> >  include/linux/cpuset.h | 4 ++--
> >  include/linux/mm.h     | 4 ++--
> >  include/linux/mmzone.h | 6 +++---
> >  include/linux/oom.h    | 2 +-
> >  include/linux/swap.h   | 2 +-
> >  kernel/cgroup/cpuset.c | 2 +-
> >  mm/internal.h          | 2 +-
> >  mm/mmzone.c            | 5 +++--
> >  mm/page_alloc.c        | 6 +++---
> >  mm/show_mem.c          | 9 ++++++---
> >  mm/vmscan.c            | 6 +++---
> >  11 files changed, 26 insertions(+), 22 deletions(-)
> >
>
> LGTM and it compiles. As long as Sashiko does not complain, feel free to
> add:

I would add caveats of:

- Complains legitimately
- And it's about this actual patch not something unrelated

:P

(Not speaking for Zi of course, but I mean just in general I feel these caveats
should be implicit :))

>
> Acked-by: Zi Yan <ziy@xxxxxxxxxx>
>
> Best Regards,
> Yan, Zi

Cheers, Lorenzo