Re: [PATCH v8 -tip 08/26] sched/fair: Snapshot the min_vruntime of CPUs on force idle

From: Peter Zijlstra
Date: Fri Oct 30 2020 - 04:42:22 EST


On Thu, Oct 29, 2020 at 10:42:29PM -0400, Joel Fernandes wrote:

> Forgot to ask, do you also need to do the task_vruntime_update() for
> the unconstrained pick?

Humm.. interesting case.

Yes, however... since in that case the picks aren't synchronized it's a
wee bit dodgy. I'll throw it on the pile together with SMT4.


Also, I'm still hoping you can make this form work:

https://lkml.kernel.org/r/20201026093131.GF2628@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx

(note that the put_prev_task() needs an additional rq argument)

That's both simpler code and faster.