From: Jamie Gennis <[email protected]>

Fix wait behavior on timeout == 0 case

Cc: Maarten Lankhorst <[email protected]>
Cc: Erik Gilling <[email protected]>
Cc: Daniel Vetter <[email protected]>
Cc: Rob Clark <[email protected]>
Cc: Sumit Semwal <[email protected]>
Cc: Greg KH <[email protected]>
Cc: [email protected]
Cc: Android Kernel Team <[email protected]>
Signed-off-by: Jamie Gennis <[email protected]>
[jstultz: Added commit message]
Signed-off-by: John Stultz <[email protected]>
---
 drivers/staging/android/sync.c |   10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/drivers/staging/android/sync.c b/drivers/staging/android/sync.c
index 9b8b0e9..b9bb974 100644
--- a/drivers/staging/android/sync.c
+++ b/drivers/staging/android/sync.c
@@ -616,10 +616,12 @@ int sync_fence_wait(struct sync_fence *fence, long 
timeout)
                return fence->status;
        }
 
-       if (fence->status == 0 && timeout > 0) {
-               pr_info("fence timeout on [%p] after %dms\n", fence,
-                       jiffies_to_msecs(timeout));
-               sync_dump();
+       if (fence->status == 0) {
+               if (timeout > 0) {
+                       pr_info("fence timeout on [%p] after %dms\n", fence,
+                               jiffies_to_msecs(timeout));
+                       sync_dump();
+               }
                return -ETIME;
        }
 
-- 
1.7.10.4

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to