enlightenment/efl
enlightenment
/
efl
8
7
Fork 14
Code Issues 13 Pull Requests 1 Projects Releases Wiki Activity

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
This commit is contained in:
Derek Foreman 2018-06-19 13:43:54 -04:00 committed by Mike Blumenkrantz
parent 023a9ca2ee
commit ee3df4efc3
1 changed files with 10 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/lib/ecore_wl2/ecore_wl2_buffer.c
View File

@ -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");
}