<p><a href="https://gerrit.osmocom.org/7906">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/7906/1/include/osmocom/core/timer_compat.h">File include/osmocom/core/timer_compat.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/7906/1/include/osmocom/core/timer_compat.h@119">Patch Set #1, Line 119:</a> <code style="font-family:monospace,monospace">#define CLOCK_BOOTTIME 7</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">so what does the magic 7 do in pre-2.6. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">We are actually not using CLOCK_BOOTTIME anywhere in our code afaik, we only provide the possibility to use it together with other clocks in the API.</p><p style="white-space: pre-wrap; word-wrap: break-word;">If CLOCK_BOOTTIME is not suported I would still efine it, then I would probably return EINVAL, as explained in man clock_gettime:<br>"EINVAL The clk_id specified is not supported on this system."</p><p style="white-space: pre-wrap; word-wrap: break-word;">If we just pass that to clock_gettime, it's going to return EINVAL anyway most probably.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/7906">change 7906</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/7906"/><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-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ia3d3a829700272ec8e5cf67379d8d85368230726 </div>
<div style="display:none"> Gerrit-Change-Number: 7906 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: lynxis lazus <lynxis@fe80.eu> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: lynxis lazus <lynxis@fe80.eu> </div>
<div style="display:none"> Gerrit-Comment-Date: Sat, 19 May 2018 12:12:44 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>