Re: [PATCH] rust: workqueue: fix SAFETY comment Arc refs in Pin<KBox<T>>

From: Miguel Ojeda

Date: Sun Apr 26 2026 - 09:39:26 EST


On Sat, Apr 25, 2026 at 3:57 PM Sagar Taunk <sagartaunk2@xxxxxxxxx> wrote:
>
> The `WorkItemPointer` implementation for `Pin<KBox<T>>` contained SAFETY
> comments that incorrectly referenced `Arc::into_raw` instead of
> `KBox::into_raw`. This implementation uses `KBox`, not `Arc`, so update
> the comments to accurately reflect the actual ownership transfer.
>
> Suggested-by: Onur Özkan <contact@xxxxxxxxxxxxx>
> Link: https://github.com/Rust-for-Linux/linux/issues/1233
> Signed-off-by: Sagar Taunk <sagartaunk2@xxxxxxxxx>

I am copying a procedural comment since I just wrote it for another patch:

For this case it doesn't matter too much, but in general, when
something is a fix, we add a Fixes: tag (and in most cases Cc: stable@
to accompany it). That helps backporting (when needed), adds context
e.g. for reviewing (at times), etc.

Thanks!

Cheers,
Miguel