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