[PATCH] drm/amdgpu: Copy MEC FW version to MEC2 if we skipped loading MEC2

[Felix Kuehling]

Am 2021-04-14 um 7:39 p.m. schrieb Joseph Greathouse:
> If we skipped loading MEC2 firmware separately from MEC, then
> MEC2 will be running the same firmware image. Copy the MEC version
> and feature numbers into MEC2 version and feature numbers. This
> is needed for things like GWS support, where we rely on knowing
> what version of firmware is running on MEC2. Leaving these MEC2
> entries blank breaks our ability to version-check enables and
> workarounds.
>
> Signed-off-by: Joseph Greathouse <Joseph.Greathouse at amd.com>
> Change-Id: Id6f672fc69452abd6ff41821a5f2240037048a6f

The patch looks good to me. I'd wait for John to approve it as well.

Reviewed-by: Felix Kuehling <Felix.Kuehling at amd.com>

John, if the patch looks good to you, please feel free to apply it to
amd-staging-drm-next in your time zone. We're in a hurry because this
fix is needed in the next ROCm release. If you believe reporting a fake
MEC2 version would cause problems for graphics, an alternative fix would
be to change kfd_gws_init to check the mec_fw_version on Arcturus.

Regards,
  Felix


> ---
>  drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
> index 06811a1f4625..a078a38c2cee 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
> @@ -1587,6 +1587,9 @@ static int gfx_v9_0_init_cp_compute_microcode(struct amdgpu_device *adev,
>  			err = 0;
>  			adev->gfx.mec2_fw = NULL;
>  		}
> +	} else {
> +		adev->gfx.mec2_fw_version = adev->gfx.mec_fw_version;
> +		adev->gfx.mec2_feature_version = adev->gfx.mec_feature_version;
>  	}
>  
>  	if (adev->firmware.load_type == AMDGPU_FW_LOAD_PSP) {