Please forget that one.
Double checking the code again I've found that this change is nonsense.
Christian.
Am 24.10.2017 um 14:44 schrieb Christian König:
From: Christian König <[email protected]>
When the scheduler drops a job we need to free the IBs as well.
Signed-off-by: Christian König <[email protected]>
Reported-by: Qu, Jim <[email protected]>
---
drivers/gpu/drm/amd/amdgpu/amdgpu_job.c | 6 +-----
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c
b/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c
index a58e3c5dd84b..2790f61842a1 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c
@@ -105,11 +105,7 @@ static void amdgpu_job_free_cb(struct amd_sched_job *s_job)
struct amdgpu_job *job = container_of(s_job, struct amdgpu_job, base);
amdgpu_ring_priority_put(job->ring, s_job->s_priority);
- dma_fence_put(job->fence);
- amdgpu_sync_free(&job->sync);
- amdgpu_sync_free(&job->dep_sync);
- amdgpu_sync_free(&job->sched_sync);
- kfree(job);
+ amdgpu_job_free(job);
}
void amdgpu_job_free(struct amdgpu_job *job)
_______________________________________________
amd-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
