Commit d14d3f53 authored by Lorenzo Stoakes's avatar Lorenzo Stoakes Committed by Andrew Morton
Browse files

mm: convert remaining users to mm_flags_*() accessors 

As part of the effort to move to mm->flags becoming a bitmap field,
convert existing users to making use of the mm_flags_*() accessors which
will, when the conversion is complete, be the only means of accessing
mm_struct flags.

No functional change intended.

Link: https://lkml.kernel.org/r/cc67a56f9a8746a8ec7d9791853dc892c1c33e0b.1755012943.git.lorenzo.stoakes@oracle.com


Signed-off-by: default avatarLorenzo Stoakes <lorenzo.stoakes@oracle.com>
Reviewed-by: default avatarLiam R. Howlett <Liam.Howlett@oracle.com>
Reviewed-by: default avatarMike Rapoport (Microsoft) <rppt@kernel.org>
Acked-by: default avatarDavid Hildenbrand <david@redhat.com>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Alexander Gordeev <agordeev@linux.ibm.com>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Al Viro <viro@zeniv.linux.org.uk>
Cc: Andreas Larsson <andreas@gaisler.com>
Cc: Andy Lutomirski <luto@kernel.org>
Cc: Arnaldo Carvalho de Melo <acme@kernel.org>
Cc: Baolin Wang <baolin.wang@linux.alibaba.com>
Cc: Barry Song <baohua@kernel.org>
Cc: Ben Segall <bsegall@google.com>
Cc: Borislav Betkov <bp@alien8.de>
Cc: Chengming Zhou <chengming.zhou@linux.dev>
Cc: Christian Borntraeger <borntraeger@linux.ibm.com>
Cc: Christian Brauner <brauner@kernel.org>
Cc: David Rientjes <rientjes@google.com>
Cc: David S. Miller <davem@davemloft.net>
Cc: Dev Jain <dev.jain@arm.com>
Cc: Dietmar Eggemann <dietmar.eggemann@arm.com>
Cc: Gerald Schaefer <gerald.schaefer@linux.ibm.com>
Cc: Heiko Carstens <hca@linux.ibm.com>
Cc: "H. Peter Anvin" <hpa@zytor.com>
Cc: Ian Rogers <irogers@google.com>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: Jan Kara <jack@suse.cz>
Cc: Jann Horn <jannh@google.com>
Cc: Jason Gunthorpe <jgg@ziepe.ca>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: John Hubbard <jhubbard@nvidia.com>
Cc: Juri Lelli <juri.lelli@redhat.com>
Cc: Kan Liang <kan.liang@linux.intel.com>
Cc: Kees Cook <kees@kernel.org>
Cc: Marc Rutland <mark.rutland@arm.com>
Cc: Mariano Pache <npache@redhat.com>
Cc: "Masami Hiramatsu (Google)" <mhiramat@kernel.org>
Cc: Mateusz Guzik <mjguzik@gmail.com>
Cc: Matthew Wilcox (Oracle) <willy@infradead.org>
Cc: Mel Gorman <mgorman <mgorman@suse.de>
Cc: Michal Hocko <mhocko@suse.com>
Cc: Namhyung kim <namhyung@kernel.org>
Cc: Oleg Nesterov <oleg@redhat.com>
Cc: Peter Xu <peterx@redhat.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Ryan Roberts <ryan.roberts@arm.com>
Cc: Shakeel Butt <shakeel.butt@linux.dev>
Cc: Steven Rostedt <rostedt@goodmis.org>
Cc: Suren Baghdasaryan <surenb@google.com>
Cc: Sven Schnelle <svens@linux.ibm.com>
Cc: Thomas Gleinxer <tglx@linutronix.de>
Cc: Valentin Schneider <vschneid@redhat.com>
Cc: Vasily Gorbik <gor@linux.ibm.com>
Cc: Vincent Guittot <vincent.guittot@linaro.org>
Cc: Vlastimil Babka <vbabka@suse.cz>
Cc: xu xin <xu.xin16@zte.com.cn>
Cc: Zi Yan <ziy@nvidia.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
parent 19148a19

fs/proc/array.c

+1 −1
Original line number Diff line number Diff line
@@ -422,7 +422,7 @@ static inline void task_thp_status(struct seq_file *m, struct mm_struct *mm) 
	bool thp_enabled = IS_ENABLED(CONFIG_TRANSPARENT_HUGEPAGE);



	if (thp_enabled)

		thp_enabled = !test_bit(MMF_DISABLE_THP, &mm->flags);

		thp_enabled = !mm_flags_test(MMF_DISABLE_THP, mm);

	seq_printf(m, "THP_enabled:\t%d\n", thp_enabled);

}

fs/proc/base.c

+2 −2
Original line number Diff line number Diff line
@@ -1163,7 +1163,7 @@ static int __set_oom_adj(struct file *file, int oom_adj, bool legacy) 
		struct task_struct *p = find_lock_task_mm(task);



		if (p) {

			if (test_bit(MMF_MULTIPROCESS, &p->mm->flags)) {

			if (mm_flags_test(MMF_MULTIPROCESS, p->mm)) {

				mm = p->mm;

				mmgrab(mm);

			}
@@ -3276,7 +3276,7 @@ static int proc_pid_ksm_stat(struct seq_file *m, struct pid_namespace *ns, 
		seq_printf(m, "ksm_merging_pages %lu\n", mm->ksm_merging_pages);

		seq_printf(m, "ksm_process_profit %ld\n", ksm_process_profit(mm));

		seq_printf(m, "ksm_merge_any: %s\n",

				test_bit(MMF_VM_MERGE_ANY, &mm->flags) ? "yes" : "no");

				mm_flags_test(MMF_VM_MERGE_ANY, mm) ? "yes" : "no");

		ret = mmap_read_lock_killable(mm);

		if (ret) {

			mmput(mm);

fs/proc/task_mmu.c

+1 −1
Original line number Diff line number Diff line
@@ -1592,7 +1592,7 @@ static inline bool pte_is_pinned(struct vm_area_struct *vma, unsigned long addr, 
		return false;

	if (!is_cow_mapping(vma->vm_flags))

		return false;

	if (likely(!test_bit(MMF_HAS_PINNED, &vma->vm_mm->flags)))

	if (likely(!mm_flags_test(MMF_HAS_PINNED, vma->vm_mm)))

		return false;

	folio = vm_normal_folio(vma, addr, pte);

	if (!folio)

kernel/fork.c

+1 −1
Original line number Diff line number Diff line
@@ -1887,7 +1887,7 @@ static void copy_oom_score_adj(u64 clone_flags, struct task_struct *tsk) 


	/* We need to synchronize with __set_oom_adj */

	mutex_lock(&oom_adj_mutex);

	set_bit(MMF_MULTIPROCESS, &tsk->mm->flags);

	mm_flags_set(MMF_MULTIPROCESS, tsk->mm);

	/* Update the values in case they were changed after copy_signal */

	tsk->signal->oom_score_adj = current->signal->oom_score_adj;

	tsk->signal->oom_score_adj_min = current->signal->oom_score_adj_min;