Re: [PATCH v2 10/12] checkpatch: Deprecate srcu_read_lock_lite() and srcu_read_unlock_lite()

Joe Perches
2025-04-01T06:53:36Z

On Mon, 2025-03-31 at 14:03 -0700, Paul E. McKenney wrote:
> Uses of srcu_read_lock_lite() and srcu_read_unlock_lite() are better
> served by the new srcu_read_lock_fast() and srcu_read_unlock_fast() APIs.
> As in srcu_read_lock_lite() and srcu_read_unlock_lite() would never have
> happened had I thought a bit harder a few months ago. Therefore, mark
> them deprecated.

Would it be better to convert the 3 existing instances?

>
> Signed-off-by: Paul E. McKenney <paulmck@kernel.org>
> ---
> scripts/checkpatch.pl | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
> index 7b28ad3317427..de8ed5efc5b16 100755
> --- a/scripts/checkpatch.pl
> +++ b/scripts/checkpatch.pl
> @@ -838,6 +838,8 @@ our %deprecated_apis = (
> "kunmap" => "kunmap_local",
> "kmap_atomic" => "kmap_local_page",
> "kunmap_atomic" => "kunmap_local",
> + "srcu_read_lock_lite" => "srcu_read_lock_fast",
> + "srcu_read_unlock_lite" => "srcu_read_unlock_fast",
> );
>
> #Create a search pattern for all these strings to speed up a loop below