Re: [PATCH v4 0/3] rust: Add pr_*_once macros134

[Boqun Feng <boqun.feng@gmail.com>]

On Thu, Jul 17, 2025 at 07:07:13PM +0300, Onur Özkan wrote:
> On Tue, 11 Feb 2025 16:42:25 +0100
> Andreas Hindborg <a.hindborg@kernel.org> wrote:
> 
> > <jens.korinth@tuta.io> writes:
> > 
> > > Hi!
> > >
> > >> Thanks for the patch! Do you plan on sending a new version? If
> > >>not, do you mind if I send v5?
> > >
> > > I think there is currently no consensus on how exactly it should be
> > > done (or at least I was confused about that). If you´re actively
> > > using the patch please go ahead! Active usage is always the best
> > > argument in such cases.
> > 
> > I was informed this patch set is the correct way to emit a warning in
> > the module_params patch set [1].
> > 
> > I did not follow all the discussion so I am not sure either. But I'll
> > look into the discussion then.
> > 
> > 
> > Best regards,
> > Andreas Hindborg
> > 
> > 
> > 
> > [1]
> > https://lore.kernel.org/rust-for-linux/20250204-module-params-v3-v5-0-bf5ec2041625@kernel.org/
> > 
> > 
> 
> I have reviewed the patch series from start to finish. I am not
> using or depending the patch actively but I can send v5 sometime
> soon (I will have some space next week) if you would like.
> 

Note that you need to use LKMM atomics [1], and you should really use
a 32bit atomic at the beginning. Thanks!

There are a few explanations on why we want to avoid use Rust native
atomics:

	https://lwn.net/Articles/993785/
	https://lore.kernel.org/rust-for-linux/CAHk-=whkQk=zq5XiMcaU3xj4v69+jyoP-y6Sywhq-TvxSSvfEA@mail.gmail.com/

[1]: https://lore.kernel.org/rust-for-linux/20250719030827.61357-1-boqun.feng@gmail.com/

Regards,
Boqun

> Regards,
> Onur