staging/android: do not let userspace trigger WARN_ON
authorGustavo Padovan <gustavo.padovan@collabora.co.uk>
Thu, 11 Aug 2016 15:26:40 +0000 (12:26 -0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 15 Aug 2016 14:39:45 +0000 (16:39 +0200)
Closing the timeline without waiting all fences to signal is not
a critical failure, it is just bad usage from userspace so avoid
calling WARN_ON in this case.

Signed-off-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/android/sw_sync.c

index 115c9174705f1b1bd2fe34dac727df54c1b9b75b..bda1f6a220c19ca7ba5f1d60f2a7321df5ede53a 100644 (file)
@@ -176,7 +176,7 @@ static void timeline_fence_release(struct fence *fence)
 
        spin_lock_irqsave(fence->lock, flags);
        list_del(&pt->child_list);
-       if (WARN_ON_ONCE(!list_empty(&pt->active_list)))
+       if (!list_empty(&pt->active_list))
                list_del(&pt->active_list);
        spin_unlock_irqrestore(fence->lock, flags);