]> git.netwichtig.de Git - user/henk/code/inspircd.git/blobdiff - src/socketengines/socketengine_kqueue.cpp
Replace SocketEngine::GetName() with INSPIRCD_SOCKETENGINE_NAME define
[user/henk/code/inspircd.git] / src / socketengines / socketengine_kqueue.cpp
index 63e16ac6e13038279433f8ea412fa6e6c69652b6..53bd741f066ecee2562797eae06e74cba3e207b5 100644 (file)
 #include <sys/event.h>
 #include <sys/time.h>
 #include "socketengine.h"
+#include <iostream>
+#include <sys/sysctl.h>
 
-/** A specialisation of the SocketEngine class, designed to use FreeBSD kqueue().
+/** A specialisation of the SocketEngine class, designed to use BSD kqueue().
  */
 class KQueueEngine : public SocketEngine
 {
@@ -34,7 +36,7 @@ private:
        int EngineHandle;
        /** These are used by kqueue() to hold socket events
         */
-       struct kevent* ke_list;
+       std::vector<struct kevent> ke_list;
        /** This is a specialised time value used by kqueue()
         */
        struct timespec ts;
@@ -49,33 +51,31 @@ public:
        void OnSetEvent(EventHandler* eh, int old_mask, int new_mask);
        virtual void DelFd(EventHandler* eh);
        virtual int DispatchEvents();
-       virtual std::string GetName();
        virtual void RecoverFromFork();
 };
 
-#include <sys/sysctl.h>
-
-KQueueEngine::KQueueEngine()
+KQueueEngine::KQueueEngine() : ke_list(1)
 {
        MAX_DESCRIPTORS = 0;
        int mib[2];
        size_t len;
 
        mib[0] = CTL_KERN;
+#ifdef KERN_MAXFILESPERPROC
        mib[1] = KERN_MAXFILESPERPROC;
+#else
+       mib[1] = KERN_MAXFILES;
+#endif
        len = sizeof(MAX_DESCRIPTORS);
        sysctl(mib, 2, &MAX_DESCRIPTORS, &len, NULL, 0);
        if (MAX_DESCRIPTORS <= 0)
        {
-               ServerInstance->Logs->Log("SOCKET", DEFAULT, "ERROR: Can't determine maximum number of open sockets!");
-               printf("ERROR: Can't determine maximum number of open sockets!\n");
-               ServerInstance->Exit(EXIT_STATUS_SOCKETENGINE);
+               ServerInstance->Logs->Log("SOCKET", LOG_DEFAULT, "ERROR: Can't determine maximum number of open sockets!");
+               std::cout << "ERROR: Can't determine maximum number of open sockets!" << std::endl;
+               ServerInstance->QuickExit(EXIT_STATUS_SOCKETENGINE);
        }
 
        this->RecoverFromFork();
-       ke_list = new struct kevent[GetMaxFds()];
-       ref = new EventHandler* [GetMaxFds()];
-       memset(ref, 0, GetMaxFds() * sizeof(EventHandler*));
 }
 
 void KQueueEngine::RecoverFromFork()
@@ -88,11 +88,11 @@ void KQueueEngine::RecoverFromFork()
        EngineHandle = kqueue();
        if (EngineHandle == -1)
        {
-               ServerInstance->Logs->Log("SOCKET",DEFAULT, "ERROR: Could not initialize socket engine. Your kernel probably does not have the proper features.");
-               ServerInstance->Logs->Log("SOCKET",DEFAULT, "ERROR: this is a fatal error, exiting now.");
-               printf("ERROR: Could not initialize socket engine. Your kernel probably does not have the proper features.\n");
-               printf("ERROR: this is a fatal error, exiting now.\n");
-               ServerInstance->Exit(EXIT_STATUS_SOCKETENGINE);
+               ServerInstance->Logs->Log("SOCKET", LOG_DEFAULT, "ERROR: Could not initialize socket engine. Your kernel probably does not have the proper features.");
+               ServerInstance->Logs->Log("SOCKET", LOG_DEFAULT, "ERROR: this is a fatal error, exiting now.");
+               std::cout << "ERROR: Could not initialize socket engine. Your kernel probably does not have the proper features." << std::endl;
+               std::cout << "ERROR: this is a fatal error, exiting now." << std::endl;
+               ServerInstance->QuickExit(EXIT_STATUS_SOCKETENGINE);
        }
        CurrentSetSize = 0;
 }
