On 11/13/25 06: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 <[email protected]> > --- > 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 | 4 ++-- > mm/show_mem.c | 9 ++++++--- > mm/vmscan.c | 6 +++--- > 11 files changed, 25 insertions(+), 21 deletions(-) > > diff --git a/include/linux/cpuset.h b/include/linux/cpuset.h > index 2ddb256187b5..548eaf7ef8d0 100644 > --- a/include/linux/cpuset.h > +++ b/include/linux/cpuset.h > @@ -80,7 +80,7 @@ extern bool cpuset_cpu_is_isolated(int cpu); > extern nodemask_t cpuset_mems_allowed(struct task_struct *p); > #define cpuset_current_mems_allowed (current->mems_allowed) > void cpuset_init_current_mems_allowed(void); > -int cpuset_nodemask_valid_mems_allowed(nodemask_t *nodemask); > +int cpuset_nodemask_valid_mems_allowed(const nodemask_t *nodemask); > > extern bool cpuset_current_node_allowed(int node, gfp_t gfp_mask); > > @@ -219,7 +219,7 @@ static inline nodemask_t cpuset_mems_allowed(struct > task_struct *p) > #define cpuset_current_mems_allowed (node_states[N_MEMORY]) > static inline void cpuset_init_current_mems_allowed(void) {}
The cleanup looks good Acked-by: Balbir Singh <[email protected]>
