diff options
author | Alex Deucher <alexander.deucher@amd.com> | 2015-09-04 00:11:54 -0400 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2015-09-04 11:04:04 -0400 |
commit | 5134e999cb72c621f48305d5c201ab30156505c1 (patch) | |
tree | 9fb6d5c1d067e9d23ee525085c743dde02eae2e7 | |
parent | 270e869d846af143f60b7404e01e785d18420007 (diff) |
drm/amdgpu: fix warning in scheduler
This should never happen so warn when the count does
not equal the expected size.
Reviewed-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
-rw-r--r-- | drivers/gpu/drm/amd/scheduler/gpu_scheduler.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/drivers/gpu/drm/amd/scheduler/gpu_scheduler.c b/drivers/gpu/drm/amd/scheduler/gpu_scheduler.c index ab8577f8ed4a..9259f1b6664c 100644 --- a/drivers/gpu/drm/amd/scheduler/gpu_scheduler.c +++ b/drivers/gpu/drm/amd/scheduler/gpu_scheduler.c | |||
@@ -332,7 +332,7 @@ static int amd_sched_main(void *param) | |||
332 | { | 332 | { |
333 | struct sched_param sparam = {.sched_priority = 1}; | 333 | struct sched_param sparam = {.sched_priority = 1}; |
334 | struct amd_gpu_scheduler *sched = (struct amd_gpu_scheduler *)param; | 334 | struct amd_gpu_scheduler *sched = (struct amd_gpu_scheduler *)param; |
335 | int r; | 335 | int r, count; |
336 | 336 | ||
337 | sched_setscheduler(current, SCHED_FIFO, &sparam); | 337 | sched_setscheduler(current, SCHED_FIFO, &sparam); |
338 | 338 | ||
@@ -361,7 +361,8 @@ static int amd_sched_main(void *param) | |||
361 | fence_put(fence); | 361 | fence_put(fence); |
362 | } | 362 | } |
363 | 363 | ||
364 | kfifo_out(&entity->job_queue, &job, sizeof(job)); | 364 | count = kfifo_out(&entity->job_queue, &job, sizeof(job)); |
365 | WARN_ON(count != sizeof(job)); | ||
365 | wake_up(&sched->job_scheduled); | 366 | wake_up(&sched->job_scheduled); |
366 | } | 367 | } |
367 | return 0; | 368 | return 0; |