<div dir="ltr">Test passes with all v2 patches, please review. I had set ->forked in parent process after fork syscall finishes, thus it was possible to have current->forked not set when task did set_pgid. <br>
</div><div dir="ltr"><br>
</div><div dir="ltr"><a href="https://patchwork.criu.org/series/1722">https://patchwork.criu.org/series/1722</a><br>
</div><div dir="ltr"><br>
</div><div dir="ltr"><br>
</div><div dir="ltr"><br>
</div><div class="wps_signature">Best regards, Tikhomirov Pavel.</div><div class="wps_quotion">Andrei Vagin <avagin@virtuozzo.com> | От: 18 июл. 2017 г. 5:38 ДП | Сообщение:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><p></p><p dir="ltr">On Thu, Jul 06, 2017 at 11:01:38AM +0300, Pavel Tikhomirov wrote:
<br>
>
<br>
>
<br>
> On 07/06/2017 08:56 AM, Andrei Vagin wrote:
<br>
> > ======================= Run zdtm/static/session01 in uns =======================
<br>
> > Start test
<br>
> > ./session01 --pidfile=session01.pid --outfile=session01.out
<br>
> > Run criu dump
<br>
> > Run criu restore
<br>
> > =[log]=> dump/zdtm/static/session01/144/1/restore.log
<br>
> > ------------------------ grep Error ------------------------
<br>
> > (00.479441) 21: nr_lazy: 0
<br>
> > (00.479447) 21: Restore on-core sigactions for 21
<br>
> > (00.479480) 21: Restoring children in alien sessions:
<br>
> > (00.479491) 21: Restoring children in our session:
<br>
> > (00.479508) 21: Error (criu/pstree.c:1259): Can't setpgid 21: item or leader is not alive
<br>
> > (00.481978) 5: Error (criu/cr-restore.c:1506): 21 exited, status=1
<br>
> > (00.483951) uns: calling do_destroy_pid_ns_helpers (-1, 0)
<br>
> > (00.487994) Error (criu/namespaces.c:1606): 204 finished unexpected: status=9
<br>
> > (00.553307) uns: calling exit_usernsd (-1, 1)
<br>
> > (00.553366) uns: daemon calls 0x4b6f00 (187, -1, 1)
<br>
> > (00.553376) uns: `- daemon exits w/ 0
<br>
> > (00.555688) uns: daemon stopped
<br>
> > (00.555708) Error (criu/cr-restore.c:2406): Restoring FAILED.
<br>
> > ------------------------ ERROR OVER ------------------------
<br>
> > ############### Test zdtm/static/session01 FAIL at CRIU restore ################
<br>
>
<br>
> My guess, these is synchronization problem ->pgrp_set is set before ->forked
<br>
> for a process, will fix it.
<br>
<br>
What is the currect status of this series?
<br>
<br>
>
<br>
> >
<br>
> > On Tue, Jul 04, 2017 at 03:25:00PM +0000, Patchwork wrote:
<br>
> > > == Series Details ==
<br>
> > >
<br>
> > > Series: rework pgid restore for pidnses (rev2)
<br>
> > > URL : <a href="https://patchwork.criu.org/series/1722">https://patchwork.criu.org/series/1722</a>/
<br>
> > > State : failure
<br>
> > >
<br>
> > > == Logs ==
<br>
> > >
<br>
> > > For more details see: <a href="https://travis-ci.org/criupatchwork/criu/builds/249965739?utm_source=github_status&utm_medium=notification">https://travis-ci.org/criupatchwork/criu/builds/249965739?utm_source=github_status&utm_medium=notification</a>
<br>
> > > _______________________________________________
<br>
> > > CRIU mailing list
<br>
> > > <a href="mailto:CRIU@openvz.org">CRIU@openvz.org</a>
<br>
> > > <a href="https://lists.openvz.org/mailman/listinfo/criu">https://lists.openvz.org/mailman/listinfo/criu</a>
<br>
>
<br>
> --
<br>
> Best regards, Tikhomirov Pavel
<br>
> Software Developer, Virtuozzo.
<br>
</p>
</blockquote></div>