Re: [PATCH v5 7/7][Update] cpufreq: schedutil: New governor based on scheduler utilization data

2016-03-19 Thread Juri Lelli
Hi Rafael, On 17/03/16 01:01, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki [...] > +static void sugov_update_commit(struct sugov_policy *sg_policy, u64 time, > + unsigned int next_freq) > +{ > + struct cpufreq_policy *policy =

Re: [PATCH v5 7/7][Update] cpufreq: schedutil: New governor based on scheduler utilization data

2016-03-19 Thread Juri Lelli
Hi Rafael, On 17/03/16 01:01, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki [...] > +static void sugov_update_commit(struct sugov_policy *sg_policy, u64 time, > + unsigned int next_freq) > +{ > + struct cpufreq_policy *policy = sg_policy->policy; > + > +

Re: [PATCH v5 7/7][Update] cpufreq: schedutil: New governor based on scheduler utilization data

2016-03-19 Thread Rafael J. Wysocki
On Thu, Mar 17, 2016 at 12:36 PM, Peter Zijlstra wrote: > On Thu, Mar 17, 2016 at 01:01:45AM +0100, Rafael J. Wysocki wrote: >> + } else if (sg_policy->next_freq != next_freq) { >> + sg_policy->work_cpu = smp_processor_id(); >> +

Re: [PATCH v5 7/7][Update] cpufreq: schedutil: New governor based on scheduler utilization data

2016-03-19 Thread Rafael J. Wysocki
On Thu, Mar 17, 2016 at 12:36 PM, Peter Zijlstra wrote: > On Thu, Mar 17, 2016 at 01:01:45AM +0100, Rafael J. Wysocki wrote: >> + } else if (sg_policy->next_freq != next_freq) { >> + sg_policy->work_cpu = smp_processor_id(); >> + irq_work_queue(_policy->irq_work); >> +

Re: [PATCH v5 7/7][Update] cpufreq: schedutil: New governor based on scheduler utilization data

2016-03-19 Thread Peter Zijlstra
On Thu, Mar 17, 2016 at 01:01:45AM +0100, Rafael J. Wysocki wrote: > + } else if (sg_policy->next_freq != next_freq) { > + sg_policy->work_cpu = smp_processor_id(); > + irq_work_queue(_policy->irq_work); > + } > +} > +static void sugov_irq_work(struct irq_work

Re: [PATCH v5 7/7][Update] cpufreq: schedutil: New governor based on scheduler utilization data

2016-03-19 Thread Peter Zijlstra
On Thu, Mar 17, 2016 at 01:01:45AM +0100, Rafael J. Wysocki wrote: > + } else if (sg_policy->next_freq != next_freq) { > + sg_policy->work_cpu = smp_processor_id(); > + irq_work_queue(_policy->irq_work); > + } > +} > +static void sugov_irq_work(struct irq_work

[PATCH v5 7/7][Update] cpufreq: schedutil: New governor based on scheduler utilization data

2016-03-19 Thread Rafael J. Wysocki
From: Rafael J. Wysocki Add a new cpufreq scaling governor, called "schedutil", that uses scheduler-provided CPU utilization information as input for making its decisions. Doing that is possible after commit 34e2c555f3e1 (cpufreq: Add mechanism for registering

[PATCH v5 7/7][Update] cpufreq: schedutil: New governor based on scheduler utilization data

2016-03-19 Thread Rafael J. Wysocki
From: Rafael J. Wysocki Add a new cpufreq scaling governor, called "schedutil", that uses scheduler-provided CPU utilization information as input for making its decisions. Doing that is possible after commit 34e2c555f3e1 (cpufreq: Add mechanism for registering utilization update callbacks) that

Re: [PATCH v5 7/7][Update] cpufreq: schedutil: New governor based on scheduler utilization data

2016-03-18 Thread Rafael J. Wysocki
On Thu, Mar 17, 2016 at 12:30 PM, Juri Lelli wrote: > Hi Rafael, > > On 17/03/16 01:01, Rafael J. Wysocki wrote: >> From: Rafael J. Wysocki > > [...] > >> +static void sugov_update_commit(struct sugov_policy *sg_policy, u64 time, >> +

Re: [PATCH v5 7/7][Update] cpufreq: schedutil: New governor based on scheduler utilization data

2016-03-18 Thread Rafael J. Wysocki
On Thu, Mar 17, 2016 at 12:30 PM, Juri Lelli wrote: > Hi Rafael, > > On 17/03/16 01:01, Rafael J. Wysocki wrote: >> From: Rafael J. Wysocki > > [...] > >> +static void sugov_update_commit(struct sugov_policy *sg_policy, u64 time, >> + unsigned int next_freq) >> +{ >>