[AMD Official Use Only]
Reviewed-by: Evan Quan <evan.quan@xxxxxxx>
> -----Original Message-----
> From: Stanley.Yang <Stanley.Yang@xxxxxxx>
> Sent: Thursday, March 10, 2022 10:02 PM
> To: amd-gfx@xxxxxxxxxxxxxxxxxxxxx; Zhang, Hawking
> <Hawking.Zhang@xxxxxxx>; Zhou1, Tao <Tao.Zhou1@xxxxxxx>; Li,
> Candice <Candice.Li@xxxxxxx>; Quan, Evan <Evan.Quan@xxxxxxx>
> Cc: Yang, Stanley <Stanley.Yang@xxxxxxx>
> Subject: [PATCH Review 1/1] drm/amd/pm: use pm mutex to protect ecc
> info table
>
> Change-Id: I6afe0332cbb20528648c38665264930d6b091c2f
> Signed-off-by: Stanley.Yang <Stanley.Yang@xxxxxxx>
> ---
> drivers/gpu/drm/amd/pm/amdgpu_dpm.c | 7 ++++++-
> 1 file changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/pm/amdgpu_dpm.c
> b/drivers/gpu/drm/amd/pm/amdgpu_dpm.c
> index 9a892d6d1d7a..89fbee568be4 100644
> --- a/drivers/gpu/drm/amd/pm/amdgpu_dpm.c
> +++ b/drivers/gpu/drm/amd/pm/amdgpu_dpm.c
> @@ -643,11 +643,16 @@ int amdgpu_dpm_get_ecc_info(struct
> amdgpu_device *adev,
> void *umc_ecc)
> {
> struct smu_context *smu = adev->powerplay.pp_handle;
> + int ret = 0;
>
> if (!is_support_sw_smu(adev))
> return -EOPNOTSUPP;
>
> - return smu_get_ecc_info(smu, umc_ecc);
> + mutex_lock(&adev->pm.mutex);
> + ret = smu_get_ecc_info(smu, umc_ecc);
> + mutex_unlock(&adev->pm.mutex);
> +
> + return ret;
> }
>
> struct amd_vce_state *amdgpu_dpm_get_vce_clock_state(struct
> amdgpu_device *adev,
> --
> 2.17.1
