Re: [PATCH v4 2/9] rust: task: add `Task::current_raw`

From: Trevor Gross
Date: Mon Feb 05 2024 - 21:18:46 EST


On Fri, Feb 2, 2024 at 5:56 AM Alice Ryhl <aliceryhl@xxxxxxxxxx> wrote:
>
> Introduces a safe function for getting a raw pointer to the current
> task.
>
> When writing bindings that need to access the current task, it is often
> more convenient to call a method that directly returns a raw pointer
> than to use the existing `Task::current` method. However, the only way
> to do that is `bindings::get_current()` which is unsafe since it calls
> into C. By introducing `Task::current_raw()`, it becomes possible to
> obtain a pointer to the current task without using unsafe.
>
> Link: https://lore.kernel.org/all/CAH5fLgjT48X-zYtidv31mox3C4_Ogoo_2cBOCmX0Ang3tAgGHA@xxxxxxxxxxxxxx/
> Reviewed-by: Benno Lossin <benno.lossin@xxxxxxxxx>
> Signed-off-by: Alice Ryhl <aliceryhl@xxxxxxxxxx>
> ---

Reviewed-by: Trevor Gross <tmgross@xxxxxxxxx>