diff options
author | Christian König <christian.koenig@amd.com> | 2016-05-20 07:06:09 -0400 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2016-07-07 14:51:21 -0400 |
commit | a7e7a93e5766dcd2929f8e2ca280967ef1572d27 (patch) | |
tree | 263514d2091a91887dcb257799dbfb68ab739108 /drivers/gpu/drm/amd/amdgpu/amdgpu_job.c | |
parent | 6fc1367582534a54476c701326f00e7d2ec81f22 (diff) |
drm/amdgpu: remove amdgpu_sync_wait
Stop hiding bugs, instead print a proper error when the scheduler
doesn't handle all dependencies.
Acked-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Christian König <christian.koenig@amd.com>
Acked-by: Chunming Zhou <david1.zhou@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_job.c')
-rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_job.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c index ddfed939c893..009e90518038 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c | |||
@@ -166,11 +166,7 @@ static struct fence *amdgpu_job_run(struct amd_sched_job *sched_job) | |||
166 | } | 166 | } |
167 | job = to_amdgpu_job(sched_job); | 167 | job = to_amdgpu_job(sched_job); |
168 | 168 | ||
169 | r = amdgpu_sync_wait(&job->sync); | 169 | BUG_ON(!amdgpu_sync_is_idle(&job->sync)); |
170 | if (r) { | ||
171 | DRM_ERROR("failed to sync wait (%d)\n", r); | ||
172 | return NULL; | ||
173 | } | ||
174 | 170 | ||
175 | trace_amdgpu_sched_run_job(job); | 171 | trace_amdgpu_sched_run_job(job); |
176 | r = amdgpu_ib_schedule(job->ring, job->num_ibs, job->ibs, | 172 | r = amdgpu_ib_schedule(job->ring, job->num_ibs, job->ibs, |