Re: [PATCH 2/2] sched: cpufreq: use rt_avg as estimate of required RT CPU capacity

2016-09-02 Thread Thomas Gleixner
On Fri, 2 Sep 2016, Juri Lelli wrote: > On 01/09/16 14:48, Steve Muckle wrote: > > On Wed, Aug 31, 2016 at 06:00:02PM +0100, Juri Lelli wrote: > > > > Another problem is that we have many semi related knobs; we have the > > > > global RT runtime limit knob, but that doesn't affect cpufreq (maybe

Re: [PATCH 2/2] sched: cpufreq: use rt_avg as estimate of required RT CPU capacity

2016-09-02 Thread Thomas Gleixner
On Fri, 2 Sep 2016, Juri Lelli wrote: > On 01/09/16 14:48, Steve Muckle wrote: > > On Wed, Aug 31, 2016 at 06:00:02PM +0100, Juri Lelli wrote: > > > > Another problem is that we have many semi related knobs; we have the > > > > global RT runtime limit knob, but that doesn't affect cpufreq (maybe

Re: [PATCH 2/2] sched: cpufreq: use rt_avg as estimate of required RT CPU capacity

2016-09-02 Thread Juri Lelli
On 01/09/16 14:48, Steve Muckle wrote: > On Wed, Aug 31, 2016 at 06:00:02PM +0100, Juri Lelli wrote: > > > Another problem is that we have many semi related knobs; we have the > > > global RT runtime limit knob, but that doesn't affect cpufreq (maybe it > > > should) > > > > Maybe we could create

Re: [PATCH 2/2] sched: cpufreq: use rt_avg as estimate of required RT CPU capacity

2016-09-02 Thread Juri Lelli
On 01/09/16 14:48, Steve Muckle wrote: > On Wed, Aug 31, 2016 at 06:00:02PM +0100, Juri Lelli wrote: > > > Another problem is that we have many semi related knobs; we have the > > > global RT runtime limit knob, but that doesn't affect cpufreq (maybe it > > > should) > > > > Maybe we could create

Re: [PATCH 2/2] sched: cpufreq: use rt_avg as estimate of required RT CPU capacity

2016-09-02 Thread Thomas Gleixner
On Wed, 31 Aug 2016, Peter Zijlstra wrote: > On Wed, Aug 31, 2016 at 06:28:10PM +0200, Thomas Gleixner wrote: > > > That is the way it's been with cpufreq and many systems (including all > > > mobile devices) rely on that to not destroy power. RT + variable cpufreq > > > is not deterministic. > >

Re: [PATCH 2/2] sched: cpufreq: use rt_avg as estimate of required RT CPU capacity

2016-09-02 Thread Thomas Gleixner
On Wed, 31 Aug 2016, Peter Zijlstra wrote: > On Wed, Aug 31, 2016 at 06:28:10PM +0200, Thomas Gleixner wrote: > > > That is the way it's been with cpufreq and many systems (including all > > > mobile devices) rely on that to not destroy power. RT + variable cpufreq > > > is not deterministic. > >

Re: [PATCH 2/2] sched: cpufreq: use rt_avg as estimate of required RT CPU capacity

2016-09-01 Thread Steve Muckle
On Wed, Aug 31, 2016 at 06:00:02PM +0100, Juri Lelli wrote: > > Another problem is that we have many semi related knobs; we have the > > global RT runtime limit knob, but that doesn't affect cpufreq (maybe it > > should) > > Maybe we could create this sort of link when using the cgroup RT >

Re: [PATCH 2/2] sched: cpufreq: use rt_avg as estimate of required RT CPU capacity

2016-09-01 Thread Steve Muckle
On Wed, Aug 31, 2016 at 06:00:02PM +0100, Juri Lelli wrote: > > Another problem is that we have many semi related knobs; we have the > > global RT runtime limit knob, but that doesn't affect cpufreq (maybe it > > should) > > Maybe we could create this sort of link when using the cgroup RT >

Re: [PATCH 2/2] sched: cpufreq: use rt_avg as estimate of required RT CPU capacity

2016-09-01 Thread Peter Zijlstra
On Wed, Aug 31, 2016 at 06:00:02PM +0100, Juri Lelli wrote: > On 31/08/16 18:40, Peter Zijlstra wrote: > > Another problem is that we have many semi related knobs; we have the > > global RT runtime limit knob, but that doesn't affect cpufreq (maybe it > > should) > > Maybe we could create this

