linux-kernel - Re: [PATCH V8 04/14] rust: Add cpumask helpers

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <87a5anv7g9.fsf@kernel.org>
Date: Sat, 15 Feb 2025 11:16:38 +0100
From: Andreas Hindborg <a.hindborg@...nel.org>
To: "Viresh Kumar" <viresh.kumar@...aro.org>
Cc: "Yury Norov" <yury.norov@...il.com>,  "Rafael J. Wysocki"
 <rafael@...nel.org>,  "Miguel Ojeda" <miguel.ojeda.sandonis@...il.com>,
  "Danilo Krummrich" <dakr@...hat.com>,  "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>,
  "Alice Ryhl" <aliceryhl@...gle.com>,  "Trevor Gross" <tmgross@...ch.edu>,
  "Rasmus Villemoes" <linux@...musvillemoes.dk>,
  <linux-pm@...r.kernel.org>,  "Vincent Guittot"
 <vincent.guittot@...aro.org>,  "Stephen Boyd" <sboyd@...nel.org>,
  "Nishanth Menon" <nm@...com>,  <rust-for-linux@...r.kernel.org>,  "Manos
 Pitsidianakis" <manos.pitsidianakis@...aro.org>,  "Erik Schilling"
 <erik.schilling@...aro.org>,  Alex Bennée
 <alex.bennee@...aro.org>,
  "Joakim Bech" <joakim.bech@...aro.org>,  "Rob Herring" <robh@...nel.org>,
  "Christoph Hellwig" <hch@....de>,  "Jason Gunthorpe" <jgg@...dia.com>,
  <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH V8 04/14] rust: Add cpumask helpers

"Viresh Kumar" <viresh.kumar@...aro.org> writes:

> On 11-02-25, 11:24, Yury Norov wrote:
>> To begin with, this is the 8th version of the same patch, but you
>> only now bothered to CC someone who is listed in MAINTAINERS. This is
>> not how the community works.
>
> Yes, this was my mistake. I assumed get_maintainers would Cc all
> relevant people, but I overlooked the fact that these are new files,
> so it didn’t include the maintainers. Unfortunately, the same issue
> occurred with the clk bindings. Miguel pointed it out at V7, and I
> corrected it immediately. There was no intention to bypass anyone.

I have to admit, I have done this as well, for the module_params series.
It feels horrible to commit this action, because it looks either
disrespectful or like an attempt to bypass people to sneak in stuff. But
as clarified this is _never_ the intention. It's simply people working
with a contribution model that they are still learning how to use. The
kernel tooling and process is not exactly intuitive to most people.

Just know that absolutely the largest fraction of contributors (I would
guess all) want to do things the right way. For all contributions, rust
or c, the patches benefit tremendously from reviews from domain experts.
The contributors appreciate this and hope for this to happen.

Best regards,
Andreas Hindborg