diff mbox

[2/2] tools/power turbostat: changed bclk discovery path for Intel Xeon x200 Architecture

Message ID 1455118517-14180-3-git-send-email-hubert.chrzaniuk@intel.com (mailing list archive)
State Accepted, archived
Delegated to: Len Brown
Headers show

Commit Message

Chrzaniuk, Hubert Feb. 10, 2016, 3:35 p.m. UTC
x200 does not enable any way to programmatically obtain bus clock
speed. Bclk for the architecture has a fixed value of 100 MHz.
At the same time x200 cannot be included in has_snb_msrs since
it does not support C7 idle state.

Signed-off-by: Hubert Chrzaniuk <hubert.chrzaniuk@intel.com>
---
 tools/power/x86/turbostat/turbostat.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Len Brown Feb. 13, 2016, 6:21 a.m. UTC | #1
Applied.

thanks,
-Len


On Wed, Feb 10, 2016 at 10:35 AM, Hubert Chrzaniuk
<hubert.chrzaniuk@intel.com> wrote:
> x200 does not enable any way to programmatically obtain bus clock
> speed. Bclk for the architecture has a fixed value of 100 MHz.
> At the same time x200 cannot be included in has_snb_msrs since
> it does not support C7 idle state.
>
> Signed-off-by: Hubert Chrzaniuk <hubert.chrzaniuk@intel.com>
> ---
>  tools/power/x86/turbostat/turbostat.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/power/x86/turbostat/turbostat.c b/tools/power/x86/turbostat/turbostat.c
> index 765c669..3a59aec 100644
> --- a/tools/power/x86/turbostat/turbostat.c
> +++ b/tools/power/x86/turbostat/turbostat.c
> @@ -2601,7 +2601,7 @@ double slm_bclk(void)
>
>  double discover_bclk(unsigned int family, unsigned int model)
>  {
> -       if (has_snb_msrs(family, model))
> +       if (has_snb_msrs(family, model) || is_knl(family, model))
>                 return 100.00;
>         else if (is_slm(family, model))
>                 return slm_bclk();
> --
> 1.8.3.1
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-pm" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/tools/power/x86/turbostat/turbostat.c b/tools/power/x86/turbostat/turbostat.c
index 765c669..3a59aec 100644
--- a/tools/power/x86/turbostat/turbostat.c
+++ b/tools/power/x86/turbostat/turbostat.c
@@ -2601,7 +2601,7 @@  double slm_bclk(void)
 
 double discover_bclk(unsigned int family, unsigned int model)
 {
-	if (has_snb_msrs(family, model))
+	if (has_snb_msrs(family, model) || is_knl(family, model))
 		return 100.00;
 	else if (is_slm(family, model))
 		return slm_bclk();