<p>Currently, check_icmp uses processing time to compute rtt. This can lead to invalid values when the machine is heavily loaded. The patch I propose uses the SO_TIMESTAMP feature of setsockopt to use the kernel reception time instead.</p>
<hr>
<h4>You can merge this Pull Request by running</h4>
<pre> git pull https://github.com/ganoze/monitoring-plugins master</pre>
<p>Or view, comment on, or merge it at:</p>
<p> <a href='https://github.com/monitoring-plugins/monitoring-plugins/pull/1232'>https://github.com/monitoring-plugins/monitoring-plugins/pull/1232</a></p>
<h4>Commit Summary</h4>
<ul>
<li>Use kernel reception time on ICMP packets to compute rtt.</li>
<li>Restore check_icmp header style (spaces at EOL).</li>
</ul>
<h4>File Changes</h4>
<ul>
<li>
<strong>M</strong>
<a href="https://github.com/monitoring-plugins/monitoring-plugins/pull/1232/files#diff-0">plugins-root/check_icmp.c</a>
(55)
</li>
</ul>
<h4>Patch Links:</h4>
<ul>
<li><a href='https://github.com/monitoring-plugins/monitoring-plugins/pull/1232.patch'>https://github.com/monitoring-plugins/monitoring-plugins/pull/1232.patch</a></li>
<li><a href='https://github.com/monitoring-plugins/monitoring-plugins/pull/1232.diff'>https://github.com/monitoring-plugins/monitoring-plugins/pull/1232.diff</a></li>
</ul>
<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br><a href='https://github.com/monitoring-plugins/monitoring-plugins/pull/1232'>Reply to this email on GitHub</a>.<img src='https://github.com/notifications/beacon/5514713__eyJzY29wZSI6Ik5ld3NpZXM6QmVhY29uIiwiZXhwaXJlcyI6MTcwNjk3NjUzMiwiZGF0YSI6eyJpZCI6MjQ4MjM1MjZ9fQ==--ae7399a64657d2c786b55ba58e7215a24366c30e.gif' height='1' width='1'></p>