Re: [PATCH] intel_pstate: track and export frequency residency stats via sysfs.

2014-09-11 Thread Dirk Brandewie
On 09/10/2014 06:04 PM, Sameer Nanda wrote: On Wed, Sep 10, 2014 at 5:04 PM, Rafael J. Wysocki wrote: On Wednesday, September 10, 2014 04:39:05 PM Anup Chenthamarakshan wrote: On Thu, Sep 11, 2014 at 12:49:48AM +0200, Rafael J. Wysocki wrote: On Wednesday, September 10, 2014 03:15:08 PM Anup

Re: [PATCH] intel_pstate: track and export frequency residency stats via sysfs.

2014-09-11 Thread Dirk Brandewie
On 09/10/2014 06:04 PM, Sameer Nanda wrote: On Wed, Sep 10, 2014 at 5:04 PM, Rafael J. Wysocki r...@rjwysocki.net wrote: On Wednesday, September 10, 2014 04:39:05 PM Anup Chenthamarakshan wrote: On Thu, Sep 11, 2014 at 12:49:48AM +0200, Rafael J. Wysocki wrote: On Wednesday, September 10,

Re: [PATCH] intel_pstate: track and export frequency residency stats via sysfs.

2014-09-10 Thread Sameer Nanda
On Wed, Sep 10, 2014 at 5:04 PM, Rafael J. Wysocki wrote: > On Wednesday, September 10, 2014 04:39:05 PM Anup Chenthamarakshan wrote: >> On Thu, Sep 11, 2014 at 12:49:48AM +0200, Rafael J. Wysocki wrote: >> > On Wednesday, September 10, 2014 03:15:08 PM Anup Chenthamarakshan wrote: >> > > >> > >

Re: [PATCH] intel_pstate: track and export frequency residency stats via sysfs.

2014-09-10 Thread Rafael J. Wysocki
On Wednesday, September 10, 2014 04:39:05 PM Anup Chenthamarakshan wrote: > On Thu, Sep 11, 2014 at 12:49:48AM +0200, Rafael J. Wysocki wrote: > > On Wednesday, September 10, 2014 03:15:08 PM Anup Chenthamarakshan wrote: > > > > > > Tools like powertop and turbostat are not present by default on

Re: [PATCH] intel_pstate: track and export frequency residency stats via sysfs.

