| Message ID | 1476969615-4421-3-git-send-email-deathsimple@vodafone.de (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Hi Christian, 2016-10-20 Christian König <deathsimple@vodafone.de>: > From: Christian König <christian.koenig@amd.com> > > This reverts commit 847b19a39e4c9b5e74c40f0842c48b41664cb43c. > > When we don't call the wait function software signaling might never be > activated. This can cause infinite polling loops with unreliable interrupt > driven hardware. > > Signed-off-by: Christian König <christian.koenig@amd.com> > Reviewed-by: Chunming Zhou <david1.zhou@amd.com> > --- > drivers/dma-buf/fence.c | 3 --- > 1 file changed, 3 deletions(-) Reviewed-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk> Gustavo
diff --git a/drivers/dma-buf/fence.c b/drivers/dma-buf/fence.c index fb915ab..64478f9 100644 --- a/drivers/dma-buf/fence.c +++ b/drivers/dma-buf/fence.c @@ -159,9 +159,6 @@ fence_wait_timeout(struct fence *fence, bool intr, signed long timeout) if (WARN_ON(timeout < 0)) return -EINVAL; - if (timeout == 0) - return fence_is_signaled(fence); - trace_fence_wait_start(fence); ret = fence->ops->wait(fence, intr, timeout); trace_fence_wait_end(fence);