Re: [PATCH V8 04/14] rust: Add cpumask helpers
From: Viresh Kumar
Date: Mon Feb 10 2025 - 23:29:23 EST
On 10-02-25, 19:02, Yury Norov wrote:
> On Thu, Feb 06, 2025 at 02:58:25PM +0530, Viresh Kumar wrote:
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index ee6709599df5..bfc1bf2ebd77 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -4021,6 +4021,7 @@ F: lib/cpumask_kunit.c
> > F: lib/find_bit.c
> > F: lib/find_bit_benchmark.c
> > F: lib/test_bitmap.c
> > +F: rust/helpers/cpumask.c
>
> Sorry what? I never committed to maintain this thing, and will
> definitely not do it for free.
>
> NAK.
Okay. I will add a separate entry for Rust cpumask stuff.
> > +#ifndef CONFIG_CPUMASK_OFFSTACK
> > +void rust_helper_free_cpumask_var(cpumask_var_t mask)
> > +{
> > + free_cpumask_var(mask);
> > +}
> > +#endif
>
> This is most likely wrong because free_cpumask_var() is declared
> unconditionally, and I suspect the rust helper should be as well.
Non-trivial C macros and inlined C functions cannot be used directly
in the Rust code and are used via functions ("helpers") that wrap
those so that they can be called from Rust.
The free_cpumask_var() function is defined as inline only for the
CONFIG_CPUMASK_OFFSTACK=n case and that's where we need this wrapper.
For the other case (CONFIG_CPUMASK_OFFSTACK=y), Rust code can directly
call free_cpumask_var() from the C code and we don't need this helper.
> Please for the next iteration keep me CCed for the whole series. I
> want to make sure you'll not make me a rust maintainer accidentally.
Sure.
--
viresh