On Mon, 21 May 2018 13:58:39 +0100
Radu Nicolau <radu.nico...@intel.com> wrote:

> On 5/21/2018 1:30 PM, Thomas Monjalon wrote:
> > 21/05/2018 13:35, Radu Nicolau:  
> >> Fixes: af75078fece3 ("first public release")
> >> Cc: sta...@dpdk.org
> >>
> >> Signed-off-by: Radu Nicolau <radu.nico...@intel.com>
> >> ---
> >> @@ -34,7 +34,8 @@ test_panic(void)
> >>            printf("Fork Failed\n");
> >>            return -1;
> >>    }
> >> -  wait(&status);
> >> +  sleep(1);
> >> +  waitpid(pid, &status, WNOHANG);  
> > Please give an explanation why you are adding this sleep.
> >  
> To give a chance to the child process to complete before calling waitpid.


Should not be necessary; waitpid does that.

Reply via email to