linux-kernel - Re: [PATCH RFC] soc: qcom: socinfo: Re-implement in Rust

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: <ekngmao2rw4mktubhpodudnkfvx5of74uvlswr7csnyxcwlmo5@cb6jfgiyg4vc>
Date: Tue, 16 Dec 2025 14:01:22 -0800
From: Bjorn Andersson <andersson@...nel.org>
To: Matthew Maurer <mmaurer@...gle.com>
Cc: Konrad Dybcio <konradybcio@...nel.org>, 
	Satya Durga Srinivasu Prabhala <satyap@...cinc.com>, Miguel Ojeda <ojeda@...nel.org>, 
	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>, 
	Trilok Soni <tsoni@...cinc.com>, linux-kernel@...r.kernel.org, linux-arm-msm@...r.kernel.org, 
	rust-for-linux@...r.kernel.org
Subject: Re: [PATCH RFC] soc: qcom: socinfo: Re-implement in Rust

On Tue, Dec 16, 2025 at 12:53:28PM -0800, Matthew Maurer wrote:
> On Tue, Dec 16, 2025 at 12:49 PM Bjorn Andersson <andersson@...nel.org> wrote:
> >
> > On Sat, Dec 13, 2025 at 12:36:00AM +0000, Matthew Maurer wrote:
> > > Re-implements qcom-socinfo driver in Rust, using `Scoped`-based DebugFS
> > > bindings.
> >
> > Okay, but why?
> >
> > https://docs.kernel.org/process/submitting-patches.html#describe-your-changes
> >
> > [..]
> 
> I'll move more of the text from the cover letter into the commit
> message, but the short version is that it's intended to be a low-risk
> driver to pilot vendor drivers in Android.
> 

The answers I'm looking for isn't in your cover letter either. I want to
know what benefits Rust provides in this particular case.

> >
> > > ---
> > > base-commit: 4b31c90f86768367c84cc6e99e1deee0ec55197b
> > > change-id: 20251029-qcom-socinfo-d8387c7fdb1c
> > > prerequisite-change-id: 20251029-soc-bindings-9b0731bcdbed:v1
> > > prerequisite-patch-id: e4da423ddabec93bd9a64004691266f9b740e0c5
> > > prerequisite-patch-id: 5097ff622f8cb1d38354674cf70c1c946ac87f6c
> > > prerequisite-change-id: 20251029-add-entropy-f57e12ebe110:v1
> > > prerequisite-patch-id: 657de204912d2c5efff9898d3f4c51e52684d8e6
> > > prerequisite-change-id: 20251212-transmute-8ab6076700a8:v1
> > > prerequisite-patch-id: 4f5f7cb002d02d232083ab5c3ce6b3cb90650bd6
> > > prerequisite-patch-id: 01a1f3d727e549c173dd142180741f5b6a3c866e
> > > prerequisite-patch-id: ff801a7ae439876b1554b4d1132d94c825bbe74f
> >
> > So, it doesn't work on a clean v6.19-rc1 tree?
> 
> Correct. Those other patchsets are patchsets I'm landing which add
> bindings used in this one.
> 

Okay, please make that explicit when submitting patches, so that the
maintainers know that your patch(es) can't be merged.

Regards,
Bjorn

> >
> > Regards,
> > Bjorn
> >
> > >
> > > Best regards,
> > > --
> > > Matthew Maurer <mmaurer@...gle.com>
> > >