[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CANiq72nkVmE_91yz82e1cRuJK8F9SPP5GpD1mT=NYLBbRTL8OQ@mail.gmail.com>
Date: Mon, 9 Feb 2026 13:48:01 +0100
From: Miguel Ojeda <miguel.ojeda.sandonis@...il.com>
To: "Christophe Leroy (CS GROUP)" <chleroy@...nel.org>
Cc: "Mukesh Kumar Chaurasiya (IBM)" <mkchauras@...il.com>, linkmauve@...kmauve.fr, ojeda@...nel.org,
boqun.feng@...il.com, gary@...yguo.net, bjorn3_gh@...tonmail.com,
lossin@...nel.org, a.hindborg@...nel.org, aliceryhl@...gle.com,
tmgross@...ch.edu, dakr@...nel.org, corbet@....net, maddy@...ux.ibm.com,
mpe@...erman.id.au, npiggin@...il.com, peterz@...radead.org,
jpoimboe@...nel.org, jbaron@...mai.com, rostedt@...dmis.org, ardb@...nel.org,
rust-for-linux@...r.kernel.org, linux-doc@...r.kernel.org,
linux-kernel@...r.kernel.org, linuxppc-dev@...ts.ozlabs.org
Subject: Re: [PATCH V4 2/3] rust: Add PowerPC support
On Mon, Feb 9, 2026 at 1:29 PM Christophe Leroy (CS GROUP)
<chleroy@...nel.org> wrote:
>
> As far as I understand the patch is complete, the problem is outside the
> kernel, it is in the libcore provided by rust or the rust compiler.
>
> That libcore seems to call __udivdi3() and __umoddi3() which are not
> provided by the kernel. Instead if should call the functions do_div()
> and friends provided by include/asm-generic/div64.h, or provide
> __udivdi3() and __umoddi3() on its own.
Yeah, but what I mean is, should we add this if it doesn't link,
especially without a note or comment of some kind? (Perhaps to your
tracking issue in GitHub?)
i.e. if I understand correctly, you prefer to not add the panicking
intrinsics to `rust/compiler_builtins.rs`, which I agree isn't ideal.
By the way, it would be nice if IBM could join one of the meetings
with have with upstream Rust to raise this topic again with them.
In any case, it is up to you, of course. My only concern on the Rust
bits is on the docs side, the rest looks fine.
Thanks!
Cheers,
Miguel
Powered by blists - more mailing lists