Re: [RFC][PATCH 5/7] cpufreq / sched: UUF_IO flag to indicate iowait condition

2016-08-06 Thread Rafael J. Wysocki
On Thursday, August 04, 2016 03:09:08 PM Steve Muckle wrote: > On Thu, Aug 04, 2016 at 11:19:00PM +0200, Rafael J. Wysocki wrote: [cut] > > > This is also an issue for the remote wakeup case where I currently have > > > another invocation of the hook in check_preempt_curr(), so I can know if >

Re: [RFC][PATCH 5/7] cpufreq / sched: UUF_IO flag to indicate iowait condition

2016-08-06 Thread Rafael J. Wysocki
On Thursday, August 04, 2016 03:09:08 PM Steve Muckle wrote: > On Thu, Aug 04, 2016 at 11:19:00PM +0200, Rafael J. Wysocki wrote: [cut] > > > This is also an issue for the remote wakeup case where I currently have > > > another invocation of the hook in check_preempt_curr(), so I can know if >

Re: [RFC][PATCH 5/7] cpufreq / sched: UUF_IO flag to indicate iowait condition

2016-08-04 Thread Steve Muckle
On Thu, Aug 04, 2016 at 11:19:00PM +0200, Rafael J. Wysocki wrote: > On Wednesday, August 03, 2016 07:24:18 PM Steve Muckle wrote: > > On Wed, Aug 03, 2016 at 12:38:20AM +0200, Rafael J. Wysocki wrote: > > > On Wed, Aug 3, 2016 at 12:02 AM, Steve Muckle > > > wrote: > >

Re: [RFC][PATCH 5/7] cpufreq / sched: UUF_IO flag to indicate iowait condition

2016-08-04 Thread Steve Muckle
On Thu, Aug 04, 2016 at 11:19:00PM +0200, Rafael J. Wysocki wrote: > On Wednesday, August 03, 2016 07:24:18 PM Steve Muckle wrote: > > On Wed, Aug 03, 2016 at 12:38:20AM +0200, Rafael J. Wysocki wrote: > > > On Wed, Aug 3, 2016 at 12:02 AM, Steve Muckle > > > wrote: > > > > On Tue, Aug 02, 2016

Re: [RFC][PATCH 5/7] cpufreq / sched: UUF_IO flag to indicate iowait condition

2016-08-04 Thread Rafael J. Wysocki
On Wednesday, August 03, 2016 07:24:18 PM Steve Muckle wrote: > On Wed, Aug 03, 2016 at 12:38:20AM +0200, Rafael J. Wysocki wrote: > > On Wed, Aug 3, 2016 at 12:02 AM, Steve Muckle > > wrote: > > > On Tue, Aug 02, 2016 at 03:37:02AM +0200, Rafael J. Wysocki wrote: > > >>

Re: [RFC][PATCH 5/7] cpufreq / sched: UUF_IO flag to indicate iowait condition

2016-08-04 Thread Rafael J. Wysocki
On Wednesday, August 03, 2016 07:24:18 PM Steve Muckle wrote: > On Wed, Aug 03, 2016 at 12:38:20AM +0200, Rafael J. Wysocki wrote: > > On Wed, Aug 3, 2016 at 12:02 AM, Steve Muckle > > wrote: > > > On Tue, Aug 02, 2016 at 03:37:02AM +0200, Rafael J. Wysocki wrote: > > >> On Tue, Aug 2, 2016 at

Re: [RFC][PATCH 5/7] cpufreq / sched: UUF_IO flag to indicate iowait condition

2016-08-03 Thread Steve Muckle
On Wed, Aug 03, 2016 at 12:38:20AM +0200, Rafael J. Wysocki wrote: > On Wed, Aug 3, 2016 at 12:02 AM, Steve Muckle wrote: > > On Tue, Aug 02, 2016 at 03:37:02AM +0200, Rafael J. Wysocki wrote: > >> On Tue, Aug 2, 2016 at 3:22 AM, Steve Muckle >

Re: [RFC][PATCH 5/7] cpufreq / sched: UUF_IO flag to indicate iowait condition

2016-08-03 Thread Steve Muckle
On Wed, Aug 03, 2016 at 12:38:20AM +0200, Rafael J. Wysocki wrote: > On Wed, Aug 3, 2016 at 12:02 AM, Steve Muckle wrote: > > On Tue, Aug 02, 2016 at 03:37:02AM +0200, Rafael J. Wysocki wrote: > >> On Tue, Aug 2, 2016 at 3:22 AM, Steve Muckle > >> wrote: > >> > On Mon, Aug 01, 2016 at

Re: [RFC][PATCH 5/7] cpufreq / sched: UUF_IO flag to indicate iowait condition

