Messages in this thread | | | Date | Thu, 27 Sep 2018 11:00:40 +0100 | From | Quentin Perret <> | Subject | Re: [PATCH v4 14/16] sched/core: uclamp: request CAP_SYS_ADMIN by default |
| |
On Tuesday 25 Sep 2018 at 17:49:56 (+0200), Peter Zijlstra wrote: > I really don't see how changing the unit changes anything. Either you > want to relate to OPPs and those are exposed in 1/1024 unit capacity > through the EAS files, or you don't and then the knob has no meaning.
FWIW, with the latest versions of the EAS series, we don't expose the capacity of the various OPPs directly any more. You have 'frequency', 'power' and a more abstract thing called 'cost' (which is useful for energy computations) in the sysfs files of the EM.
We decided to remove the 'capacity' file to simplify things quite a bit, and because it wasn't helping much. I'm talking about this discussion w/ Dietmar on v4:
https://lore.kernel.org/lkml/20180717141955.GA4496@e108498-lin.cambridge.arm.com/
But it is also true that we could add back that 'capacity' file if it makes sense for uclamp. It's just an additional show() function in energy_model.c, so no problem on that side per se.
The only problem is if you want to use uclamp w/o EAS, I guess ...
Thanks, Quentin
| |