Re: [PATCH 2/2] sched: cpufreq: use rt_avg as estimate of required RT CPU capacity

2016-09-01 Thread Peter Zijlstra
On Wed, Aug 31, 2016 at 06:00:02PM +0100, Juri Lelli wrote: > On 31/08/16 18:40, Peter Zijlstra wrote: > > Another problem is that we have many semi related knobs; we have the > > global RT runtime limit knob, but that doesn't affect cpufreq (maybe it > > should) > > Maybe we could create this

Re: [PATCH 2/2] sched: cpufreq: use rt_avg as estimate of required RT CPU capacity

2016-08-31 Thread Rafael J. Wysocki
On Wednesday, August 31, 2016 06:40:09 PM Peter Zijlstra wrote: > On Wed, Aug 31, 2016 at 06:28:10PM +0200, Thomas Gleixner wrote: > > > That is the way it's been with cpufreq and many systems (including all > > > mobile devices) rely on that to not destroy power. RT + variable cpufreq > > > is

Re: [PATCH 2/2] sched: cpufreq: use rt_avg as estimate of required RT CPU capacity

2016-08-31 Thread Rafael J. Wysocki
On Wednesday, August 31, 2016 06:40:09 PM Peter Zijlstra wrote: > On Wed, Aug 31, 2016 at 06:28:10PM +0200, Thomas Gleixner wrote: > > > That is the way it's been with cpufreq and many systems (including all > > > mobile devices) rely on that to not destroy power. RT + variable cpufreq > > > is

Re: [PATCH 2/2] sched: cpufreq: use rt_avg as estimate of required RT CPU capacity

2016-08-31 Thread Thomas Gleixner
On Wed, 31 Aug 2016, Steve Muckle wrote: > On Wed, Aug 31, 2016 at 04:39:07PM +0200, Peter Zijlstra wrote: > > On Fri, Aug 26, 2016 at 11:40:48AM -0700, Steve Muckle wrote: > > > A policy of going to fmax on any RT activity will be detrimental > > > for power on many platforms. Often RT accounts

Re: [PATCH 2/2] sched: cpufreq: use rt_avg as estimate of required RT CPU capacity

2016-08-31 Thread Thomas Gleixner
On Wed, 31 Aug 2016, Steve Muckle wrote: > On Wed, Aug 31, 2016 at 04:39:07PM +0200, Peter Zijlstra wrote: > > On Fri, Aug 26, 2016 at 11:40:48AM -0700, Steve Muckle wrote: > > > A policy of going to fmax on any RT activity will be detrimental > > > for power on many platforms. Often RT accounts

Re: [PATCH 2/2] sched: cpufreq: use rt_avg as estimate of required RT CPU capacity

2016-08-31 Thread Juri Lelli
On 31/08/16 18:40, Peter Zijlstra wrote: > On Wed, Aug 31, 2016 at 06:28:10PM +0200, Thomas Gleixner wrote: > > > That is the way it's been with cpufreq and many systems (including all > > > mobile devices) rely on that to not destroy power. RT + variable cpufreq > > > is not deterministic. > > >

Re: [PATCH 2/2] sched: cpufreq: use rt_avg as estimate of required RT CPU capacity

2016-08-31 Thread Juri Lelli
On 31/08/16 18:40, Peter Zijlstra wrote: > On Wed, Aug 31, 2016 at 06:28:10PM +0200, Thomas Gleixner wrote: > > > That is the way it's been with cpufreq and many systems (including all > > > mobile devices) rely on that to not destroy power. RT + variable cpufreq > > > is not deterministic. > > >

Re: [PATCH 2/2] sched: cpufreq: use rt_avg as estimate of required RT CPU capacity

2016-08-31 Thread Peter Zijlstra
On Wed, Aug 31, 2016 at 06:28:10PM +0200, Thomas Gleixner wrote: > > That is the way it's been with cpufreq and many systems (including all > > mobile devices) rely on that to not destroy power. RT + variable cpufreq > > is not deterministic. > > > > Given we don't have good constraints on RT

Re: [PATCH 2/2] sched: cpufreq: use rt_avg as estimate of required RT CPU capacity

2016-08-31 Thread Peter Zijlstra
On Wed, Aug 31, 2016 at 06:28:10PM +0200, Thomas Gleixner wrote: > > That is the way it's been with cpufreq and many systems (including all > > mobile devices) rely on that to not destroy power. RT + variable cpufreq > > is not deterministic. > > > > Given we don't have good constraints on RT

