You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As Lina and Janne reported, there was a bug in AllocLayout::new(). Although it's fixed, it would be better to have something unit/doc tests about it like this.
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.
Just sent in the patch! The thread should be here.
Thank you for making these opportunities for people who want to contribute but aren't familiar with the process!
Activity
LordGoatius commentedon Nov 29, 2024
Hi, I'm new to the kernel development process and I'd like to claim this.
ojeda commentedon Nov 29, 2024
Of course, please go ahead!
rust: alloc: Add doctest for `ArrayLayout`
LordGoatius commentedon Dec 3, 2024
Just sent in the patch! The thread should be here.
Thank you for making these opportunities for people who want to contribute but aren't familiar with the process!
ojeda commentedon Dec 3, 2024
You're welcome! It is nice to hear they are useful :)
rust: alloc: Add doctest for `ArrayLayout`
rust: alloc: Add doctest for `ArrayLayout`
rust: alloc: Add doctest for `ArrayLayout`
rust: alloc: add doctest for `ArrayLayout::new()`
rust: alloc: add doctest for `ArrayLayout::new()`
rust: alloc: add doctest for `ArrayLayout::new()`
rust: alloc: add doctest for `ArrayLayout::new()`