On Tue, Jul 07, 2026 at 12:47:01PM -0700, Stanislav Kinsburskii wrote:

> +int hmm_range_fault_unlocked_timeout(struct hmm_range *range,
> +                                  unsigned long timeout)
> +{
> +     struct mm_struct *mm = range->notifier->mm;
> +     unsigned long deadline = 0;
> +     int locked, ret;
> +
> +     if (timeout)
> +             deadline = jiffies + timeout;
> +
> +     do {
> +             if (fatal_signal_pending(current))
> +                     return -EINTR;
> +
> +             if (timeout && time_after(jiffies, deadline))
> +                     return -EBUSY;

I really dislike there is a timeout here, HMM is supposed to be more
deterministic. GUP doesn't have a timeout, what is this about?

Jason

Reply via email to