[Public]
Thanks for clarification, Lijo.
Reviewed-by: Guchun Chen <guchun.c...@amd.com>
Regards,
Guchun
-----Original Message-----
From: Lazar, Lijo <lijo.la...@amd.com>
Sent: Thursday, November 25, 2021 9:32 PM
To: Chen, Guchun <guchun.c...@amd.com>; amd-gfx@lists.freedesktop.org
Cc: Deucher, Alexander <alexander.deuc...@amd.com>; Limonciello, Mario
<mario.limoncie...@amd.com>; Zhang, Hawking <hawking.zh...@amd.com>
Subject: Re: [PATCH] drm/amd/pm: Add warning for unexpected PG requests
On 11/25/2021 6:52 PM, Chen, Guchun wrote:
> [Public]
>
> Use dev_warn to be mGPU friendly?
The intention is to get a trace as well along with that. There are multiple
paths to this function.
Thanks,
Lijo
>
> Regards,
> Guchun
>
> -----Original Message-----
> From: amd-gfx <amd-gfx-boun...@lists.freedesktop.org> On Behalf Of
> Lijo Lazar
> Sent: Thursday, November 25, 2021 7:51 PM
> To: amd-gfx@lists.freedesktop.org
> Cc: Deucher, Alexander <alexander.deuc...@amd.com>; Limonciello, Mario
> <mario.limoncie...@amd.com>; Zhang, Hawking <hawking.zh...@amd.com>
> Subject: [PATCH] drm/amd/pm: Add warning for unexpected PG requests
>
> Ideally power gate/ungate requests shouldn't come when smu block is
> uninitialized. Add a WARN message to check the origins if such a thing ever
> happens.
>
> Signed-off-by: Lijo Lazar <lijo.la...@amd.com>
> ---
> drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c
> b/drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c
> index e156add7b560..e0f8ab8be975 100644
> --- a/drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c
> +++ b/drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c
> @@ -277,8 +277,11 @@ static int smu_dpm_set_power_gate(void *handle,
> struct smu_context *smu = handle;
> int ret = 0;
>
> - if (!smu->pm_enabled || !smu->adev->pm.dpm_enabled)
> + if (!smu->pm_enabled || !smu->adev->pm.dpm_enabled) {
> + WARN(true, "SMU uninitialized but power %s requested for %u!\n",
> + gate ? "gate" : "ungate", block_type);
> return -EOPNOTSUPP;
> + }
>
> switch (block_type) {
> /*
> --
> 2.25.1
>
