[PATCH 3/3] reservation: revert "wait only with non-zero timeout specified (v3)"

Christian König deathsimple at vodafone.de
Mon Oct 17 13:55:18 UTC 2016


From: Christian König <christian.koenig at amd.com>

This reverts commit fb8b7d2b9d80e1e71f379e57355936bd2b024be9.

Otherwise signaling might never be activated on the fences. This can
result in infinite waiting with hardware which has unreliable interrupts.

Signed-off-by: Christian König <christian.koenig at amd.com>
---
 drivers/dma-buf/reservation.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/drivers/dma-buf/reservation.c b/drivers/dma-buf/reservation.c
index 9566a62..bb97b04 100644
--- a/drivers/dma-buf/reservation.c
+++ b/drivers/dma-buf/reservation.c
@@ -381,9 +381,6 @@ long reservation_object_wait_timeout_rcu(struct reservation_object *obj,
 	unsigned seq, shared_count, i = 0;
 	long ret = timeout;
 
-	if (!timeout)
-		return reservation_object_test_signaled_rcu(obj, wait_all);
-
 retry:
 	fence = NULL;
 	shared_count = 0;
-- 
2.5.0



More information about the dri-devel mailing list