Hi Thara,
On Mon, Dec 19, 2016 at 10:17:29AM -0500, Thara Gopinath wrote:
[...]
But what is missing is handling of misfit task. Can we not handle misfit task as a separate condition in update_sd_lb? i.e in the above example if either CPU A or CPU B has a misfit task, set the overutilization flag for the next level SD which is equivalent to setting the flag in RD in this case.
Agree, we can do this for misfit task :)
IIUC, the idea of your patch is firstly to use SD level 2 flag to present "inner" overutilized, then later in load balance flow to check if need set rd->overutilized flag for outer 'overutilized'. So for 'misfit' case, we need wait until load balance flow to check it and set rd->overutilized flag.
This is why I suggest to use 'discrete' flags in corresponding SD level to present outer 'overutilized', so we can set flag at the first place for outer 'overutilized' but not delay until in load balance flow.
Thanks, Leo Yan