| Message ID | 20250320020740.1631171-17-contact@antoniohickey.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | None | expand |
"Antonio Hickey" <contact@antoniohickey.com> writes: > Replacing all occurrences of `addr_of_mut!(place)` with > `&raw mut place`. > > This will allow us to reduce macro complexity, and improve consistency > with existing reference syntax as `&raw mut` is similar to `&mut` > making it fit more naturally with other existing code. > > Suggested-by: Benno Lossin <benno.lossin@proton.me> > Link: https://github.com/Rust-for-Linux/linux/issues/1148 > Signed-off-by: Antonio Hickey <contact@antoniohickey.com> Acked-by: Andreas Hindborg <a.hindborg@kernel.org> Best regards, Andreas Hindborg
On Thu Mar 20, 2025 at 3:07 AM CET, Antonio Hickey wrote: > Replacing all occurrences of `addr_of_mut!(place)` with > `&raw mut place`. > > This will allow us to reduce macro complexity, and improve consistency > with existing reference syntax as `&raw mut` is similar to `&mut` > making it fit more naturally with other existing code. > > Suggested-by: Benno Lossin <benno.lossin@proton.me> > Link: https://github.com/Rust-for-Linux/linux/issues/1148 > Signed-off-by: Antonio Hickey <contact@antoniohickey.com> Reviewed-by: Benno Lossin <benno.lossin@proton.me> --- Cheers, Benno > --- > rust/kernel/block/mq/request.rs | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/rust/kernel/block/mq/request.rs b/rust/kernel/block/mq/request.rs index 7943f43b9575..4a5b7ec914ef 100644 --- a/rust/kernel/block/mq/request.rs +++ b/rust/kernel/block/mq/request.rs @@ -12,7 +12,7 @@ }; use core::{ marker::PhantomData, - ptr::{addr_of_mut, NonNull}, + ptr::NonNull, sync::atomic::{AtomicU64, Ordering}, }; @@ -187,7 +187,7 @@ pub(crate) fn refcount(&self) -> &AtomicU64 { pub(crate) unsafe fn refcount_ptr(this: *mut Self) -> *mut AtomicU64 { // SAFETY: Because of the safety requirements of this function, the // field projection is safe. - unsafe { addr_of_mut!((*this).refcount) } + unsafe { &raw mut (*this).refcount } } }
Replacing all occurrences of `addr_of_mut!(place)` with `&raw mut place`. This will allow us to reduce macro complexity, and improve consistency with existing reference syntax as `&raw mut` is similar to `&mut` making it fit more naturally with other existing code. Suggested-by: Benno Lossin <benno.lossin@proton.me> Link: https://github.com/Rust-for-Linux/linux/issues/1148 Signed-off-by: Antonio Hickey <contact@antoniohickey.com> --- rust/kernel/block/mq/request.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)