Re: [PATCH V2] powercap/drivers/idle_injection: Add an idle injection framework

2018-05-15 Thread Daniel Lezcano
On Tue, May 15, 2018 at 10:42:41AM +0530, viresh kumar wrote: > On 11-05-18, 13:55, Daniel Lezcano wrote: > > On Fri, May 11, 2018 at 03:02:21PM +0530, viresh kumar wrote: > > > On 10-05-18, 14:26, Daniel Lezcano wrote: > > > > +int idle_injection_start(struct idle_injection_device *ii_dev) > > >

Re: [PATCH V2] powercap/drivers/idle_injection: Add an idle injection framework

2018-05-15 Thread Daniel Lezcano
On Tue, May 15, 2018 at 10:42:41AM +0530, viresh kumar wrote: > On 11-05-18, 13:55, Daniel Lezcano wrote: > > On Fri, May 11, 2018 at 03:02:21PM +0530, viresh kumar wrote: > > > On 10-05-18, 14:26, Daniel Lezcano wrote: > > > > +int idle_injection_start(struct idle_injection_device *ii_dev) > > >

Re: [PATCH V2] powercap/drivers/idle_injection: Add an idle injection framework

2018-05-14 Thread Viresh Kumar
On 11-05-18, 13:55, Daniel Lezcano wrote: > On Fri, May 11, 2018 at 03:02:21PM +0530, viresh kumar wrote: > > On 10-05-18, 14:26, Daniel Lezcano wrote: > > > +int idle_injection_start(struct idle_injection_device *ii_dev) > > > +{ > > > + if (!atomic_read(_dev->idle_duration_ms)) > > > +

Re: [PATCH V2] powercap/drivers/idle_injection: Add an idle injection framework

2018-05-14 Thread Viresh Kumar
On 11-05-18, 13:55, Daniel Lezcano wrote: > On Fri, May 11, 2018 at 03:02:21PM +0530, viresh kumar wrote: > > On 10-05-18, 14:26, Daniel Lezcano wrote: > > > +int idle_injection_start(struct idle_injection_device *ii_dev) > > > +{ > > > + if (!atomic_read(_dev->idle_duration_ms)) > > > +

Re: [PATCH V2] powercap/drivers/idle_injection: Add an idle injection framework

2018-05-11 Thread Daniel Lezcano
On Fri, May 11, 2018 at 03:02:21PM +0530, viresh kumar wrote: > On 10-05-18, 14:26, Daniel Lezcano wrote: > > Initially, the cpu_cooling device for ARM was changed by adding a new > > policy inserting idle cycles. The intel_powerclamp driver does a > > similar action. > > > > Instead of

Re: [PATCH V2] powercap/drivers/idle_injection: Add an idle injection framework

2018-05-11 Thread Daniel Lezcano
On Fri, May 11, 2018 at 03:02:21PM +0530, viresh kumar wrote: > On 10-05-18, 14:26, Daniel Lezcano wrote: > > Initially, the cpu_cooling device for ARM was changed by adding a new > > policy inserting idle cycles. The intel_powerclamp driver does a > > similar action. > > > > Instead of

Re: [PATCH V2] powercap/drivers/idle_injection: Add an idle injection framework

2018-05-11 Thread Viresh Kumar
On 10-05-18, 14:26, Daniel Lezcano wrote: > Initially, the cpu_cooling device for ARM was changed by adding a new > policy inserting idle cycles. The intel_powerclamp driver does a > similar action. > > Instead of implementing idle injections privately in the cpu_cooling > device, move the idle

Re: [PATCH V2] powercap/drivers/idle_injection: Add an idle injection framework

2018-05-11 Thread Viresh Kumar
On 10-05-18, 14:26, Daniel Lezcano wrote: > Initially, the cpu_cooling device for ARM was changed by adding a new > policy inserting idle cycles. The intel_powerclamp driver does a > similar action. > > Instead of implementing idle injections privately in the cpu_cooling > device, move the idle

[PATCH V2] powercap/drivers/idle_injection: Add an idle injection framework

2018-05-10 Thread Daniel Lezcano
Initially, the cpu_cooling device for ARM was changed by adding a new policy inserting idle cycles. The intel_powerclamp driver does a similar action. Instead of implementing idle injections privately in the cpu_cooling device, move the idle injection code in a dedicated framework and give the

[PATCH V2] powercap/drivers/idle_injection: Add an idle injection framework

2018-05-10 Thread Daniel Lezcano
Initially, the cpu_cooling device for ARM was changed by adding a new policy inserting idle cycles. The intel_powerclamp driver does a similar action. Instead of implementing idle injections privately in the cpu_cooling device, move the idle injection code in a dedicated framework and give the