Commit 357ef5b3 authored by Andrew Martin's avatar Andrew Martin Committed by Alex Deucher
Browse files

drm/amdgpu: Failed to check various return code 



Clean up code to quiet the compiler on us failing to check the return
code.

Signed-off-by: default avatarAndrew Martin <Andrew.Martin@amd.com>
Reviewed-by: default avatarHarish Kasiviswanathan <Harish.Kasiviswanathan@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 88a45aa6

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

+1 −1
Original line number Diff line number Diff line
@@ -368,7 +368,7 @@ void amdgpu_amdkfd_free_gtt_mem(struct amdgpu_device *adev, void **mem_obj) 
{

	struct amdgpu_bo **bo = (struct amdgpu_bo **) mem_obj;



	amdgpu_bo_reserve(*bo, true);

	(void)amdgpu_bo_reserve(*bo, true);

	amdgpu_bo_kunmap(*bo);

	amdgpu_bo_unpin(*bo);

	amdgpu_bo_unreserve(*bo);

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

+7 −7
Original line number Diff line number Diff line
@@ -730,7 +730,7 @@ kfd_mem_dmaunmap_userptr(struct kgd_mem *mem, 
		return;



	amdgpu_bo_placement_from_domain(bo, AMDGPU_GEM_DOMAIN_CPU);

	ttm_bo_validate(&bo->tbo, &bo->placement, &ctx);

	(void)ttm_bo_validate(&bo->tbo, &bo->placement, &ctx);



	dma_unmap_sgtable(adev->dev, ttm->sg, direction, 0);

	sg_free_table(ttm->sg);
@@ -779,7 +779,7 @@ kfd_mem_dmaunmap_sg_bo(struct kgd_mem *mem, 
	}



	amdgpu_bo_placement_from_domain(bo, AMDGPU_GEM_DOMAIN_CPU);

	ttm_bo_validate(&bo->tbo, &bo->placement, &ctx);

	(void)ttm_bo_validate(&bo->tbo, &bo->placement, &ctx);



	dir = mem->alloc_flags & KFD_IOC_ALLOC_MEM_FLAGS_WRITABLE ?

				DMA_BIDIRECTIONAL : DMA_TO_DEVICE;
@@ -989,7 +989,7 @@ static int kfd_mem_attach(struct amdgpu_device *adev, struct kgd_mem *mem, 
		if (!attachment[i])

			continue;

		if (attachment[i]->bo_va) {

			amdgpu_bo_reserve(bo[i], true);

			(void)amdgpu_bo_reserve(bo[i], true);

			if (--attachment[i]->bo_va->ref_count == 0)

				amdgpu_vm_bo_del(adev, attachment[i]->bo_va);

			amdgpu_bo_unreserve(bo[i]);
@@ -1259,11 +1259,11 @@ static int unmap_bo_from_gpuvm(struct kgd_mem *mem, 
		return -EBUSY;

	}



	amdgpu_vm_bo_unmap(adev, bo_va, entry->va);

	(void)amdgpu_vm_bo_unmap(adev, bo_va, entry->va);



	amdgpu_vm_clear_freed(adev, vm, &bo_va->last_pt_update);

	(void)amdgpu_vm_clear_freed(adev, vm, &bo_va->last_pt_update);



	amdgpu_sync_fence(sync, bo_va->last_pt_update);

	(void)amdgpu_sync_fence(sync, bo_va->last_pt_update);



	return 0;

}
@@ -2352,7 +2352,7 @@ void amdgpu_amdkfd_gpuvm_unmap_gtt_bo_from_kernel(struct kgd_mem *mem) 
{

	struct amdgpu_bo *bo = mem->bo;



	amdgpu_bo_reserve(bo, true);

	(void)amdgpu_bo_reserve(bo, true);

	amdgpu_bo_kunmap(bo);

	amdgpu_bo_unpin(bo);

	amdgpu_bo_unreserve(bo);