RE: [PATCH 1/2] tools/power turbostat: Correct SNB_C1/C3_AUTO_UNDEMOTE defines

[Brown, Len]

You just did it:-)

-Original Message-
From: Matt Turner [mailto:matts...@gmail.com] 
Sent: Saturday, April 07, 2018 6:25 PM
To: Brown, Len 
Cc: Thomas Gleixner ; Ingo Molnar ; H. 
Peter Anvin ; Borislav Petkov ; LKML 
; x...@kernel.org; Matt Turner 

Subject: Re: [PATCH 1/2] tools/power turbostat: Correct SNB_C1/C3_AUTO_UNDEMOTE 
defines

On Tue, Feb 13, 2018 at 11:12 AM, Matt Turner  wrote:
> According to the Intel Software Developers' Manual, Vol. 4, Order No.
> 335592, these macros have been reversed since they were added.
>
> Fixes: 889facbee3e6 ("tools/power turbostat: v3.0: monitor Watts and 
> Temperature")
> Signed-off-by: Matt Turner 

Is there something I need to do to ensure these two trivial patches get picked 
up?

Re: [PATCH 1/2] tools/power turbostat: Correct SNB_C1/C3_AUTO_UNDEMOTE defines

[Matt Turner]

On Tue, Feb 13, 2018 at 11:12 AM, Matt Turner  wrote:
> According to the Intel Software Developers' Manual, Vol. 4, Order No.
> 335592, these macros have been reversed since they were added.
>
> Fixes: 889facbee3e6 ("tools/power turbostat: v3.0: monitor Watts and 
> Temperature")
> Signed-off-by: Matt Turner 

Is there something I need to do to ensure these two trivial patches
get picked up?

[PATCH 1/2] tools/power turbostat: Correct SNB_C1/C3_AUTO_UNDEMOTE defines

[Matt Turner]

According to the Intel Software Developers' Manual, Vol. 4, Order No.
335592, these macros have been reversed since they were added.

Fixes: 889facbee3e6 ("tools/power turbostat: v3.0: monitor Watts and 
Temperature")
Signed-off-by: Matt Turner 
---
 tools/power/x86/turbostat/turbostat.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tools/power/x86/turbostat/turbostat.c 
b/tools/power/x86/turbostat/turbostat.c
index bd9c6b31a504..e55dc2e626c4 100644
--- a/tools/power/x86/turbostat/turbostat.c
+++ b/tools/power/x86/turbostat/turbostat.c
@@ -2071,8 +2071,8 @@ dump_nhm_cst_cfg(void)
 
get_msr(base_cpu, MSR_PKG_CST_CONFIG_CONTROL, &msr);
 
-#define SNB_C1_AUTO_UNDEMOTE  (1UL << 27)
-#define SNB_C3_AUTO_UNDEMOTE  (1UL << 28)
+#define SNB_C3_AUTO_UNDEMOTE  (1UL << 27)
+#define SNB_C1_AUTO_UNDEMOTE  (1UL << 28)
 
fprintf(outf, "cpu%d: MSR_PKG_CST_CONFIG_CONTROL: 0x%08llx", base_cpu, 
msr);
 
-- 
2.13.6