| From 2f470e9fd77e885ee2d0201a91b9f926c72f743e Mon Sep 17 00:00:00 2001 |
| From: Sasha Levin <sashal@kernel.org> |
| Date: Thu, 28 Mar 2024 16:54:54 -0300 |
| Subject: rust: kernel: require `Send` for `Module` implementations |
| |
| From: Wedson Almeida Filho <walmeida@microsoft.com> |
| |
| [ Upstream commit 323617f649c0966ad5e741e47e27e06d3a680d8f ] |
| |
| The thread that calls the module initialisation code when a module is |
| loaded is not guaranteed [in fact, it is unlikely] to be the same one |
| that calls the module cleanup code on module unload, therefore, `Module` |
| implementations must be `Send` to account for them moving from one |
| thread to another implicitly. |
| |
| Signed-off-by: Wedson Almeida Filho <walmeida@microsoft.com> |
| Reviewed-by: Alice Ryhl <aliceryhl@google.com> |
| Reviewed-by: Benno Lossin <benno.lossin@proton.me> |
| Cc: stable@vger.kernel.org # 6.8.x: df70d04d5697: rust: phy: implement `Send` for `Registration` |
| Cc: stable@vger.kernel.org |
| Fixes: 247b365dc8dc ("rust: add `kernel` crate") |
| Link: https://lore.kernel.org/r/20240328195457.225001-3-wedsonaf@gmail.com |
| Signed-off-by: Miguel Ojeda <ojeda@kernel.org> |
| Signed-off-by: Sasha Levin <sashal@kernel.org> |
| --- |
| rust/kernel/lib.rs | 2 +- |
| 1 file changed, 1 insertion(+), 1 deletion(-) |
| |
| diff --git a/rust/kernel/lib.rs b/rust/kernel/lib.rs |
| index abd46261d3855..43cf5f6bde9c2 100644 |
| --- a/rust/kernel/lib.rs |
| +++ b/rust/kernel/lib.rs |
| @@ -37,7 +37,7 @@ |
| /// The top level entrypoint to implementing a kernel module. |
| /// |
| /// For any teardown or cleanup operations, your type may implement [`Drop`]. |
| -pub trait Module: Sized + Sync { |
| +pub trait Module: Sized + Sync + Send { |
| /// Called at module initialization time. |
| /// |
| /// Use this method to perform whatever setup or registration your module |
| -- |
| 2.43.0 |
| |