@@ -100,8 +100,6 @@ void KQueueEngine::RecoverFromFork()
 KQueueEngine::~KQueueEngine()
 {
        this->Close(EngineHandle);
-       delete[] ref;
-       delete[] ke_list;
 }
 
 bool KQueueEngine::AddFd(EventHandler* eh, int event_mask)
@@ -111,7 +109,7 @@ bool KQueueEngine::AddFd(EventHandler* eh, int event_mask)
        if ((fd < 0) || (fd > GetMaxFds() - 1))
                return false;
 
-       if (ref[fd])
+       if (!SocketEngine::AddFd(eh))
                return false;
 
        // We always want to read from the socket...
@@ -121,17 +119,18 @@ bool KQueueEngine::AddFd(EventHandler* eh, int event_mask)
        int i = kevent(EngineHandle, &ke, 1, 0, 0, NULL);
        if (i == -1)
        {
-               ServerInstance->Logs->Log("SOCKET",DEFAULT,"Failed to add fd: %d %s",
+               ServerInstance->Logs->Log("SOCKET", LOG_DEFAULT, "Failed to add fd: %d %s",
                                          fd, strerror(errno));
                return false;
        }
 
-       ref[fd] = eh;
+       ServerInstance->Logs->Log("SOCKET", LOG_DEBUG, "New file descriptor: %d", fd);
+
        SocketEngine::SetEventMask(eh, event_mask);
        OnSetEvent(eh, 0, event_mask);
        CurrentSetSize++;
+       ResizeDouble(ke_list);
 
-       ServerInstance->Logs->Log("SOCKET",DEBUG,"New file descriptor: %d", fd);
        return true;
 }
 
@@ -141,7 +140,7 @@ void KQueueEngine::DelFd(EventHandler* eh)
 
        if ((fd < 0) || (fd > GetMaxFds() - 1))
        {
-               ServerInstance->Logs->Log("SOCKET",DEFAULT,"DelFd() on invalid fd: %d", fd);
+               ServerInstance->Logs->Log("SOCKET", LOG_DEFAULT, "DelFd() on invalid fd: %d", fd);
                return;
        }
 
@@ -158,14 +157,14 @@ void KQueueEngine::DelFd(EventHandler* eh)
 
        if (j < 0)
        {
-               ServerInstance->Logs->Log("SOCKET",DEFAULT,"Failed to remove fd: %d %s",
+               ServerInstance->Logs->Log("SOCKET", LOG_DEFAULT, "Failed to remove fd: %d %s",
                                          fd, strerror(errno));
        }
 
+       SocketEngine::DelFd(eh);
        CurrentSetSize--;
-       ref[fd] = NULL;
 
-       ServerInstance->Logs->Log("SOCKET",DEBUG,"Remove file descriptor: %d", fd);
+       ServerInstance->Logs->Log("SOCKET", LOG_DEBUG, "Remove file descriptor: %d", fd);
 }
 
 void KQueueEngine::OnSetEvent(EventHandler* eh, int old_mask, int new_mask)
@@ -176,10 +175,9 @@ void KQueueEngine::OnSetEvent(EventHandler* eh, int old_mask, int new_mask)
                struct kevent ke;
                EV_SET(&ke, eh->GetFd(), EVFILT_WRITE, EV_ADD, 0, 0, NULL);
                int i = kevent(EngineHandle, &ke, 1, 0, 0, NULL);
