[v8,05/29] rust: alloc: make `allocator` module public

Message ID	20241001150008.183102-6-dakr@kernel.org (mailing list archive)
State	New
Headers	show Return-Path: <owner-linux-mm@kvack.org> From: Danilo Krummrich <dakr@kernel.org> To: ojeda@kernel.org, alex.gaynor@gmail.com, wedsonaf@gmail.com, boqun.feng@gmail.com, gary@garyguo.net, bjorn3_gh@protonmail.com, benno.lossin@proton.me, a.hindborg@samsung.com, aliceryhl@google.com, akpm@linux-foundation.org Cc: daniel.almeida@collabora.com, faith.ekstrand@collabora.com, boris.brezillon@collabora.com, lina@asahilina.net, mcanal@igalia.com, zhiw@nvidia.com, cjia@nvidia.com, jhubbard@nvidia.com, airlied@redhat.com, ajanulgu@redhat.com, lyude@redhat.com, linux-kernel@vger.kernel.org, rust-for-linux@vger.kernel.org, linux-mm@kvack.org, Danilo Krummrich <dakr@kernel.org> Subject: [PATCH v8 05/29] rust: alloc: make `allocator` module public Date: Tue, 1 Oct 2024 16:59:40 +0200 Message-ID: <20241001150008.183102-6-dakr@kernel.org> In-Reply-To: <20241001150008.183102-1-dakr@kernel.org> References: <20241001150008.183102-1-dakr@kernel.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Sender: owner-linux-mm@kvack.org Precedence: bulk
Series	Generic `Allocator` support for Rust \| expand [v8,00/29] Generic `Allocator` support for Rust [v8,01/29] rust: alloc: add `Allocator` trait [v8,02/29] rust: alloc: separate `aligned_size` from `krealloc_aligned` [v8,03/29] rust: alloc: rename `KernelAllocator` to `Kmalloc` [v8,04/29] rust: alloc: implement `ReallocFunc` [v8,05/29] rust: alloc: make `allocator` module public [v8,06/29] rust: alloc: implement `Allocator` for `Kmalloc` [v8,07/29] rust: alloc: add module `allocator_test` [v8,08/29] rust: alloc: implement `Vmalloc` allocator [v8,09/29] rust: alloc: implement `KVmalloc` allocator [v8,10/29] rust: alloc: add __GFP_NOWARN to `Flags` [v8,11/29] rust: alloc: implement kernel `Box` [v8,12/29] rust: treewide: switch to our kernel `Box` type [v8,13/29] rust: alloc: remove extension of std's `Box` [v8,14/29] rust: alloc: add `Box` to prelude [v8,15/29] rust: alloc: introduce `ArrayLayout` [v8,16/29] rust: alloc: implement kernel `Vec` type [v8,17/29] rust: alloc: implement `IntoIterator` for `Vec` [v8,18/29] rust: alloc: implement `collect` for `IntoIter` [v8,19/29] rust: treewide: switch to the kernel `Vec` type [v8,20/29] rust: alloc: remove `VecExt` extension [v8,21/29] rust: alloc: add `Vec` to prelude [v8,22/29] rust: error: use `core::alloc::LayoutError` [v8,23/29] rust: error: check for config `test` in `Error::name` [v8,24/29] rust: alloc: implement `contains` for `Flags` [v8,25/29] rust: alloc: implement `Cmalloc` in module allocator_test [v8,26/29] rust: str: test: replace `alloc::format` [v8,27/29] rust: alloc: update module comment of alloc.rs [v8,28/29] kbuild: rust: remove the `alloc` crate and `GlobalAlloc` [v8,29/29] MAINTAINERS: add entry for the Rust `alloc` module

Message ID

20241001150008.183102-6-dakr@kernel.org (mailing list archive)

State

New

Headers

From: Danilo Krummrich <dakr@kernel.org>
To: ojeda@kernel.org,
	alex.gaynor@gmail.com,
	wedsonaf@gmail.com,
	boqun.feng@gmail.com,
	gary@garyguo.net,
	bjorn3_gh@protonmail.com,
	benno.lossin@proton.me,
	a.hindborg@samsung.com,
	aliceryhl@google.com,
	akpm@linux-foundation.org
