[LTP] [PATCH v2 1/2] clone08: convert to new LTP API

Alexey Kodanev alexey.kodanev@oracle.com
Tue Jul 18 14:00:30 CEST 2017


On 06/22/2017 06:01 PM, Cyril Hrubis wrote:
> Hi!
>>  
>> -	stopped_flag = 0;
>> -	child = clone_child(&test_cases[t], 1);
>> +	child = clone_child(&test_cases[t]);
>>  
>>  	/* give the kernel scheduler chance to run the CLONE_STOPPED thread*/
>>  	for (i = 0; i < 100; i++) {
>> @@ -246,23 +178,22 @@ static void test_clone_stopped(int t)
>>  		usleep(1000);
>>  	}
> Can't we check the /proc/$pid/state here to assert that the child is in
> 'T' state here? We do have PROCESS_STATE_WAIT() for that.

Hi Cyril,

Sure, changed the loop to the macro and applied.

Thanks,
Alexey

> Otherwise this looks good.
>



More information about the ltp mailing list