diff options
author | Danilo Krummrich <dakr@kernel.org> | 2025-03-14 17:09:07 +0100 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2025-03-17 08:04:25 +0100 |
commit | 4d320e30ee04c25c660eca2bb33e846ebb71a79a (patch) | |
tree | 57a65743c3a8b5ccbeff41420267ce9bc1b3be51 /rust/kernel/types.rs | |
parent | 7b948a2af6b5d64a25c14da8f63d8084ea527cd9 (diff) | |
download | linux-4d320e30ee04c25c660eca2bb33e846ebb71a79a.tar.gz linux-4d320e30ee04c25c660eca2bb33e846ebb71a79a.tar.bz2 linux-4d320e30ee04c25c660eca2bb33e846ebb71a79a.zip |
rust: platform: fix unrestricted &mut platform::Device
As by now, platform::Device is implemented as:
#[derive(Clone)]
pub struct Device(ARef<device::Device>);
This may be convenient, but has the implication that drivers can call
device methods that require a mutable reference concurrently at any
point of time.
Instead define platform::Device as
pub struct Device<Ctx: DeviceContext = Normal>(
Opaque<bindings::platform_dev>,
PhantomData<Ctx>,
);
and manually implement the AlwaysRefCounted trait.
With this we can implement methods that should only be called from
bus callbacks (such as probe()) for platform::Device<Core>. Consequently,
we make this type accessible in bus callbacks only.
Arbitrary references taken by the driver are still of type
ARef<platform::Device> and hence don't provide access to methods that are
reserved for bus callbacks.
Fixes: 683a63befc73 ("rust: platform: add basic platform device / driver abstractions")
Reviewed-by: Benno Lossin <benno.lossin@proton.me>
Signed-off-by: Danilo Krummrich <dakr@kernel.org>
Acked-by: Boqun Feng <boqun.feng@gmail.com>
Link: https://lore.kernel.org/r/20250314160932.100165-5-dakr@kernel.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'rust/kernel/types.rs')
0 files changed, 0 insertions, 0 deletions