<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 Tue, Sep 13, 2022 at 5:34 PM luoxiaoyu (C) <<a href="mailto:luoxiaoyu9@huawei.com">luoxiaoyu9@huawei.com</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">Maybe I have a question to ask for.<br>
As my understand, tst_reap_children() has been called in API even test hit tst_brk, right?<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">To precisely, when hitting a tst_brk, the main process will invoke do_test_cleanup()</div><div class="gmail_default" style="font-size:small">and finally call tst_test->cleanup function from the specific testcase, to recover the</div><div class="gmail_default" style="font-size:small">original environment as much as they can.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">In pidfd_send_signal03, the tst_reap_children will be invoked in that break situation.</div><div class="gmail_default" style="font-size:small">But we shouldn't say "it has been called in API", they're totally different concepts.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">see: <a href="https://github.com/linux-test-project/ltp/blob/master/lib/tst_test.c#L339">https://github.com/linux-test-project/ltp/blob/master/lib/tst_test.c#L339</a></div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Regards,<br>
Luo<br>
-----origin mail-----<br>
Re: [LTP] [PATCH] pty04.c/pidfd_send_signal03.c: Drop redundant tst_reap_children()<br>
<br>
> On Tue, Sep 13, 2022 at 2:42 PM Petr Vorel <<a href="mailto:pvorel@suse.cz" target="_blank">pvorel@suse.cz</a>> wrote:<br>
<br>
> > Hi Luo,<br>
<br>
> > > Drop redundant calling of tst_reap_children() in cleanup() since <br>
> > > it has been called in the API (run_tests()).<br>
<br>
> > LGTM, I suppose that was a mistake.<br>
<br>
<br>
> Hmm, I slightly think that reflact a cautious thought from the author, <br>
> in case of test hit tst_brk unexpectly with invoke cleanup it can also <br>
> do reap children.<br>
<br>
> So, it's probably no harmful to keep tst_reap_children there.<br>
Ah, thx for info. I set status rejected in patchwork.<br>
<br>
Kind regards,<br>
Petr<br>
<br>
</blockquote></div><br clear="all"><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>