From: Lucas Stach Date: Thu, 4 Oct 2018 09:37:00 +0000 (+0200) Subject: drm/etnaviv: fix bogus fence complete check in timeout handler X-Git-Tag: rel_imx_4.19.35_1.1.0~8531 X-Git-Url: https://git.somdevices.com/?a=commitdiff_plain;h=3ff21b59ffd294c04fd83994e71417247f0d2ee7;p=linux.git drm/etnaviv: fix bogus fence complete check in timeout handler commit 6fce3a406108ee6c8a61e2a33e52e9198a626ea0 upstream. The GPU hardware fences and the job out-fences are on different timelines so it's wrong to compare them. Fix this by only looking at the out-fence. Cc: Fixes: 2c83a726d6fb (drm/etnaviv: bring back progress check in job timeout handler) Signed-off-by: Lucas Stach Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/gpu/drm/etnaviv/etnaviv_sched.c b/drivers/gpu/drm/etnaviv/etnaviv_sched.c index 69e9b431bf1f..e5a9fae31ab7 100644 --- a/drivers/gpu/drm/etnaviv/etnaviv_sched.c +++ b/drivers/gpu/drm/etnaviv/etnaviv_sched.c @@ -93,7 +93,7 @@ static void etnaviv_sched_timedout_job(struct drm_sched_job *sched_job) * If the GPU managed to complete this jobs fence, the timout is * spurious. Bail out. */ - if (fence_completed(gpu, submit->out_fence->seqno)) + if (dma_fence_is_signaled(submit->out_fence)) return; /*