Re: [PATCH v3 01/14] sched/core: uclamp: extend sched_setattr to support utilization clamping

2018-08-17 Thread Quentin Perret
On Friday 17 Aug 2018 at 11:57:31 (+0100), Patrick Bellasi wrote: > On 17-Aug 11:34, Quentin Perret wrote: > > Hi Patrick, > > > > On Thursday 09 Aug 2018 at 16:23:13 (+0100), Patrick Bellasi wrote: > > > On 09-Aug 11:50, Juri Lelli wrote: > > > > On 09/08/18 10:14, Patrick Bellasi wrote: > > > >

Re: [PATCH v3 01/14] sched/core: uclamp: extend sched_setattr to support utilization clamping

2018-08-17 Thread Patrick Bellasi
On 17-Aug 11:34, Quentin Perret wrote: > Hi Patrick, > > On Thursday 09 Aug 2018 at 16:23:13 (+0100), Patrick Bellasi wrote: > > On 09-Aug 11:50, Juri Lelli wrote: > > > On 09/08/18 10:14, Patrick Bellasi wrote: > > > > On 07-Aug 14:35, Juri Lelli wrote: > > > > > On 06/08/18 17:39, Patrick Bellas

Re: [PATCH v3 01/14] sched/core: uclamp: extend sched_setattr to support utilization clamping

2018-08-17 Thread Quentin Perret
Hi Patrick, On Thursday 09 Aug 2018 at 16:23:13 (+0100), Patrick Bellasi wrote: > On 09-Aug 11:50, Juri Lelli wrote: > > On 09/08/18 10:14, Patrick Bellasi wrote: > > > On 07-Aug 14:35, Juri Lelli wrote: > > > > On 06/08/18 17:39, Patrick Bellasi wrote: > > [...] > > > > 1) make CAP_SYS_NICE pro

Re: [PATCH v3 01/14] sched/core: uclamp: extend sched_setattr to support utilization clamping

2018-08-13 Thread Juri Lelli
On 13/08/18 13:14, Patrick Bellasi wrote: > On 07-Aug 11:59, Juri Lelli wrote: > > Hi, > > > > Minor comments below. > > > > On 06/08/18 17:39, Patrick Bellasi wrote: > > > > [...] > > > > > + * > > > + * Task Utilization Attributes > > > + * === > > > + * > > > + * A su

Re: [PATCH v3 01/14] sched/core: uclamp: extend sched_setattr to support utilization clamping

2018-08-13 Thread Patrick Bellasi
On 07-Aug 11:59, Juri Lelli wrote: > Hi, > > Minor comments below. > > On 06/08/18 17:39, Patrick Bellasi wrote: > > [...] > > > + * > > + * Task Utilization Attributes > > + * === > > + * > > + * A subset of sched_attr attributes allows to specify the utilization > > w

Re: [PATCH v3 01/14] sched/core: uclamp: extend sched_setattr to support utilization clamping

2018-08-10 Thread Juri Lelli
On 09/08/18 16:23, Patrick Bellasi wrote: > On 09-Aug 11:50, Juri Lelli wrote: > > On 09/08/18 10:14, Patrick Bellasi wrote: > > > On 07-Aug 14:35, Juri Lelli wrote: > > > > On 06/08/18 17:39, Patrick Bellasi wrote: > > [...] > > > > 1) make CAP_SYS_NICE protected the clamp groups, with an option

Re: [PATCH v3 01/14] sched/core: uclamp: extend sched_setattr to support utilization clamping

2018-08-09 Thread Patrick Bellasi
On 09-Aug 11:50, Juri Lelli wrote: > On 09/08/18 10:14, Patrick Bellasi wrote: > > On 07-Aug 14:35, Juri Lelli wrote: > > > On 06/08/18 17:39, Patrick Bellasi wrote: [...] > > 1) make CAP_SYS_NICE protected the clamp groups, with an optional boot > >time parameter to relax this check > > It

Re: [PATCH v3 01/14] sched/core: uclamp: extend sched_setattr to support utilization clamping

