Re: [PATCH 1/3] intel_pstate: Clarify average performance computation

2016-05-11 Thread Rafael J. Wysocki
On Wed, May 11, 2016 at 7:01 AM, Srinivas Pandruvada wrote: > On Tue, 2016-05-10 at 22:57 +0200, Rafael J. Wysocki wrote: >> > > > > > [...] > >> --- >> From: Rafael J. Wysocki >> Subject: [PATCH] intel_pstate: Clarify average

Re: [PATCH 1/3] intel_pstate: Clarify average performance computation

2016-05-11 Thread Rafael J. Wysocki
On Wed, May 11, 2016 at 7:01 AM, Srinivas Pandruvada wrote: > On Tue, 2016-05-10 at 22:57 +0200, Rafael J. Wysocki wrote: >> > > > > > [...] > >> --- >> From: Rafael J. Wysocki >> Subject: [PATCH] intel_pstate: Clarify average performance >> computation >> >> The core_pct_busy field of struct

Re: [PATCH 1/3] intel_pstate: Clarify average performance computation

2016-05-10 Thread Srinivas Pandruvada
On Tue, 2016-05-10 at 22:57 +0200, Rafael J. Wysocki wrote: > > > > [...] > --- > From: Rafael J. Wysocki > Subject: [PATCH] intel_pstate: Clarify average performance > computation > > The core_pct_busy field of struct sample actually contains the > average

Re: [PATCH 1/3] intel_pstate: Clarify average performance computation

2016-05-10 Thread Srinivas Pandruvada
On Tue, 2016-05-10 at 22:57 +0200, Rafael J. Wysocki wrote: > > > > [...] > --- > From: Rafael J. Wysocki > Subject: [PATCH] intel_pstate: Clarify average performance > computation > > The core_pct_busy field of struct sample actually contains the > average performace during the last sampling

Re: [PATCH 1/3] intel_pstate: Clarify average performance computation

2016-05-10 Thread Rafael J. Wysocki
On Tuesday, May 10, 2016 12:58:04 PM Srinivas Pandruvada wrote: > On Tue, 2016-05-10 at 21:21 +0200, Rafael J. Wysocki wrote: > > On Tue, May 10, 2016 at 3:18 AM, Srinivas Pandruvada > > wrote: > > > > > > On Sat, 2016-05-07 at 01:44 +0200, Rafael J. Wysocki

Re: [PATCH 1/3] intel_pstate: Clarify average performance computation

2016-05-10 Thread Rafael J. Wysocki
On Tuesday, May 10, 2016 12:58:04 PM Srinivas Pandruvada wrote: > On Tue, 2016-05-10 at 21:21 +0200, Rafael J. Wysocki wrote: > > On Tue, May 10, 2016 at 3:18 AM, Srinivas Pandruvada > > wrote: > > > > > > On Sat, 2016-05-07 at 01:44 +0200, Rafael J. Wysocki wrote: > > > > > > > > From: Rafael

Re: [PATCH 1/3] intel_pstate: Clarify average performance computation

2016-05-10 Thread Srinivas Pandruvada
On Tue, 2016-05-10 at 21:21 +0200, Rafael J. Wysocki wrote: > On Tue, May 10, 2016 at 3:18 AM, Srinivas Pandruvada > wrote: > > > > On Sat, 2016-05-07 at 01:44 +0200, Rafael J. Wysocki wrote: > > > > > > From: Rafael J. Wysocki >

Re: [PATCH 1/3] intel_pstate: Clarify average performance computation

2016-05-10 Thread Srinivas Pandruvada
On Tue, 2016-05-10 at 21:21 +0200, Rafael J. Wysocki wrote: > On Tue, May 10, 2016 at 3:18 AM, Srinivas Pandruvada > wrote: > > > > On Sat, 2016-05-07 at 01:44 +0200, Rafael J. Wysocki wrote: > > > > > > From: Rafael J. Wysocki > > > > > > The core_pct_busy field of struct sample actually

Re: [PATCH 1/3] intel_pstate: Clarify average performance computation

2016-05-10 Thread Rafael J. Wysocki
On Tue, May 10, 2016 at 3:18 AM, Srinivas Pandruvada wrote: > On Sat, 2016-05-07 at 01:44 +0200, Rafael J. Wysocki wrote: >> From: Rafael J. Wysocki >> >> The core_pct_busy field of struct sample actually contains the >> average

Re: [PATCH 1/3] intel_pstate: Clarify average performance computation

2016-05-10 Thread Rafael J. Wysocki
On Tue, May 10, 2016 at 3:18 AM, Srinivas Pandruvada wrote: > On Sat, 2016-05-07 at 01:44 +0200, Rafael J. Wysocki wrote: >> From: Rafael J. Wysocki >> >> The core_pct_busy field of struct sample actually contains the >> average performace during the last sampling period (in percent) >> and not

Re: [PATCH 1/3] intel_pstate: Clarify average performance computation

2016-05-09 Thread Srinivas Pandruvada
On Sat, 2016-05-07 at 01:44 +0200, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki > > The core_pct_busy field of struct sample actually contains the > average performace during the last sampling period (in percent) > and not the utilization of the core as

Re: [PATCH 1/3] intel_pstate: Clarify average performance computation

2016-05-09 Thread Srinivas Pandruvada
On Sat, 2016-05-07 at 01:44 +0200, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki > > The core_pct_busy field of struct sample actually contains the > average performace during the last sampling period (in percent) > and not the utilization of the core as suggested by its name > which is

[PATCH 1/3] intel_pstate: Clarify average performance computation

2016-05-06 Thread Rafael J. Wysocki
From: Rafael J. Wysocki The core_pct_busy field of struct sample actually contains the average performace during the last sampling period (in percent) and not the utilization of the core as suggested by its name which is confusing. For this reason, change the name of

[PATCH 1/3] intel_pstate: Clarify average performance computation

2016-05-06 Thread Rafael J. Wysocki
From: Rafael J. Wysocki The core_pct_busy field of struct sample actually contains the average performace during the last sampling period (in percent) and not the utilization of the core as suggested by its name which is confusing. For this reason, change the name of that field to core_avg_perf