Revert "dma-buf/sync-file: Avoid enable fence signaling if poll(.timeout=0)"
authorGustavo Padovan <gustavo.padovan@collabora.co.uk>
Fri, 18 Nov 2016 08:26:43 +0000 (17:26 +0900)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Fri, 18 Nov 2016 09:35:58 +0000 (10:35 +0100)
This reverts commit ecebca79f6976ddaddfd054d699272515869ea28.

Do not enable fence callback on poll() when using fence_array causes the
fence_array to not signal.

For now we will revert the change and enable signaling everytime time
poll is called with timeout=0 as well.

Cc: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
Acked-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: http://patchwork.freedesktop.org/patch/msgid/1479457603-30758-1-git-send-email-gustavo@padovan.org
drivers/dma-buf/sync_file.c

index 69d8ef98d34c0dd446a2fe20bfa75a6384f5a4ff..6d802f2d2881e21f2c02b5bc3c99abac53e8ed78 100644 (file)
@@ -308,8 +308,7 @@ static unsigned int sync_file_poll(struct file *file, poll_table *wait)
 
        poll_wait(file, &sync_file->wq, wait);
 
-       if (!poll_does_not_wait(wait) &&
-           !test_and_set_bit(POLL_ENABLED, &sync_file->fence->flags)) {
+       if (!test_and_set_bit(POLL_ENABLED, &sync_file->fence->flags)) {
                if (dma_fence_add_callback(sync_file->fence, &sync_file->cb,
                                           fence_check_cb_func) < 0)
                        wake_up_all(&sync_file->wq);