[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CACQBu=XbepNPkeGW649Ogf9PNQ59kebatpM8XTgFN162TVj9-g@mail.gmail.com>
Date: Wed, 19 Mar 2025 17:11:15 +0100
From: Burak Emir <bqe@...gle.com>
To: Yury Norov <yury.norov@...il.com>
Cc: Miguel Ojeda <miguel.ojeda.sandonis@...il.com>,
Rasmus Villemoes <linux@...musvillemoes.dk>, Viresh Kumar <viresh.kumar@...aro.org>,
Miguel Ojeda <ojeda@...nel.org>, Alex Gaynor <alex.gaynor@...il.com>,
Boqun Feng <boqun.feng@...il.com>, Gary Guo <gary@...yguo.net>,
Björn Roy Baron <bjorn3_gh@...tonmail.com>,
Benno Lossin <benno.lossin@...ton.me>, Andreas Hindborg <a.hindborg@...nel.org>,
Alice Ryhl <aliceryhl@...gle.com>, Trevor Gross <tmgross@...ch.edu>, rust-for-linux@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH] rust: adds bindings for bitmap.c and bitops.c
On Wed, Mar 19, 2025 at 5:07 PM Yury Norov <yury.norov@...il.com> wrote:
>
> On Wed, Mar 19, 2025 at 04:43:50PM +0100, Miguel Ojeda wrote:
> > On Wed, Mar 19, 2025 at 2:58 PM Burak Emir <bqe@...gle.com> wrote:
> > >
> > > Thanks, I was unsure about this. I understand the desire to not add dead code.
> > > Yet, next-20250318 does now contain helpers for `cpumask.c` without
> > > corresponding `cpumask.rs`
> > >
> > > So, I will include this in v5 of the rebased series, but with BITOPS
> > > API BINDINGS [RUST] in its own patch.
> > > Thanks for bearing with me : )
> >
> > No worries at all! Perhaps wait a bit in case I misunderstood and Yury
> > wants to clarify.
>
> Yes, the general idea is to be able look through the all software pieces
> in a single series - from C wrappers all the way down to rust end users.
>
> This way I'll make sure you are using the API in a right way. And I
> think I already helped you in that department, right?
Yep, thank you for the timely reviews with the comments. The series got
quite a few better. I'd love to be as timely with sending back the new versions,
still getting used to the conventions, workflow and limitations of the
infrastructure.
Powered by blists - more mailing lists