From: Zhang X. <zha...@cn...> - 2009-09-02 05:38:50
|
I tested ltp mbind case and found the result is error as follows: mbind01 0 TINFO : (case00) START EXPECT: return value(ret)=0 errno=0 (Success) RESULT: return value(ret)=0 errno=0 (Success) mbind01 0 TINFO : (case00) END => OK mbind01 0 TINFO : (case01) START EXPECT: return value(ret)=-1 errno=22 (Invalid argument) RESULT: return value(ret)=-1 errno=22 (Invalid argument) mbind01 0 TINFO : (case01) END => NG mbind01 0 TINFO : (case02) START EXPECT: return value(ret)=-1 errno=22 (Invalid argument) RESULT: return value(ret)=-1 errno=22 (Invalid argument) mbind01 0 TINFO : (case02) END => NG mbind01 0 TINFO : (case03) START EXPECT: return value(ret)=0 errno=0 (Success) RESULT: return value(ret)=0 errno=0 (Success) mbind01 0 TINFO : (case03) END => OK mbind01 0 TINFO : (case04) START EXPECT: return value(ret)=-1 errno=22 (Invalid argument) RESULT: return value(ret)=-1 errno=22 (Invalid argument) mbind01 0 TINFO : (case04) END => NG mbind01 0 TINFO : (case05) START EXPECT: return value(ret)=0 errno=0 (Success) RESULT: return value(ret)=0 errno=0 (Success) mbind01 0 TINFO : (case05) END => OK mbind01 0 TINFO : (case06) START EXPECT: return value(ret)=0 errno=0 (Success), r/w check=OK RESULT: return value(ret)=0 errno=0 (Success), r/w check=NG mbind01 0 TINFO : (case06) END => OK mbind01 0 TINFO : (case07) START EXPECT: return value(ret)=0 errno=0 (Success) RESULT: return value(ret)=0 errno=0 (Success) mbind01 0 TINFO : (case07) END => OK mbind01 0 TINFO : (case08) START EXPECT: return value(ret)=-1 errno=22 (Invalid argument) RESULT: return value(ret)=-1 errno=22 (Invalid argument) mbind01 0 TINFO : (case08) END => NG mbind01 0 TINFO : (case09) START EXPECT: return value(ret)=-1 errno=22 (Invalid argument) RESULT: return value(ret)=-1 errno=22 (Invalid argument) mbind01 0 TINFO : (case09) END => NG mbind01 0 TINFO : (case10) START mbind01 1 TFAIL : get_mempolicy failed - errno = 14 : Bad address In case01,case02,case04,case08,case09, the expect is same with the result. So these cases should be OK instead of NG. In sourse code, TEST_RETURN should get value by "ret". By the way, the "switch/case RESULT_OK" missed "break". Signed-off-by: Zhang Xiliang <zha...@cn...> --- testcases/kernel/syscalls/mbind/mbind01.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/testcases/kernel/syscalls/mbind/mbind01.c b/testcases/kernel/syscalls/mbind/mbind01.c index 2b861ac..94c2dd0 100644 --- a/testcases/kernel/syscalls/mbind/mbind01.c +++ b/testcases/kernel/syscalls/mbind/mbind01.c @@ -389,6 +389,7 @@ int main(int ac, char **av) { int ret; tst_resm(TINFO,"(case%02d) START", i); ret = do_test(&tcase[i]); + TEST_RETURN = ret; tst_resm(TINFO,"(case%02d) END => %s", i, ( TEST_RETURN== 0) ? "OK" : "NG"); result |= ret; } @@ -399,6 +400,7 @@ int main(int ac, char **av) { switch(result) { case RESULT_OK: tst_resm(TPASS, "mbind call succeeded --OK-- "); + break; default: tst_resm(TFAIL, "%s failed - errno = %d : %s --NG--", TCID, TEST_ERRNO, strerror(TEST_ERRNO)); |