On 2018-04-18 12:06 PM, Christian König wrote: > We don't have s_fence as local variable here. > > Signed-off-by: Christian König <christian.koenig at amd.com> > --- > drivers/gpu/drm/scheduler/gpu_scheduler.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/scheduler/gpu_scheduler.c b/drivers/gpu/drm/scheduler/gpu_scheduler.c > index 5de79bbb12c8..f4b862503710 100644 > --- a/drivers/gpu/drm/scheduler/gpu_scheduler.c > +++ b/drivers/gpu/drm/scheduler/gpu_scheduler.c > @@ -402,7 +402,7 @@ drm_sched_entity_pop_job(struct drm_sched_entity *entity) > dma_fence_set_error(&sched_job->s_fence->finished, -ECANCELED); > > dma_fence_put(entity->last_scheduled); > - entity->last_scheduled = dma_fence_get(&s_fence->finished); > + entity->last_scheduled = dma_fence_get(&sched_job->s_fence->finished); > > spsc_queue_pop(&entity->job_queue); > return sched_job; > There's no need to wait for a review before pushing such an obvious and trivial fix for a compile failure. -- Earthling Michel Dänzer | http://www.amd.com Libre software enthusiast | Mesa and X developer
