On 02/21/2014 03:51 PM, Francesco RUNDO wrote:
> Changed exit code from TFAIL to TCONF  in case of unsupported
> syscall. The "madvise" syscall needs kernel support as well as
> fs support as -for specific advice- it requires specific filesystem
> operations such as fallocate().

Could you paste the running log on not supported system?

Thanks,
Wanlong Gao

> So, if kernel support is missed or fs doesn't provide specific
> operation needed for such "advice" scenario of that syscall, the right
> exit code (according to LTP defs) is not TFAIL as it is not a failure
> but it is TCONF as the syscall is unsupported.
> 
> Signed-off-by: Francesco Rundo <[email protected]>
> ---
>  testcases/kernel/syscalls/madvise/madvise03.c |   17 +++++++++++++----
>  1 files changed, 13 insertions(+), 4 deletions(-)
> 
> diff --git a/testcases/kernel/syscalls/madvise/madvise03.c 
> b/testcases/kernel/syscalls/madvise/madvise03.c
> index b0c17e8..23dd9a6 100644
> --- a/testcases/kernel/syscalls/madvise/madvise03.c
> +++ b/testcases/kernel/syscalls/madvise/madvise03.c
> @@ -160,10 +160,19 @@ static void cleanup(void)
>  static void check_and_print(char *advice)
>  {
>       if (TEST_RETURN == -1) {
> -             tst_resm(TFAIL,
> -                      "madvise test for %s failed with "
> -                      "return = %ld, errno = %d : %s",
> -                      advice, TEST_RETURN, TEST_ERRNO, strerror(TEST_ERRNO));
> +             if (TEST_ERRNO != ENOTSUP) {
> +                     tst_resm(TFAIL,
> +                              "madvise test for %s failed with "
> +                              "return = %ld, errno = %d : %s",
> +                              advice, TEST_RETURN, TEST_ERRNO,
> +                              strerror(TEST_ERRNO));
> +             } else {
> +                     tst_resm(TCONF,
> +                             "madvise test for %s unsupported "
> +                             "return = %ld, errno = %d : %s",
> +                             advice, TEST_RETURN, TEST_ERRNO,
> +                             strerror(TEST_ERRNO));
> +             }
>       } else if (STD_FUNCTIONAL_TEST) {
>               tst_resm(TPASS, "madvise test for %s PASSED", advice);
>       }
> 


------------------------------------------------------------------------------
Managing the Performance of Cloud-Based Applications
Take advantage of what the Cloud has to offer - Avoid Common Pitfalls.
Read the Whitepaper.
http://pubads.g.doubleclick.net/gampad/clk?id=121054471&iu=/4140/ostg.clktrk
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list

Reply via email to