diff mbox

tools/power turbostat: Disable HWP OOB status

Message ID 1478902499-21293-1-git-send-email-srinivas.pandruvada@linux.intel.com (mailing list archive)
State Superseded, archived
Headers show

Commit Message

Srinivas Pandruvada Nov. 11, 2016, 10:14 p.m. UTC
Display if the HWP is enabled in OOB (Out of band) mode.

Signed-off-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
---
 tools/power/x86/turbostat/turbostat.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

Comments

Rafael J. Wysocki Nov. 11, 2016, 10:18 p.m. UTC | #1
On Fri, Nov 11, 2016 at 11:14 PM, Srinivas Pandruvada
<srinivas.pandruvada@linux.intel.com> wrote:
> Display if the HWP is enabled in OOB (Out of band) mode.

Hmm.  Should the subject be "tools/power turbostat: Display HWP OOB status"?

> Signed-off-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
> ---
>  tools/power/x86/turbostat/turbostat.c | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/tools/power/x86/turbostat/turbostat.c b/tools/power/x86/turbostat/turbostat.c
> index 3e199b5..27fb0f0 100644
> --- a/tools/power/x86/turbostat/turbostat.c
> +++ b/tools/power/x86/turbostat/turbostat.c
> @@ -3174,10 +3174,11 @@ void decode_misc_pwr_mgmt_msr(void)
>                 return;
>
>         if (!get_msr(base_cpu, MSR_MISC_PWR_MGMT, &msr))
> -               fprintf(outf, "cpu%d: MSR_MISC_PWR_MGMT: 0x%08llx (%sable-EIST_Coordination %sable-EPB)\n",
> +               fprintf(outf, "cpu%d: MSR_MISC_PWR_MGMT: 0x%08llx (%sable-EIST_Coordination %sable-EPB %sable-OOB)\n",
>                         base_cpu, msr,
>                         msr & (1 << 0) ? "DIS" : "EN",
> -                       msr & (1 << 1) ? "EN" : "DIS");
> +                       msr & (1 << 1) ? "EN" : "DIS",
> +                       msr & (1 << 8) ? "EN" : "DIS");
>  }
>
>  void process_cpuid()
> --

Thanks,
Rafael
--
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
Srinivas Pandruvada Nov. 11, 2016, 10:25 p.m. UTC | #2
On Fri, 2016-11-11 at 23:18 +0100, Rafael J. Wysocki wrote:
> On Fri, Nov 11, 2016 at 11:14 PM, Srinivas Pandruvada
> <srinivas.pandruvada@linux.intel.com> wrote:
> > 
> > Display if the HWP is enabled in OOB (Out of band) mode.
> 
> Hmm.  Should the subject be "tools/power turbostat: Display HWP OOB
> status"?
yikes. I don't know how I wrote this. This should be "Display" instead
of "Disable"

Thanks,
Srinivas

> 
> > 
> > Signed-off-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel
> > .com>
> > ---
> >  tools/power/x86/turbostat/turbostat.c | 5 +++--
> >  1 file changed, 3 insertions(+), 2 deletions(-)
> > 
> > diff --git a/tools/power/x86/turbostat/turbostat.c
> > b/tools/power/x86/turbostat/turbostat.c
> > index 3e199b5..27fb0f0 100644
> > --- a/tools/power/x86/turbostat/turbostat.c
> > +++ b/tools/power/x86/turbostat/turbostat.c
> > @@ -3174,10 +3174,11 @@ void decode_misc_pwr_mgmt_msr(void)
> >                 return;
> > 
> >         if (!get_msr(base_cpu, MSR_MISC_PWR_MGMT, &msr))
> > -               fprintf(outf, "cpu%d: MSR_MISC_PWR_MGMT: 0x%08llx
> > (%sable-EIST_Coordination %sable-EPB)\n",
> > +               fprintf(outf, "cpu%d: MSR_MISC_PWR_MGMT: 0x%08llx
> > (%sable-EIST_Coordination %sable-EPB %sable-OOB)\n",
> >                         base_cpu, msr,
> >                         msr & (1 << 0) ? "DIS" : "EN",
> > -                       msr & (1 << 1) ? "EN" : "DIS");
> > +                       msr & (1 << 1) ? "EN" : "DIS",
> > +                       msr & (1 << 8) ? "EN" : "DIS");
> >  }
> > 
> >  void process_cpuid()
> > --
> 
> Thanks,
> Rafael
> --
> 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
--
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 3e199b5..27fb0f0 100644
--- a/tools/power/x86/turbostat/turbostat.c
+++ b/tools/power/x86/turbostat/turbostat.c
@@ -3174,10 +3174,11 @@  void decode_misc_pwr_mgmt_msr(void)
 		return;
 
 	if (!get_msr(base_cpu, MSR_MISC_PWR_MGMT, &msr))
-		fprintf(outf, "cpu%d: MSR_MISC_PWR_MGMT: 0x%08llx (%sable-EIST_Coordination %sable-EPB)\n",
+		fprintf(outf, "cpu%d: MSR_MISC_PWR_MGMT: 0x%08llx (%sable-EIST_Coordination %sable-EPB %sable-OOB)\n",
 			base_cpu, msr,
 			msr & (1 << 0) ? "DIS" : "EN",
-			msr & (1 << 1) ? "EN" : "DIS");
+			msr & (1 << 1) ? "EN" : "DIS",
+			msr & (1 << 8) ? "EN" : "DIS");
 }
 
 void process_cpuid()