-               if (i < 0) {
-                       ServerInstance->Logs->Log("SOCKET",DEFAULT,"Failed to mark for writing: %d %s",
+               if (i < 0)
+                       ServerInstance->Logs->Log("SOCKET", LOG_DEFAULT, "Failed to mark for writing: %d %s",
                                                  eh->GetFd(), strerror(errno));
-               }
        }
        else if ((old_mask & FD_WANT_POLL_WRITE) && !(new_mask & FD_WANT_POLL_WRITE))
        {
@@ -187,10 +185,9 @@ void KQueueEngine::OnSetEvent(EventHandler* eh, int old_mask, int new_mask)
                struct kevent ke;
                EV_SET(&ke, eh->GetFd(), EVFILT_WRITE, EV_DELETE, 0, 0, NULL);
                int i = kevent(EngineHandle, &ke, 1, 0, 0, NULL);
-               if (i < 0) {
-                       ServerInstance->Logs->Log("SOCKET",DEFAULT,"Failed to mark for writing: %d %s",
+               if (i < 0)
+                       ServerInstance->Logs->Log("SOCKET", LOG_DEFAULT, "Failed to mark for writing: %d %s",
                                                  eh->GetFd(), strerror(errno));
-               }
        }
        if ((new_mask & (FD_WANT_FAST_WRITE | FD_WANT_SINGLE_WRITE)) && !(old_mask & (FD_WANT_FAST_WRITE | FD_WANT_SINGLE_WRITE)))
        {
@@ -198,10 +195,9 @@ void KQueueEngine::OnSetEvent(EventHandler* eh, int old_mask, int new_mask)
                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 < 0) {
-                       ServerInstance->Logs->Log("SOCKET",DEFAULT,"Failed to mark for writing: %d %s",
+               if (i < 0)
+                       ServerInstance->Logs->Log("SOCKET", LOG_DEFAULT, "Failed to mark for writing: %d %s",
                                                  eh->GetFd(), strerror(errno));
-               }
        }
 }
 
@@ -210,23 +206,33 @@ int KQueueEngine::DispatchEvents()
        ts.tv_nsec = 0;
        ts.tv_sec = 1;
 
-       int i = kevent(EngineHandle, NULL, 0, &ke_list[0], GetMaxFds(), &ts);
+       int i = kevent(EngineHandle, NULL, 0, &ke_list[0], ke_list.size(), &ts);
        ServerInstance->UpdateTime();
 
+       if (i < 0)
+               return i;
+
        TotalEvents += i;
 
        for (int j = 0; j < i; j++)
        {
-               EventHandler* eh = ref[ke_list[j].ident];
+               struct kevent& kev = ke_list[j];
+
+               // Copy these in case the vector gets resized and kev invalidated
+               const int fd = kev.ident;
+               const short filter = kev.filter;
+
+               EventHandler* eh = GetRef(fd);
                if (!eh)
                        continue;
-               if (ke_list[j].flags & EV_EOF)
+
+               if (kev.flags & EV_EOF)
                {
                        ErrorEvents++;
-                       eh->HandleEvent(EVENT_ERROR, ke_list[j].fflags);
+                       eh->HandleEvent(EVENT_ERROR, kev.fflags);
                        continue;
                }
-               if (ke_list[j].filter == EVFILT_WRITE)
+               if (filter == EVFILT_WRITE)
                {
                        WriteEvents++;
                        /* When mask is FD_WANT_FAST_WRITE or FD_WANT_SINGLE_WRITE,
@@ -236,12 +242,8 @@ int KQueueEngine::DispatchEvents()
                        const int bits_to_clr = FD_WANT_SINGLE_WRITE | FD_WANT_FAST_WRITE | FD_WRITE_WILL_BLOCK;
                        SetEventMask(eh, eh->GetEventMask() & ~bits_to_clr);
                        eh->HandleEvent(EVENT_WRITE);
-
-                       if (eh != ref[ke_list[j].ident])
-                               // whoops, deleted out from under us
-                               continue;
                }
-               if (ke_list[j].filter == EVFILT_READ)
+               else if (filter == EVFILT_READ)
                {
                        ReadEvents++;
                        SetEventMask(eh, eh->GetEventMask() & ~FD_READ_WILL_BLOCK);
@@ -252,11 +254,6 @@ int KQueueEngine::DispatchEvents()
        return i;
 }
 
-std::string KQueueEngine::GetName()
-{
-       return "kqueue";
-}
-
 SocketEngine* CreateSocketEngine()
 {
        return new KQueueEngine;