[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <8AA2058F-B9CA-4B5F-99B5-8C719C4630C3@collabora.com>
Date: Wed, 27 Aug 2025 10:48:35 -0300
From: Daniel Almeida <daniel.almeida@...labora.com>
To: Andreas Hindborg <a.hindborg@...nel.org>
Cc: Boqun Feng <boqun.feng@...il.com>,
Miguel Ojeda <ojeda@...nel.org>,
Alex Gaynor <alex.gaynor@...il.com>,
Gary Guo <gary@...yguo.net>,
Björn Roy Baron <bjorn3_gh@...tonmail.com>,
Benno Lossin <lossin@...nel.org>,
Alice Ryhl <aliceryhl@...gle.com>,
Trevor Gross <tmgross@...ch.edu>,
Danilo Krummrich <dakr@...nel.org>,
Jens Axboe <axboe@...nel.dk>,
Breno Leitao <leitao@...ian.org>,
linux-block@...r.kernel.org,
rust-for-linux@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH v6 07/18] rust: configfs: re-export `configfs_attrs` from
`configfs` module
> On 22 Aug 2025, at 09:14, Andreas Hindborg <a.hindborg@...nel.org> wrote:
>
> Re-export `configfs_attrs` from `configfs` module, so that users can import
> the macro from the `configfs` module rather than the root of the `kernel`
> crate.
>
> Also update users to import from the new path.
>
> Reviewed-by: Alice Ryhl <aliceryhl@...gle.com>
> Signed-off-by: Andreas Hindborg <a.hindborg@...nel.org>
> ---
> rust/kernel/configfs.rs | 2 ++
> samples/rust/rust_configfs.rs | 2 +-
> 2 files changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/rust/kernel/configfs.rs b/rust/kernel/configfs.rs
> index 2736b798cdc6..318a2f073d1c 100644
> --- a/rust/kernel/configfs.rs
> +++ b/rust/kernel/configfs.rs
> @@ -1039,3 +1039,5 @@ macro_rules! configfs_attrs {
> };
>
> }
> +
> +pub use crate::configfs_attrs;
> diff --git a/samples/rust/rust_configfs.rs b/samples/rust/rust_configfs.rs
> index af04bfa35cb2..ad364fb93e53 100644
> --- a/samples/rust/rust_configfs.rs
> +++ b/samples/rust/rust_configfs.rs
> @@ -5,7 +5,7 @@
> use kernel::alloc::flags;
> use kernel::c_str;
> use kernel::configfs;
> -use kernel::configfs_attrs;
> +use kernel::configfs::configfs_attrs;
> use kernel::new_mutex;
> use kernel::page::PAGE_SIZE;
> use kernel::prelude::*;
>
> --
> 2.47.2
>
>
>
Reviewed-by: Daniel Almeida <daniel.almeida@...labora.com>
Powered by blists - more mailing lists