On 08-Jun 15:26, Jon Medhurst (Tixy) wrote:
On Wed, 2016-06-08 at 14:51 +0100, Ryan Harkin wrote:
On 8 June 2016 at 10:39, Chris Redpath Chris.Redpath@arm.com wrote:
[...]
I've been toying with the idea of having an EAS-Android.conf as well, which is where we could put these things which are probably necessary for Android builds but not necessarily for barebones - what do you think about having an additional config fragment?
In theory, android.conf should have everything Android needs; it has CGROUPS already. Then EAS.conf all the EAS specific stuff. Unless there are EAS things you want to turn on/off for Android and not for other distros?
Tixy would have a better view than me about how to split these, so it's best to work with him.
android.conf has the configs AOSP recommend, I don't want to modify it for EAS. So if there really is Android specific configuration for EAS then yes, we could have an extra config fragment for that. (Though it seems strange to me that EAS would be designed to be tuned by kernel configs rather than by system parameters determined at run time).
Indeed, it is not. We are talking about a couple of configuration options which are related to SchedTune, which has two different working mode: global boosting vs per-task boosting (the second mode is an extension of the first one, with a slightly different userspace API).
The global boosting mode is an option provided just to allows SchedTune support on system compiled without CGroups support. Do they exists? Basically it provides the minimum functionalities for example to tune SchedFreq to be more like the ondemand or the performance governors.
The per-task boosting support is definitively the most interesting one, especially when it comes down to Android integration. Thus, IMO a default configuration for all systems in general should be SchedTune with per-task support, which requires CGroups enabled.
Cheers Patrick
-- Tixy
eas-dev mailing list eas-dev@lists.linaro.org https://lists.linaro.org/mailman/listinfo/eas-dev
-- #include <best/regards.h>
Patrick Bellasi