<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">That brings us back to the discussion on what is more important regarding logging: delaying logging to safe some resources or to receive the logging content as soon as possible.<br>I get your point too regarding logging. We could enable/disable NODELAY dynamically based on whether logging is enabled or not, that would be an option.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I'm not sure I'm following you.  If you are logging to the VTY, then this means it's an interactive user logged in via telnet.  The latency of NAGLE certainly won't matter, as the user will never notice the delay caused by nagle.  At least not unless you have a *very* high RTT (VSAT?) VTY connection, as the NAGLE delay should typically be a maximum of 1RTT.</p><p style="white-space: pre-wrap; word-wrap: break-word;">If your concern is that log output would not be written in case ofa crash: I doubt NAGLE has any influence on that either.  You have made the write() syscall on the socket "synchronously", and TCP will have to deliver that to the peer? Or does a close() on the socket mean you loose unackonwledged data that still is in the buffer?</p><p><a href="https://gerrit.osmocom.org/c/libosmocore/+/20575">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmocore/+/20575">change 20575</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/libosmocore/+/20575"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmocore </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I34f8d3ac70045e48613f67e3c79c2ff49c4d2794 </div>
<div style="display:none"> Gerrit-Change-Number: 20575 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 12 Oct 2020 16:03:47 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>