Cc: daniel.almeida@collabora.com,
	faith.ekstrand@collabora.com,
	boris.brezillon@collabora.com,
	lina@asahilina.net,
	mcanal@igalia.com,
	zhiw@nvidia.com,
	cjia@nvidia.com,
	jhubbard@nvidia.com,
	airlied@redhat.com,
	ajanulgu@redhat.com,
	lyude@redhat.com,
	linux-kernel@vger.kernel.org,
	rust-for-linux@vger.kernel.org,
	linux-mm@kvack.org,
	Danilo Krummrich <dakr@kernel.org>
Subject: [PATCH v8 05/29] rust: alloc: make `allocator` module public
Date: Tue,  1 Oct 2024 16:59:40 +0200
Message-ID: <20241001150008.183102-6-dakr@kernel.org>
In-Reply-To: <20241001150008.183102-1-dakr@kernel.org>
References: <20241001150008.183102-1-dakr@kernel.org>
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Sender: owner-linux-mm@kvack.org
Precedence: bulk

Series

Generic `Allocator` support for Rust | expand

Commit Message

Danilo Krummrich Oct. 1, 2024, 2:59 p.m. UTC

Subsequent patches implement allocators such as `Kmalloc`, `Vmalloc`,
`KVmalloc`; we need them to be available outside of the kernel crate as
well.

Signed-off-by: Danilo Krummrich <dakr@kernel.org>
---
 rust/kernel/alloc.rs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Benno Lossin Oct. 1, 2024, 4:01 p.m. UTC | #1

On 01.10.24 16:59, Danilo Krummrich wrote:
> Subsequent patches implement allocators such as `Kmalloc`, `Vmalloc`,
> `KVmalloc`; we need them to be available outside of the kernel crate as
> well.
> 
> Signed-off-by: Danilo Krummrich <dakr@kernel.org>

Reviewed-by: Benno Lossin <benno.lossin@proton.me>

---
Cheers,
Benno

> ---
>  rust/kernel/alloc.rs | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/rust/kernel/alloc.rs b/rust/kernel/alloc.rs
> index da49c5306431..91b2a26faf3d 100644
> --- a/rust/kernel/alloc.rs
> +++ b/rust/kernel/alloc.rs
> @@ -4,7 +4,7 @@
> 
>  #[cfg(not(test))]
>  #[cfg(not(testlib))]
> -mod allocator;
> +pub mod allocator;
>  pub mod box_ext;
>  pub mod vec_ext;
> 
> --
> 2.46.1
>

Gary Guo Oct. 1, 2024, 6:25 p.m. UTC | #2

On Tue,  1 Oct 2024 16:59:40 +0200
Danilo Krummrich <dakr@kernel.org> wrote:

> Subsequent patches implement allocators such as `Kmalloc`, `Vmalloc`,
> `KVmalloc`; we need them to be available outside of the kernel crate as
> well.
> 
> Signed-off-by: Danilo Krummrich <dakr@kernel.org>

Reviewed-by: Gary Guo <gary@garyguo.net>

> ---
>  rust/kernel/alloc.rs | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/rust/kernel/alloc.rs b/rust/kernel/alloc.rs
> index da49c5306431..91b2a26faf3d 100644
> --- a/rust/kernel/alloc.rs
> +++ b/rust/kernel/alloc.rs
> @@ -4,7 +4,7 @@
>  
>  #[cfg(not(test))]
>  #[cfg(not(testlib))]
> -mod allocator;
> +pub mod allocator;
>  pub mod box_ext;
>  pub mod vec_ext;
>

diff --git a/rust/kernel/alloc.rs b/rust/kernel/alloc.rs
index da49c5306431..91b2a26faf3d 100644
--- a/rust/kernel/alloc.rs
+++ b/rust/kernel/alloc.rs
@@ -4,7 +4,7 @@ 
 
 #[cfg(not(test))]
 #[cfg(not(testlib))]
-mod allocator;
+pub mod allocator;
 pub mod box_ext;
 pub mod vec_ext;

[v8,05/29] rust: alloc: make `allocator` module public

Commit Message

Comments

Patch