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