Re: [PATCH 2/2] sched: cpufreq: use rt_avg as estimate of required RT CPU capacity

2016-08-31 Thread Steve Muckle
On Wed, Aug 31, 2016 at 04:39:07PM +0200, Peter Zijlstra wrote: > On Fri, Aug 26, 2016 at 11:40:48AM -0700, Steve Muckle wrote: > > A policy of going to fmax on any RT activity will be detrimental > > for power on many platforms. Often RT accounts for only a small amount > > of CPU activity so

Re: [PATCH 2/2] sched: cpufreq: use rt_avg as estimate of required RT CPU capacity

2016-08-31 Thread Steve Muckle
On Wed, Aug 31, 2016 at 04:39:07PM +0200, Peter Zijlstra wrote: > On Fri, Aug 26, 2016 at 11:40:48AM -0700, Steve Muckle wrote: > > A policy of going to fmax on any RT activity will be detrimental > > for power on many platforms. Often RT accounts for only a small amount > > of CPU activity so

Re: [PATCH 2/2] sched: cpufreq: use rt_avg as estimate of required RT CPU capacity

2016-08-31 Thread Steve Muckle
On Wed, Aug 31, 2016 at 03:31:07AM +0200, Rafael J. Wysocki wrote: > On Friday, August 26, 2016 11:40:48 AM Steve Muckle wrote: > > A policy of going to fmax on any RT activity will be detrimental > > for power on many platforms. Often RT accounts for only a small amount > > of CPU activity so

Re: [PATCH 2/2] sched: cpufreq: use rt_avg as estimate of required RT CPU capacity

2016-08-31 Thread Steve Muckle
On Wed, Aug 31, 2016 at 03:31:07AM +0200, Rafael J. Wysocki wrote: > On Friday, August 26, 2016 11:40:48 AM Steve Muckle wrote: > > A policy of going to fmax on any RT activity will be detrimental > > for power on many platforms. Often RT accounts for only a small amount > > of CPU activity so

Re: [PATCH 2/2] sched: cpufreq: use rt_avg as estimate of required RT CPU capacity

2016-08-31 Thread Peter Zijlstra
On Fri, Aug 26, 2016 at 11:40:48AM -0700, Steve Muckle wrote: > A policy of going to fmax on any RT activity will be detrimental > for power on many platforms. Often RT accounts for only a small amount > of CPU activity so sending the CPU frequency to fmax is overkill. Worse > still, some

Re: [PATCH 2/2] sched: cpufreq: use rt_avg as estimate of required RT CPU capacity

2016-08-31 Thread Peter Zijlstra
On Fri, Aug 26, 2016 at 11:40:48AM -0700, Steve Muckle wrote: > A policy of going to fmax on any RT activity will be detrimental > for power on many platforms. Often RT accounts for only a small amount > of CPU activity so sending the CPU frequency to fmax is overkill. Worse > still, some

Re: [PATCH 2/2] sched: cpufreq: use rt_avg as estimate of required RT CPU capacity

2016-08-30 Thread Rafael J. Wysocki
On Friday, August 26, 2016 11:40:48 AM Steve Muckle wrote: > A policy of going to fmax on any RT activity will be detrimental > for power on many platforms. Often RT accounts for only a small amount > of CPU activity so sending the CPU frequency to fmax is overkill. Worse > still, some platforms

Re: [PATCH 2/2] sched: cpufreq: use rt_avg as estimate of required RT CPU capacity

2016-08-30 Thread Rafael J. Wysocki
On Friday, August 26, 2016 11:40:48 AM Steve Muckle wrote: > A policy of going to fmax on any RT activity will be detrimental > for power on many platforms. Often RT accounts for only a small amount > of CPU activity so sending the CPU frequency to fmax is overkill. Worse > still, some platforms

[PATCH 2/2] sched: cpufreq: use rt_avg as estimate of required RT CPU capacity

2016-08-26 Thread Steve Muckle
A policy of going to fmax on any RT activity will be detrimental for power on many platforms. Often RT accounts for only a small amount of CPU activity so sending the CPU frequency to fmax is overkill. Worse still, some platforms may not be able to even complete the CPU frequency change before the

[PATCH 2/2] sched: cpufreq: use rt_avg as estimate of required RT CPU capacity

2016-08-26 Thread Steve Muckle
A policy of going to fmax on any RT activity will be detrimental for power on many platforms. Often RT accounts for only a small amount of CPU activity so sending the CPU frequency to fmax is overkill. Worse still, some platforms may not be able to even complete the CPU frequency change before the