2014-09-10 Thread Anup Chenthamarakshan
On Thu, Sep 11, 2014 at 12:49:48AM +0200, Rafael J. Wysocki wrote: > On Wednesday, September 10, 2014 03:15:08 PM Anup Chenthamarakshan wrote: > > > > Tools like powertop and turbostat are not present by default on all systems, > > so it is not always possible to use them :( > > Which systems

Re: [PATCH] intel_pstate: track and export frequency residency stats via sysfs.

2014-09-10 Thread Rafael J. Wysocki
On Wednesday, September 10, 2014 03:15:08 PM Anup Chenthamarakshan wrote: > On Wed, Sep 10, 2014 at 09:39:30AM -0700, Dirk Brandewie wrote: > > On 09/09/2014 04:22 PM, Anup Chenthamarakshan wrote: > > >On Tue, Sep 09, 2014 at 08:15:13AM -0700, Dirk Brandewie wrote: > > >>On 09/08/2014 05:10 PM,

Re: [PATCH] intel_pstate: track and export frequency residency stats via sysfs.

2014-09-10 Thread Anup Chenthamarakshan
On Wed, Sep 10, 2014 at 09:39:30AM -0700, Dirk Brandewie wrote: > On 09/09/2014 04:22 PM, Anup Chenthamarakshan wrote: > >On Tue, Sep 09, 2014 at 08:15:13AM -0700, Dirk Brandewie wrote: > >>On 09/08/2014 05:10 PM, Anup Chenthamarakshan wrote: > >>>Exported stats appear in >

Re: [PATCH] intel_pstate: track and export frequency residency stats via sysfs.

2014-09-10 Thread Dirk Brandewie
On 09/09/2014 04:22 PM, Anup Chenthamarakshan wrote: On Tue, Sep 09, 2014 at 08:15:13AM -0700, Dirk Brandewie wrote: On 09/08/2014 05:10 PM, Anup Chenthamarakshan wrote: Exported stats appear in /devices/system/cpu/intel_pstate/time_in_state as follows: ## CPU 0 40 3647 50 24342

Re: [PATCH] intel_pstate: track and export frequency residency stats via sysfs.

2014-09-10 Thread Viresh Kumar
On 10 September 2014 05:01, Anup Chenthamarakshan wrote: > Here's a comparison of time taken to run intel_pstate_set_pstate() using > different approaches: > average time for a transition > no stats 4.7us > intel_pstate-stats

Re: [PATCH] intel_pstate: track and export frequency residency stats via sysfs.

2014-09-10 Thread Viresh Kumar
On 10 September 2014 05:01, Anup Chenthamarakshan an...@chromium.org wrote: Here's a comparison of time taken to run intel_pstate_set_pstate() using different approaches: average time for a transition no stats 4.7us

Re: [PATCH] intel_pstate: track and export frequency residency stats via sysfs.

2014-09-10 Thread Dirk Brandewie
On 09/09/2014 04:22 PM, Anup Chenthamarakshan wrote: On Tue, Sep 09, 2014 at 08:15:13AM -0700, Dirk Brandewie wrote: On 09/08/2014 05:10 PM, Anup Chenthamarakshan wrote: Exported stats appear in sysfs/devices/system/cpu/intel_pstate/time_in_state as follows: ## CPU 0 40 3647 50 24342

Re: [PATCH] intel_pstate: track and export frequency residency stats via sysfs.

2014-09-10 Thread Anup Chenthamarakshan
On Wed, Sep 10, 2014 at 09:39:30AM -0700, Dirk Brandewie wrote: On 09/09/2014 04:22 PM, Anup Chenthamarakshan wrote: On Tue, Sep 09, 2014 at 08:15:13AM -0700, Dirk Brandewie wrote: On 09/08/2014 05:10 PM, Anup Chenthamarakshan wrote: Exported stats appear in

Re: [PATCH] intel_pstate: track and export frequency residency stats via sysfs.

2014-09-10 Thread Rafael J. Wysocki
On Wednesday, September 10, 2014 03:15:08 PM Anup Chenthamarakshan wrote: On Wed, Sep 10, 2014 at 09:39:30AM -0700, Dirk Brandewie wrote: On 09/09/2014 04:22 PM, Anup Chenthamarakshan wrote: On Tue, Sep 09, 2014 at 08:15:13AM -0700, Dirk Brandewie wrote: On 09/08/2014 05:10 PM, Anup

Re: [PATCH] intel_pstate: track and export frequency residency stats via sysfs.

2014-09-10 Thread Anup Chenthamarakshan
On Thu, Sep 11, 2014 at 12:49:48AM +0200, Rafael J. Wysocki wrote: On Wednesday, September 10, 2014 03:15:08 PM Anup Chenthamarakshan wrote: Tools like powertop and turbostat are not present by default on all systems, so it is not always possible to use them :( Which systems are you

Re: [PATCH] intel_pstate: track and export frequency residency stats via sysfs.

2014-09-10 Thread Rafael J. Wysocki
On Wednesday, September 10, 2014 04:39:05 PM Anup Chenthamarakshan wrote: On Thu, Sep 11, 2014 at 12:49:48AM +0200, Rafael J. Wysocki wrote: On Wednesday, September 10, 2014 03:15:08 PM Anup Chenthamarakshan wrote: Tools like powertop and turbostat are not present by default on all

Re: [PATCH] intel_pstate: track and export frequency residency stats via sysfs.

2014-09-10 Thread Sameer Nanda
On Wed, Sep 10, 2014 at 5:04 PM, Rafael J. Wysocki r...@rjwysocki.net wrote: On Wednesday, September 10, 2014 04:39:05 PM Anup Chenthamarakshan wrote: On Thu, Sep 11, 2014 at 12:49:48AM +0200, Rafael J. Wysocki wrote: On Wednesday, September 10, 2014 03:15:08 PM Anup Chenthamarakshan wrote:

Re: [PATCH] intel_pstate: track and export frequency residency stats via sysfs.

2014-09-09 Thread Anup Chenthamarakshan
On Tue, Sep 09, 2014 at 11:56:18AM +0530, Viresh Kumar wrote: > On Tue, Sep 9, 2014 at 11:02 AM, Anup Chenthamarakshan > wrote: > > I had initially tried reusing cpufreq_stats.c to export stats. > > Calling cpufreq_stats_update() via the cpufreq notifier added > > some amount of overhead while

Re: [PATCH] intel_pstate: track and export frequency residency stats via sysfs.

2014-09-09 Thread Anup Chenthamarakshan
On Tue, Sep 09, 2014 at 08:15:13AM -0700, Dirk Brandewie wrote: > On 09/08/2014 05:10 PM, Anup Chenthamarakshan wrote: > > Exported stats appear in > > /devices/system/cpu/intel_pstate/time_in_state as follows: > > > > ## CPU 0 > > 40 3647 > > 50 24342 > > 60 144150 > > 70 202469

Re: [PATCH] intel_pstate: track and export frequency residency stats via sysfs.

2014-09-09 Thread Dirk Brandewie
On 09/08/2014 05:10 PM, Anup Chenthamarakshan wrote: > Exported stats appear in > /devices/system/cpu/intel_pstate/time_in_state as follows: > > ## CPU 0 > 40 3647 > 50 24342 > 60 144150 > 70 202469 > ## CPU 1 > 40 4813 > 50 22628 > 60 149564 > 70 211885 > 80

Re: [PATCH] intel_pstate: track and export frequency residency stats via sysfs.

2014-09-09 Thread Viresh Kumar
On Tue, Sep 9, 2014 at 11:02 AM, Anup Chenthamarakshan wrote: > I had initially tried reusing cpufreq_stats.c to export stats. > Calling cpufreq_stats_update() via the cpufreq notifier added > some amount of overhead while switching frequencies. Specifically, > looking up the index of the new

Re: [PATCH] intel_pstate: track and export frequency residency stats via sysfs.

2014-09-09 Thread Viresh Kumar
On Tue, Sep 9, 2014 at 11:02 AM, Anup Chenthamarakshan an...@chromium.org wrote: I had initially tried reusing cpufreq_stats.c to export stats. Calling cpufreq_stats_update() via the cpufreq notifier added some amount of overhead while switching frequencies. Specifically, looking up the index

Re: [PATCH] intel_pstate: track and export frequency residency stats via sysfs.

2014-09-09 Thread Dirk Brandewie
On 09/08/2014 05:10 PM, Anup Chenthamarakshan wrote: Exported stats appear in sysfs/devices/system/cpu/intel_pstate/time_in_state as follows: ## CPU 0 40 3647 50 24342 60 144150 70 202469 ## CPU 1 40 4813 50 22628 60 149564 70 211885 80 173890

Re: [PATCH] intel_pstate: track and export frequency residency stats via sysfs.

2014-09-09 Thread Anup Chenthamarakshan
On Tue, Sep 09, 2014 at 08:15:13AM -0700, Dirk Brandewie wrote: On 09/08/2014 05:10 PM, Anup Chenthamarakshan wrote: Exported stats appear in sysfs/devices/system/cpu/intel_pstate/time_in_state as follows: ## CPU 0 40 3647 50 24342 60 144150 70 202469 ## CPU 1

Re: [PATCH] intel_pstate: track and export frequency residency stats via sysfs.

2014-09-09 Thread Anup Chenthamarakshan
On Tue, Sep 09, 2014 at 11:56:18AM +0530, Viresh Kumar wrote: On Tue, Sep 9, 2014 at 11:02 AM, Anup Chenthamarakshan an...@chromium.org wrote: I had initially tried reusing cpufreq_stats.c to export stats. Calling cpufreq_stats_update() via the cpufreq notifier added some amount of

Re: [PATCH] intel_pstate: track and export frequency residency stats via sysfs.

2014-09-08 Thread Anup Chenthamarakshan
On Mon, Sep 8, 2014 at 10:03 PM, Viresh Kumar wrote: > On 9 September 2014 05:40, Anup Chenthamarakshan wrote: >> Exported stats appear in >> /devices/system/cpu/intel_pstate/time_in_state as follows: >> >> ## CPU 0 >> 40 3647 >> 50 24342 >> 60 144150 >> 70 202469 >> ## CPU 1 >>

Re: [PATCH] intel_pstate: track and export frequency residency stats via sysfs.

2014-09-08 Thread Viresh Kumar
On 9 September 2014 05:40, Anup Chenthamarakshan wrote: > Exported stats appear in > /devices/system/cpu/intel_pstate/time_in_state as follows: > > ## CPU 0 > 40 3647 > 50 24342 > 60 144150 > 70 202469 > ## CPU 1 > 40 4813 > 50 22628 > 60 149564 > 70 211885 >

Re: [PATCH] intel_pstate: track and export frequency residency stats via sysfs.

2014-09-08 Thread Viresh Kumar
On 9 September 2014 05:40, Anup Chenthamarakshan an...@chromium.org wrote: Exported stats appear in sysfs/devices/system/cpu/intel_pstate/time_in_state as follows: ## CPU 0 40 3647 50 24342 60 144150 70 202469 ## CPU 1 40 4813 50 22628 60 149564 70

Re: [PATCH] intel_pstate: track and export frequency residency stats via sysfs.

2014-09-08 Thread Anup Chenthamarakshan
On Mon, Sep 8, 2014 at 10:03 PM, Viresh Kumar viresh.ku...@linaro.org wrote: On 9 September 2014 05:40, Anup Chenthamarakshan an...@chromium.org wrote: Exported stats appear in sysfs/devices/system/cpu/intel_pstate/time_in_state as follows: ## CPU 0 40 3647 50 24342 60 144150