<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Sep 22, 2021 at 5:31 PM Cyril Hrubis <<a href="mailto:chrubis@suse.cz">chrubis@suse.cz</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi!<br>
> > > Agreed, that sounds like a great method.<br>
> ><br>
> > Should I send v3 or can I commit v2 with this change?<br>
> ><br>
> <br>
> V3 please, I will help test it all.<br>
<br>
Ok, will send ASAP.<br></blockquote><div><span class="gmail_default" style="font-size:small">Thanks.</span></div><div><span class="gmail_default" style="font-size:small"></span> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> I just think of setsid(), is that works and better than setpgid(0, 0)?<br>
<br>
For moving the process out of the process group setpgid() is enough, we<br>
use it in the lib/tst_test.c for the fork_testrun() in order to track<br>
all children of the test process as well.<br>
<br>
I do not think that we need a new session here, that would probably have<br>
strange side effects, for instance it would isolate the tst_timeout_kill<br>
process from the terminal the tests were started in.<br></blockquote><div><br></div><div class="gmail_default" style="font-size:small">Ok, thanks! Let's go with the safe method setpgid(0, 0).</div></div><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>