Page MenuHomePhabricator

Bump DEFAULT_MAX_PEER_CONNECTIONS to 4096
ClosedPublic

Authored by deadalnix on Dec 4 2020, 16:49.

Details

Reviewers
majcosta
Group Reviewers
Restricted Project
Commits
rABC537d1a1e42c2: Bump DEFAULT_MAX_PEER_CONNECTIONS to 4096
Summary

With avalanche, we'll have to keep in touch with more nodes.

Test Plan
ninja all check-all

Kept a node running for a while and check that resource consumption is not out of wack.

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Reduce the max connection used by default in integration tests as to not exaust the available file descriptors when many nodes are running at once.

This revision is now accepted and ready to land.Dec 4 2020, 17:26
src/net.h
91 ↗(On Diff #26195)

4096*

Fix warnings in bitcoin-qt and log due to an error in the the required descriptor count calcuation.

This error exists in Core too, but never triggers because the system pretty much always provide enough descriptor by default and the value doesn't need to be increased.

This revision was landed with ongoing or failed builds.Dec 5 2020, 00:58
This revision was automatically updated to reflect the committed changes.