Re: [PATCH 3/6] cpufreq: schedutil: ensure max frequency while running RT/DL tasks

2017-04-07 Thread Patrick Bellasi
On 07-Apr 17:30, Peter Zijlstra wrote: > On Thu, Mar 02, 2017 at 03:45:04PM +, Patrick Bellasi wrote: > > + struct task_struct *curr = cpu_curr(smp_processor_id()); > > Isn't that a weird way of writing 'current' ? Right... (cough)... it's a new fangled way. :-/ Will cleanup before reposti

Re: [PATCH 3/6] cpufreq: schedutil: ensure max frequency while running RT/DL tasks

2017-04-07 Thread Peter Zijlstra
On Thu, Mar 02, 2017 at 03:45:04PM +, Patrick Bellasi wrote: > + struct task_struct *curr = cpu_curr(smp_processor_id()); Isn't that a weird way of writing 'current' ?

Re: [PATCH 3/6] cpufreq: schedutil: ensure max frequency while running RT/DL tasks

2017-03-17 Thread Patrick Bellasi
On 16-Mar 00:32, Rafael J. Wysocki wrote: > On Wed, Mar 15, 2017 at 3:40 PM, Patrick Bellasi > wrote: > > On 15-Mar 12:52, Rafael J. Wysocki wrote: > >> On Friday, March 03, 2017 12:38:30 PM Patrick Bellasi wrote: > >> > On 03-Mar 14:01, Viresh Kumar wrote: > >> > > On 02-03-17, 15:45, Patrick Bel

Re: [PATCH 3/6] cpufreq: schedutil: ensure max frequency while running RT/DL tasks

2017-03-15 Thread Rafael J. Wysocki
On Wed, Mar 15, 2017 at 3:40 PM, Patrick Bellasi wrote: > On 15-Mar 12:52, Rafael J. Wysocki wrote: >> On Friday, March 03, 2017 12:38:30 PM Patrick Bellasi wrote: >> > On 03-Mar 14:01, Viresh Kumar wrote: >> > > On 02-03-17, 15:45, Patrick Bellasi wrote: >> > > > diff --git a/kernel/sched/cpufreq

Re: [PATCH 3/6] cpufreq: schedutil: ensure max frequency while running RT/DL tasks

2017-03-15 Thread Patrick Bellasi
On 15-Mar 12:52, Rafael J. Wysocki wrote: > On Friday, March 03, 2017 12:38:30 PM Patrick Bellasi wrote: > > On 03-Mar 14:01, Viresh Kumar wrote: > > > On 02-03-17, 15:45, Patrick Bellasi wrote: > > > > diff --git a/kernel/sched/cpufreq_schedutil.c > > > > b/kernel/sched/cpufreq_schedutil.c > > >

Re: [PATCH 3/6] cpufreq: schedutil: ensure max frequency while running RT/DL tasks

2017-03-15 Thread Rafael J. Wysocki
On Friday, March 03, 2017 12:38:30 PM Patrick Bellasi wrote: > On 03-Mar 14:01, Viresh Kumar wrote: > > On 02-03-17, 15:45, Patrick Bellasi wrote: > > > diff --git a/kernel/sched/cpufreq_schedutil.c > > > b/kernel/sched/cpufreq_schedutil.c > > > @@ -293,15 +305,29 @@ static void sugov_update_share

Re: [PATCH 3/6] cpufreq: schedutil: ensure max frequency while running RT/DL tasks

2017-03-03 Thread Patrick Bellasi
On 03-Mar 14:01, Viresh Kumar wrote: > On 02-03-17, 15:45, Patrick Bellasi wrote: > > diff --git a/kernel/sched/cpufreq_schedutil.c > > b/kernel/sched/cpufreq_schedutil.c > > @@ -293,15 +305,29 @@ static void sugov_update_shared(struct > > update_util_data *hook, u64 time, > > if (curr == sg_

Re: [PATCH 3/6] cpufreq: schedutil: ensure max frequency while running RT/DL tasks

2017-03-03 Thread Viresh Kumar
On 02-03-17, 15:45, Patrick Bellasi wrote: > diff --git a/kernel/sched/cpufreq_schedutil.c > b/kernel/sched/cpufreq_schedutil.c > @@ -293,15 +305,29 @@ static void sugov_update_shared(struct update_util_data > *hook, u64 time, > if (curr == sg_policy->thread) > goto done; >

[PATCH 3/6] cpufreq: schedutil: ensure max frequency while running RT/DL tasks

2017-03-02 Thread Patrick Bellasi
The policy in use for RT/DL tasks sets the maximum frequency when a task in these classes calls for a cpufreq_update_this_cpu(). However, the current implementation might cause a frequency drop while a RT/DL task is still running, just because for example a FAIR task wakes up and is enqueued in th