Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1349570

Re: [PATCH 5/6] cpufreq: Support for fast frequency switching

Show all headers | View raw

On Thu, Mar 3, 2016 at 12:16 PM, Peter Zijlstra <peterz@infradead.org> wrote:
> On Wed, Mar 02, 2016 at 03:12:33AM +0100, Rafael J. Wysocki wrote:
>> The most important change from the previous version is that the
>> ->fast_switch() callback takes an additional "relation" argument
>> and now the governor can use it to choose a selection method.
>
>> +unsigned int acpi_cpufreq_fast_switch(struct cpufreq_policy *policy,
>> +                                   unsigned int target_freq,
>> +                                   unsigned int relation)
>
> Would it make sense to replace the {target_freq, relation} pair with
> something like the CPPC {min_freq, max_freq} pair?

Yes, it would in general, but since I use __cpufreq_driver_target() in
the "slow driver" case, that would need to be reworked too for
consistency.  So I'd prefer to do that later.

> Then you could use the closest frequency to max provided it is larger
> than min.
>
> This communicates more actual information in the same number of
> parameters and would thereby allow for a more flexible (better)
> frequency selection.

Agreed.

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 5/6] cpufreq: Support for fast frequency switching "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2016-03-02 03:30 +0100
  Re: [PATCH 5/6] cpufreq: Support for fast frequency switching Viresh Kumar <viresh.kumar@linaro.org> - 2016-03-03 07:10 +0100
    Re: [PATCH 5/6] cpufreq: Support for fast frequency switching "Rafael J. Wysocki" <rafael@kernel.org> - 2016-03-04 03:20 +0100
  Re: [PATCH 5/6] cpufreq: Support for fast frequency switching Peter Zijlstra <peterz@infradead.org> - 2016-03-03 12:20 +0100
    Re: [PATCH 5/6] cpufreq: Support for fast frequency switching "Rafael J. Wysocki" <rafael@kernel.org> - 2016-03-03 20:40 +0100
  Re: [PATCH 5/6] cpufreq: Support for fast frequency switching Peter Zijlstra <peterz@infradead.org> - 2016-03-03 12:20 +0100
    Re: [PATCH 5/6] cpufreq: Support for fast frequency switching "Rafael J. Wysocki" <rafael@kernel.org> - 2016-03-03 22:00 +0100
      Re: [PATCH 5/6] cpufreq: Support for fast frequency switching Peter Zijlstra <peterz@infradead.org> - 2016-03-03 22:20 +0100

csiph-web