* [PATCH] turbostat: fix PC6 displaying on some systems
@ 2021-10-04 10:52 Artem Bityutskiy
2021-11-10 6:57 ` Artem Bityutskiy
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Artem Bityutskiy @ 2021-10-04 10:52 UTC (permalink / raw)
To: Rafael J. Wysocki; +Cc: Linux PM Mailing List
From: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
'MSR_PKG_CST_CONFIG_CONTROL' encodes the deepest allowed package C-state limit,
and turbostat decodes it.
Before this patch: turbostat does not recognize value "3" on Ice Lake Xeon
(ICX), treats it as "unknown", and does not display any package C-states in the
results table.
After this patch: turbostat recognizes value 3 on ICX, treats it as "PC6", and
correctly displays package C-states in the results table.
Signed-off-by: Artem Bityutskiy <artem.bityutskiy@linux.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 47d3ba895d6d..bc5ae0872fed 100644
--- a/tools/power/x86/turbostat/turbostat.c
+++ b/tools/power/x86/turbostat/turbostat.c
@@ -2323,7 +2323,7 @@ int skx_pkg_cstate_limits[16] =
};
int icx_pkg_cstate_limits[16] =
- { PCL__0, PCL__2, PCL__6, PCLRSV, PCLRSV, PCLRSV, PCLRSV, PCLUNL, PCLRSV, PCLRSV, PCLRSV, PCLRSV, PCLRSV, PCLRSV,
+ { PCL__0, PCL__2, PCL__6, PCL__6, PCLRSV, PCLRSV, PCLRSV, PCLUNL, PCLRSV, PCLRSV, PCLRSV, PCLRSV, PCLRSV, PCLRSV,
PCLRSV, PCLRSV
};
--
2.31.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] turbostat: fix PC6 displaying on some systems
2021-10-04 10:52 [PATCH] turbostat: fix PC6 displaying on some systems Artem Bityutskiy
@ 2021-11-10 6:57 ` Artem Bityutskiy
2021-11-24 15:16 ` Artem Bityutskiy
2022-03-06 14:51 ` Artem Bityutskiy
2 siblings, 0 replies; 5+ messages in thread
From: Artem Bityutskiy @ 2021-11-10 6:57 UTC (permalink / raw)
To: Rafael J. Wysocki, Len Brown, Len Brown; +Cc: Linux PM Mailing List
On Mon, 2021-10-04 at 13:52 +0300, Artem Bityutskiy wrote:
> From: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
>
> 'MSR_PKG_CST_CONFIG_CONTROL' encodes the deepest allowed package C-state limit,
> and turbostat decodes it.
>
> Before this patch: turbostat does not recognize value "3" on Ice Lake Xeon
> (ICX), treats it as "unknown", and does not display any package C-states in the
> results table.
>
> After this patch: turbostat recognizes value 3 on ICX, treats it as "PC6", and
> correctly displays package C-states in the results table.
>
> Signed-off-by: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
Hi,
any chance for this patch to be picked?
Thanks,
Artem.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] turbostat: fix PC6 displaying on some systems
2021-10-04 10:52 [PATCH] turbostat: fix PC6 displaying on some systems Artem Bityutskiy
2021-11-10 6:57 ` Artem Bityutskiy
@ 2021-11-24 15:16 ` Artem Bityutskiy
2022-03-06 14:51 ` Artem Bityutskiy
2 siblings, 0 replies; 5+ messages in thread
From: Artem Bityutskiy @ 2021-11-24 15:16 UTC (permalink / raw)
To: Rafael J. Wysocki; +Cc: Linux PM Mailing List
Hi, any feed-back on this patch, or a chance for it to be picked up?
Thanks!
On Mon, 2021-10-04 at 13:52 +0300, Artem Bityutskiy wrote:
> From: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
>
> 'MSR_PKG_CST_CONFIG_CONTROL' encodes the deepest allowed package C-state
> limit,
> and turbostat decodes it.
>
> Before this patch: turbostat does not recognize value "3" on Ice Lake Xeon
> (ICX), treats it as "unknown", and does not display any package C-states in
> the
> results table.
>
> After this patch: turbostat recognizes value 3 on ICX, treats it as "PC6", and
> correctly displays package C-states in the results table.
>
> Signed-off-by: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] turbostat: fix PC6 displaying on some systems
2021-10-04 10:52 [PATCH] turbostat: fix PC6 displaying on some systems Artem Bityutskiy
2021-11-10 6:57 ` Artem Bityutskiy
2021-11-24 15:16 ` Artem Bityutskiy
@ 2022-03-06 14:51 ` Artem Bityutskiy
2 siblings, 0 replies; 5+ messages in thread
From: Artem Bityutskiy @ 2022-03-06 14:51 UTC (permalink / raw)
To: Rafael J. Wysocki; +Cc: Linux PM Mailing List
On Mon, 2021-10-04 at 13:52 +0300, Artem Bityutskiy wrote:
> From: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
>
> 'MSR_PKG_CST_CONFIG_CONTROL' encodes the deepest allowed package C-state limit,
> and turbostat decodes it.
>
> Before this patch: turbostat does not recognize value "3" on Ice Lake Xeon
> (ICX), treats it as "unknown", and does not display any package C-states in the
> results table.
>
> After this patch: turbostat recognizes value 3 on ICX, treats it as "PC6", and
> correctly displays package C-states in the results table.
>
> Signed-off-by: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
Hi Rafael,
would you please take a look at this patch and possibly apply it? Thanks!
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] turbostat: fix PC6 displaying on some systems
@ 2022-03-08 11:33 me_gmail
0 siblings, 0 replies; 5+ messages in thread
From: me_gmail @ 2022-03-08 11:33 UTC (permalink / raw)
To: Rafael J. Wysocki, Linux PM Mailing List; +Cc: Len Brown, Yu Chen, Kin Cho
From: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
'MSR_PKG_CST_CONFIG_CONTROL' encodes the deepest allowed package C-state limit,
and turbostat decodes it.
Before this patch: turbostat does not recognize value "3" on Ice Lake Xeon
(ICX) and Sapphire Rapids Xeon (SPR), treats it as "unknown", and does not
display any package C-states in the results table.
After this patch: turbostat recognizes value 3 on ICX and SPR, treats it as
"PC6", and correctly displays package C-states in the results table.
Signed-off-by: Artem Bityutskiy <artem.bityutskiy@linux.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 47d3ba895d6d..bc5ae0872fed 100644
--- a/tools/power/x86/turbostat/turbostat.c
+++ b/tools/power/x86/turbostat/turbostat.c
@@ -2323,7 +2323,7 @@ int skx_pkg_cstate_limits[16] =
};
int icx_pkg_cstate_limits[16] =
- { PCL__0, PCL__2, PCL__6, PCLRSV, PCLRSV, PCLRSV, PCLRSV, PCLUNL, PCLRSV, PCLRSV, PCLRSV, PCLRSV, PCLRSV, PCLRSV,
+ { PCL__0, PCL__2, PCL__6, PCL__6, PCLRSV, PCLRSV, PCLRSV, PCLUNL, PCLRSV, PCLRSV, PCLRSV, PCLRSV, PCLRSV, PCLRSV,
PCLRSV, PCLRSV
};
--
2.31.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
end of thread, other threads:[~2022-03-08 11:33 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-10-04 10:52 [PATCH] turbostat: fix PC6 displaying on some systems Artem Bityutskiy
2021-11-10 6:57 ` Artem Bityutskiy
2021-11-24 15:16 ` Artem Bityutskiy
2022-03-06 14:51 ` Artem Bityutskiy
2022-03-08 11:33 me_gmail
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.