Commit dd649566 authored by Tvrtko Ursulin's avatar Tvrtko Ursulin Committed by Alex Deucher
Browse files

drm/amdgpu: Remove duplicated "context still alive" check 



When amdgpu_ctx_mgr_fini() calls amdgpu_ctx_mgr_entity_fini() it contains
the exact same "context still alive" check as it will do next. Remove the
duplicated copy.

Reviewed-by: default avatarChristian König <christian.koenig@amd.com>
Signed-off-by: default avatarTvrtko Ursulin <tvrtko.ursulin@igalia.com>
Cc: Alex Deucher <alexander.deucher@amd.com>
Cc: Christian König <christian.koenig@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 16f2c942

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

+0 −12
Original line number Diff line number Diff line
@@ -949,19 +949,7 @@ static void amdgpu_ctx_mgr_entity_fini(struct amdgpu_ctx_mgr *mgr) 


void amdgpu_ctx_mgr_fini(struct amdgpu_ctx_mgr *mgr)

{

	struct amdgpu_ctx *ctx;

	struct idr *idp;

	uint32_t id;



	amdgpu_ctx_mgr_entity_fini(mgr);



	idp = &mgr->ctx_handles;



	idr_for_each_entry(idp, ctx, id) {

		if (kref_put(&ctx->refcount, amdgpu_ctx_fini) != 1)

			DRM_ERROR("ctx %p is still alive\n", ctx);

	}



	idr_destroy(&mgr->ctx_handles);

	mutex_destroy(&mgr->lock);

}