[PATCH 8/8] drm/amd/powerplay: add Renoir support for gpu metrics export
Nirmoy
nirmodas at amd.com
Thu Jul 30 07:26:54 UTC 2020
On 7/30/20 5:28 AM, Evan Quan wrote:
> Add Renoir gpu metrics export interface.
>
> Change-Id: Ic83265536eeaa9e458dc395b2be18ea49da4c68a
> Signed-off-by: Evan Quan <evan.quan at amd.com>
> Reviewed-by: Alex Deucher <alexander.deucher at amd.com>
> ---
> drivers/gpu/drm/amd/powerplay/inc/smu_v12_0.h | 2 +
> drivers/gpu/drm/amd/powerplay/renoir_ppt.c | 95 ++++++++++++++++++-
> drivers/gpu/drm/amd/powerplay/smu_v12_0.c | 10 ++
> 3 files changed, 104 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/powerplay/inc/smu_v12_0.h b/drivers/gpu/drm/amd/powerplay/inc/smu_v12_0.h
> index 02de3b6199e5..fa2e8cb07967 100644
> --- a/drivers/gpu/drm/amd/powerplay/inc/smu_v12_0.h
> +++ b/drivers/gpu/drm/amd/powerplay/inc/smu_v12_0.h
> @@ -60,5 +60,7 @@ int smu_v12_0_set_soft_freq_limited_range(struct smu_context *smu, enum smu_clk_
>
> int smu_v12_0_set_driver_table_location(struct smu_context *smu);
>
> +void smu_v12_0_init_gpu_metrics_v2_0(struct gpu_metrics_v2_0 *gpu_metrics);
> +
> #endif
> #endif
> diff --git a/drivers/gpu/drm/amd/powerplay/renoir_ppt.c b/drivers/gpu/drm/amd/powerplay/renoir_ppt.c
> index 575ae4be98a2..8d73781775bc 100644
> --- a/drivers/gpu/drm/amd/powerplay/renoir_ppt.c
> +++ b/drivers/gpu/drm/amd/powerplay/renoir_ppt.c
> @@ -166,18 +166,32 @@ static int renoir_init_smc_tables(struct smu_context *smu)
>
> smu_table->clocks_table = kzalloc(sizeof(DpmClocks_t), GFP_KERNEL);
> if (!smu_table->clocks_table)
> - return -ENOMEM;
> + goto err0_out;
>
> smu_table->metrics_table = kzalloc(sizeof(SmuMetrics_t), GFP_KERNEL);
> if (!smu_table->metrics_table)
> - return -ENOMEM;
> + goto err1_out;
> smu_table->metrics_time = 0;
>
> smu_table->watermarks_table = kzalloc(sizeof(Watermarks_t), GFP_KERNEL);
> if (!smu_table->watermarks_table)
> - return -ENOMEM;
> + goto err2_out;
> +
> + smu_table->gpu_metrics_table_size = sizeof(struct gpu_metrics_v2_0);
> + smu_table->gpu_metrics_table = kzalloc(smu_table->gpu_metrics_table_size, GFP_KERNEL);
> + if (!smu_table->gpu_metrics_table)
> + goto err3_out;
>
> return 0;
> +
> +err3_out:
> + kfree(smu_table->watermarks_table);
> +err2_out:
> + kfree(smu_table->metrics_table);
> +err1_out:
> + kfree(smu_table->clocks_table);
> +err0_out:
> + return -ENOMEM;
> }
>
> /**
> @@ -995,6 +1009,80 @@ static bool renoir_is_dpm_running(struct smu_context *smu)
>
> }
>
> +static ssize_t renoir_get_gpu_metrics(struct smu_context *smu,
> + void **table)
> +{
> + struct smu_table_context *smu_table = &smu->smu_table;
> + struct gpu_metrics_v2_0 *gpu_metrics =
> + (struct gpu_metrics_v2_0 *)smu_table->gpu_metrics_table;
> + SmuMetrics_t metrics;
> + int ret = 0;
> +
> + ret = renoir_get_metrics_table(smu, &metrics);
> + if (ret)
> + return ret;
> +
> + smu_v12_0_init_gpu_metrics_v2_0(gpu_metrics);
> +
> + gpu_metrics->temperature_gfx = metrics.GfxTemperature;
> + gpu_metrics->temperature_soc = metrics.SocTemperature;
> + gpu_metrics->temperature_core[0] = metrics.CoreTemperature[0];
> + gpu_metrics->temperature_core[1] = metrics.CoreTemperature[1];
> + gpu_metrics->temperature_core[2] = metrics.CoreTemperature[2];
> + gpu_metrics->temperature_core[3] = metrics.CoreTemperature[3];
> + gpu_metrics->temperature_core[4] = metrics.CoreTemperature[4];
> + gpu_metrics->temperature_core[5] = metrics.CoreTemperature[5];
> + gpu_metrics->temperature_core[6] = metrics.CoreTemperature[6];
> + gpu_metrics->temperature_core[7] = metrics.CoreTemperature[7];
memcpy() would this much more compact.
> + gpu_metrics->temperature_l3[0] = metrics.L3Temperature[0];
> + gpu_metrics->temperature_l3[1] = metrics.L3Temperature[1];
> +
> + gpu_metrics->average_gfx_activity = metrics.AverageGfxActivity;
> + gpu_metrics->average_mm_activity = metrics.AverageUvdActivity;
> +
> + gpu_metrics->average_socket_power = metrics.CurrentSocketPower;
> + gpu_metrics->average_cpu_power = metrics.Power[0];
> + gpu_metrics->average_soc_power = metrics.Power[1];
> + gpu_metrics->average_core_power[0] = metrics.CorePower[0];
> + gpu_metrics->average_core_power[1] = metrics.CorePower[1];
> + gpu_metrics->average_core_power[2] = metrics.CorePower[2];
> + gpu_metrics->average_core_power[3] = metrics.CorePower[3];
> + gpu_metrics->average_core_power[4] = metrics.CorePower[4];
> + gpu_metrics->average_core_power[5] = metrics.CorePower[5];
> + gpu_metrics->average_core_power[6] = metrics.CorePower[6];
> + gpu_metrics->average_core_power[7] = metrics.CorePower[7];
Same as above.
> +
> + gpu_metrics->average_gfxclk_frequency = metrics.AverageGfxclkFrequency;
> + gpu_metrics->average_socclk_frequency = metrics.AverageSocclkFrequency;
> + gpu_metrics->average_fclk_frequency = metrics.AverageFclkFrequency;
> + gpu_metrics->average_vclk_frequency = metrics.AverageVclkFrequency;
> +
> + gpu_metrics->current_gfxclk = metrics.ClockFrequency[CLOCK_GFXCLK];
> + gpu_metrics->current_socclk = metrics.ClockFrequency[CLOCK_SOCCLK];
> + gpu_metrics->current_uclk = metrics.ClockFrequency[CLOCK_UMCCLK];
> + gpu_metrics->current_fclk = metrics.ClockFrequency[CLOCK_FCLK];
> + gpu_metrics->current_vclk = metrics.ClockFrequency[CLOCK_VCLK];
> + gpu_metrics->current_dclk = metrics.ClockFrequency[CLOCK_DCLK];
> + gpu_metrics->current_coreclk[0] = metrics.CoreFrequency[0];
> + gpu_metrics->current_coreclk[1] = metrics.CoreFrequency[1];
> + gpu_metrics->current_coreclk[2] = metrics.CoreFrequency[2];
> + gpu_metrics->current_coreclk[3] = metrics.CoreFrequency[3];
> + gpu_metrics->current_coreclk[4] = metrics.CoreFrequency[4];
> + gpu_metrics->current_coreclk[5] = metrics.CoreFrequency[5];
> + gpu_metrics->current_coreclk[6] = metrics.CoreFrequency[6];
> + gpu_metrics->current_coreclk[7] = metrics.CoreFrequency[7];
Same as above.
> + gpu_metrics->current_l3clk[0] = metrics.L3Frequency[0];
> + gpu_metrics->current_l3clk[1] = metrics.L3Frequency[1];
> +
> + gpu_metrics->throttle_status = metrics.ThrottlerStatus;
> +
> + gpu_metrics->fan_pwm = metrics.FanPwm;
> +
> + *table = (void *)gpu_metrics;
> +
> + return sizeof(struct gpu_metrics_v2_0);
> +}
> +
> static const struct pptable_funcs renoir_ppt_funcs = {
> .set_power_state = NULL,
> .print_clk_levels = renoir_print_clk_levels,
> @@ -1029,6 +1117,7 @@ static const struct pptable_funcs renoir_ppt_funcs = {
> .is_dpm_running = renoir_is_dpm_running,
> .get_pp_feature_mask = smu_cmn_get_pp_feature_mask,
> .set_pp_feature_mask = smu_cmn_set_pp_feature_mask,
> + .get_gpu_metrics = renoir_get_gpu_metrics,
> };
>
> void renoir_set_ppt_funcs(struct smu_context *smu)
> diff --git a/drivers/gpu/drm/amd/powerplay/smu_v12_0.c b/drivers/gpu/drm/amd/powerplay/smu_v12_0.c
> index 31456437bb18..533b0f04c85c 100644
> --- a/drivers/gpu/drm/amd/powerplay/smu_v12_0.c
> +++ b/drivers/gpu/drm/amd/powerplay/smu_v12_0.c
> @@ -274,3 +274,13 @@ int smu_v12_0_set_driver_table_location(struct smu_context *smu)
>
> return ret;
> }
> +
> +void smu_v12_0_init_gpu_metrics_v2_0(struct gpu_metrics_v2_0 *gpu_metrics)
> +{
> + memset(gpu_metrics, 0xFF, sizeof(struct gpu_metrics_v2_0));
> +
> + gpu_metrics->common_header.structure_size =
> + sizeof(struct gpu_metrics_v2_0);
> + gpu_metrics->common_header.format_revision = 2;
> + gpu_metrics->common_header.content_revision = 0;
> +}
More information about the amd-gfx
mailing list