Commit 2d2ccc4c authored by Felix Kuehling's avatar Felix Kuehling Committed by Alex Deucher
Browse files

drm/amdkfd: Fix error handling in kfd_criu_restore_events 



mutex_unlock before the exit label because all the error code paths that
jump there didn't take that lock. This fixes unbalanced locking errors
in case of restore errors.

Fixes: 40e8a766 ("drm/amdkfd: CRIU checkpoint and restore events")
Signed-off-by: default avatarFelix Kuehling <Felix.Kuehling@amd.com>
Reviewed-by: default avatarRajneesh Bhardwaj <rajneesh.bhardwaj@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent bcde9dad

drivers/gpu/drm/amd/amdkfd/kfd_events.c

+1 −2
Original line number Diff line number Diff line
@@ -506,6 +506,7 @@ int kfd_criu_restore_event(struct file *devkfd, 
		ret = create_other_event(p, ev, &ev_priv->event_id);

		break;

	}

	mutex_unlock(&p->event_mutex);



exit:

	if (ret)
@@ -513,8 +514,6 @@ int kfd_criu_restore_event(struct file *devkfd, 


	kfree(ev_priv);



	mutex_unlock(&p->event_mutex);



	return ret;

}