Feature #2021

Better server.event-handler defaults

Added by Olaf-van-der-Spek almost 5 years ago. Updated over 4 years ago.

Status:FixedStart date:2009-07-05
Priority:NormalDue date:
Assignee:-% Done:

100%

Category:core
Target version:1.4.24
Missing in 1.5.x:

Description

Could the server.event-handler default to a better option?
On Linux, this would be epoll. This should be easy to do at run-time.

Associated revisions

Revision 2612
Added by stbuehler over 4 years ago

Use linux-epoll by default if available (fixes #2021, thx Olaf van der Spek)

Revision 2613
Added by stbuehler over 4 years ago

Use linux-epoll by default if available (fixes #2021)

History

#1 Updated by stbuehler almost 5 years ago

  • Target version changed from 1.4.24 to 1.4.x

#2 Updated by Olaf-van-der-Spek almost 5 years ago

  • % Done changed from 0 to 10
--- orig/lighttpd-1.4.23/src/configfile.c       2009-04-26 19:51:55.000000000 +0200
+++ lighttpd-1.4.23/src/configfile.c    2009-07-24 12:57:21.000000000 +0200
@@ -1135,15 +1135,15 @@
                 * - select works everywhere
                 * - linux-* are experimental
                 */
+#ifdef USE_LINUX_EPOLL
+               { FDEVENT_HANDLER_LINUX_SYSEPOLL, "linux-sysepoll" },
+#endif
 #ifdef USE_POLL
                { FDEVENT_HANDLER_POLL,           "poll" },
 #endif
 #ifdef USE_SELECT
                { FDEVENT_HANDLER_SELECT,         "select" },
 #endif
-#ifdef USE_LINUX_EPOLL
-               { FDEVENT_HANDLER_LINUX_SYSEPOLL, "linux-sysepoll" },
-#endif
 #ifdef USE_LINUX_SIGIO
                { FDEVENT_HANDLER_LINUX_RTSIG,    "linux-rtsig" },
 #endif

Hmm, redmine's diff parser seems broken...

#3 Updated by stbuehler over 4 years ago

  • Status changed from New to Fixed
  • % Done changed from 10 to 100

Applied in changeset r2612.

#4 Updated by stbuehler over 4 years ago

  • Target version changed from 1.4.x to 1.4.24

Also available in: Atom