Re: [PATCH v3 4/6] sched/rt: fast switch to maximum frequency when RT tasks are scheduled

2017-12-06 Thread Vincent Guittot
On 6 December 2017 at 12:38, Patrick Bellasi wrote: > Hi Vincent, > > On 06-Dec 10:39, Vincent Guittot wrote: >> Hi Patrick, >> >> On 30 November 2017 at 12:47, Patrick Bellasi >> wrote: > > [...] > >> > static void dequeue_task_rt(struct rq

Re: [PATCH v3 4/6] sched/rt: fast switch to maximum frequency when RT tasks are scheduled

2017-12-06 Thread Vincent Guittot
On 6 December 2017 at 12:38, Patrick Bellasi wrote: > Hi Vincent, > > On 06-Dec 10:39, Vincent Guittot wrote: >> Hi Patrick, >> >> On 30 November 2017 at 12:47, Patrick Bellasi >> wrote: > > [...] > >> > static void dequeue_task_rt(struct rq *rq, struct task_struct *p, int >> > flags) >> > @@

Re: [PATCH v3 4/6] sched/rt: fast switch to maximum frequency when RT tasks are scheduled

2017-12-06 Thread Patrick Bellasi
Hi Vincent, On 06-Dec 10:39, Vincent Guittot wrote: > Hi Patrick, > > On 30 November 2017 at 12:47, Patrick Bellasi wrote: [...] > > static void dequeue_task_rt(struct rq *rq, struct task_struct *p, int > > flags) > > @@ -1564,6 +1564,9 @@ pick_next_task_rt(struct

Re: [PATCH v3 4/6] sched/rt: fast switch to maximum frequency when RT tasks are scheduled

2017-12-06 Thread Patrick Bellasi
Hi Vincent, On 06-Dec 10:39, Vincent Guittot wrote: > Hi Patrick, > > On 30 November 2017 at 12:47, Patrick Bellasi wrote: [...] > > static void dequeue_task_rt(struct rq *rq, struct task_struct *p, int > > flags) > > @@ -1564,6 +1564,9 @@ pick_next_task_rt(struct rq *rq, struct task_struct

Re: [PATCH v3 4/6] sched/rt: fast switch to maximum frequency when RT tasks are scheduled

2017-12-06 Thread Vincent Guittot
Hi Patrick, On 30 November 2017 at 12:47, Patrick Bellasi wrote: > Currently schedutil updates are triggered for the RT class using a single > call place, which is part of the rt::update_curr_rt() used in: > > - dequeue_task_rt: > but it does not make sense to set the

Re: [PATCH v3 4/6] sched/rt: fast switch to maximum frequency when RT tasks are scheduled

2017-12-06 Thread Vincent Guittot
Hi Patrick, On 30 November 2017 at 12:47, Patrick Bellasi wrote: > Currently schedutil updates are triggered for the RT class using a single > call place, which is part of the rt::update_curr_rt() used in: > > - dequeue_task_rt: > but it does not make sense to set the schedutil's

Re: [PATCH v3 4/6] sched/rt: fast switch to maximum frequency when RT tasks are scheduled

2017-11-30 Thread Juri Lelli
Hi, On 30/11/17 11:47, Patrick Bellasi wrote: > Currently schedutil updates are triggered for the RT class using a single > call place, which is part of the rt::update_curr_rt() used in: > > - dequeue_task_rt: > but it does not make sense to set the schedutil's SCHED_CPUFREQ_RT in > case the

Re: [PATCH v3 4/6] sched/rt: fast switch to maximum frequency when RT tasks are scheduled

2017-11-30 Thread Juri Lelli
Hi, On 30/11/17 11:47, Patrick Bellasi wrote: > Currently schedutil updates are triggered for the RT class using a single > call place, which is part of the rt::update_curr_rt() used in: > > - dequeue_task_rt: > but it does not make sense to set the schedutil's SCHED_CPUFREQ_RT in > case the

[PATCH v3 4/6] sched/rt: fast switch to maximum frequency when RT tasks are scheduled

2017-11-30 Thread Patrick Bellasi
Currently schedutil updates are triggered for the RT class using a single call place, which is part of the rt::update_curr_rt() used in: - dequeue_task_rt: but it does not make sense to set the schedutil's SCHED_CPUFREQ_RT in case the next task should not be an RT one - put_prev_task_rt:

[PATCH v3 4/6] sched/rt: fast switch to maximum frequency when RT tasks are scheduled

2017-11-30 Thread Patrick Bellasi
Currently schedutil updates are triggered for the RT class using a single call place, which is part of the rt::update_curr_rt() used in: - dequeue_task_rt: but it does not make sense to set the schedutil's SCHED_CPUFREQ_RT in case the next task should not be an RT one - put_prev_task_rt: