[v1] Implement RCU Tasks Trace in terms of SRCU-fast and optimize

[PATCH 20/34] checkpatch: Deprecate rcu_read_{,un}lock_trace()

Posted by Paul E. McKenney 1 week, 1 day ago

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@kernel.org>
Cc: Andy Whitcroft <apw@canonical.com>
Cc: Joe Perches <joe@perches.com>
Cc: Dwaipayan Ray <dwaipayanray1@gmail.com>
Cc: Lukas Bulwahn <lukas.bulwahn@gmail.com>
Cc: Andrii Nakryiko <andrii@kernel.org>
Cc: Alexei Starovoitov <ast@kernel.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: <bpf@vger.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
-- 
2.40.1

Re: [PATCH 20/34] checkpatch: Deprecate rcu_read_{,un}lock_trace()

Posted by Joe Perches 1 week, 1 day ago

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@kernel.org>

Acked-by: Joe Perches <joe@perches.com>

> Cc: Andy Whitcroft <apw@canonical.com>
> Cc: Joe Perches <joe@perches.com>
> Cc: Dwaipayan Ray <dwaipayanray1@gmail.com>
> Cc: Lukas Bulwahn <lukas.bulwahn@gmail.com>
> Cc: Andrii Nakryiko <andrii@kernel.org>
> Cc: Alexei Starovoitov <ast@kernel.org>
> Cc: Peter Zijlstra <peterz@infradead.org>
> Cc: <bpf@vger.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

Re: [PATCH 20/34] checkpatch: Deprecate rcu_read_{,un}lock_trace()

Posted by Paul E. McKenney 1 week, 1 day ago

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@kernel.org>
> 
> Acked-by: Joe Perches <joe@perches.com>

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

							Thanx, Paul

> > Cc: Andy Whitcroft <apw@canonical.com>
> > Cc: Joe Perches <joe@perches.com>
> > Cc: Dwaipayan Ray <dwaipayanray1@gmail.com>
> > Cc: Lukas Bulwahn <lukas.bulwahn@gmail.com>
> > Cc: Andrii Nakryiko <andrii@kernel.org>
> > Cc: Alexei Starovoitov <ast@kernel.org>
> > Cc: Peter Zijlstra <peterz@infradead.org>
> > Cc: <bpf@vger.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