Commit ad7c088e authored by Jesse.Zhang's avatar Jesse.Zhang Committed by Alex Deucher
Browse files

drm/amdgpu: Fix API status offset for MES queue reset 



The mes_v11_0_reset_hw_queue and mes_v12_0_reset_hw_queue functions were
using the wrong union type (MESAPI__REMOVE_QUEUE) when getting the offset
for api_status. Since these functions handle queue reset operations, they
should use MESAPI__RESET union instead.

This fixes the polling of API status during hardware queue reset operations
in the MES for both v11 and v12 versions.

Signed-off-by: default avatarJesse Zhang <jesse.zhang@amd.com>
Reviewed-By: default avatarShaoyun.liu <Shaoyun.liu@amd.com>
Reviewed-by: default avatarPrike Liang <Prike.Liang@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 3a2191ef

drivers/gpu/drm/amd/amdgpu/mes_v11_0.c

+1 −1
Original line number Diff line number Diff line
@@ -497,7 +497,7 @@ static int mes_v11_0_reset_hw_queue(struct amdgpu_mes *mes, 


	return mes_v11_0_submit_pkt_and_poll_completion(mes,

			&mes_reset_queue_pkt, sizeof(mes_reset_queue_pkt),

			offsetof(union MESAPI__REMOVE_QUEUE, api_status));

			offsetof(union MESAPI__RESET, api_status));

}



static int mes_v11_0_map_legacy_queue(struct amdgpu_mes *mes,

drivers/gpu/drm/amd/amdgpu/mes_v12_0.c

+1 −1
Original line number Diff line number Diff line
@@ -517,7 +517,7 @@ static int mes_v12_0_reset_hw_queue(struct amdgpu_mes *mes, 


	return mes_v12_0_submit_pkt_and_poll_completion(mes, pipe,

			&mes_reset_queue_pkt, sizeof(mes_reset_queue_pkt),

			offsetof(union MESAPI__REMOVE_QUEUE, api_status));

			offsetof(union MESAPI__RESET, api_status));

}



static int mes_v12_0_map_legacy_queue(struct amdgpu_mes *mes,