Re: [PATCH] sched/fair: Use rq->lock when checking cfs_rq list presence

From: Odin Ugedal
Date: Wed Oct 13 2021 - 14:52:50 EST


Hi,

tir. 12. okt. 2021 kl. 19:31 skrev Tao Zhou <tao.zhou@xxxxxxxxx>:
>
> Er.. this is considered specific to this fix I think. Normal unthrottle(not
> race with delete, avg maybe used in after) also need the normal avg decay.

Yeah, it was more meant as a way to express my idea, should probably have
said that more explicitly. It would essentially be a revert of a7b359fc6a37
("sched/fair: Correctly insert cfs_rq's to list on unthrottle"),
while "temporary" fixing the problem it fixed, by removing the load from the tg.

But yeah, it would still need to decay the load properly to _actually_ be fully
correct and work as it does today. But in most cases nr_running>0 on unthrottle,
but yeah, it is definitely not always the case.

Thanks
Odin