[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CANiq72nTiKkL-QC245Te3D-RRQmzVwbFRsc1aacUJXX91REFJg@mail.gmail.com>
Date: Sun, 23 Mar 2025 23:38:31 +0100
From: Miguel Ojeda <miguel.ojeda.sandonis@...il.com>
To: Tamir Duberstein <tamird@...il.com>
Cc: 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>,
Danilo Krummrich <dakr@...nel.org>, rust-for-linux@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2] rust: macros: fix `make rusttest` build on macOS
On Mon, Feb 10, 2025 at 3:51 PM Tamir Duberstein <tamird@...il.com> wrote:
>
> Do not emit `#[link_section = ".modinfo"]` on macOS (i.e. when building
> userspace tests); .modinfo is not a legal section specifier in mach-o.
>
> Before this change tests failed to compile:
>
> ---- ../rust/macros/lib.rs - module (line 66) stdout ----
> rustc-LLVM ERROR: Global variable '_ZN8rust_out13__module_init13__module_init27__MY_DEVICE_DRIVER_MODULE_017h141f80536770e0d4E' has an invalid section specifier '.modinfo': mach-o section specifier requires a segment and section separated by a comma.
> Couldn't compile the test.
> ---- ../rust/macros/lib.rs - module (line 33) stdout ----
> rustc-LLVM ERROR: Global variable '_ZN8rust_out13__module_init13__module_init20__MY_KERNEL_MODULE_017h5d79189564b41e07E' has an invalid section specifier '.modinfo': mach-o section specifier requires a segment and section separated by a comma.
> Couldn't compile the test.
>
> Signed-off-by: Tamir Duberstein <tamird@...il.com>
Applied to `rust-next` -- thanks!
Given the `cfg_attr`, I think this should not change/break anything
non-macOS we have, so I think it should be safe to apply.
Cheers,
Miguel
Powered by blists - more mailing lists