Commit cc63bcfd authored by Jesse.zhang@amd.com's avatar Jesse.zhang@amd.com Committed by Alex Deucher
Browse files

drm/amdgpu: Fix SDMA engine reset logic 



The scheduler should restart only if the reset operation
succeeds  This ensures that new tasks are only submitted
to the queues after a successful reset.

Fixes: 4c02f730 ("drm/amdgpu: Introduce conditional user queue suspension for SDMA resets")
Suggested-by: default avatarAlex Deucher <alexander.deucher@amd.com>
Signed-off-by: default avatarJesse.Zhang <Jesse.zhang@amd.com>
Reviewed-by: default avatarTim Huang <tim.huang@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 1cfeb60e

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

+1 −1
Original line number Diff line number Diff line
@@ -609,7 +609,7 @@ int amdgpu_sdma_reset_engine(struct amdgpu_device *adev, uint32_t instance_id, b 
	 * if they were stopped by this function. This allows new tasks

	 * to be submitted to the queues after the reset is complete.

	 */

	if (ret) {

	if (!ret) {

		if (gfx_sched_stopped && amdgpu_ring_sched_ready(gfx_ring)) {

			drm_sched_wqueue_start(&gfx_ring->sched);

		}