Commit 4bef1abe authored by Alice Wong's avatar Alice Wong Committed by Alex Deucher

drm/amdgpu/psp: Return failure when firmware failed to load in SRIOV

In SRIOV, PSP will block incompatible firmware from loading. When this happens,
driver should be prevented from continue initialization and start cleanup.
Return failure in psp_cmd_submit_buf when firmware load failed in SRIOV.
Signed-off-by: default avatarAlice Wong <shiwei.wong@amd.com>
Reviewed-by: default avatarSashank Saye <sashank.saye@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 3cc69021
...@@ -630,10 +630,11 @@ psp_cmd_submit_buf(struct psp_context *psp, ...@@ -630,10 +630,11 @@ psp_cmd_submit_buf(struct psp_context *psp,
DRM_WARN("psp gfx command %s(0x%X) failed and response status is (0x%X)\n", DRM_WARN("psp gfx command %s(0x%X) failed and response status is (0x%X)\n",
psp_gfx_cmd_name(psp->cmd_buf_mem->cmd_id), psp->cmd_buf_mem->cmd_id, psp_gfx_cmd_name(psp->cmd_buf_mem->cmd_id), psp->cmd_buf_mem->cmd_id,
psp->cmd_buf_mem->resp.status); psp->cmd_buf_mem->resp.status);
/* If we load CAP FW, PSP must return 0 under SRIOV /* If any firmware (including CAP) load fails under SRIOV, it should
* also return failure in case of timeout * return failure to stop the VF from initializing.
* Also return failure in case of timeout
*/ */
if ((ucode && (ucode->ucode_id == AMDGPU_UCODE_ID_CAP)) || !timeout) { if ((ucode && amdgpu_sriov_vf(psp->adev)) || !timeout) {
ret = -EINVAL; ret = -EINVAL;
goto exit; goto exit;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment