X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/5a557d1ea0c466ea01d8d934d3de8800a625b86a..a95f37bc57cd0318ee2ae5cc94ac447cce47a5cf:/src/unix/epolldispatcher.cpp diff --git a/src/unix/epolldispatcher.cpp b/src/unix/epolldispatcher.cpp index f509c7be25..d42c4fea3b 100644 --- a/src/unix/epolldispatcher.cpp +++ b/src/unix/epolldispatcher.cpp @@ -23,6 +23,7 @@ #include "wx/unix/private/epolldispatcher.h" #include "wx/unix/private.h" +#include "wx/stopwatch.h" #ifndef WX_PRECOMP #include "wx/log.h" @@ -158,30 +159,63 @@ bool wxEpollDispatcher::UnregisterFD(int fd) return true; } -bool wxEpollDispatcher::Dispatch(int timeout) +int +wxEpollDispatcher::DoPoll(epoll_event *events, int numEvents, int timeout) const +{ + // the code below relies on TIMEOUT_INFINITE being -1 so that we can pass + // timeout value directly to epoll_wait() which interprets -1 as meaning to + // wait forever and would need to be changed if the value of + // TIMEOUT_INFINITE ever changes + wxCOMPILE_TIME_ASSERT( TIMEOUT_INFINITE == -1, UpdateThisCode ); + + wxMilliClock_t timeEnd; + if ( timeout > 0 ) + timeEnd = wxGetLocalTimeMillis(); + + int rc; + for ( ;; ) + { + rc = epoll_wait(m_epollDescriptor, events, numEvents, timeout); + if ( rc != -1 || errno != EINTR ) + break; + + // we got interrupted, update the timeout and restart + if ( timeout > 0 ) + { + timeout = wxMilliClockToLong(timeEnd - wxGetLocalTimeMillis()); + if ( timeout < 0 ) + return 0; + } + } + + return rc; +} + +bool wxEpollDispatcher::HasPending() const +{ + epoll_event event; + + // NB: it's not really clear if epoll_wait() can return a number greater + // than the number of events passed to it but just in case it can, use + // >= instead of == here, see #10397 + return DoPoll(&event, 1, 0) >= 1; +} + +int wxEpollDispatcher::Dispatch(int timeout) { epoll_event events[16]; - const int e_num = epoll_wait - ( - m_epollDescriptor, - events, - WXSIZEOF(events), - timeout == TIMEOUT_INFINITE ? -1 : timeout - ); + const int rc = DoPoll(events, WXSIZEOF(events), timeout); - if ( e_num == -1 ) + if ( rc == -1 ) { - if ( errno != EINTR ) - { - wxLogSysError(_("Waiting for IO on epoll descriptor %d failed"), - m_epollDescriptor); - return false; - } + wxLogSysError(_("Waiting for IO on epoll descriptor %d failed"), + m_epollDescriptor); + return -1; } - bool gotEvents = false; - for ( epoll_event *p = events; p < events + e_num; p++ ) + int numEvents = 0; + for ( epoll_event *p = events; p < events + rc; p++ ) { wxFDIOHandler * const handler = (wxFDIOHandler *)(p->data.ptr); if ( !handler ) @@ -203,10 +237,10 @@ bool wxEpollDispatcher::Dispatch(int timeout) else continue; - gotEvents = true; + numEvents++; } - return gotEvents; + return numEvents; } #endif // wxUSE_EPOLL_DISPATCHER