<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr">вт, 3 апр 2018 г., 14:24 Pavel Tikhomirov &lt;<a href="mailto:ptikhomirov@virtuozzo.com">ptikhomirov@virtuozzo.com</a>&gt;:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
On 04/03/2018 04:11 PM, Dmitry Safonov wrote:<br>
&gt; 2018-04-03 10:34 GMT+01:00 Pavel Tikhomirov &lt;<a href="mailto:ptikhomirov@virtuozzo.com" target="_blank" rel="noreferrer">ptikhomirov@virtuozzo.com</a>&gt;:<br>
&gt; [..]<br>
&gt;&gt; +<br>
&gt;&gt; +#define CSEC_PER_SEC 100<br>
&gt;&gt; +<br>
&gt;&gt; +__maybe_unused int parse_uptime(u_int64_t *upt)<br>
&gt;<br>
&gt; Nit: can we use uint64_t?<br>
&gt; I see u_int64_t the first time and git says we&#39;ve it only in one other place.<br>
<br>
Oops, I brought it from encode_time - will use uint64_t instead, thanks!<br>
<br>
&gt;<br>
&gt;&gt; +{<br>
&gt;&gt; +       unsigned long sec, csec;<br>
&gt;&gt; +       FILE *f;<br>
&gt;&gt; +<br>
&gt;&gt; +       f = fopen(&quot;/proc/uptime&quot;, &quot;r&quot;);<br>
&gt;&gt; +       if (!f) {<br>
&gt;&gt; +               pr_perror(&quot;Failed to fopen /proc/uptime&quot;);<br>
&gt;&gt; +               return -1;<br>
&gt;&gt; +       }<br>
&gt;&gt; +<br>
&gt;&gt; +       if (fscanf(f, &quot;%lu.%2lu&quot;, &amp;sec, &amp;csec) != 2) {<br>
&gt;&gt; +               pr_perror(&quot;Failed to parse /proc/uptime&quot;);<br>
&gt;&gt; +               fclose(f);<br>
&gt;&gt; +               return -1;<br>
&gt;&gt; +       }<br>
&gt;&gt; +<br>
&gt;&gt; +       *upt = sec * USEC_PER_SEC + csec * (USEC_PER_SEC / CSEC_PER_SEC);<br>
&gt;<br>
&gt; It&#39;s not clear from the commit message, why do you save time in usec,<br>
&gt; rather than in csec?<br>
<br>
I thought it would be better to have usec in image, so when/if we have<br>
more precise uptime we can just reuse same variable in image and won&#39;t<br>
need adding another one.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">I don&#39;t mind. As you&#39;ll resend anyway - could you mention</div><div dir="auto">that in commit message, please? ;)</div><div dir="auto"><br></div><div dir="auto"> Thanks,</div><div dir="auto">              Dmitry</div><div dir="auto">(answering from phone, sorry for html)</div></div>