On Wed, Feb 09, 2022 at 06:22:18PM -0700, Shuah Khan <skhan@xxxxxxxxxxxxxxxxxxx> wrote:
> Please see comments below. A bit of a concern with how long this test
> will run. Did you time it?
It runs around 1 s, I didn't measure it and I used it manually only.
> How long does this test now run for with this loop?
I kept this sleep to space output from individual tasks for
better readability of output. It's not necessary for the sake of the
test. I'll remove it in next version.
> > - for (i = 0; i < NR_CHILDS; i++) {
> > + for (i = 0; i < NR_CHILDREN; i++) {
> > if (WIFEXITED(wstatus[i]))
> > warnx("(pid=%d): pid %d exited, status=%d",
> > pid, -child[i], WEXITSTATUS(wstatus[i]));
> >
>
> Please a add few more comments in the code path.
Hehe, this is inherited from the original version. (True, it's not
overly clear on its own.)
Michal
