Hi!
> + if (reap_children(-1, WNOHANG, fork_kid_pid, MAXKIDS)) {
> + tst_res(TFAIL, "reap_children() failed");
> + return;
Same applies here, the TFAIL here is redundant.
The rest looks good and the same applies to the rest of the patches.
--
Cyril Hrubis
chrubis@suse.cz