diff options
author | fanquake <fanquake@gmail.com> | 2023-02-15 16:03:55 +0000 |
---|---|---|
committer | fanquake <fanquake@gmail.com> | 2023-02-15 16:10:46 +0000 |
commit | 5ecd14a31cad496e58fbc4aca9af0e9a001689d5 (patch) | |
tree | a237a04b9e35c895c93823bda03e1c7a66b3b416 /.gitattributes | |
parent | a65d2259f15b332f09d77f68759b37de32c5bc79 (diff) | |
parent | 691eaf8873fe2f189153ca637506a0291504c97a (diff) |
Merge bitcoin/bitcoin#26844: Net: Pass `MSG_MORE` flag when sending non-final network messages (round 2)
691eaf8873fe2f189153ca637506a0291504c97a Pass MSG_MORE flag when sending non-final network messages (Matt Whitlock)
Pull request description:
**N.B.:** This is my second attempt at introducing this optimization. #12519 (2018) was closed in deference to switching to doing gathering socket writes using `sendmsg(2)`, which I agree would have superior performance due to fewer syscalls, but that work was apparently abandoned in late 2018. Ever since, Bitcoin Core has continued writing tons of runt packets to the wire. Can we proceed with my halfway solution for now?
----
Since Nagle's algorithm is disabled, each and every call to `send(2)` can potentially generate a separate TCP segment on the wire. This is especially inefficient when sending the tiny header preceding each message payload.
Linux implements a `MSG_MORE` flag that tells the kernel not to push the passed data immediately to the connected peer but rather to collect it in the socket's internal transmit buffer where it can be combined with data from successive calls to `send(2)`. Where available, specify this flag when calling `send(2)` in `CConnman::SocketSendData(CNode &)` if the data buffer being sent is not the last one in `node.vSendMsg`.
ACKs for top commit:
sipa:
ACK 691eaf8873fe2f189153ca637506a0291504c97a
vasild:
ACK 691eaf8873fe2f189153ca637506a0291504c97a
Tree-SHA512: 9a7f46bc12edbf78d488f05d1c46760110a24c95af74b627d2604fcd198fa3f511c5956bac36d0034e88c632d432f7d394147e667a11b027af0a30f70a546d70
Diffstat (limited to '.gitattributes')
0 files changed, 0 insertions, 0 deletions