<div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Aug 8, 2018 at 8:36 PM, Cyril Hrubis <span dir="ltr"><<a href="mailto:chrubis@suse.cz" target="_blank">chrubis@suse.cz</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi!<br>
<span class="gmail-">> This patch set are base on Cyril's new patch(lib: Fix result<br>
> propagation after exec() + tst_reinit()) to rewrite all exec[lvpe]<br>
> testcase in new API. <br>
<br>
</span>I was thinking about doing exactly this yesterday, it's nice to see it<br>
done so fast :-).<br>
<br>
The patches 1-5 are pushed with two changes:<br>
<br>
* There is no need to set the .needs_root flag for exec*01 testcases<br>
<br>
* It's wrong to do waitpid() for the child that reports the results<br>
  (and it's even written in the test-writing-guidelines) that is because<br>
  tst_brk() is propagated using the exit value, hence children that use<br>
  the test library API has to be waited for in the test library (which<br>
  happens automatically once the main test process exits the test<br>
  function)<br></blockquote><div><br></div><div class="gmail_default" style="font-size:small">Yes, that's right. Thanks for the explanation.</div><div class="gmail_default" style="font-size:small"></div><div class="gmail_default"><span style="font-size:small">And, I have looked though all the comments in patch 6/6, there're all </span>pertinent advice.  Agreed.</div></div><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div>
</div></div>