Re: [PATCH v2 6/7] intel-speed-select: Implement 'perf-profile info' on CascadeLake-N

From: Srinivas Pandruvada
Date: Fri Oct 04 2019 - 16:14:48 EST


On Thu, 2019-10-03 at 08:11 -0400, Prarit Bhargava wrote:
> Add functionality for perf-profile info on CascadeLake-N.
Also the display is not correct. It will show redundant info for both
packages.

#intel-speed-select perf-profile info
Intel(R) Speed Select Technology
Executing on CPU model:85[0x55]
package-0
die-0
cpu-0
perf-profile-level-0
cpu-count:20
enable-cpu-mask:00000000,00000000
enable-cpu-list:none
thermal-design-power-ratio:23
base-frequency(MHz):2300
speed-select-turbo-freq:unsupported
speed-select-base-freq:enabled
speed-select-base-freq
high-priority-base-frequency(MHz):2700000
high-priority-cpu-mask:0000000e,8c00e8c0
high-priority-cpu-list:6,7,11,13,14,15,26,27,31,33,34,35
low-priority-base-frequency(MHz):2100000
package-1
die-0
cpu-20
perf-profile-level-0
cpu-count:20
enable-cpu-mask:00000000,00000000
enable-cpu-list:none
thermal-design-power-ratio:23
base-frequency(MHz):2300
speed-select-turbo-freq:unsupported
speed-select-base-freq:enabled
speed-select-base-freq
high-priority-base-frequency(MHz):2700000
high-priority-cpu-mask:0000000e,8c00e8c0
high-priority-cpu-list:6,7,11,13,14,15,26,27,31,33,34,35
low-priority-base-frequency(MHz):2100000

Thanks,
Srinivas

>
> Signed-off-by: Prarit Bhargava <prarit@xxxxxxxxxx>
> ---
> .../x86/intel-speed-select/isst-config.c | 20 +++++++++++++++
> ----
> .../x86/intel-speed-select/isst-display.c | 12 +++++++++++
> tools/power/x86/intel-speed-select/isst.h | 1 +
> 3 files changed, 29 insertions(+), 4 deletions(-)
>
> diff --git a/tools/power/x86/intel-speed-select/isst-config.c
> b/tools/power/x86/intel-speed-select/isst-config.c
> index 3ab0edade5ec..d9b580139a07 100644
> --- a/tools/power/x86/intel-speed-select/isst-config.c
> +++ b/tools/power/x86/intel-speed-select/isst-config.c
> @@ -836,6 +836,12 @@ static void clx_n_config(void)
> ctdp_level->fact_enabled = 0;
> }
>
> +static void dump_cn_config_for_cpu(int cpu, void *arg1, void *arg2,
> + void *arg3, void *arg4)
> +{
> + isst_ctdp_display_information(cpu, outf, tdp_level,
> &clx_n_pkg_dev);
> +}
> +
> static void dump_isst_config_for_cpu(int cpu, void *arg1, void
> *arg2,
> void *arg3, void *arg4)
> {
> @@ -854,6 +860,8 @@ static void dump_isst_config_for_cpu(int cpu,
> void *arg1, void *arg2,
>
> static void dump_isst_config(int arg)
> {
> + void *fn;
> +
> if (cmd_help) {
> fprintf(stderr,
> "Print Intel(R) Speed Select Technology
> Performance profile configuration\n");
> @@ -865,14 +873,17 @@ static void dump_isst_config(int arg)
> exit(0);
> }
>
> + if (!is_clx_n_platform())
> + fn = dump_isst_config_for_cpu;
> + else
> + fn = dump_cn_config_for_cpu;
> +
> isst_ctdp_display_information_start(outf);
>
> if (max_target_cpus)
> - for_each_online_target_cpu_in_set(dump_isst_config_for_
> cpu,
> - NULL, NULL, NULL,
> NULL);
> + for_each_online_target_cpu_in_set(fn, NULL, NULL, NULL,
> NULL);
> else
> - for_each_online_package_in_set(dump_isst_config_for_cpu
> , NULL,
> - NULL, NULL, NULL);
> + for_each_online_package_in_set(fn, NULL, NULL, NULL,
> NULL);
>
> isst_ctdp_display_information_end(outf);
> }
> @@ -1633,6 +1644,7 @@ static void get_clos_assoc(int arg)
> }
>
> static struct process_cmd_struct clx_n_cmds[] = {
> + { "perf-profile", "info", dump_isst_config, 0 },
> { NULL, NULL, NULL, 0 }
> };
>
> diff --git a/tools/power/x86/intel-speed-select/isst-display.c
> b/tools/power/x86/intel-speed-select/isst-display.c
> index 1caa7ae25245..8309810e7425 100644
> --- a/tools/power/x86/intel-speed-select/isst-display.c
> +++ b/tools/power/x86/intel-speed-select/isst-display.c
> @@ -202,6 +202,9 @@ static void _isst_pbf_display_information(int
> cpu, FILE *outf, int level,
> pbf_info->p1_low * DISP_FREQ_MULTIPLIER);
> format_and_print(outf, disp_level + 1, header, value);
>
> + if (is_clx_n_platform())
> + return;
> +
> snprintf(header, sizeof(header), "tjunction-temperature(C)");
> snprintf(value, sizeof(value), "%d", pbf_info->t_prochot);
> format_and_print(outf, disp_level + 1, header, value);
> @@ -375,6 +378,15 @@ void isst_ctdp_display_information(int cpu, FILE
> *outf, int tdp_level,
> snprintf(value, sizeof(value), "unsupported");
> format_and_print(outf, base_level + 4, header, value);
>
> + if (is_clx_n_platform()) {
> + if (ctdp_level->pbf_support)
> + _isst_pbf_display_information(cpu,
> outf,
> + tdp_level
> ,
> + &ctdp_level-
> >pbf_info,
> + base_leve
> l + 4);
> + continue;
> + }
> +
> snprintf(header, sizeof(header), "thermal-design-
> power(W)");
> snprintf(value, sizeof(value), "%d", ctdp_level-
> >pkg_tdp);
> format_and_print(outf, base_level + 4, header, value);
> diff --git a/tools/power/x86/intel-speed-select/isst.h
> b/tools/power/x86/intel-speed-select/isst.h
> index 0dcae17b3945..bef27bd6138e 100644
> --- a/tools/power/x86/intel-speed-select/isst.h
> +++ b/tools/power/x86/intel-speed-select/isst.h
> @@ -239,4 +239,5 @@ extern void isst_display_result(int cpu, FILE
> *outf, char *feature, char *cmd,
> extern int isst_clos_get_clos_information(int cpu, int *enable, int
> *type);
> extern void isst_clos_display_clos_information(int cpu, FILE *outf,
> int clos_enable, int
> type);
> +extern int is_clx_n_platform(void);
> #endif