Hi,
I tried "quotactl01" testcases, and found two problems.
Second one occurred after first one's revise.
1) In the result of "quotactl01", "Success" are displayed in spite of
failing in the test:
------------
quotactl01 1 TFAIL : cmd=0x800002: TEST_ERRNO=???(0): Success
quotactl01 2 TFAIL : cmd=0x800003: TEST_ERRNO=???(0): Success
quotactl01 3 TFAIL : cmd=0x800007: TEST_ERRNO=???(0): Success
quotactl01 4 TFAIL : cmd=0x800008: TEST_ERRNO=???(0): Success
quotactl01 5 TFAIL : cmd=0x800005: TEST_ERRNO=???(0): Success
quotactl01 6 TFAIL : cmd=0x800006: TEST_ERRNO=???(0): Success
quotactl01 7 TFAIL : cmd=0x800004: TEST_ERRNO=???(0): Success
quotactl01 8 TFAIL : cmd=0x800001: TEST_ERRNO=???(0): Success
------------
This problem occurred because the return value of system-call
"(syscall(__NR_quotactl, cmd[i], ...))" is not judged correctly
in "${LTPROOT}/testcases/kernel/syscalls/quotactl/quotactl01.c".
Above errors are caused by TFAIL and messages are printed by TTERRNO.
And there is only one place (Line 206) which uses TFAIL in "quotactl01.c"
The following patch can fix this problem:
============
--- quotactl01.c 2009-12-20 09:36:35.000000000 +0900
+++ quotactl01.c.new 2010-01-04 17:11:17.000000000 +0900
@@ -177,7 +177,6 @@
int id = getuid();
int newtid = -1;
int result;
- int ret;
int i;
int lc; /* loop counter */
char *msg; /* message returned from
parse_opts */
@@ -199,10 +198,10 @@
for (i = 0; i <= 7; i++){
- ret = syscall(__NR_quotactl, cmd[i],
+ TEST(syscall(__NR_quotactl, cmd[i],
(const char *)NULL, id,
- (caddr_t)NULL);
- if (ret != 0) {
+ (caddr_t)NULL));
+ if (TEST_RETURN != 0) {
tst_resm(TFAIL|TTERRNO, "cmd=0x%x",
cmd[i]);
} else {
tst_resm(TPASS, "quotactl call
succeeded");
============
However, new problem occurred after applying above patch.
2) The test failed with the following error in my system:
------------
quotactl01 1 TFAIL : cmd=0x800002: TEST_ERRNO=EFAULT(14): Bad address
quotactl01 2 TFAIL : cmd=0x800003: TEST_ERRNO=EFAULT(14): Bad address
quotactl01 3 TFAIL : cmd=0x800007: TEST_ERRNO=EFAULT(14): Bad address
quotactl01 4 TFAIL : cmd=0x800008: TEST_ERRNO=EFAULT(14): Bad address
quotactl01 5 TFAIL : cmd=0x800005: TEST_ERRNO=EFAULT(14): Bad address
quotactl01 6 TFAIL : cmd=0x800006: TEST_ERRNO=EFAULT(14): Bad address
quotactl01 7 TFAIL : cmd=0x800004: TEST_ERRNO=EFAULT(14): Bad address
quotactl01 8 TFAIL : cmd=0x800001: TEST_ERRNO=EFAULT(14): Bad address
------------
"EFAULT" means that there is an error in "addr".
The format of "quotactl" is following:
------------
int quotactl(int cmd, const char *special, int id, caddr_t addr);
------------
Therefore, "EFAULT" means that 4th argument of "quotactl()" has problem.
In "${LTPROOT}/testcases/kernel/syscalls/quotactl/quotactl01.c", 4th
argument of "quotactl()" is "NULL":
Is it right?
I'm glad if I get your help.
Regards--
-Tomonori Mitani
------------------------------------------------------------------------------
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list
