Re: [PATCH v2 1/5] rust: task: add safe schedule_timeout() wrapper
From: Miguel Ojeda
Date: Tue Nov 10 2026 - 08:02:37 EST
On Thu, Nov 5, 2026 at 2:00 AM Danilo Krummrich <dakr@xxxxxxxxxx> wrote:
>
> +/// Timeouts at or above [`MAX_SCHEDULE_TIMEOUT`] mean an indefinite wait. In that case, the return
> +/// value is `MAX_SCHEDULE_TIMEOUT` as [`Jiffies`].
[`MAX_SCHEDULE_TIMEOUT`]
By the way, should we even allow higher values? Clamping is OK, but I
wonder if we should take the chance to make those EB by adding a
`debug_assert!`, before we have callers relying on that.
With that considered:
Acked-by: Miguel Ojeda <ojeda@xxxxxxxxxx>
Cheers,
Miguel