Re: Rust kernel policy
From: Jarkko Sakkinen
Date: Wed Feb 19 2025 - 01:20:52 EST
On Tue, 2025-02-18 at 13:22 -0800, Boqun Feng wrote:
> FWIW, usually Rust code has doc tests allowing you to run it with
> kunit,
> see:
>
> https://docs.kernel.org/rust/testing.html
I know this document and this was what I used to compile DMA patches.
Then I ended up into "no test, no go" state :-)
I put this is way. If that is enough, or perhaps combined with
submitting-patches.rst, why this email thread exists?
>
> , I took a look at the DMA patches, there is one doc test, but
> unfortunately it's only a function definition, i.e. it won't run
> these
> DMA bindings.
>
> I agree that test payload should be provided, there must be something
> mentioning this in Documentation/process/submitting-patches.rst
> already?
Partly yes. This what was exactly what I was wondering when I read
through the thread, i.e. why no one is speaking about tests :-)
>
> Regards,
> Boqun
Thanks for responding, definitely not picking a fight here. I
actually just wanted to help, and doing kernel QA is the best
possible way to take the first baby steps on a new subsystem,
and sort of area where I'm professional already as a kernel
maintainer.
BR, Jarkko