linux-kernel - Re: [PATCH] rust: remove spurious compiler

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 for Android: free password hash cracker in your pocket

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

Message-ID: <CAJ-ks9m=OiqFXWDoQA+WAuTvASXWZwi+YcxHsihrgYHzpFS1CA@mail.gmail.com>
Date: Mon, 12 May 2025 16:21:33 -0400
From: Tamir Duberstein <tamird@...il.com>
To: Miguel Ojeda <ojeda@...nel.org>, Lukas Fischer <kernel@...o11oo.de>, 
	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>, 
	Danilo Krummrich <dakr@...nel.org>
Cc: rust-for-linux@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] rust: remove spurious compiler_builtins dependents

On Thu, Apr 24, 2025 at 6:40 AM Tamir Duberstein <tamird@...il.com> wrote:
>
> On Tue, Apr 8, 2025 at 2:52 PM Tamir Duberstein <tamird@...il.com> wrote:
> >
> > The dependency on `compiler_builtins.o` was first added in commit
> > 2f7ab1267dc9 ("Kbuild: add Rust support") to `alloc` which matches the
> > standard library[0] but was copied to other targets in:
> > - commit ecaa6ddff2fd ("rust: add `build_error` crate")
> > - commit d072acda4862 ("rust: use custom FFI integer types")
> > - commit 4e1746656839 ("rust: uapi: Add UAPI crate")
> > - commit d7659acca7a3 ("rust: add pin-init crate build infrastructure")
> >
> > The alloc crate was removed in commit 392e34b6bc22 ("kbuild: rust:
> > remove the `alloc` crate and `GlobalAlloc`"). As fas as I can tell none
> > of the other dependencies are required; it is only required that
> > compiler_builtins be linked into the rust-enabled kernel. In the
> > standard library, compiler_builtins is a dependency of std[1].
> >
> > Remove these dependency edges. Add a dependency edge from
> > `compiler_builtins` to `core` to `scripts/generate_rust_analyzer.py` to
> > match `rust/Makefile`. This has been incorrect since commit 8c4555ccc55c
> > ("scripts: add `generate_rust_analyzer.py`")
> >
> > Link: https://github.com/rust-lang/rust/blob/f820b75feef00654924c9351a2faca8d34818339/library/alloc/Cargo.toml#L19 [0]
> > Link: https://github.com/rust-lang/rust/blob/f820b75feef00654924c9351a2faca8d34818339/library/std/Cargo.toml#L21 [1]
> > Link: https://rust-for-linux.zulipchat.com/#narrow/channel/288089-General/topic/rust-analyzer.20improvements/near/510200959
> > Signed-off-by: Tamir Duberstein <tamird@...il.com>
>
> @Miguel do you have opinions on this?

Gentle ping.