Opened 9 years ago

Closed 8 years ago

#5478 enhancement closed fixed (fixed)

Use epoll as default reactor on Linux

Reported by: Itamar Turner-Trauring Owned by: Itamar Turner-Trauring
Priority: normal Milestone: Twisted-12.1
Component: core Keywords:
Cc: Thijs Triemstra Branch: branches/epoll-default-5478
branch-diff, diff-cov, branch-cov, buildbot
Author: itamar

Description

epoll is most scalable event loop on Linux; it should be the default.

This depends on #4429 being fixed, sadly, otherwise we'll randomly break working applications that used twisted.internet.stdio.

Change History (6)

comment:1 Changed 8 years ago by Itamar Turner-Trauring

Milestone: Twisted-12.1
Owner: set to Itamar Turner-Trauring

comment:2 Changed 8 years ago by itamarst

Author: itamarst
Branch: branches/epoll-default-5478

(In [34331]) Branching to 'epoll-default-5478'

comment:3 Changed 8 years ago by Itamar Turner-Trauring

Author: itamarstitamar
Keywords: review added
Owner: Itamar Turner-Trauring deleted

Ready for review. Note that this can only be merged after #4429 is merged.

http://buildbot.twistedmatrix.com/boxes-supported?branch=/branches/epoll-default-5478

comment:4 Changed 8 years ago by Thijs Triemstra

Cc: Thijs Triemstra added

The choosing-reactor howto mentions epoll for linux 2.6 a couple of times, but it's supported on newer versions as well of course.

comment:5 Changed 8 years ago by therve

Keywords: review removed
Owner: set to Itamar Turner-Trauring

Looks good, please merge.

comment:6 Changed 8 years ago by itamarst

Resolution: fixed
Status: newclosed

(In [34355]) Merge epoll-default-5478: epoll is now default reactor on Linux.

Author: itamar Review: therve Fixes: #5478

epoll is the default on linux; if not available, poll is used. Also, platforms where poll was default will now fall back to select.

Note: See TracTickets for help on using tickets.