ecore_wl2: Better error handling for dmabuf sync ioctls

Summary:
Turns out these can fail with EINTR or EAGAIN, and we're supposed
to try again.

Reviewers: zmike

Reviewed By: zmike

Subscribers: cedric, #committers, zmike

Tags: #efl

Differential Revision: https://phab.enlightenment.org/D6250

1 files changed, 10 insertions, 2 deletions

diff --git a/src/lib/ecore_wl2/ecore_wl2_buffer.c b/src/lib/ecore_wl2/ecore_wl2_buffer.c
index 498e3091f5..8e2e164546 100644
--- a/src/lib/ecore_wl2/ecore_wl2_buffer.c
+++ b/src/lib/ecore_wl2/ecore_wl2_buffer.c
@@ -111,7 +111,11 @@ _dmabuf_lock(Ecore_Wl2_Buffer *b)
    struct dma_buf_sync s;
 
    s.flags = DMA_BUF_SYNC_START | DMA_BUF_SYNC_RW;
-   ret = ioctl(b->fd, DMA_BUF_IOCTL_SYNC, &s);
+   do
+     {
+        ret = ioctl(b->fd, DMA_BUF_IOCTL_SYNC, &s);
+     } while (ret && ((errno == EAGAIN) || (errno == EINTR)));
+
    if (ret) WRN("Failed to lock dmabuf");
 }
 
@@ -122,7 +126,11 @@ _dmabuf_unlock(Ecore_Wl2_Buffer *b)
    struct dma_buf_sync s;
 
    s.flags = DMA_BUF_SYNC_END | DMA_BUF_SYNC_RW;
-   ret = ioctl(b->fd, DMA_BUF_IOCTL_SYNC, &s);
+   do
+     {
+        ret = ioctl(b->fd, DMA_BUF_IOCTL_SYNC, &s);
+     } while (ret && ((errno == EAGAIN) || (errno == EINTR)));
+
    if (ret) WRN("Failed to unlock dmabuf");
 }