mirror of
git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2025-09-04 20:19:47 +08:00
drm/amdgpu: drop the amdgpu_device argument from amdgpu_ib_free
It's unused. Signed-off-by: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com> Reviewed-by: Alex Deucher <alexander.deucher@amd.com> Reviewed-by: Christian König <christian.koenig@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
This commit is contained in:
parent
2ae520cb12
commit
0014952b17
@ -89,16 +89,14 @@ int amdgpu_ib_get(struct amdgpu_device *adev, struct amdgpu_vm *vm,
|
||||
/**
|
||||
* amdgpu_ib_free - free an IB (Indirect Buffer)
|
||||
*
|
||||
* @adev: amdgpu_device pointer
|
||||
* @ib: IB object to free
|
||||
* @f: the fence SA bo need wait on for the ib alloation
|
||||
*
|
||||
* Free an IB (all asics).
|
||||
*/
|
||||
void amdgpu_ib_free(struct amdgpu_device *adev, struct amdgpu_ib *ib,
|
||||
struct dma_fence *f)
|
||||
void amdgpu_ib_free(struct amdgpu_ib *ib, struct dma_fence *f)
|
||||
{
|
||||
amdgpu_sa_bo_free(adev, &ib->sa_bo, f);
|
||||
amdgpu_sa_bo_free(&ib->sa_bo, f);
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -267,7 +267,7 @@ void amdgpu_job_free_resources(struct amdgpu_job *job)
|
||||
f = NULL;
|
||||
|
||||
for (i = 0; i < job->num_ibs; ++i)
|
||||
amdgpu_ib_free(NULL, &job->ibs[i], f);
|
||||
amdgpu_ib_free(&job->ibs[i], f);
|
||||
}
|
||||
|
||||
static void amdgpu_job_free_cb(struct drm_sched_job *s_job)
|
||||
|
@ -342,8 +342,7 @@ int amdgpu_sa_bo_manager_start(struct amdgpu_device *adev,
|
||||
int amdgpu_sa_bo_new(struct amdgpu_sa_manager *sa_manager,
|
||||
struct drm_suballoc **sa_bo,
|
||||
unsigned int size);
|
||||
void amdgpu_sa_bo_free(struct amdgpu_device *adev,
|
||||
struct drm_suballoc **sa_bo,
|
||||
void amdgpu_sa_bo_free(struct drm_suballoc **sa_bo,
|
||||
struct dma_fence *fence);
|
||||
#if defined(CONFIG_DEBUG_FS)
|
||||
void amdgpu_sa_bo_dump_debug_info(struct amdgpu_sa_manager *sa_manager,
|
||||
|
@ -462,8 +462,7 @@ int amdgpu_ib_get(struct amdgpu_device *adev, struct amdgpu_vm *vm,
|
||||
unsigned size,
|
||||
enum amdgpu_ib_pool_type pool,
|
||||
struct amdgpu_ib *ib);
|
||||
void amdgpu_ib_free(struct amdgpu_device *adev, struct amdgpu_ib *ib,
|
||||
struct dma_fence *f);
|
||||
void amdgpu_ib_free(struct amdgpu_ib *ib, struct dma_fence *f);
|
||||
int amdgpu_ib_schedule(struct amdgpu_ring *ring, unsigned num_ibs,
|
||||
struct amdgpu_ib *ibs, struct amdgpu_job *job,
|
||||
struct dma_fence **f);
|
||||
|
@ -93,8 +93,7 @@ int amdgpu_sa_bo_new(struct amdgpu_sa_manager *sa_manager,
|
||||
return 0;
|
||||
}
|
||||
|
||||
void amdgpu_sa_bo_free(struct amdgpu_device *adev, struct drm_suballoc **sa_bo,
|
||||
struct dma_fence *fence)
|
||||
void amdgpu_sa_bo_free(struct drm_suballoc **sa_bo, struct dma_fence *fence)
|
||||
{
|
||||
if (sa_bo == NULL || *sa_bo == NULL) {
|
||||
return;
|
||||
|
@ -503,7 +503,7 @@ static int amdgpu_vce_get_create_msg(struct amdgpu_ring *ring, uint32_t handle,
|
||||
ib->ptr[i] = 0x0;
|
||||
|
||||
r = amdgpu_job_submit_direct(job, ring, &f);
|
||||
amdgpu_ib_free(ring->adev, &ib_msg, f);
|
||||
amdgpu_ib_free(&ib_msg, f);
|
||||
if (r)
|
||||
goto err;
|
||||
|
||||
|
@ -586,7 +586,7 @@ static int amdgpu_vcn_dec_send_msg(struct amdgpu_ring *ring,
|
||||
if (r)
|
||||
goto err_free;
|
||||
|
||||
amdgpu_ib_free(adev, ib_msg, f);
|
||||
amdgpu_ib_free(ib_msg, f);
|
||||
|
||||
if (fence)
|
||||
*fence = dma_fence_get(f);
|
||||
@ -597,7 +597,7 @@ static int amdgpu_vcn_dec_send_msg(struct amdgpu_ring *ring,
|
||||
err_free:
|
||||
amdgpu_job_free(job);
|
||||
err:
|
||||
amdgpu_ib_free(adev, ib_msg, f);
|
||||
amdgpu_ib_free(ib_msg, f);
|
||||
return r;
|
||||
}
|
||||
|
||||
@ -779,7 +779,7 @@ static int amdgpu_vcn_dec_sw_send_msg(struct amdgpu_ring *ring,
|
||||
if (r)
|
||||
goto err_free;
|
||||
|
||||
amdgpu_ib_free(adev, ib_msg, f);
|
||||
amdgpu_ib_free(ib_msg, f);
|
||||
|
||||
if (fence)
|
||||
*fence = dma_fence_get(f);
|
||||
@ -790,7 +790,7 @@ static int amdgpu_vcn_dec_sw_send_msg(struct amdgpu_ring *ring,
|
||||
err_free:
|
||||
amdgpu_job_free(job);
|
||||
err:
|
||||
amdgpu_ib_free(adev, ib_msg, f);
|
||||
amdgpu_ib_free(ib_msg, f);
|
||||
return r;
|
||||
}
|
||||
|
||||
@ -1020,7 +1020,7 @@ int amdgpu_vcn_enc_ring_test_ib(struct amdgpu_ring *ring, long timeout)
|
||||
r = 0;
|
||||
|
||||
error:
|
||||
amdgpu_ib_free(adev, &ib, fence);
|
||||
amdgpu_ib_free(&ib, fence);
|
||||
dma_fence_put(fence);
|
||||
|
||||
return r;
|
||||
|
@ -834,7 +834,7 @@ static int vpe_ring_test_ib(struct amdgpu_ring *ring, long timeout)
|
||||
ret = (le32_to_cpu(adev->wb.wb[index]) == test_pattern) ? 0 : -EINVAL;
|
||||
|
||||
err1:
|
||||
amdgpu_ib_free(adev, &ib, NULL);
|
||||
amdgpu_ib_free(&ib, NULL);
|
||||
dma_fence_put(f);
|
||||
err0:
|
||||
amdgpu_device_wb_free(adev, index);
|
||||
|
@ -698,7 +698,7 @@ static int cik_sdma_ring_test_ib(struct amdgpu_ring *ring, long timeout)
|
||||
r = -EINVAL;
|
||||
|
||||
err1:
|
||||
amdgpu_ib_free(adev, &ib, NULL);
|
||||
amdgpu_ib_free(&ib, NULL);
|
||||
dma_fence_put(f);
|
||||
err0:
|
||||
amdgpu_device_wb_free(adev, index);
|
||||
|
@ -4036,7 +4036,7 @@ static int gfx_v10_0_ring_test_ib(struct amdgpu_ring *ring, long timeout)
|
||||
else
|
||||
r = -EINVAL;
|
||||
err2:
|
||||
amdgpu_ib_free(adev, &ib, NULL);
|
||||
amdgpu_ib_free(&ib, NULL);
|
||||
dma_fence_put(f);
|
||||
err1:
|
||||
amdgpu_device_wb_free(adev, index);
|
||||
|
@ -615,7 +615,7 @@ static int gfx_v11_0_ring_test_ib(struct amdgpu_ring *ring, long timeout)
|
||||
r = -EINVAL;
|
||||
err2:
|
||||
if (!ring->is_mes_queue)
|
||||
amdgpu_ib_free(adev, &ib, NULL);
|
||||
amdgpu_ib_free(&ib, NULL);
|
||||
dma_fence_put(f);
|
||||
err1:
|
||||
if (!ring->is_mes_queue)
|
||||
|
@ -513,7 +513,7 @@ static int gfx_v12_0_ring_test_ib(struct amdgpu_ring *ring, long timeout)
|
||||
r = -EINVAL;
|
||||
err2:
|
||||
if (!ring->is_mes_queue)
|
||||
amdgpu_ib_free(adev, &ib, NULL);
|
||||
amdgpu_ib_free(&ib, NULL);
|
||||
dma_fence_put(f);
|
||||
err1:
|
||||
if (!ring->is_mes_queue)
|
||||
|
@ -1910,7 +1910,7 @@ static int gfx_v6_0_ring_test_ib(struct amdgpu_ring *ring, long timeout)
|
||||
r = -EINVAL;
|
||||
|
||||
error:
|
||||
amdgpu_ib_free(adev, &ib, NULL);
|
||||
amdgpu_ib_free(&ib, NULL);
|
||||
dma_fence_put(f);
|
||||
return r;
|
||||
}
|
||||
|
@ -2330,7 +2330,7 @@ static int gfx_v7_0_ring_test_ib(struct amdgpu_ring *ring, long timeout)
|
||||
r = -EINVAL;
|
||||
|
||||
error:
|
||||
amdgpu_ib_free(adev, &ib, NULL);
|
||||
amdgpu_ib_free(&ib, NULL);
|
||||
dma_fence_put(f);
|
||||
return r;
|
||||
}
|
||||
|
@ -914,7 +914,7 @@ static int gfx_v8_0_ring_test_ib(struct amdgpu_ring *ring, long timeout)
|
||||
r = -EINVAL;
|
||||
|
||||
err2:
|
||||
amdgpu_ib_free(adev, &ib, NULL);
|
||||
amdgpu_ib_free(&ib, NULL);
|
||||
dma_fence_put(f);
|
||||
err1:
|
||||
amdgpu_device_wb_free(adev, index);
|
||||
@ -1656,7 +1656,7 @@ static int gfx_v8_0_do_edc_gpr_workarounds(struct amdgpu_device *adev)
|
||||
RREG32(sec_ded_counter_registers[i]);
|
||||
|
||||
fail:
|
||||
amdgpu_ib_free(adev, &ib, NULL);
|
||||
amdgpu_ib_free(&ib, NULL);
|
||||
dma_fence_put(f);
|
||||
|
||||
return r;
|
||||
|
@ -1243,7 +1243,7 @@ static int gfx_v9_0_ring_test_ib(struct amdgpu_ring *ring, long timeout)
|
||||
r = -EINVAL;
|
||||
|
||||
err2:
|
||||
amdgpu_ib_free(adev, &ib, NULL);
|
||||
amdgpu_ib_free(&ib, NULL);
|
||||
dma_fence_put(f);
|
||||
err1:
|
||||
amdgpu_device_wb_free(adev, index);
|
||||
@ -4788,7 +4788,7 @@ static int gfx_v9_0_do_edc_gpr_workarounds(struct amdgpu_device *adev)
|
||||
}
|
||||
|
||||
fail:
|
||||
amdgpu_ib_free(adev, &ib, NULL);
|
||||
amdgpu_ib_free(&ib, NULL);
|
||||
dma_fence_put(f);
|
||||
|
||||
return r;
|
||||
|
@ -412,7 +412,7 @@ static int gfx_v9_4_2_run_shader(struct amdgpu_device *adev,
|
||||
r = amdgpu_ib_schedule(ring, 1, ib, NULL, fence_ptr);
|
||||
if (r) {
|
||||
dev_err(adev->dev, "ib submit failed (%d).\n", r);
|
||||
amdgpu_ib_free(adev, ib, NULL);
|
||||
amdgpu_ib_free(ib, NULL);
|
||||
}
|
||||
return r;
|
||||
}
|
||||
@ -611,16 +611,16 @@ static int gfx_v9_4_2_do_sgprs_init(struct amdgpu_device *adev)
|
||||
}
|
||||
|
||||
disp2_failed:
|
||||
amdgpu_ib_free(adev, &disp_ibs[2], NULL);
|
||||
amdgpu_ib_free(&disp_ibs[2], NULL);
|
||||
dma_fence_put(fences[2]);
|
||||
disp1_failed:
|
||||
amdgpu_ib_free(adev, &disp_ibs[1], NULL);
|
||||
amdgpu_ib_free(&disp_ibs[1], NULL);
|
||||
dma_fence_put(fences[1]);
|
||||
disp0_failed:
|
||||
amdgpu_ib_free(adev, &disp_ibs[0], NULL);
|
||||
amdgpu_ib_free(&disp_ibs[0], NULL);
|
||||
dma_fence_put(fences[0]);
|
||||
pro_end:
|
||||
amdgpu_ib_free(adev, &wb_ib, NULL);
|
||||
amdgpu_ib_free(&wb_ib, NULL);
|
||||
|
||||
if (r)
|
||||
dev_info(adev->dev, "Init SGPRS Failed\n");
|
||||
@ -687,10 +687,10 @@ static int gfx_v9_4_2_do_vgprs_init(struct amdgpu_device *adev)
|
||||
}
|
||||
|
||||
disp_failed:
|
||||
amdgpu_ib_free(adev, &disp_ib, NULL);
|
||||
amdgpu_ib_free(&disp_ib, NULL);
|
||||
dma_fence_put(fence);
|
||||
pro_end:
|
||||
amdgpu_ib_free(adev, &wb_ib, NULL);
|
||||
amdgpu_ib_free(&wb_ib, NULL);
|
||||
|
||||
if (r)
|
||||
dev_info(adev->dev, "Init VGPRS Failed\n");
|
||||
|
@ -503,7 +503,7 @@ static int gfx_v9_4_3_ring_test_ib(struct amdgpu_ring *ring, long timeout)
|
||||
r = -EINVAL;
|
||||
|
||||
err2:
|
||||
amdgpu_ib_free(adev, &ib, NULL);
|
||||
amdgpu_ib_free(&ib, NULL);
|
||||
dma_fence_put(f);
|
||||
err1:
|
||||
amdgpu_device_wb_free(adev, index);
|
||||
|
@ -633,7 +633,7 @@ static int sdma_v2_4_ring_test_ib(struct amdgpu_ring *ring, long timeout)
|
||||
r = -EINVAL;
|
||||
|
||||
err1:
|
||||
amdgpu_ib_free(adev, &ib, NULL);
|
||||
amdgpu_ib_free(&ib, NULL);
|
||||
dma_fence_put(f);
|
||||
err0:
|
||||
amdgpu_device_wb_free(adev, index);
|
||||
|
@ -906,7 +906,7 @@ static int sdma_v3_0_ring_test_ib(struct amdgpu_ring *ring, long timeout)
|
||||
else
|
||||
r = -EINVAL;
|
||||
err1:
|
||||
amdgpu_ib_free(adev, &ib, NULL);
|
||||
amdgpu_ib_free(&ib, NULL);
|
||||
dma_fence_put(f);
|
||||
err0:
|
||||
amdgpu_device_wb_free(adev, index);
|
||||
|
@ -1565,7 +1565,7 @@ static int sdma_v4_0_ring_test_ib(struct amdgpu_ring *ring, long timeout)
|
||||
r = -EINVAL;
|
||||
|
||||
err1:
|
||||
amdgpu_ib_free(adev, &ib, NULL);
|
||||
amdgpu_ib_free(&ib, NULL);
|
||||
dma_fence_put(f);
|
||||
err0:
|
||||
amdgpu_device_wb_free(adev, index);
|
||||
|
@ -1130,7 +1130,7 @@ static int sdma_v4_4_2_ring_test_ib(struct amdgpu_ring *ring, long timeout)
|
||||
r = -EINVAL;
|
||||
|
||||
err1:
|
||||
amdgpu_ib_free(adev, &ib, NULL);
|
||||
amdgpu_ib_free(&ib, NULL);
|
||||
dma_fence_put(f);
|
||||
err0:
|
||||
amdgpu_device_wb_free(adev, index);
|
||||
|
@ -1194,7 +1194,7 @@ static int sdma_v5_0_ring_test_ib(struct amdgpu_ring *ring, long timeout)
|
||||
r = -EINVAL;
|
||||
|
||||
err1:
|
||||
amdgpu_ib_free(adev, &ib, NULL);
|
||||
amdgpu_ib_free(&ib, NULL);
|
||||
dma_fence_put(f);
|
||||
err0:
|
||||
if (!ring->is_mes_queue)
|
||||
|
@ -1050,7 +1050,7 @@ static int sdma_v5_2_ring_test_ib(struct amdgpu_ring *ring, long timeout)
|
||||
r = -EINVAL;
|
||||
|
||||
err1:
|
||||
amdgpu_ib_free(adev, &ib, NULL);
|
||||
amdgpu_ib_free(&ib, NULL);
|
||||
dma_fence_put(f);
|
||||
err0:
|
||||
if (!ring->is_mes_queue)
|
||||
|
@ -1063,7 +1063,7 @@ static int sdma_v6_0_ring_test_ib(struct amdgpu_ring *ring, long timeout)
|
||||
r = -EINVAL;
|
||||
|
||||
err1:
|
||||
amdgpu_ib_free(adev, &ib, NULL);
|
||||
amdgpu_ib_free(&ib, NULL);
|
||||
dma_fence_put(f);
|
||||
err0:
|
||||
if (!ring->is_mes_queue)
|
||||
|
@ -1108,7 +1108,7 @@ static int sdma_v7_0_ring_test_ib(struct amdgpu_ring *ring, long timeout)
|
||||
r = -EINVAL;
|
||||
|
||||
err1:
|
||||
amdgpu_ib_free(adev, &ib, NULL);
|
||||
amdgpu_ib_free(&ib, NULL);
|
||||
dma_fence_put(f);
|
||||
err0:
|
||||
if (!ring->is_mes_queue)
|
||||
|
@ -286,7 +286,7 @@ static int si_dma_ring_test_ib(struct amdgpu_ring *ring, long timeout)
|
||||
r = -EINVAL;
|
||||
|
||||
err1:
|
||||
amdgpu_ib_free(adev, &ib, NULL);
|
||||
amdgpu_ib_free(&ib, NULL);
|
||||
dma_fence_put(f);
|
||||
err0:
|
||||
amdgpu_device_wb_free(adev, index);
|
||||
|
Loading…
Reference in New Issue
Block a user