X-Git-Url: https://git.netwichtig.de/gitweb/?a=blobdiff_plain;f=src%2Fsocketengines%2Fsocketengine_select.cpp;h=bc7b5c63fcd4aa77365eef479651e17b6fcc6cfe;hb=78db7544d26cdeffeb2bd8045529fe90bd5d852d;hp=24c3d56378288fcea1ba257760c64d5a12737896;hpb=3079075ad62820fd6661c0368970178d71bbfe98;p=user%2Fhenk%2Fcode%2Finspircd.git diff --git a/src/socketengines/socketengine_select.cpp b/src/socketengines/socketengine_select.cpp index 24c3d5637..bc7b5c63f 100644 --- a/src/socketengines/socketengine_select.cpp +++ b/src/socketengines/socketengine_select.cpp @@ -1,183 +1,185 @@ -/* +------------------------------------+ - * | Inspire Internet Relay Chat Daemon | - * +------------------------------------+ +/* + * InspIRCd -- Internet Relay Chat Daemon * - * InspIRCd: (C) 2002-2008 InspIRCd Development Team - * See: http://www.inspircd.org/wiki/index.php/Credits + * Copyright (C) 2013-2015 Attila Molnar + * Copyright (C) 2013, 2017, 2019 Sadie Powell + * Copyright (C) 2012 Robby + * Copyright (C) 2012 ChrisTX + * Copyright (C) 2011, 2014 Adam + * Copyright (C) 2009-2010 Daniel De Graaf + * Copyright (C) 2008 Robin Burchell + * Copyright (C) 2007 Dennis Friis + * Copyright (C) 2006-2008 Craig Edwards * - * This program is free but copyrighted software; see - * the file COPYING for details. + * This file is part of InspIRCd. InspIRCd is free software: you can + * redistribute it and/or modify it under the terms of the GNU General Public + * License as published by the Free Software Foundation, version 2. * - * --------------------------------------------------- + * This program is distributed in the hope that it will be useful, but WITHOUT + * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS + * FOR A PARTICULAR PURPOSE. See the GNU General Public License for more + * details. + * + * You should have received a copy of the GNU General Public License + * along with this program. If not, see . */ + #include "inspircd.h" -#ifndef WINDOWS + +#ifndef _WIN32 #include -#endif // WINDOWS -#include "socketengines/socketengine_select.h" +#endif // _WIN32 + +/** A specialisation of the SocketEngine class, designed to use traditional select(). + */ +namespace +{ + fd_set ReadSet, WriteSet, ErrSet; + int MaxFD = 0; +} + +void SocketEngine::Init() +{ +#ifdef _WIN32 + // Set up winsock. + WSADATA wsadata; + WSAStartup(MAKEWORD(2,2), &wsadata); +#endif + MaxSetSize = FD_SETSIZE; -SelectEngine::SelectEngine(InspIRCd* Instance) : SocketEngine(Instance) + FD_ZERO(&ReadSet); + FD_ZERO(&WriteSet); + FD_ZERO(&ErrSet); +} + +void SocketEngine::Deinit() { - EngineHandle = 0; - CurrentSetSize = 0; - - writeable = new bool [GetMaxFds()]; - memset(writeable, 0, sizeof(writeable)); - ref = new EventHandler* [GetMaxFds()]; - memset(ref, 0, GetMaxFds() * sizeof(EventHandler*)); - ev = new EventHandler* [GetMaxFds()]; } -SelectEngine::~SelectEngine() +void SocketEngine::RecoverFromFork() { - delete[] ref; - delete[] ev; } -bool SelectEngine::AddFd(EventHandler* eh) +bool SocketEngine::AddFd(EventHandler* eh, int event_mask) { int fd = eh->GetFd(); - if ((fd < 0) || (fd > GetMaxFds() - 1)) + + if (fd < 0) return false; - if (GetRemainingFds() <= 1) + if (static_cast(fd) >= GetMaxFds()) return false; - if (ref[fd]) + if (!SocketEngine::AddFdRef(eh)) return false; - fds[fd] = fd; - ref[fd] = eh; - CurrentSetSize++; + eh->SetEventMask(event_mask); + OnSetEvent(eh, 0, event_mask); + FD_SET(fd, &ErrSet); + if (fd > MaxFD) + MaxFD = fd; - ServerInstance->Logs->Log("SOCKET",DEBUG,"New file descriptor: %d", fd); + ServerInstance->Logs->Log("SOCKET", LOG_DEBUG, "New file descriptor: %d", fd); return true; } -void SelectEngine::WantWrite(EventHandler* eh) -{ - writeable[eh->GetFd()] = true; -} - -bool SelectEngine::DelFd(EventHandler* eh, bool force) +void SocketEngine::DelFd(EventHandler* eh) { int fd = eh->GetFd(); - if ((fd < 0) || (fd > GetMaxFds() - 1)) - return false; - - std::map::iterator t = fds.find(fd); - if (t != fds.end()) - fds.erase(t); + if (fd < 0) + return; - CurrentSetSize--; - ref[fd] = NULL; + if (static_cast(fd) >= GetMaxFds()) + return; - ServerInstance->Logs->Log("SOCKET",DEBUG,"Remove file descriptor: %d", fd); - return true; -} + SocketEngine::DelFdRef(eh); -int SelectEngine::GetMaxFds() -{ - return FD_SETSIZE; -} + FD_CLR(fd, &ReadSet); + FD_CLR(fd, &WriteSet); + FD_CLR(fd, &ErrSet); + if (fd == MaxFD) + --MaxFD; -int SelectEngine::GetRemainingFds() -{ - return GetMaxFds() - CurrentSetSize; + ServerInstance->Logs->Log("SOCKET", LOG_DEBUG, "Remove file descriptor: %d", fd); } -int SelectEngine::DispatchEvents() +void SocketEngine::OnSetEvent(EventHandler* eh, int old_mask, int new_mask) { - int result = 0; - timeval tval; - int sresult = 0; - socklen_t codesize; - int errcode; - - FD_ZERO(&wfdset); - FD_ZERO(&rfdset); - FD_ZERO(&errfdset); + int fd = eh->GetFd(); + int diff = old_mask ^ new_mask; - for (std::map::iterator a = fds.begin(); a != fds.end(); a++) + if (diff & (FD_WANT_POLL_READ | FD_WANT_FAST_READ)) { - if (ref[a->second]->Readable()) - FD_SET (a->second, &rfdset); + if (new_mask & (FD_WANT_POLL_READ | FD_WANT_FAST_READ)) + FD_SET(fd, &ReadSet); else - FD_SET (a->second, &wfdset); - if (writeable[a->second]) - FD_SET (a->second, &wfdset); - - FD_SET (a->second, &errfdset); + FD_CLR(fd, &ReadSet); + } + if (diff & (FD_WANT_POLL_WRITE | FD_WANT_FAST_WRITE | FD_WANT_SINGLE_WRITE)) + { + if (new_mask & (FD_WANT_POLL_WRITE | FD_WANT_FAST_WRITE | FD_WANT_SINGLE_WRITE)) + FD_SET(fd, &WriteSet); + else + FD_CLR(fd, &WriteSet); } +} +int SocketEngine::DispatchEvents() +{ + timeval tval; tval.tv_sec = 1; tval.tv_usec = 0; - sresult = select(FD_SETSIZE, &rfdset, &wfdset, &errfdset, &tval); + fd_set rfdset = ReadSet, wfdset = WriteSet, errfdset = ErrSet; - if (sresult > 0) - { - for (std::map::iterator a = fds.begin(); a != fds.end(); a++) - { - if ((FD_ISSET (a->second, &rfdset)) || (FD_ISSET (a->second, &wfdset)) || FD_ISSET (a->second, &errfdset)) - { - ev[result++] = ref[a->second]; - } - } - } + int sresult = select(MaxFD + 1, &rfdset, &wfdset, &errfdset, &tval); + ServerInstance->UpdateTime(); - /** An event handler may remove its own descriptor from the list, therefore it is not - * safe to directly iterate over the list and dispatch events there with STL iterators. - * Thats a shame because it makes this code slower and more resource intensive, but maybe - * the user should stop using select(), as select() smells anyway. - */ - for (int i = 0; i < result; i++) + for (int i = 0, j = sresult; i <= MaxFD && j > 0; i++) { - if (ev[i]) + int has_read = FD_ISSET(i, &rfdset), has_write = FD_ISSET(i, &wfdset), has_error = FD_ISSET(i, &errfdset); + + if (!(has_read || has_write || has_error)) + continue; + + --j; + + EventHandler* ev = GetRef(i); + if (!ev) + continue; + + if (has_error) { - if (FD_ISSET (ev[i]->GetFd(), &errfdset)) - { - ErrorEvents++; + stats.ErrorEvents++; - if (getsockopt(ev[i]->GetFd(), SOL_SOCKET, SO_ERROR, (char*)&errcode, &codesize) < 0) - errcode = errno; + socklen_t codesize = sizeof(int); + int errcode = 0; + if (getsockopt(i, SOL_SOCKET, SO_ERROR, (char*)&errcode, &codesize) < 0) + errcode = errno; - ev[i]->HandleEvent(EVENT_ERROR, errcode); + ev->OnEventHandlerError(errcode); + continue; + } + if (has_read) + { + ev->SetEventMask(ev->GetEventMask() & ~FD_READ_WILL_BLOCK); + ev->OnEventHandlerRead(); + if (ev != GetRef(i)) continue; - } - else - { - if (writeable[ev[i]->GetFd()]) - { - WriteEvents++; - writeable[ev[i]->GetFd()] = false; - ev[i]->HandleEvent(EVENT_WRITE); - } - else - { - if (ev[i]->Readable()) - { - ReadEvents++; - ev[i]->HandleEvent(EVENT_READ); - } - else - { - WriteEvents++; - ev[i]->HandleEvent(EVENT_WRITE); - } - } - } } - } - return result; -} + if (has_write) + { + int newmask = (ev->GetEventMask() & ~(FD_WRITE_WILL_BLOCK | FD_WANT_SINGLE_WRITE)); + SocketEngine::OnSetEvent(ev, ev->GetEventMask(), newmask); + ev->SetEventMask(newmask); + ev->OnEventHandlerWrite(); + } + } -std::string SelectEngine::GetName() -{ - return "select"; + return sresult; }