From fdf90c0ffbf608ed08665eaffa5c750fa5a5bfee Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Tue, 31 Oct 2023 20:42:52 +0000 Subject: ds: move maxevents further down the stack The epoll implementation is the only one which respects the limit (kevent would, but IO::KQueue does not). In any case, I'm not a fan of the maxevents=1000 historical default since it leads to fairness problems with shared non-blocking listeners across multiple daemon workers. --- lib/PublicInbox/DSPoll.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'lib/PublicInbox/DSPoll.pm') diff --git a/lib/PublicInbox/DSPoll.pm b/lib/PublicInbox/DSPoll.pm index 0446df4c..b947f756 100644 --- a/lib/PublicInbox/DSPoll.pm +++ b/lib/PublicInbox/DSPoll.pm @@ -18,7 +18,7 @@ use Errno (); sub new { bless {}, __PACKAGE__ } # fd => events sub ep_wait { - my ($self, $maxevents, $timeout_msec, $events) = @_; + my ($self, $timeout_msec, $events) = @_; my (@pset, $n, $fd, $revents, $nval); while (my ($fd, $events) = each %$self) { my $pevents = $events & EPOLLIN ? POLLIN : 0; -- cgit v1.2.3-24-ge0c7