Jesus, look who's the commit whore today. More header updates, and removal of namespa...

[user/henk/code/inspircd.git] / src / socketengine_kqueue.cpp

diff --git a/src/socketengine_kqueue.cpp b/src/socketengine_kqueue.cpp
index 0792e792053fd153cd23f9f15fe132b971d1b1c0..02cef0e44d494f9e04955159d567efcb5b8860bb 100644 (file)
--- a/src/socketengine_kqueue.cpp
+++ b/src/socketengine_kqueue.cpp
@@ -2,12 +2,9 @@
  *       | Inspire Internet Relay Chat Daemon |
  *       +------------------------------------+
  *
- *  InspIRCd is copyright (C) 2002-2006 ChatSpike-Dev.
- *                       E-mail:
- *                <brain@chatspike.net>
- *                <Craig@chatspike.net>
+ *  InspIRCd: (C) 2002-2007 InspIRCd Development Team
+ * See: http://www.inspircd.org/wiki/index.php/Credits
  *
- * Written by Craig Edwards, Craig McLure, and others.
  * This program is free but copyrighted software; see
  *            the file COPYING for details.
  *
@@ -45,22 +42,22 @@ bool KQueueEngine::AddFd(EventHandler* eh)
 {
        int fd = eh->GetFd();
 
-       ServerInstance->Log(DEFAULT,"KQueueEngine::AddFd(%d)",fd);
+       ServerInstance->Log(DEBUG,"KQueueEngine::AddFd(%d)",fd);
 
        if ((fd < 0) || (fd > MAX_DESCRIPTORS))
        {
-               ServerInstance->Log(DEFAULT,"ERROR: FD of %d added above max of %d",fd,MAX_DESCRIPTORS);
+               ServerInstance->Log(DEBUG,"ERROR: FD of %d added above max of %d",fd,MAX_DESCRIPTORS);
                return false;
        }
        if (GetRemainingFds() <= 1)
        {
-               ServerInstance->Log(DEFAULT,"ERROR: System out of file descriptors!");
+               ServerInstance->Log(DEBUG,"ERROR: System out of file descriptors!");
                return false;
        }
 
        if (ref[fd])
        {
-               ServerInstance->Log(DEFAULT,"ERROR: Slot already occupied");
+               ServerInstance->Log(DEBUG,"ERROR: Slot already occupied");
                return false;
        }
 
@@ -70,6 +67,7 @@ bool KQueueEngine::AddFd(EventHandler* eh)
        struct kevent ke;
        ServerInstance->Log(DEBUG,"kqueue: Add socket to events, kq=%d socket=%d",EngineHandle,fd);
        EV_SET(&ke, fd, eh->Readable() ? EVFILT_READ : EVFILT_WRITE, EV_ADD, 0, 0, NULL);
+
        int i = kevent(EngineHandle, &ke, 1, 0, 0, NULL);
        if (i == -1)
        {
@@ -91,19 +89,38 @@ bool KQueueEngine::DelFd(EventHandler* eh)
                return false;
 
        struct kevent ke;
-       EV_SET(&ke, fd, eh->Readable() ? EVFILT_READ : EVFILT_WRITE, EV_DELETE, 0, 0, NULL);
+       EV_SET(&ke, eh->GetFd(), EVFILT_READ, EV_DELETE, 0, 0, NULL);
+
        int i = kevent(EngineHandle, &ke, 1, 0, 0, NULL);
-       if (i == -1)
-       {
-               ServerInstance->Log(DEBUG,"kqueue: Failed to remove socket from queue!");
+       
+       EV_SET(&ke, eh->GetFd(), EVFILT_WRITE, EV_DELETE, 0, 0, NULL);
+
+       int j = kevent(EngineHandle, &ke, 1, 0, 0, NULL);
+
+       if ((j < 0) && (i < 0))
                return false;
-       }
 
        CurrentSetSize--;
        ref[fd] = NULL;
+
        return true;
 }
 
+void KQueueEngine::WantWrite(EventHandler* eh)
+{
+       /** When changing an item in a kqueue, there is no 'modify' call
+        * as in epoll. Instead, we add the item again, and this overwrites
+        * the original setting rather than adding it twice. See man kqueue.
+        */
+       struct kevent ke;
+       EV_SET(&ke, eh->GetFd(), EVFILT_WRITE, EV_ADD | EV_ONESHOT, 0, 0, NULL);
+       int i = kevent(EngineHandle, &ke, 1, 0, 0, NULL);
+       if (i == -1)
+       {
+               ServerInstance->Log(DEBUG,"kqueue: Unable to set fd %d for wanting write", eh->GetFd());
+       }
+}
+
 int KQueueEngine::GetMaxFds()
 {
        return MAX_DESCRIPTORS;
@@ -121,8 +138,39 @@ int KQueueEngine::DispatchEvents()
        int i = kevent(EngineHandle, NULL, 0, &ke_list[0], MAX_DESCRIPTORS, &ts);
        for (int j = 0; j < i; j++)
        {
-               ServerInstance->Log(DEBUG,"Handle %s event on fd %d",ref[ke_list[j].ident]->Readable() ? "read" : "write", ref[ke_list[j].ident]->GetFd());
-               ref[ke_list[j].ident]->HandleEvent(ref[ke_list[j].ident]->Readable() ? EVENT_READ : EVENT_WRITE);
+               ServerInstance->Log(DEBUG,"Handle %s event on fd %d",ke_list[j].flags & EVFILT_WRITE ? "write" : "read", ke_list[j].ident);
+               if (ke_list[j].flags & EV_EOF)
+               {
+                       ServerInstance->Log(DEBUG,"kqueue: Error on FD %d", ke_list[j].ident);
+                       /* We love you kqueue, oh yes we do *sings*!
+                        * kqueue gives us the error number directly in the EOF state!
+                        * Unlike smelly epoll and select, where we have to getsockopt
+                        * to get the error, this saves us time and cpu cycles. Go BSD!
+                        */
+                       if (ref[ke_list[j].ident])
+                               ref[ke_list[j].ident]->HandleEvent(EVENT_ERROR, ke_list[j].fflags);
+                       continue;
+               }
+               if (ke_list[j].flags & EVFILT_WRITE)
+               {
+                       /* This looks wrong but its right. As above, theres no modify 
+                        * call in kqueue. See the manpage.
+                        */
+                       struct kevent ke;
+                       EV_SET(&ke, ke_list[j].ident, EVFILT_READ, EV_ADD, 0, 0, NULL);
+                       int i = kevent(EngineHandle, &ke, 1, 0, 0, NULL);
+                       if (i == -1)
+                       {
+                               ServerInstance->Log(DEBUG,"kqueue: Unable to set fd %d back to just wanting to read!", ke_list[j].ident);
+                       }
+                       if (ref[ke_list[j].ident])
+                               ref[ke_list[j].ident]->HandleEvent(EVENT_WRITE);
+               }
+               else
+               {
+                       if (ref[ke_list[j].ident])
+                               ref[ke_list[j].ident]->HandleEvent(EVENT_READ);
+               }
        }
 
        return i;