[PATCH v4 6/6] rust: remove `#[allow(clippy::non_send_fields_in_send_ty)]`

Onur Özkan posted 6 patches 3 months, 1 week ago
[PATCH v4 6/6] rust: remove `#[allow(clippy::non_send_fields_in_send_ty)]`
Posted by Onur Özkan 3 months, 1 week ago
Just like how we did in [1], we no longer need this lint
as it's no longer part of Clippy's `suspicious` lint group.

Link: https://github.com/torvalds/linux/commit/5e7c9b84ad08cc7a41b2ddbbbaccb60057da3860 [1]

Signed-off-by: Onur Özkan <work@onurozkan.dev>
---
 rust/kernel/cpufreq.rs | 1 -
 1 file changed, 1 deletion(-)

diff --git a/rust/kernel/cpufreq.rs b/rust/kernel/cpufreq.rs
index 11b03e9d7e89..97de9b0573da 100644
--- a/rust/kernel/cpufreq.rs
+++ b/rust/kernel/cpufreq.rs
@@ -907,7 +907,6 @@ fn register_em(_policy: &mut Policy) {
 /// or CPUs, so it is safe to share it.
 unsafe impl<T: Driver> Sync for Registration<T> {}

-#[allow(clippy::non_send_fields_in_send_ty)]
 /// SAFETY: Registration with and unregistration from the cpufreq subsystem can happen from any
 /// thread.
 unsafe impl<T: Driver> Send for Registration<T> {}
--
2.50.0

Re: [PATCH v4 6/6] rust: remove `#[allow(clippy::non_send_fields_in_send_ty)]`
Posted by Miguel Ojeda 3 months, 1 week ago
On Tue, Jul 1, 2025 at 7:36 AM Onur Özkan <work@onurozkan.dev> wrote:
>
> Just like how we did in [1], we no longer need this lint
> as it's no longer part of Clippy's `suspicious` lint group.
>
> Link: https://github.com/torvalds/linux/commit/5e7c9b84ad08cc7a41b2ddbbbaccb60057da3860 [1]

Normally this format is used:

    ... did in commit 5e7c9b84ad08 ("rust: sync: remove unneeded
`#[allow(clippy::non_send_fields_in_send_ty)]`"), we no longer ...

Maintainers may want to fix this when they apply the patch, so I
wouldn't send a new version just for this (but if you do, then please
reword it).

Cheers,
Miguel