[PATCH] drm/amdgpu: Do not init ta microcode from guest side

[Zhang, Hawking]

[AMD Official Use Only - AMD Internal Distribution Only]

Yes, this applies to all types of Tas

Regards,
Hawking

-----Original Message-----
From: Lazar, Lijo <Lijo.Lazar at amd.com>
Sent: Monday, August 12, 2024 12:52
To: Zhang, Hawking <Hawking.Zhang at amd.com>; amd-gfx at lists.freedesktop.org; Zhou1, Tao <Tao.Zhou1 at amd.com>
Subject: Re: [PATCH] drm/amdgpu: Do not init ta microcode from guest side



On 8/12/2024 8:52 AM, Hawking Zhang wrote:
> TA should not be loaded from guest side.

Does this apply to XGMI TA?

Thanks,
Lijo

>
> Signed-off-by: Hawking Zhang <Hawking.Zhang at amd.com>
> Reviewed-by: Shiwu Zhang <shiwu.zhang at amd.com>
> ---
>  drivers/gpu/drm/amd/amdgpu/psp_v13_0.c | 8 +++++---
>  1 file changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/psp_v13_0.c b/drivers/gpu/drm/amd/amdgpu/psp_v13_0.c
> index 85ec9e35690a..749d8143b1e7 100644
> --- a/drivers/gpu/drm/amd/amdgpu/psp_v13_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/psp_v13_0.c
> @@ -132,9 +132,11 @@ static int psp_v13_0_init_microcode(struct psp_context *psp)
>                   (adev->emu_flags & AMDGPU_EMU_dGPU_SIDEWINDER))
>                       break;
>               /* It's not necessary to load ras ta on Guest side */
> -             err = psp_init_ta_microcode(psp, ucode_prefix);
> -             if (err)
> -                     return err;
> +             if (!amdgpu_sriov_vf(adev)) {
> +                     err = psp_init_ta_microcode(psp, ucode_prefix);
> +                     if (err)
> +                             return err;
> +             }
>               break;
>       default:
>               BUG();