Add support for listen(2) backlog definition #1368

Closed
wants to merge 2 commits into
from

Projects

None yet

3 participants

@nmerdan

In high RPS environments, the default listen backlog is not sufficient, so
giving users the power to configure it is the right approach, especially
since it requires only minor modifications to the code.

@nmerdan nmerdan Add support for listen(2) backlog definition
In high RPS environments, the default listen backlog is not sufficient, so
giving users the power to configure it is the right approach, especially
since it requires only minor modifications to the code.
b0d9a5a
@sstewartgallus sstewartgallus commented on the diff Jun 30, 2014
src/anet.c
if (bind(s,sa,len) == -1) {
anetSetError(err, "bind: %s", strerror(errno));
close(s);
return ANET_ERR;
}
- /* Use a backlog of 512 entries. We pass 511 to the listen() call because
@sstewartgallus
sstewartgallus Jun 30, 2014

Why not put the comment in the configuration file?

@mattsta

Added in d76aa96 (renamed to tcp-backlog later)

@mattsta mattsta closed this Aug 1, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment