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@xxxxxxx>
> 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@xxxxxxx>
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) {
_______________________________________________
amd-gfx mailing list
amd-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
