linux-kernel - Re: [PATCH 20/34] checkpatch: Deprecate rcu_read_{,un}lock

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <6777d949-e1f7-454d-8c10-247dcbf8c6fa@paulmck-laptop>
Date: Tue, 23 Sep 2025 10:01:06 -0700
From: "Paul E. McKenney" <paulmck@...nel.org>
To: Joe Perches <joe@...ches.com>
Cc: rcu@...r.kernel.org, Andrew Morton <akpm@...ux-foundation.org>,
	linux-kernel@...r.kernel.org, kernel-team@...a.com,
	rostedt@...dmis.org, Andy Whitcroft <apw@...onical.com>,
	Dwaipayan Ray <dwaipayanray1@...il.com>,
	Lukas Bulwahn <lukas.bulwahn@...il.com>,
	Andrii Nakryiko <andrii@...nel.org>,
	Alexei Starovoitov <ast@...nel.org>,
	Peter Zijlstra <peterz@...radead.org>, bpf@...r.kernel.org
Subject: Re: [PATCH 20/34] checkpatch: Deprecate rcu_read_{,un}lock_trace()

On Tue, Sep 23, 2025 at 08:47:13AM -0700, Joe Perches wrote:
> On Tue, 2025-09-23 at 07:20 -0700, Paul E. McKenney wrote:
> > Uses of rcu_read_lock_trace() and rcu_read_unlock_trace()
> > are better served by the new rcu_read_lock_tasks_trace() and
> > rcu_read_unlock_tasks_trace() APIs.  Therefore, mark the old APIs as
> > deprecated.
> > 
> > Signed-off-by: Paul E. McKenney <paulmck@...nel.org>
> 
> Acked-by: Joe Perches <joe@...ches.com>

Thank you, and I will apply on my next rebase.

							Thanx, Paul

> > Cc: Andy Whitcroft <apw@...onical.com>
> > Cc: Joe Perches <joe@...ches.com>
> > Cc: Dwaipayan Ray <dwaipayanray1@...il.com>
> > Cc: Lukas Bulwahn <lukas.bulwahn@...il.com>
> > Cc: Andrii Nakryiko <andrii@...nel.org>
> > Cc: Alexei Starovoitov <ast@...nel.org>
> > Cc: Peter Zijlstra <peterz@...radead.org>
> > Cc: <bpf@...r.kernel.org>
> > ---
> >  scripts/checkpatch.pl | 2 ++
> >  1 file changed, 2 insertions(+)
> > 
> > diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
> > index e722dd6fa8ef3d..3bb7d35a5cfcba 100755
> > --- a/scripts/checkpatch.pl
> > +++ b/scripts/checkpatch.pl
> > @@ -860,6 +860,8 @@ our %deprecated_apis = (
> >  	"kunmap"				=> "kunmap_local",
> >  	"kmap_atomic"				=> "kmap_local_page",
> >  	"kunmap_atomic"				=> "kunmap_local",
> > +	"rcu_read_lock_trace"			=> "rcu_read_lock_tasks_trace",
> > +	"rcu_read_unlock_trace"			=> "rcu_read_unlock_tasks_trace",
> >  );
> >  
> >  #Create a search pattern for all these strings to speed up a loop below