On Fri, Nov 26 2021 at 22:59, Peter Zijlstra wrote: > On Fri, Nov 26, 2021 at 10:08:14PM +0100, Thomas Gleixner wrote: >> > + if (timo) >> > + hrtimer_sleeper_start_expires(&timeout, HRTIMER_MODE_ABS); >> > + >> > + freezable_schedule(); >> >> You can replace the whole hrtimer foo with >> >> if (!schedule_hrtimeout_range_clock(timo ? &timo : NULL, >> tsk->timer_slack_ns, >> HRTIMER_MODE_ABS, >> tsk->umcg_clock)) { >> ret = -ETIMEOUT; >> break; >> } > > That seems to loose the freezable crud.. then again, since we're > interruptible, that shouldn't matter. Lemme go do that. We could add a freezable wrapper for that if necessary. Thanks, tglx