linux-kernel - Re: [PATCH v7] scripts: checkpatch: move Rust-specific lints to separate file

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: <CANiq72kXaFkL34g0V3isOp39VqfN5Oaj9O_A0o1ipERFWhWi2w@mail.gmail.com>
Date: Sat, 7 Feb 2026 19:46:03 +0100
From: Miguel Ojeda <miguel.ojeda.sandonis@...il.com>
To: Jason Hall <jason.kei.hall@...il.com>
Cc: Joe Perches <joe@...ches.com>, rust-for-linux@...r.kernel.org, 
	linux-kernel@...r.kernel.org, Miguel Ojeda <ojeda@...nel.org>
Subject: Re: [PATCH v7] scripts: checkpatch: move Rust-specific lints to
 separate file

On Sat, Feb 7, 2026 at 5:53 PM Jason Hall <jason.kei.hall@...il.com> wrote:
>
> Create scripts/rust_checkpatch.pl for Rust-specific linting logic.
> Add a conditional loading hook in scripts/checkpatch.pl to call it.
> This will allow Rust linting logic to be maintained independently.
>
> Add a new entry to the MAINTAINERS file to track this new file.
>
> Suggested-by: Joe Perches <joe@...ches.com>
> Suggested-by: Miguel Ojeda <ojeda@...nel.org>
> Signed-off-by: Jason Hall <jason.kei.hall@...il.com>

This is better, and perhaps the maintainers decide to take it on its
own, but what I meant by splitting in 2 patches before was that this
should be a series of 2 patches. The first would be this one, and the
second would add the lint. Otherwise, the first one doesn't really do
much, since there would be no lint yet.

https://docs.kernel.org/process/submitting-patches.html#the-canonical-patch-format
explain a bit how patch series work.

In addition, you are still not Cc'ing all the maintainers and
reviewers. Please do so by Cc'ing them -- the addresses are in
`MAINTAINERS` file, and you should look at the "RUST" and "CHECKPATCH"
entries.

Thanks!

Cheers,
Miguel