Re: [LTP] [PATCH] setrlimit01: wait for all children to exit in test3

[Carmelo AMOROSO <carmelo.amoroso@st.com>]

On 20/03/2012 16.39, Salvatore CRO' wrote:
> In test3, parent just wait for one child to exit but it
> actually has 10.
> As it could happen there may be other children around that
> didn't get yet the chance to exit, they will get signaled
> by ltp-pan (SIGTERM) and pollute the output of subsequent
> test (setrlimit02).
> Parent definitely needs to wait for all children to exit.
> 
> Signed-off-by: Salvatore Cro <salvatore.cro@st.com>

Acked-by: Carmelo Amoroso <carmelo.amoroso@st.com>

> ---
>  testcases/kernel/syscalls/setrlimit/setrlimit01.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/testcases/kernel/syscalls/setrlimit/setrlimit01.c b/testcases/kernel/syscalls/setrlimit/setrlimit01.c
> index ac32450..430835c 100644
> --- a/testcases/kernel/syscalls/setrlimit/setrlimit01.c
> +++ b/testcases/kernel/syscalls/setrlimit/setrlimit01.c
> @@ -267,7 +267,7 @@ void test3()
>  			exit(0);
>  		}
>  	}
> -	waitpid(pid, &status, 0);
> +	while(wait(&status) > 0) { /* no-op */ ; }
>  	if (WEXITSTATUS(status) != 0) {
>  		tst_resm(TFAIL, "RLIMIT_NPROC functionality is not correct");
>  	} else {


------------------------------------------------------------------------------
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list