Re: [PATCH v11 1/5] sched/core: uclamp: Extend CPU's cgroup controller

2019-07-18 Thread Patrick Bellasi
On 18-Jul 07:52, Tejun Heo wrote: > Hello, Patrick. > > On Mon, Jul 08, 2019 at 09:43:53AM +0100, Patrick Bellasi wrote: > > +static inline void cpu_uclamp_print(struct seq_file *sf, > > + enum uclamp_id clamp_id) > > +{ > > + struct task_group *tg; > > + u64 util

Re: [PATCH v11 1/5] sched/core: uclamp: Extend CPU's cgroup controller

2019-07-18 Thread Tejun Heo
Hello, Patrick. On Mon, Jul 08, 2019 at 09:43:53AM +0100, Patrick Bellasi wrote: > +static inline void cpu_uclamp_print(struct seq_file *sf, > + enum uclamp_id clamp_id) > +{ > + struct task_group *tg; > + u64 util_clamp; > + u64 percent; > + u32 rem

Re: [PATCH v11 1/5] sched/core: uclamp: Extend CPU's cgroup controller

2019-07-15 Thread Patrick Bellasi
On 08-Jul 12:08, Quentin Perret wrote: > Hi Patrick, Hi Quentin! > On Monday 08 Jul 2019 at 09:43:53 (+0100), Patrick Bellasi wrote: > > +static inline int uclamp_scale_from_percent(char *buf, u64 *value) > > +{ > > + *value = SCHED_CAPACITY_SCALE; > > + > > + buf = strim(buf); > > + if (st

Re: [PATCH v11 1/5] sched/core: uclamp: Extend CPU's cgroup controller

2019-07-08 Thread Quentin Perret
Hi Patrick, On Monday 08 Jul 2019 at 09:43:53 (+0100), Patrick Bellasi wrote: > +static inline int uclamp_scale_from_percent(char *buf, u64 *value) > +{ > + *value = SCHED_CAPACITY_SCALE; > + > + buf = strim(buf); > + if (strncmp("max", buf, 4)) { > + s64 percent; > +

[PATCH v11 1/5] sched/core: uclamp: Extend CPU's cgroup controller

2019-07-08 Thread Patrick Bellasi
The cgroup CPU bandwidth controller allows to assign a specified (maximum) bandwidth to the tasks of a group. However this bandwidth is defined and enforced only on a temporal base, without considering the actual frequency a CPU is running on. Thus, the amount of computation completed by a task wit