]> git.ipfire.org Git - thirdparty/haproxy.git/commit
MEDIUM: evports: permit to report multiple events at once
authorWilly Tarreau <w@1wt.eu>
Wed, 17 Apr 2024 14:37:04 +0000 (16:37 +0200)
committerWilly Tarreau <w@1wt.eu>
Wed, 17 Apr 2024 14:37:04 +0000 (16:37 +0200)
commite6662bf706de55303fe0757294e9386d8f95861b
treeef313d552e3e623c119222fa4047a171ea48b8c2
parent36d92dcd9b62c5af0d7499c07479d6995565db9f
MEDIUM: evports: permit to report multiple events at once

Since the beginning in 2.0 the nevlist parameter was set to 1 before
calling port_getn(), which means that a single FD event will be reported
per polling loop. This is extremely inefficient, and all the code was
designed to use global.tune.maxpollevents. It looks like it's a leftover
of a temporary debugging change. No apparent issues were found by setting
it to a higher value, so better do that.

That code is not much used nowadays with Solaris disappearing from the
landscape, so even if this definitely was a bug, it's preferable not to
backport that fix as it could uncover other subtle bugs that were never
raised yet.
src/ev_evports.c