Re: [PATCH] sched/fair: util_est: fast ramp-up EWMA on utilization increases
From: Douglas Raillard
Date: Mon Oct 14 2019 - 12:16:07 EST
Hi Peter,
On 10/14/19 3:52 PM, Peter Zijlstra wrote:
The energy aware schedutil patches remimded me this was still pending.
On Fri, Aug 02, 2019 at 10:47:25AM +0100, Patrick Bellasi wrote:
Hi Peter, Vincent,
is there anything different I can do on this?
I think both Vincent and me are basically fine with the patch, it was
the Changelog/explanation for it that sat uneasy.
Specifically I think the 'confusion' around the PELT invariance stuff
doesn't help.
I think that if you present it simply as making util_est directly follow
upward motion and only decay on downward -- and the rationale for it --
then it should be fine.
random idea: Since these things are much easier to understand by looking at a graph
of util over time, we may agree on some mailing-list-friendly way to convey graphs.
For example, a simple CSV with:
* before/after delimiters (line of # or =)
* graph title
* one point per signal transition, so that it can be plotted with gnuplot style "steps" or matplotlib drawstyle='steps-post'
* consistent column names:
- time: in seconds (scientific notation for nanoseconds)
- activation: 1 when the task is actually running, 0 otherwise
(so it can be turned into transparent coloured bands like using gnuplot filledcurves, like in [1])
- util: util_avg of the task being talked about
The delimiters allow writing a scripts to render graphs directly out of an mbox file or ML archive URL.
This won't solve the issue for the commit message itself, but that may ease the ML discussions.
[1] https://lisa-linux-integrated-system-analysis.readthedocs.io/en/master/trace_analysis.html#lisa.analysis.tasks.TasksAnalysis.plot_task_activation
Cheers,
Douglas