do not create a kqueue everytime we call poll
This commit is contained in:
parent
22dd32d4e9
commit
0834198e74
@ -52,10 +52,11 @@ namespace ix
|
|||||||
PollResultType Socket::poll(bool readyToRead,
|
PollResultType Socket::poll(bool readyToRead,
|
||||||
int timeoutMs,
|
int timeoutMs,
|
||||||
int sockfd,
|
int sockfd,
|
||||||
const SelectInterruptPtr& selectInterrupt)
|
const SelectInterruptPtr& selectInterrupt,
|
||||||
|
int kqueuefd)
|
||||||
{
|
{
|
||||||
#if defined(__APPLE__)
|
#if defined(__APPLE__)
|
||||||
int kqueuefd = kqueue();
|
// FIXME int kqueuefd = kqueue();
|
||||||
|
|
||||||
struct kevent ke;
|
struct kevent ke;
|
||||||
EV_SET(&ke, sockfd, (readyToRead) ? EVFILT_READ : EVFILT_WRITE, EV_ADD, 0, 0, NULL);
|
EV_SET(&ke, sockfd, (readyToRead) ? EVFILT_READ : EVFILT_WRITE, EV_ADD, 0, 0, NULL);
|
||||||
@ -145,7 +146,8 @@ namespace ix
|
|||||||
#endif
|
#endif
|
||||||
|
|
||||||
free(events);
|
free(events);
|
||||||
::close(kqueuefd);
|
|
||||||
|
// ::close(kqueuefd); //FMXE
|
||||||
|
|
||||||
return pollResult;
|
return pollResult;
|
||||||
#else
|
#else
|
||||||
@ -254,7 +256,7 @@ namespace ix
|
|||||||
}
|
}
|
||||||
|
|
||||||
bool readyToRead = true;
|
bool readyToRead = true;
|
||||||
return poll(readyToRead, timeoutMs, _sockfd, _selectInterrupt);
|
return poll(readyToRead, timeoutMs, _sockfd, _selectInterrupt, _kqueuefd);
|
||||||
}
|
}
|
||||||
|
|
||||||
PollResultType Socket::isReadyToWrite(int timeoutMs)
|
PollResultType Socket::isReadyToWrite(int timeoutMs)
|
||||||
@ -265,7 +267,7 @@ namespace ix
|
|||||||
}
|
}
|
||||||
|
|
||||||
bool readyToRead = false;
|
bool readyToRead = false;
|
||||||
return poll(readyToRead, timeoutMs, _sockfd, _selectInterrupt);
|
return poll(readyToRead, timeoutMs, _sockfd, _selectInterrupt, _kqueuefd);
|
||||||
}
|
}
|
||||||
|
|
||||||
// Wake up from poll/select by writing to the pipe which is watched by select
|
// Wake up from poll/select by writing to the pipe which is watched by select
|
||||||
|
@ -101,7 +101,8 @@ namespace ix
|
|||||||
static PollResultType poll(bool readyToRead,
|
static PollResultType poll(bool readyToRead,
|
||||||
int timeoutMs,
|
int timeoutMs,
|
||||||
int sockfd,
|
int sockfd,
|
||||||
const SelectInterruptPtr& selectInterrupt);
|
const SelectInterruptPtr& selectInterrupt,
|
||||||
|
int kqueuefd);
|
||||||
|
|
||||||
|
|
||||||
// Used as special codes for pipe communication
|
// Used as special codes for pipe communication
|
||||||
|
@ -66,7 +66,10 @@ namespace ix
|
|||||||
int timeoutMs = 10;
|
int timeoutMs = 10;
|
||||||
bool readyToRead = false;
|
bool readyToRead = false;
|
||||||
auto selectInterrupt = std::make_unique<SelectInterrupt>();
|
auto selectInterrupt = std::make_unique<SelectInterrupt>();
|
||||||
PollResultType pollResult = Socket::poll(readyToRead, timeoutMs, fd, selectInterrupt);
|
|
||||||
|
int kqueuefd = kqueue();
|
||||||
|
PollResultType pollResult = Socket::poll(readyToRead, timeoutMs, fd, selectInterrupt, kqueuefd);
|
||||||
|
::close(kqueuefd);
|
||||||
|
|
||||||
if (pollResult == PollResultType::Timeout)
|
if (pollResult == PollResultType::Timeout)
|
||||||
{
|
{
|
||||||
|
@ -259,8 +259,11 @@ namespace ix
|
|||||||
int timeoutMs = 10;
|
int timeoutMs = 10;
|
||||||
bool readyToRead = true;
|
bool readyToRead = true;
|
||||||
auto selectInterrupt = std::make_unique<SelectInterrupt>();
|
auto selectInterrupt = std::make_unique<SelectInterrupt>();
|
||||||
|
|
||||||
|
int kqueuefd = kqueue();
|
||||||
PollResultType pollResult =
|
PollResultType pollResult =
|
||||||
Socket::poll(readyToRead, timeoutMs, _serverFd, selectInterrupt);
|
Socket::poll(readyToRead, timeoutMs, _serverFd, selectInterrupt, kqueuefd);
|
||||||
|
::close(kqueuefd);
|
||||||
|
|
||||||
if (pollResult == PollResultType::Error)
|
if (pollResult == PollResultType::Error)
|
||||||
{
|
{
|
||||||
|
Loading…
Reference in New Issue
Block a user