Re: [PATCH v2] rust: sync: optimize rust symbol generation for CondVar
From: Miguel Ojeda
Date: Mon Mar 24 2025 - 05:38:46 EST
On Mon, Mar 24, 2025 at 7:19 AM Kunwu Chan <kunwu.chan@xxxxxxxxx> wrote:
>
> Link: https://github.com/Rust-for-Linux/linux/issues/1145
> Suggested-by: Alice Ryhl <aliceryhl@xxxxxxxxxx>
Nit: I typically use Link after Suggested-by, to mimic what the docs
require about Reported-by with Link. (No need to resend a new version
just for this :)
> - Remove '#[inline]' for notify()
It is good that the commit matches the log now, though I wonder if in
the future we may want the `#[inline]` for `notify` anyway, even if
LLVM does that one on its own today.
Cheers,
Miguel