Messages in this thread | | | Date | Mon, 10 Jul 2017 12:24:43 +0530 | From | Viresh Kumar <> | Subject | Re: [PATCH v2 02/10] cpufreq: provide data for frequency-invariant load-tracking support |
| |
On 08-07-17, 14:09, Rafael J. Wysocki wrote: > I'm sort of wondering how many is "a lot" really. For instance, do you really > want all of the existing ARM platforms to use the new stuff even though > it may regress things there in principle?
That's a valid question and we must (maybe we already have) have a policy for such changes. I thought that such changes (which are so closely bound to the scheduler) must be at least done at the architecture level and not really at platform level. And so doing it widely (like done in this patch) maybe the right thing to do.
> Anyway, if everyone agrees that doing it in the core is the way to go (Peter?), > why don't you introduce a __weak function for setting policy->cur and > override it from your arch so as to call arch_set_freq_scale() from there?
I agree. I wanted to suggest that earlier but somehow forgot to mention this.
-- viresh
| |