Re: [LTP] [PATCH] syscalls/clone08: Skip test_clone_stopped after kernel 2.6.25

[Xiaoguang Wang <wangxg.fnst@cn.fujitsu.com>]

Hi,

On 04/29/2014 04:44 PM, shuang.qiu@oracle.com wrote:
> From: Shuang Qiu <shuang.qiu@oracle.com>
>
> According to CLONE manual page "From Linux 2.6.25 this flag is
> deprecated.You probably never wanted to use it, you certainly shouldn’t
> be using it,  and  soon  it  will  go away",skip test_clone_stopped
> after kernel 2.6.25.
>
> Signed-off-by: Shuang Qiu <shuang.qiu@oracle.com>
> ---
>  testcases/kernel/syscalls/clone/clone08.c |    9 +++++++++
>  1 files changed, 9 insertions(+), 0 deletions(-)
>
> diff --git a/testcases/kernel/syscalls/clone/clone08.c b/testcases/kernel/syscalls/clone/clone08.c
> index 14ebf60..610986d 100644
> --- a/testcases/kernel/syscalls/clone/clone08.c
> +++ b/testcases/kernel/syscalls/clone/clone08.c
> @@ -84,15 +84,24 @@ int main(int ac, char **av)
>  {
>  	char *msg;
>  	int i, lc;
> +	int clone_stopped = 0;
>  
>  	msg = parse_opts(ac, av, NULL, NULL);
>  	if (msg != NULL)
>  		tst_brkm(TBROK, NULL, "OPTION PARSING ERROR - %s", msg);
>  
>  	setup();
> +
> +#ifdef CLONE_STOPPED
> +	if (tst_kvercmp(2, 6, 25) > 0)
> +		clone_stopped =1;
> +#endif
> +		
>  	for (lc = 0; TEST_LOOPING(lc); lc++) {
>  		tst_count = 0;
>  		for (i = 0; i < TST_TOTAL; i++) {
> +			if (clone_stopped == 1 && i ==3)
> +				continue;
>  			tst_resm(TINFO, "running %s", test_cases[i].name);
>  			test_cases[i].testfunc(i);
>  		}
I think you should do the kernel version comparison in test_clone_stopped(), and if
kernel is after 2.6.25, you may print a TCONF with tst_resm() and return.

Here you use "i == 3", but if other elements are added in struct test_case arry,
the index for test_clone_stopped may change.

Regards,
Xiaoguang Wang


------------------------------------------------------------------------------
"Accelerate Dev Cycles with Automated Cross-Browser Testing - For FREE
Instantly run your Selenium tests across 300+ browser/OS combos.  Get 
unparalleled scalability from the best Selenium testing platform available.
Simple to use. Nothing to install. Get started now for free."
http://p.sf.net/sfu/SauceLabs
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list