<p><a href="https://gerrit.osmocom.org/c/libosmocore/+/25414">View Change</a></p><p>3 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/libosmocore/+/25414/1/include/osmocom/core/rate_ctr.h">File include/osmocom/core/rate_ctr.h:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/libosmocore/+/25414/1/include/osmocom/core/rate_ctr.h@8">Patch Set #1, Line 8:</a> <code style="font-family:monospace,monospace">stdatomic</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">which c standard do we minimally depend on with  this? should we check for that in autoconf?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">gnu11, as set 7 months ago in configure.ac by oliver, just like libosmocore.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/libosmocore/+/25414/1/src/rate_ctr.c">File src/rate_ctr.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/libosmocore/+/25414/1/src/rate_ctr.c@72">Patch Set #1, Line 72:</a> <code style="font-family:monospace,monospace">s</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">so every thread gets their own set of rate counters [counter groups]. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">They should be initialized in one thread, and only used by other threads. There is also the matter of the ratecounter cb if you really wanted to share the list among multiple threads, you probably wouldn't want one _per thread_ operating on a shared list either...<br>So yes, osmotrx needs fixing, if we keep it like this.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/libosmocore/+/25414/1/src/rate_ctr.c@470">Patch Set #1, Line 470:</a> <code style="font-family:monospace,monospace">osmo_fds</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">no osmo_fds here?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">copypasta mistake from the other source file that is handled like this</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmocore/+/25414">change 25414</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/+/25414"/><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: I117f89add4798a250b5758543f1fb3e01d974205 </div>
<div style="display:none"> Gerrit-Change-Number: 25414 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Hoernchen <ewild@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 09 Sep 2021 17:36:52 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>