[LTP] [PATCH V2 04/11] syscalls/waitpid: call reap_children() via TST_TRACE()

Stanislav Kholmanskikh stanislav.kholmanskikh@oracle.com
Thu Aug 25 13:56:48 CEST 2016



On 08/25/2016 02:20 PM, Cyril Hrubis wrote:
> Hi!
> This is obviously correct.
> 
>> diff --git a/testcases/kernel/syscalls/waitpid/waitpid11.c b/testcases/kernel/syscalls/waitpid/waitpid11.c
>> index 9b51e04..1067bca 100644
>> --- a/testcases/kernel/syscalls/waitpid/waitpid11.c
>> +++ b/testcases/kernel/syscalls/waitpid/waitpid11.c
>> @@ -50,11 +50,12 @@ static void do_child_1(void)
>>  
>>  	TST_CHECKPOINT_WAKE2(0, MAXKIDS);
>>  
>> -	if (reap_children(0, 0, fork_kid_pid + (MAXKIDS / 2), MAXKIDS / 2))
>> +	if (TST_TRACE(reap_children(0, 0, fork_kid_pid + (MAXKIDS / 2),
>> +				    MAXKIDS / 2)))
>>  		return;
> 
> Hmm, if we drop the useless parenthesis and spaces around / we will fit
> into 80 chars:
> 
> 	if (TST_TRACE(reap_children(0, 0, fork_kid_pid + MAXKIDS/2, MAXKIDS/2)))
> 		return;
> 
Ok.


More information about the ltp mailing list