<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Aug 10, 2015 at 11:16 AM, Tycho Andersen <span dir="ltr">&lt;<a href="mailto:tycho.andersen@canonical.com" target="_blank">tycho.andersen@canonical.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Mon, Aug 10, 2015 at 03:00:42PM +0000, Hui Kang wrote:<br>
&gt; The process to be restored may not have certain cgroup dir, e.g<br>
&gt; cpu/[PID]/cpu.shares. Currently restoring  will fail in closing<br>
&gt; the file. If this is the case, the fileno is -1 and we can ignore<br>
&gt; this and let the restore proceed.<br>
&gt;<br>
&gt; Signed-off-by: Hui Kang &lt;<a href="mailto:hkang.sunysb@gmail.com">hkang.sunysb@gmail.com</a>&gt;<br>
&gt; ---<br>
&gt;  cgroup.c | 2 +-<br>
&gt;  1 file changed, 1 insertion(+), 1 deletion(-)<br>
&gt;<br>
&gt; diff --git a/cgroup.c b/cgroup.c<br>
&gt; index a4e0146..f4e49b3 100644<br>
&gt; --- a/cgroup.c<br>
&gt; +++ b/cgroup.c<br>
&gt; @@ -974,7 +974,7 @@ static int restore_cgroup_prop(const CgroupPropEntry * cg_prop_entry_p,<br>
&gt;               return -1;<br>
&gt;       }<br>
&gt;<br>
&gt; -     if (fclose(f) != 0) {<br>
&gt; +     if ((fclose(f) != 0) &amp;&amp; (fileno(f) &gt;= 0)) {<br>
<br>
</span>If we put these in the other order we can avoid the fclose all<br>
together, I think.<br>
<br></blockquote><div><br></div><div>Thanks for your suggestions. However, the fileno only turns to -1 after fclose(f). This is because file is closed.<br>Prior to that, fileno(f) returns 5, which looks normal for cgroup/cpu/cpus.share.<br><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Tycho<br>
<span class=""><br>
&gt;               pr_perror(&quot;Failed closing %s&quot;, path);<br>
&gt;               return -1;<br>
&gt;       }<br>
&gt; --<br>
&gt; 1.9.1<br>
&gt;<br>
</span>&gt; _______________________________________________<br>
&gt; CRIU mailing list<br>
&gt; <a href="mailto:CRIU@openvz.org">CRIU@openvz.org</a><br>
&gt; <a href="https://lists.openvz.org/mailman/listinfo/criu" rel="noreferrer" target="_blank">https://lists.openvz.org/mailman/listinfo/criu</a><br>
</blockquote></div><br></div></div>