<p style="white-space: pre-wrap; word-wrap: break-word;"><br>> Maybe the timer can be disabled by default, so that no runtime overhead in terms of getsockopt is incurred.  But there should be _one_ switch, provided by libosmocore, which will make all TCP sockets serving remote connections provide those statistics.</p><p style="white-space: pre-wrap; word-wrap: break-word;">To be more precise: There should be one switch, exposed to the user via the VTY, where he can enable or disable TCP statistics for all remote sockets serving signaling traffic (so we're excluding CTRL + VTY here as those are OAM and not part of the normal signaling)</p><p style="white-space: pre-wrap; word-wrap: break-word;">V8 of this patch does not introduce any overhead to osmo_select_main, all the getsockopt calls are in a timer.  So from my point of view, I don't understand where your strong reaction is coming from</p><ul><li>there is a separate list of TCP sockets. only applies to those.  Adds one small dynamic allocation on socket creation time.  No runtime overhead during processing of traffic of those sockets</li><li>the actual statistics polling via getsockopt happens in a timer</li></ul><p style="white-space: pre-wrap; word-wrap: break-word;">So IMHO, there are only two topics open:</p><p style="white-space: pre-wrap; word-wrap: break-word;">1) ensuring that the timer doesn't tage ages in case we have many TCP sockets by not polling the full list on every expiration, spreading the load over multiple timer iterations</p><p style="white-space: pre-wrap; word-wrap: break-word;">2) thinking about setting the default timer to "off", requiring explicit user interaction to activate this feature via VTY/config<br></p><p><a href="https://gerrit.osmocom.org/c/libosmocore/+/26454">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmocore/+/26454">change 26454</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/+/26454"/><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: I1416f95aff2adcf13689646b7574845de169fa3d </div>
<div style="display:none"> Gerrit-Change-Number: 26454 </div>
<div style="display:none"> Gerrit-PatchSet: 9 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: daniel <dwillmann@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 16 Dec 2021 11:05:15 +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>