Re: [PATCH v2 0/3] rust: use &raw [mut] place instead of addr_of[_mut]!(place)

[Benno Lossin <benno.lossin@proton.me>]

On Fri Mar 14, 2025 at 4:45 AM CET, Antonio Hickey wrote:
> This patch set enables the `raw_ref_op` feature, which became stable in
> Rust 1.82. It then adds the previous macros for getting a raw pointer to a
> place to the disallowed macros in `.clippy.toml`. Finally it replaces
> all occurences of `addr_of!(place)` and `addr_of_mut!(place)` with 
> `&raw const place` and `&raw mut place`.
>
> 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>
>
>
> Antonio Hickey (3):
>   rust: enable `raw_ref_op` feature
>   rust: clippy: disable `addr_of!` and `addr_of_mut` macros
>   rust: replace `addr_of[_mut]!` with `&raw [mut]`
>
>  .clippy.toml                           |  4 ++++
>  rust/kernel/block/mq/request.rs        |  4 ++--
>  rust/kernel/faux.rs                    |  4 ++--
>  rust/kernel/fs/file.rs                 |  2 +-
>  rust/kernel/init.rs                    |  8 ++++----
>  rust/kernel/init/macros.rs             | 28 +++++++++++++-------------
>  rust/kernel/jump_label.rs              |  4 ++--
>  rust/kernel/kunit.rs                   |  4 ++--
>  rust/kernel/lib.rs                     |  2 ++
>  rust/kernel/list.rs                    |  2 +-
>  rust/kernel/list/impl_list_item_mod.rs |  6 +++---
>  rust/kernel/net/phy.rs                 |  4 ++--
>  rust/kernel/pci.rs                     |  4 ++--
>  rust/kernel/platform.rs                |  4 +---
>  rust/kernel/rbtree.rs                  | 22 ++++++++++----------
>  rust/kernel/sync/arc.rs                |  2 +-
>  rust/kernel/task.rs                    |  4 ++--
>  rust/kernel/workqueue.rs               |  8 ++++----
>  18 files changed, 60 insertions(+), 56 deletions(-)

Usually when sending a new version, you send it as its own email thread.
That way it's easier to distinguish the two versions. (no need to resend
now, but for the future)

---
Cheers,
Benno