search

Rust-for-Linux / linux

Adding support for the Rust language to the Linux kernel.
https://rust-for-linux.com
Other
4k stars 431 forks source link

Impl `fmt::Display` for kernel `Box<T, A>` #1126

Open fbq opened 1 month ago

fbq commented 1 month ago

Currently fmt::Display is not implemented for Box<T, A>, due to we have our own allocator implementation. The following code should just work if we have impl fmt::Display for that:

let v = KBox::new(42, GFP_KERNEL)?;
println!("{}", v);

This requires submitting a proper patch to the LKML and the Rust for Linux mailing list. Please recall to test your changes (including generating the documentation if changed, running the Rust doctests if changed, etc.), to use a proper title for the commit, to sign your commit under the Developer's Certificate of Origin and to add a Suggested-by: tag and a Link: tag to this issue. Please see https://rust-for-linux.com/contributing for details.

Please take this issue only if you are new to the kernel development process and you would like to use it as a test to submit your first patch to the kernel. Please do not take it if you do not plan to make other contributions to the kernel.

JCKeep commented 1 month ago

I am new to the Kernel development and I would like to take it if is possible.

ojeda commented 1 month ago

Sure, please go ahead!