linux-kernel - Re: [PATCH] checkpatch: add IDR to the deprecated list

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] [day] [month] [year] [list]

Message-ID: <1189b6b0a67eef57f824f200121c8d46e73e5f24.camel@perches.com>
Date: Sat, 01 Nov 2025 10:43:21 -0700
From: Joe Perches <joe@...ches.com>
To: Carlos López <clopez@...e.de>, 
	linux-kernel@...r.kernel.org, Andrew Morton <akpm@...ux-foundation.org>
Cc: willy@...radead.org, Dan Williams <dan.j.williams@...el.com>, Andy
 Whitcroft <apw@...onical.com>, Dwaipayan Ray <dwaipayanray1@...il.com>,
 Lukas Bulwahn	 <lukas.bulwahn@...il.com>
Subject: Re: [PATCH] checkpatch: add IDR to the deprecated list

On Fri, 2025-10-31 at 12:19 +0100, Carlos López wrote:
> As of commit 85656ec193e9, the IDR interface is marked as deprecated
> in the documentation, but no checks are made in that regard for new
> code. Add the existing IDR initialization APIs to the deprecated list
> in checkpatch, so that if new code is introduced using these APIs, a
> warning is emitted.
> 
> Suggested-by: Dan Williams <dan.j.williams@...el.com>
> Signed-off-by: Carlos López <clopez@...e.de>
> ---
>  scripts/checkpatch.pl | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
> index 92669904eecc..bc72fa66c0ef 100755
> --- a/scripts/checkpatch.pl
> +++ b/scripts/checkpatch.pl
> @@ -860,6 +860,10 @@ our %deprecated_apis = (
>  	"kunmap"				=> "kunmap_local",
>  	"kmap_atomic"				=> "kmap_local_page",
>  	"kunmap_atomic"				=> "kunmap_local",
> +	#These should be enough to drive away new IDR users
> +	"DEFINE_IDR"				=> "DEFINE_XARRAY",
> +	"idr_init"				=> "xa_init",
> +	"idr_init_base"				=> "xa_init_flags"
>  );
>  
>  #Create a search pattern for all these strings to speed up a loop below

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