Re: [LTP] Some of the testcases which are resulting in FAIL even though they might not be

[Subrata M. <su...@li...>]

On Fri, 2009-06-12 at 14:08 +0200, Jiri Palecek wrote: 
> On Wednesday 10 June 2009 14:06:25 Munipradeep Beerakam wrote:
> > Hi,
> 
> Hi,
> 
> > Here is the list of testcases in May 2009 LTP, which *may* result in
> > FAIL even though it is a configuration error, not the error due to the
> > functionality we are testing. Those tests should give "CONF" message,
> > not "FAIL". I ran LTP on 2.6.18 kernel, x86 architecture.
> > 
> > 
> >    3. get_mempolicy01
> > Part of code, which is not correct
> > tst_resm(TFAIL, "set_mempolicy() failed - errno = %d : %s",TEST_ERRNO,
> > strerror(TEST_ERRNO));
> > ...
> > ...
> > tst_resm(TFAIL, "malloc failed - errno = %d : %s",TEST_ERRNO,
> > strerror(TEST_ERRNO));
> > 
> >    4. mq_open01
> > Part of code, which is not correct
> > if (TEST_RETURN < 0) {
> >                         tst_resm(TFAIL, "mq_getsetattr failed - errno =
> > %d : %s",TEST_ERRNO, strerror(TEST_ERRNO));
> 
> Not that I'd say this code is stellar, it may even be incorrect, but changing these to TCONF wouldn't be correct either. These error messages should be TCONF only if errno is ENOSYS, otherwise, they should be reported as TBROK, because it would be an error that prevents the test to run. The best option, however, would probably be to add a tst_kvercmp check for kernel versions that contain the respective functions.
> 

Muni,

Can you work out a patch on this lines. I would help you to do so if
required.

Regards--
Subrata

> Regards
>     Jiri Palecek
> 
> ------------------------------------------------------------------------------
> Crystal Reports - New Free Runtime and 30 Day Trial
> Check out the new simplified licensing option that enables unlimited
> royalty-free distribution of the report engine for externally facing 
> server and web deployment.
> http://p.sf.net/sfu/businessobjects
> _______________________________________________
> Ltp-list mailing list
> Ltp...@li...
> https://lists.sourceforge.net/lists/listinfo/ltp-list