Re: [PATCH v2 19/22] rust: pin-init: miscellaneous synchronization with the user-space version

[Fiona Behrens]

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.