Messages in this thread | | | Date | Wed, 22 Oct 2014 12:28:57 +0200 | From | Peter Zijlstra <> | Subject | Re: [RESEND PATCH 2/3 v5] sched: Rewrite per entity runnable load average tracking |
| |
So I would really like to hear from others on this, as it stands I tend to rather like this code -- and it removes a fair amount of icky lines of code.
That said, it does change semantics slightly in that it removes the blocked/runnable split and last time we added the blocked component to the load balancer we had regressions -- now Yuyand and Wu ran this through some testing (AIM7 iirc) and didn't see horrid, but I can't quite remember where we saw the biggest fail back when.
Also, Morten is a little worried about changing all this code seeing how the existing stuff has proven itself over the past few years and feels its somewhat understood. Then again, going by that we'd never change anything.
Ben, Paul, can you guys give this a go?
| |