Re: [PATCH v2 19/22] rust: pin-init: miscellaneous synchronization with the user-space version
From: Fiona Behrens
Date: Thu Mar 13 2025 - 07:24:44 EST
Benno Lossin <benno.lossin@xxxxxxxxx> writes:
> Remove the last differences between the kernel version and the
> user-space version.
>
> Signed-off-by: Benno Lossin <benno.lossin@xxxxxxxxx>
> Reviewed-by: Andreas Hindborg <a.hindborg@xxxxxxxxxx>
> Tested-by: Andreas Hindborg <a.hindborg@xxxxxxxxxx>
Reviewed-by: Fiona Behrens <me@xxxxxxxxxx>
> ---
> rust/pin-init/internal/src/lib.rs | 5 ++---
> rust/pin-init/src/__internal.rs | 2 +-
> rust/pin-init/src/lib.rs | 2 --
> 3 files changed, 3 insertions(+), 6 deletions(-)
>
> diff --git a/rust/pin-init/internal/src/lib.rs b/rust/pin-init/internal/src/lib.rs
> index 30e145f80bc0..babe5e878550 100644
> --- a/rust/pin-init/internal/src/lib.rs
> +++ b/rust/pin-init/internal/src/lib.rs
> @@ -14,6 +14,8 @@
> //
> // Remove once we have `proc_macro2` in the kernel.
> #![allow(clippy::useless_conversion)]
> +// Documentation is done in the pin-init crate instead.
> +#![allow(missing_docs)]
>
> use proc_macro::TokenStream;
>
> @@ -30,19 +32,16 @@
> mod pinned_drop;
> mod zeroable;
>
> -#[allow(missing_docs)]
> #[proc_macro_attribute]
> pub fn pin_data(inner: TokenStream, item: TokenStream) -> TokenStream {
> pin_data::pin_data(inner.into(), item.into()).into()
> }
>
> -#[allow(missing_docs)]
> #[proc_macro_attribute]
> pub fn pinned_drop(args: TokenStream, input: TokenStream) -> TokenStream {
> pinned_drop::pinned_drop(args.into(), input.into()).into()
> }
>
> -#[allow(missing_docs)]
> #[proc_macro_derive(Zeroable)]
> pub fn derive_zeroable(input: TokenStream) -> TokenStream {
> zeroable::derive(input.into()).into()
> diff --git a/rust/pin-init/src/__internal.rs b/rust/pin-init/src/__internal.rs
> index 7f7744d48575..557b5948cddc 100644
> --- a/rust/pin-init/src/__internal.rs
> +++ b/rust/pin-init/src/__internal.rs
> @@ -14,7 +14,7 @@
> ///
> /// [nomicon]: https://doc.rust-lang.org/nomicon/subtyping.html
> /// [this table]: https://doc.rust-lang.org/nomicon/phantom-data.html#table-of-phantomdata-patterns
> -pub(super) type Invariant<T> = PhantomData<fn(*mut T) -> *mut T>;
> +pub(crate) type Invariant<T> = PhantomData<fn(*mut T) -> *mut T>;
>
> /// Module-internal type implementing `PinInit` and `Init`.
> ///
> diff --git a/rust/pin-init/src/lib.rs b/rust/pin-init/src/lib.rs
> index a00288133ae3..45880ffa81bb 100644
> --- a/rust/pin-init/src/lib.rs
> +++ b/rust/pin-init/src/lib.rs
> @@ -349,8 +349,6 @@
> /// }
> /// }
> /// ```
> -///
> -/// [`pin_init!`]: crate::pin_init
> pub use ::pin_init_internal::pin_data;
>
> /// Used to implement `PinnedDrop` safely.