[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAPRMd3kXUJC6rC_X4i41dWNpS2tx4aEXFmBuEwncXmdJewinDA@mail.gmail.com>
Date: Sun, 27 Jul 2025 13:00:00 +0530
From: Shankari Anand <shankari.ak0208@...il.com>
To: Miguel Ojeda <miguel.ojeda.sandonis@...il.com>
Cc: linux-kernel@...r.kernel.org, rust-for-linux@...r.kernel.org,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>, Dave Ertman <david.m.ertman@...el.com>,
Ira Weiny <ira.weiny@...el.com>, Leon Romanovsky <leon@...nel.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 <lossin@...nel.org>, Andreas Hindborg <a.hindborg@...nel.org>,
Alice Ryhl <aliceryhl@...gle.com>, Trevor Gross <tmgross@...ch.edu>,
Danilo Krummrich <dakr@...nel.org>, "Rafael J . Wysocki" <rafael@...nel.org>,
Abdiel Janulgue <abdiel.janulgue@...il.com>, Daniel Almeida <daniel.almeida@...labora.com>,
Robin Murphy <robin.murphy@....com>, Viresh Kumar <vireshk@...nel.org>, Nishanth Menon <nm@...com>,
Stephen Boyd <sboyd@...nel.org>, Bjorn Helgaas <bhelgaas@...gle.com>,
Krzysztof Wilczyński <kwilczynski@...nel.org>
Subject: Re: [PATCH 7/7] rust: kernel: update ARef and AlwaysRefCounted
imports from sync::aref
> It has been only a week (and we are close to the merge window, thus
> maintainers are typically dealing with other things/patches):
> https://docs.kernel.org/process/submitting-patches.html#don-t-get-discouraged-or-impatient
Apologies for the early ping, I was unsure about the appropriate way
to split the patches and wanted to confirm before proceeding.
> Having said that, I am not sure why this one in particular has e.g.
> `opp.rs` and `devres.rs` -- those are different
> entries/trees/maintainers in `MAINTAINERS`. So it would likely help to
> perform a split that doesn't involve different subsystems.
Sure, I'll carefully split the patches accordingly in the next version
to align with the relevant maintainers and trees.
> By the way, if you meant to send the patches as completely independent
> ones, then I would suggest avoiding the series notation (e.g. "7/7")
> -- at least I see them in Lore as separate threads:
Regarding the series notation (e.g., "7/7"), I included it to indicate
the overall scope of changes as part of the same effort. However,
since the patches are independent and target different subsystems,
I’ll avoid that format going forward, as suggested.
Thanks for your guidance.
~ Shankari
On Fri, Jul 25, 2025 at 7:33 PM Miguel Ojeda
<miguel.ojeda.sandonis@...il.com> wrote:
>
> On Fri, Jul 25, 2025 at 3:02 PM Shankari Anand
> <shankari.ak0208@...il.com> wrote:
> >
> > Can this be picked up for review? Should I be splitting each core
> > kernel file into a separate patch?
>
> It has been only a week (and we are close to the merge window, thus
> maintainers are typically dealing with other things/patches):
>
> https://docs.kernel.org/process/submitting-patches.html#don-t-get-discouraged-or-impatient
>
> Having said that, I am not sure why this one in particular has e.g.
> `opp.rs` and `devres.rs` -- those are different
> entries/trees/maintainers in `MAINTAINERS`. So it would likely help to
> perform a split that doesn't involve different subsystems.
>
> By the way, if you meant to send the patches as completely independent
> ones, then I would suggest avoiding the series notation (e.g. "7/7")
> -- at least I see them in Lore as separate threads:
>
> https://lore.kernel.org/rust-for-linux/20250717073450.15090-1-shankari.ak0208@gmail.com/
>
> Thanks!
>
> Cheers,
> Miguel
Powered by blists - more mailing lists