<p>neels <strong>uploaded patch set #4</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/c/libosmocore/+/25464">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">refactor stat_item: get rid of FIFO and "skipped" error<br><br>Intead of attempting to store all distinct values of a reporting period,<br>just store min, max, last as well as a sum and N of each reporting<br>period.<br><br>This gets rid of error messages like<br><br>  DLSTATS ERROR stat_item.c:285 num_bts:oml_connected: 44 stats values skipped<br><br>while at the same time more accurately reporting the max value for each<br>reporting period. (So far stats_item only reports the max value; keep<br>that part unchanged, as shown in stats_test.c.)<br><br>With the other so far unused values (min, sum), we are ready to also<br>report the minimum value as well as an average value per reporting<br>period in the future, if/when our stats reporter allows for it.<br><br>Store the complete record of the previous reporting period. So far we<br>only compare the 'max' value, but like this we are ready to also see<br>changes in min, last and average value between reporting periods.<br><br>This patch breaks API by removing:<br>- struct members osmo_stats_item.stats_next_id, .last_offs and .values[]<br>- struct definition osmo_stats_item_value<br>In libosmocore, we do have a policy of never breaking API. But since the<br>above should never be accessed by users of the osmo_stats_item API -- or<br>if they are, would not yield useful results --, we decided to make an<br>exception in this case. The alternative would be to introduce a new<br>osmo_stats_item2 API and maintaining an unused legacy osmo_stats_item<br>forever, but we decided that the effort is not worth it. There are no<br>known users of the removed items.<br><br>Related: SYS#5542<br>Change-Id: I137992a5479fc39bbceb6c6c2af9c227bd33b39b<br>---<br>M TODO-RELEASE<br>M include/osmocom/core/stat_item.h<br>M src/stat_item.c<br>M src/stats.c<br>M tests/stats/stats_test.c<br>M tests/stats/stats_test.err<br>6 files changed, 312 insertions(+), 310 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/64/25464/4</pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmocore/+/25464">change 25464</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/+/25464"/><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: I137992a5479fc39bbceb6c6c2af9c227bd33b39b </div>
<div style="display:none"> Gerrit-Change-Number: 25464 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@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-CC: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>