2018-08-09 Thread Randy Dunlap
On 08/09/2018 01:39 AM, Patrick Bellasi wrote: > On 06-Aug 09:50, Randy Dunlap wrote: >> Hi, > > Hi Randy, > >> On 08/06/2018 09:39 AM, Patrick Bellasi wrote: >>> diff --git a/init/Kconfig b/init/Kconfig >>> index 041f3a022122..1d45a6877d6f 100644 >>> --- a/init/Kconfig >>> +++ b/init/Kconfig >>>

Re: [PATCH v3 01/14] sched/core: uclamp: extend sched_setattr to support utilization clamping

2018-08-09 Thread Juri Lelli
On 09/08/18 10:14, Patrick Bellasi wrote: > On 07-Aug 14:35, Juri Lelli wrote: > > On 06/08/18 17:39, Patrick Bellasi wrote: > > > > [...] > > > > > @@ -4218,6 +4245,13 @@ static int __sched_setscheduler(struct task_struct > > > *p, > > > return retval; > > > } > > > > > >

Re: [PATCH v3 01/14] sched/core: uclamp: extend sched_setattr to support utilization clamping

2018-08-09 Thread Patrick Bellasi
On 07-Aug 14:35, Juri Lelli wrote: > On 06/08/18 17:39, Patrick Bellasi wrote: > > [...] > > > @@ -4218,6 +4245,13 @@ static int __sched_setscheduler(struct task_struct > > *p, > > return retval; > > } > > > > + /* Configure utilization clamps for the task */ > > +

Re: [PATCH v3 01/14] sched/core: uclamp: extend sched_setattr to support utilization clamping

2018-08-09 Thread Patrick Bellasi
On 06-Aug 09:50, Randy Dunlap wrote: > Hi, Hi Randy, > On 08/06/2018 09:39 AM, Patrick Bellasi wrote: > > diff --git a/init/Kconfig b/init/Kconfig > > index 041f3a022122..1d45a6877d6f 100644 > > --- a/init/Kconfig > > +++ b/init/Kconfig > > @@ -583,6 +583,25 @@ config HAVE_UNSTABLE_SCHED_CLOCK >

Re: [PATCH v3 01/14] sched/core: uclamp: extend sched_setattr to support utilization clamping

2018-08-07 Thread Juri Lelli
On 06/08/18 17:39, Patrick Bellasi wrote: [...] > @@ -4218,6 +4245,13 @@ static int __sched_setscheduler(struct task_struct *p, > return retval; > } > > + /* Configure utilization clamps for the task */ > + if (attr->sched_flags & SCHED_FLAG_UTIL_CLAMP) { > +

Re: [PATCH v3 01/14] sched/core: uclamp: extend sched_setattr to support utilization clamping

2018-08-07 Thread Juri Lelli
Hi, Minor comments below. On 06/08/18 17:39, Patrick Bellasi wrote: [...] > + * > + * Task Utilization Attributes > + * === > + * > + * A subset of sched_attr attributes allows to specify the utilization which > + * should be expected by a task. These attributes allows t

Re: [PATCH v3 01/14] sched/core: uclamp: extend sched_setattr to support utilization clamping

2018-08-06 Thread Randy Dunlap
Hi, On 08/06/2018 09:39 AM, Patrick Bellasi wrote: > diff --git a/init/Kconfig b/init/Kconfig > index 041f3a022122..1d45a6877d6f 100644 > --- a/init/Kconfig > +++ b/init/Kconfig > @@ -583,6 +583,25 @@ config HAVE_UNSTABLE_SCHED_CLOCK > config GENERIC_SCHED_CLOCK > bool > > +menu "Schedule

[PATCH v3 01/14] sched/core: uclamp: extend sched_setattr to support utilization clamping

2018-08-06 Thread Patrick Bellasi
The SCHED_DEADLINE scheduling class provides an advanced and formal model to define tasks requirements which can be translated into proper decisions for both task placements and frequencies selections. Other classes have a more simplified model which is essentially based on the relatively simple co