2016-08-02 Thread Rafael J. Wysocki
On Wed, Aug 3, 2016 at 12:02 AM, Steve Muckle wrote: > On Tue, Aug 02, 2016 at 03:37:02AM +0200, Rafael J. Wysocki wrote: >> On Tue, Aug 2, 2016 at 3:22 AM, Steve Muckle wrote: >> > On Mon, Aug 01, 2016 at 01:37:23AM +0200, Rafael J. Wysocki

Re: [RFC][PATCH 5/7] cpufreq / sched: UUF_IO flag to indicate iowait condition

2016-08-02 Thread Rafael J. Wysocki
On Wed, Aug 3, 2016 at 12:02 AM, Steve Muckle wrote: > On Tue, Aug 02, 2016 at 03:37:02AM +0200, Rafael J. Wysocki wrote: >> On Tue, Aug 2, 2016 at 3:22 AM, Steve Muckle wrote: >> > On Mon, Aug 01, 2016 at 01:37:23AM +0200, Rafael J. Wysocki wrote: >> > ... >> >> For this purpose, define a new

Re: [RFC][PATCH 5/7] cpufreq / sched: UUF_IO flag to indicate iowait condition

2016-08-02 Thread Steve Muckle
On Tue, Aug 02, 2016 at 03:37:02AM +0200, Rafael J. Wysocki wrote: > On Tue, Aug 2, 2016 at 3:22 AM, Steve Muckle wrote: > > On Mon, Aug 01, 2016 at 01:37:23AM +0200, Rafael J. Wysocki wrote: > > ... > >> For this purpose, define a new cpufreq_update_util() flag > >>

Re: [RFC][PATCH 5/7] cpufreq / sched: UUF_IO flag to indicate iowait condition

2016-08-02 Thread Steve Muckle
On Tue, Aug 02, 2016 at 03:37:02AM +0200, Rafael J. Wysocki wrote: > On Tue, Aug 2, 2016 at 3:22 AM, Steve Muckle wrote: > > On Mon, Aug 01, 2016 at 01:37:23AM +0200, Rafael J. Wysocki wrote: > > ... > >> For this purpose, define a new cpufreq_update_util() flag > >> UUF_IO and modify

Re: [RFC][PATCH 5/7] cpufreq / sched: UUF_IO flag to indicate iowait condition

2016-08-01 Thread Rafael J. Wysocki
On Tue, Aug 2, 2016 at 3:22 AM, Steve Muckle wrote: > On Mon, Aug 01, 2016 at 01:37:23AM +0200, Rafael J. Wysocki wrote: > ... >> For this purpose, define a new cpufreq_update_util() flag >> UUF_IO and modify enqueue_task_fair() to pass that flag to >>

Re: [RFC][PATCH 5/7] cpufreq / sched: UUF_IO flag to indicate iowait condition

2016-08-01 Thread Rafael J. Wysocki
On Tue, Aug 2, 2016 at 3:22 AM, Steve Muckle wrote: > On Mon, Aug 01, 2016 at 01:37:23AM +0200, Rafael J. Wysocki wrote: > ... >> For this purpose, define a new cpufreq_update_util() flag >> UUF_IO and modify enqueue_task_fair() to pass that flag to >> cpufreq_update_util() in the in_iowait case.

Re: [RFC][PATCH 5/7] cpufreq / sched: UUF_IO flag to indicate iowait condition

2016-08-01 Thread Steve Muckle
On Mon, Aug 01, 2016 at 01:37:23AM +0200, Rafael J. Wysocki wrote: ... > For this purpose, define a new cpufreq_update_util() flag > UUF_IO and modify enqueue_task_fair() to pass that flag to > cpufreq_update_util() in the in_iowait case. That generally > requires cpufreq_update_util() to be

Re: [RFC][PATCH 5/7] cpufreq / sched: UUF_IO flag to indicate iowait condition

2016-08-01 Thread Steve Muckle
On Mon, Aug 01, 2016 at 01:37:23AM +0200, Rafael J. Wysocki wrote: ... > For this purpose, define a new cpufreq_update_util() flag > UUF_IO and modify enqueue_task_fair() to pass that flag to > cpufreq_update_util() in the in_iowait case. That generally > requires cpufreq_update_util() to be

[RFC][PATCH 5/7] cpufreq / sched: UUF_IO flag to indicate iowait condition

2016-07-31 Thread Rafael J. Wysocki
From: Rafael J. Wysocki Testing indicates that it is possible to improve performace significantly without increasing energy consumption too much by teaching cpufreq governors to bump up the CPU performance level if the in_iowait flag is set for the task in

[RFC][PATCH 5/7] cpufreq / sched: UUF_IO flag to indicate iowait condition

2016-07-31 Thread Rafael J. Wysocki
From: Rafael J. Wysocki Testing indicates that it is possible to improve performace significantly without increasing energy consumption too much by teaching cpufreq governors to bump up the CPU performance level if the in_iowait flag is set for the task in enqueue_task_fair(). For this purpose,