From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mta1.migadu.com (out-166.mta1.migadu.com [95.215.58.166]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id F3320468C31 for ; Mon, 21 Sep 2026 09:21:21 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=95.215.58.166 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789982484; cv=none; b=RIbIFOLXLZbAOGw8lPdarKEZZ2z5E9jcst/D3VWsnc6CQ5KdF/xH767EhWR1ebktpWtbqu5u5ERTi0DhUadktNr048ig8yA3NDBqdLDbpWsKnVWa35J1Uo6gQ21vPoYFfuJEVqTrdArNazBUp9tpu8kkJnB/kMB7JaxOpjcQla0= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789982484; c=relaxed/simple; bh=SoD6rbNlmdBhL/S9+WOTUPuzd+nRVogzcLA/JJFQNZM=; h=From:To:Cc:Subject:Date:Message-Id:MIME-Version:Content-Type; b=dLcm78XhmiT/kN7qPtJ3+qp6dPSxic71RCdTf7i1WYVqGp8F42vCpytx7iPzJtyvxvoWtEmA9Vh0QbpdQGOPytIN7vld5cFQRqNbsArj3yBjtlbJ6blk1h+Y9s0tp7WuyBX5G6khDRevxZcoGa797/O5YZFX3IhVQiWb3+zv3us= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.dev; spf=pass smtp.mailfrom=linux.dev; dkim=pass (1024-bit key) header.d=linux.dev header.i=@linux.dev header.b=bPUBDz2V; arc=none smtp.client-ip=95.215.58.166 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.dev Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linux.dev Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linux.dev header.i=@linux.dev header.b="bPUBDz2V" X-Envelope-To: linux-kernel@vger.kernel.org DKIM-Signature: a=rsa-sha256; bh=SoD6rbNlmdBhL/S9+WOTUPuzd+nRVogzcLA/JJFQNZM=; c=simple/simple; d=linux.dev; h=from:to:subject:date:message-id:mime-version:content-type; s=key1; t=1789982480; v=1; x=1790587280; b=bPUBDz2VIyGxyOartNajvdI+c+JjY7hBzrcjVlofzPUL37v9FEcKoJqURJ+2UuSja2hv7j/h PmlAHbUFkOIMZTdyjDVpYocVAyStm7FniK73fDL63jeUrpUu/tN6Glsm7osTxofWvzw2iwtdgsy UEvtC8siMo/2TMxAN9LtXYvg= X-Envelope-To: linux-kernel@vger.kernel.org Received: by smtp.migadu.com with ESMTPS id b70590be7f446065; Mon, 21 Sep 2026 09:21:19 +0000 X-Mizu-Trace-ID: b70590be7f446065 X-Migadu-Flow: FLOW_OUT From: Ye Liu To: Andrew Morton , Michal Hocko , Peter Zijlstra , "Paul E. McKenney" , Ingo Molnar , Steven Rostedt , Josh Poimboeuf , =?UTF-8?q?Micka=C3=ABl=20Sala=C3=BCn?= , Oleg Nesterov , Thomas Gleixner Cc: Ye Liu , David Hildenbrand , Miaohe Lin , Naoya Horiguchi , "Rafael J. Wysocki" , =?UTF-8?q?G=C3=BCnther=20Noack?= , Lorenzo Stoakes , SJ Park , Gregory Price , linux-mm@kvack.org, linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org, rcu@vger.kernel.org, linux-trace-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-security-module@vger.kernel.org, rientjes@google.com, shakeel.butt@linux.dev, xu.xin16@zte.com.cn, chengming.zhou@linux.dev, will@kernel.org, boqun@kernel.org, frederic@kernel.org, neeraj.upadhyay@kernel.org, joelagnelf@nvidia.com, josh@joshtriplett.org, urezki@gmail.com, juri.lelli@redhat.com, vincent.guittot@linaro.org, tony.luck@intel.com, reinette.chatre@intel.com, x86@kernel.org, paul@paul-moore.com, jmorris@namei.org, serge@hallyn.com, pavel@kernel.org, lance.yang@linux.dev, mhiramat@kernel.org, pmladek@suse.com, longman@redhat.com, mathieu.desnoyers@efficios.com, jiangshanlai@gmail.com, qiang.zhang@linux.dev, dietmar.eggemann@arm.com, bsegall@google.com, mgorman@suse.de, vschneid@redhat.com, kprateek.nayak@amd.com, mark.rutland@arm.com, Dave.Martin@arm.com, james.morse@arm.com, babu.moger@amd.com Subject: [PATCH v4 00/15] sched: introduce for_each_process_rculock and for_each_thread_rculock Date: Mon, 21 Sep 2026 17:20:46 +0800 Message-Id: <20260921092101.89285-1-ye.liu@linux.dev> X-Mailer: git-send-email 2.25.1 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit From: Ye Liu Introduce for_each_process_rculock(), for_each_thread_rculock() and for_each_process_thread_rculock() macros that combine the existing iteration macros with scoped_guard(rcu), so that the RCU read lock is automatically acquired before iteration and released when the loop exits — including via break, goto, or return. The rest of the series converts manual rcu_read_lock()/ rcu_read_unlock() and guard(rcu)() pairs across mm/, kernel/, fs/, lib/ and security/ to use the new macros. Suggested by Michal Hocko for the oom_kill path [2]. Signed-off-by: Ye Liu --- Changes since v3 [4]: - Fix patch 5 (cpu/hotplug): title and commit message incorrectly said "thread iterator" and "for_each_thread_rculock" while the code converts for_each_process() to for_each_process_rculock(). Note: Patches 4, 11, and 13 received automated reviews from sashiko-bot in the v3 thread [4] that were verified as false positives: - Patch 4: claimed the folio lock is dropped before collect_procs(), leading to a NULL deref on folio->mapping; the lock is held in all call paths. - Patch 11: claimed mixing goto/scope-based cleanup in alloc_retstack_tasklist(); the goto free label handles memory cleanup, not lock management. - Patch 13: claimed mixing goto/scope-based cleanup in __set_oom_adj(); goto err_unlock manages oom_adj_mutex while scoped_guard(rcu) manages the RCU read lock — different resources. Changes since v2 [3]: - Split the kernel/ patch into per-subsystem patches, as the combined patch could not be applied (Peter Zijlstra, Steven Rostedt). - Move changelog below the --- separator so it is excluded from the commit message (Lorenzo Stoakes). - Indent loop body in macro definitions to show scoped_guard scope (Lorenzo Stoakes). Changes since v1 [1]: - Rename macros from *_rcu to *_rculock, as suggested by Steven Rostedt and acked by Thomas Gleixner, to avoid confusion with existing *_rcu() list iterators that expect the caller to hold the RCU read lock. - Improve the comment on for_each_process_thread_rculock() to document that 'break' only exits the inner loop and 'goto' is needed to exit both loops (Thomas Gleixner). - Rename the stale 'unlock:' label to 'out:' in hung_task.c, as noted by Günther Noack. - Clarify in patch 4 that page_pgoff() is safe outside the RCU read-side critical section (SJ Park). - CC all relevant maintainers on every patch (Lorenzo Stoakes). - Drop the mm: prefix from patch 1, as the macros are in include/linux/sched/signal.h (Michal Hocko). [1] https://lore.kernel.org/all/20260904083001.553587-1-ye.liu@linux.dev/ [2] https://lore.kernel.org/all/20260813092933.562028-1-ye.liu@linux.dev/ [3] https://lore.kernel.org/all/20260907081334.1152889-1-ye.liu@linux.dev/ [4] https://lore.kernel.org/all/20260911075800.491472-1-ye.liu@linux.dev/ Ye Liu (15): sched: introduce for_each_process_rculock and for_each_thread_rculock mm/oom_kill: convert process/thread iterators to for_each_*_rculock mm/ksm: convert process iterator to for_each_process_rculock mm/memory-failure: convert process iterator to for_each_process_rculock cpu/hotplug: convert process iterator to for_each_process_rculock freezer: convert thread iterator to for_each_thread_rculock hung_task: convert process/thread iterators to for_each_*_rculock locking/lockdep: convert process/thread iterators to for_each_*_rculock rcu: convert process/thread iterator to for_each_process_thread_rculock sched: convert process/thread iterators to for_each_*_rculock tracing/fgraph: convert process/thread iterator to for_each_process_thread_rculock unwind: convert process/thread iterator to for_each_process_thread_rculock fs: convert process/thread iterators to for_each_*_rculock lib: convert process iterator to for_each_process_rculock security/landlock: convert thread iterator to for_each_thread_rculock fs/proc/base.c | 4 +--- fs/resctrl/rdtgroup.c | 8 ++------ include/linux/sched/signal.h | 26 ++++++++++++++++++++++++++ kernel/cpu.c | 4 +--- kernel/freezer.c | 4 +--- kernel/hung_task.c | 11 ++++------- kernel/locking/lockdep.c | 4 +--- kernel/rcu/update.c | 4 +--- kernel/sched/core.c | 3 +-- kernel/sched/debug.c | 4 +--- kernel/trace/fgraph.c | 8 ++------ kernel/unwind/deferred.c | 3 +-- lib/is_single_threaded.c | 5 +---- mm/ksm.c | 4 +--- mm/memory-failure.c | 16 ++++------------ mm/oom_kill.c | 20 +++++--------------- security/landlock/tsync.c | 8 ++------ 17 files changed, 55 insertions(+), 81 deletions(-) -- 2.25.1