From cfb27a680cd22b993b3ff5e5257154aa3358dbbd Mon Sep 17 00:00:00 2001 From: Alex Zhuravlev Date: Mon, 29 Apr 2019 11:21:13 +0300 Subject: [PATCH] LU-2233 tests: improve tests sanityn/40-47 sanity/40-46 usually take 800-900s which is almost a half of the whole sanityn pass. 99.(9)% of time the tests just wait to ensure specific order the operations execute in. the patch changes cfs_fail_timeout_set() so that it can interrupt waiting if fail_loc is set to 0 - polling with 1/10s frequency is used. the tests itself are modified to reset fail_loc. to be able to do so both operations (referenced as OP1 and OP2 in the tests) are run in background. once started and then ensured with pdo_sched() helper that MDS threads got to the blocking points, we can interrupt OP1 and do usual checks. ONLY=40-47 sh sanityn.sh take: 1017s before and 78s after. Lustre-change: https://review.whamcloud.com/4392 Lustre-commit: 743b85a32e24cff0c77dff739691043970a0901e LU-12470 tests: increase pdirops timeout There are pretty regular failures of the sanityn pdirops test_40-47. Increase the timeout slightly to reduces the frequency of failures. Lustre-change: https://review.whamcloud.com/37304 Lustre-commit: b35f50c96c608ba650a5b3cf29fa129e01025549 Test-Parameters: trivial testlist=sanityn,sanityn,sanityn,sanityn Test-Parameters: testlist=sanityn,sanityn,sanityn,sanityn,sanityn Signed-off-by: Alex Zhuravlev Signed-off-by: Andreas Dilger Reviewed-by: Mike Pershin Reviewed-by: Alex Zhuravlev Reviewed-by: Andreas Dilger Signed-off-by: Etienne AUJAMES Change-Id: Ib8aec2b4517a6f84402ccae66f6d5ceac6d73d85 Reviewed-on: https://review.whamcloud.com/44391 Tested-by: jenkins Tested-by: Maloo Reviewed-by: Oleg Drokin --- libcfs/libcfs/fail.c | 19 +- lustre/tests/sanityn.sh | 511 ++++++++++++++++++++++++++++++------------------ 2 files changed, 331 insertions(+), 199 deletions(-) diff --git a/libcfs/libcfs/fail.c b/libcfs/libcfs/fail.c index 12addb2..a441a45 100644 --- a/libcfs/libcfs/fail.c +++ b/libcfs/libcfs/fail.c @@ -121,16 +121,23 @@ EXPORT_SYMBOL(__cfs_fail_check_set); int __cfs_fail_timeout_set(__u32 id, __u32 value, int ms, int set) { + ktime_t till = ktime_add_ms(ktime_get(), ms); int ret = 0; ret = __cfs_fail_check_set(id, value, set); if (ret && likely(ms > 0)) { - CERROR("cfs_fail_timeout id %x sleeping for %dms\n", - id, ms); - set_current_state(TASK_UNINTERRUPTIBLE); - schedule_timeout(cfs_time_seconds(ms) / 1000); - set_current_state(TASK_RUNNING); - CERROR("cfs_fail_timeout id %x awake\n", id); + CERROR("cfs_fail_timeout id %x sleeping for %dms\n", id, ms); + while (ktime_before(ktime_get(), till)) { + schedule_timeout_uninterruptible(cfs_time_seconds(1) + / 10); + set_current_state(TASK_RUNNING); + if (!cfs_fail_loc) { + CERROR("cfs_fail_timeout interrupted\n"); + break; + } + } + if (cfs_fail_loc) + CERROR("cfs_fail_timeout id %x awake\n", id); } return ret; } diff --git a/lustre/tests/sanityn.sh b/lustre/tests/sanityn.sh index 3e17f99..73c03c8 100755 --- a/lustre/tests/sanityn.sh +++ b/lustre/tests/sanityn.sh @@ -1467,6 +1467,14 @@ test_39d() { # LU-7310 } run_test 39d "sync write should update mtime" +pdo_sched() { + # how long 40-47 take with specific delay + # sleep 0.1 # 78s + # sleep 0.2 # 103s + # sleep 0.3 # 124s + sleep 0.5 # 164s +} + # for pdo testing, we must cancel MDT-MDT locks as well as client locks to # avoid unexpected delays due to previous tests pdo_lru_clear() { @@ -1484,7 +1492,7 @@ pdo_lru_clear() { check_pdo_conflict() { local pid=$1 local conflict=0 - sleep 1 # to ensure OP1 is finished on client if OP2 is blocked by OP1 + pdo_sched # to ensure OP1 is finished on client if OP2 is blocked by OP1 if [[ `ps --pid $pid | wc -l` == 1 ]]; then conflict=1 echo "Conflict" @@ -1503,8 +1511,7 @@ test_40a() { do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 touch $DIR2 mkdir $DIR1/$tfile & - PID1=$! - sleep 1 + PID1=$!; pdo_sched touch $DIR2/$tfile-2 check_pdo_conflict $PID1 || error "create is blocked" mkdir $DIR2/$tfile-3 @@ -1521,6 +1528,7 @@ test_40a() { # all operations above shouldn't wait the first one check_pdo_conflict $PID1 || error "parallel operation is blocked" + do_facet $SINGLEMDS lctl set_param fail_loc=0 wait $PID1 rm -rf $DIR/$tfile* return 0 @@ -1533,8 +1541,7 @@ test_40b() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 touch $DIR1/$tfile & - PID1=$! - sleep 1 + PID1=$!; pdo_sched # open|create touch $DIR2/$tfile-2 check_pdo_conflict $PID1 || error "create is blocked" @@ -1552,6 +1559,7 @@ test_40b() { # all operations above shouldn't wait the first one check_pdo_conflict $PID1 || error "parallel operation is blocked" + do_facet $SINGLEMDS lctl set_param fail_loc=0 wait $PID1 rm -rf $DIR/$tfile* return 0 @@ -1565,8 +1573,7 @@ test_40c() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 link $DIR1/$tfile $DIR1/$tfile-0 & - PID1=$! - sleep 1 + PID1=$!; pdo_sched # open|create touch $DIR2/$tfile-2 check_pdo_conflict $PID1 || error "create is blocked" @@ -1584,6 +1591,7 @@ test_40c() { # all operations above shouldn't wait the first one check_pdo_conflict $PID1 || error "parallel operation is blocked" + do_facet $SINGLEMDS lctl set_param fail_loc=0 wait $PID1 rm -rf $DIR/$tfile* return 0 @@ -1597,8 +1605,7 @@ test_40d() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 rm $DIR1/$tfile & - PID1=$! - sleep 1 + PID1=$!; pdo_sched # open|create touch $DIR2/$tfile-2 check_pdo_conflict $PID1 || error "create is blocked" @@ -1616,6 +1623,7 @@ test_40d() { # all operations above shouldn't wait the first one check_pdo_conflict $PID1 || error "parallel operation is blocked" + do_facet $SINGLEMDS lctl set_param fail_loc=0 wait $PID1 return 0 } @@ -1628,8 +1636,7 @@ test_40e() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 mv $DIR1/$tfile $DIR1/$tfile-0 & - PID1=$! - sleep 1 + PID1=$!; pdo_sched # open|create touch $DIR2/$tfile-2 check_pdo_conflict $PID1 || error "create is blocked" @@ -1645,6 +1652,7 @@ test_40e() { # all operations above shouldn't wait the first one check_pdo_conflict $PID1 || error "parallel operation is blocked" + do_facet $SINGLEMDS lctl set_param fail_loc=0 wait $PID1 rm -rf $DIR/$tfile* return 0 @@ -1657,10 +1665,12 @@ test_41a() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 $MULTIOP $DIR1/$tfile oO_CREAT:O_RDWR:c & - PID1=$! - sleep 1 - mkdir $DIR2/$tfile && error "mkdir must fail" + PID1=$! ; pdo_sched + mkdir $DIR2/$tfile & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; echo "mkdir isn't blocked"; } + wait $PID2 ; [ $? -ne 0 ] || error "mkdir must fail" rm -rf $DIR/$tfile* return 0 } @@ -1671,10 +1681,12 @@ test_41b() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 $MULTIOP $DIR1/$tfile oO_CREAT:O_RDWR:c & - PID1=$! - sleep 1 - $MULTIOP $DIR2/$tfile oO_CREAT:O_EXCL:c && error "create must fail" + PID1=$! ; pdo_sched + $MULTIOP $DIR2/$tfile oO_CREAT:O_EXCL:c & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "create isn't blocked"; } + wait $PID2 ; [ $? -ne 0 ] || error "create must fail" rm -rf $DIR/$tfile* return 0 } @@ -1686,10 +1698,12 @@ test_41c() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 $MULTIOP $DIR1/$tfile oO_CREAT:O_RDWR:c & - PID1=$! - sleep 1 - link $DIR2/$tfile-2 $DIR2/$tfile && error "link must fail" + PID1=$! ; pdo_sched + link $DIR2/$tfile-2 $DIR2/$tfile & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "link isn't blocked"; } + wait $PID2 ; [ $? -ne 0 ] || error "link must fail" rm -rf $DIR/$tfile* return 0 } @@ -1700,10 +1714,12 @@ test_41d() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 $MULTIOP $DIR1/$tfile oO_CREAT:O_RDWR:c & - PID1=$! - sleep 1 - rm $DIR2/$tfile || error "unlink must succeed" + PID1=$! ; pdo_sched + rm $DIR2/$tfile & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "unlink isn't blocked"; } + wait $PID2 ; [ $? -eq 0 ] || error "unlink must succeed" rm -rf $DIR/$tfile* return 0 } @@ -1715,10 +1731,12 @@ test_41e() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 $MULTIOP $DIR1/$tfile oO_CREAT:O_RDWR:c & - PID1=$! - sleep 1 - mv $DIR2/$tfile-2 $DIR2/$tfile || error "rename must succeed" + PID1=$! ; pdo_sched + mv $DIR2/$tfile-2 $DIR2/$tfile & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "rename isn't blocked"; } + wait $PID2 ; [ $? -eq 0 ] || error "rename must succeed" rm -rf $DIR/$tfile* return 0 } @@ -1729,10 +1747,12 @@ test_41f() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 $MULTIOP $DIR1/$tfile oO_CREAT:O_RDWR:c & - PID1=$! - sleep 1 - mv $DIR2/$tfile $DIR2/$tfile-2 || error "rename must succeed" + PID1=$! ; pdo_sched + mv $DIR2/$tfile $DIR2/$tfile-2 & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "rename isn't blocked"; } + wait $PID2 ; [ $? -eq 0 ] || error "rename must succeed" rm -rf $DIR/$tfile* return 0 } @@ -1743,10 +1763,12 @@ test_41g() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 $MULTIOP $DIR1/$tfile oO_CREAT:O_RDWR:c & - PID1=$! - sleep 1 - stat $DIR2/$tfile > /dev/null || error "stat must succeed" + PID1=$! ; pdo_sched + stat $DIR2/$tfile > /dev/null & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "getattr isn't blocked"; } + wait $PID2 ; [ $? -eq 0 ] || error "stat must succeed" rm -rf $DIR/$tfile* return 0 } @@ -1757,10 +1779,12 @@ test_41h() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 $MULTIOP $DIR1/$tfile oO_CREAT:O_RDWR:c & - PID1=$! - sleep 1 - ls -lia $DIR2/ > /dev/null + PID1=$! ; pdo_sched + ls -lia $DIR2/ > /dev/null & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "readdir isn't blocked"; } + wait $PID2 rm -rf $DIR/$tfile* return 0 } @@ -1823,10 +1847,12 @@ test_42a() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 mkdir $DIR1/$tfile & - PID1=$! - sleep 1 - mkdir $DIR2/$tfile && error "mkdir must fail" + PID1=$! ; pdo_sched + mkdir $DIR2/$tfile & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "mkdir isn't blocked"; } + wait $PID2 ; [ $? -ne 0 ] || error "mkdir must fail" rm -rf $DIR/$tfile* return 0 } @@ -1837,10 +1863,12 @@ test_42b() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 mkdir $DIR1/$tfile & - PID1=$! - sleep 1 - $MULTIOP $DIR2/$tfile oO_CREAT:O_EXCL:c && error "create must fail" + PID1=$! ; pdo_sched + $MULTIOP $DIR2/$tfile oO_CREAT:O_EXCL:c & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "create isn't blocked"; } + wait $PID2 ; [ $? -ne 0 ] || error "create must fail" rm -rf $DIR/$tfile* return 0 } @@ -1852,10 +1880,12 @@ test_42c() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 mkdir $DIR1/$tfile & - PID1=$! - sleep 1 - link $DIR2/$tfile-2 $DIR2/$tfile && error "link must fail" + PID1=$! ; pdo_sched + link $DIR2/$tfile-2 $DIR2/$tfile & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "link isn't blocked"; } + wait $PID2 ; [ $? -ne 0 ] || error "link must fail" rm -rf $DIR/$tfile* return 0 } @@ -1866,10 +1896,12 @@ test_42d() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 mkdir $DIR1/$tfile & - PID1=$! - sleep 1 - rmdir $DIR2/$tfile || error "unlink must succeed" + PID1=$! ; pdo_sched + rmdir $DIR2/$tfile & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "unlink isn't blocked"; } + wait $PID2 ; [ $? -eq 0 ] || error "unlink must succeed" rm -rf $DIR/$tfile* return 0 } @@ -1881,10 +1913,12 @@ test_42e() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 mkdir $DIR1/$tfile & - PID1=$! - sleep 1 - mv -T $DIR2/$tfile-2 $DIR2/$tfile && error "rename must fail" + PID1=$! ; pdo_sched + mv -T $DIR2/$tfile-2 $DIR2/$tfile & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "rename isn't blocked"; } + wait $PID2 ; [ $? -ne 0 ] || error "rename must fail" rm -rf $DIR/$tfile* return 0 } @@ -1895,10 +1929,12 @@ test_42f() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 mkdir $DIR1/$tfile & - PID1=$! - sleep 1 - mv $DIR2/$tfile $DIR2/$tfile-2 || error "rename must succeed" + PID1=$! ; pdo_sched + mv $DIR2/$tfile $DIR2/$tfile-2 & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "rename isn't blocked"; } + wait $PID2 ; [ $? -eq 0 ] || error "rename must succeed" rm -rf $DIR/$tfile* return 0 } @@ -1909,10 +1945,12 @@ test_42g() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 mkdir $DIR1/$tfile & - PID1=$! - sleep 1 - stat $DIR2/$tfile > /dev/null || error "stat must succeed" + PID1=$! ; pdo_sched + stat $DIR2/$tfile > /dev/null & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "getattr isn't blocked"; } + wait $PID2 ; [ $? -eq 0 ] || error "stat must succeed" rm -rf $DIR/$tfile* return 0 } @@ -1923,10 +1961,12 @@ test_42h() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 mkdir $DIR1/$tfile & - PID1=$! - sleep 1 - ls -lia $DIR2/ > /dev/null + PID1=$! ; pdo_sched + ls -lia $DIR2/ > /dev/null & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "readdir isn't blocked"; } + wait $PID2 rm -rf $DIR/$tfile* return 0 } @@ -1939,10 +1979,12 @@ test_43a() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 rm $DIR1/$tfile & - PID1=$! - sleep 1 - mkdir $DIR2/$tfile || error "mkdir must succeed" + PID1=$! ; pdo_sched + mkdir $DIR2/$tfile & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "mkdir isn't blocked"; } + wait $PID2 ; [ $? -eq 0 ] || error "mkdir must succeed" rm -rf $DIR/$tfile* return 0 } @@ -1954,10 +1996,12 @@ test_43b() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 rm $DIR1/$tfile & - PID1=$! - sleep 1 - $MULTIOP $DIR2/$tfile oO_CREAT:O_EXCL:c || error "create must succeed" + PID1=$! ; pdo_sched + $MULTIOP $DIR2/$tfile oO_CREAT:O_EXCL:c & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "create isn't blocked"; } + wait $PID2 ; [ $? -eq 0 ] || error "create must succeed" rm -rf $DIR/$tfile* return 0 } @@ -1970,10 +2014,12 @@ test_43c() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 rm $DIR1/$tfile & - PID1=$! - sleep 1 - link $DIR2/$tfile-2 $DIR2/$tfile || error "link must succeed" + PID1=$! ; pdo_sched + link $DIR2/$tfile-2 $DIR2/$tfile & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "link isn't blocked"; } + wait $PID2 ; [ $? -eq 0 ] || error "link must succeed" rm -rf $DIR/$tfile* return 0 } @@ -1985,10 +2031,12 @@ test_43d() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 rm $DIR1/$tfile & - PID1=$! - sleep 1 - rm $DIR2/$tfile && error "unlink must fail" + PID1=$! ; pdo_sched + rm $DIR2/$tfile & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "unlink isn't blocked"; } + wait $PID2 ; [ $? -ne 0 ] || error "unlink must fail" rm -rf $DIR/$tfile* return 0 } @@ -2001,10 +2049,12 @@ test_43e() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 rm $DIR1/$tfile & - PID1=$! - sleep 1 - mv -u $DIR2/$tfile-2 $DIR2/$tfile || error "rename must succeed" + PID1=$! ; pdo_sched + mv -u $DIR2/$tfile-2 $DIR2/$tfile & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "rename isn't blocked"; } + wait $PID2 ; [ $? -eq 0 ] || error "rename must succeed" rm -rf $DIR/$tfile* return 0 } @@ -2016,10 +2066,12 @@ test_43f() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 rm $DIR1/$tfile & - PID1=$! - sleep 1 - mv $DIR2/$tfile $DIR2/$tfile-2 && error "rename must fail" + PID1=$! ; pdo_sched + mv $DIR2/$tfile $DIR2/$tfile-2 & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "rename isn't blocked"; } + wait $PID2 ; [ $? -ne 0 ] || error "rename must fail" rm -rf $DIR/$tfile* return 0 } @@ -2031,10 +2083,12 @@ test_43g() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 rm $DIR1/$tfile & - PID1=$! - sleep 1 - stat $DIR2/$tfile > /dev/null && error "stat must fail" + PID1=$! ; pdo_sched + stat $DIR2/$tfile > /dev/null & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "getattr isn't blocked"; } + wait $PID2 ; [ $? -ne 0 ] || error "stat must fail" rm -rf $DIR/$tfile* return 0 } @@ -2046,10 +2100,12 @@ test_43h() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 rm $DIR1/$tfile & - PID1=$! - sleep 1 - ls -lia $DIR2/ > /dev/null + PID1=$! ; pdo_sched + ls -lia $DIR2/ > /dev/null & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "readdir isn't blocked"; } + wait $PID2 rm -rf $DIR/$tfile* return 0 } @@ -2062,11 +2118,13 @@ test_43i() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 rm $DIR1/$tfile & - PID1=$! - sleep 1 - $LFS mkdir -i 1 $DIR2/$tfile || error "remote mkdir must succeed" + PID1=$! ; pdo_sched + $LFS mkdir -i 1 $DIR2/$tfile & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "remote mkdir isn't blocked"; } + wait $PID2 ; [ $? -eq 0 ] || error "remote mkdir must succeed" rm -rf $DIR/$tfile* return 0 } @@ -2085,7 +2143,7 @@ sub_test_43k() { do_nodes $(comma_list $(mdts_nodes)) \ "lctl set_param -n fail_loc=${fail_loc} || true" &>/dev/null echo content > $DIR1/$tdir/$tfile & PID1=$! - sleep 0.5 + pdo_sched multiop $DIR2/$tdir/$tfile u & PID2=$! wait $PID1 || @@ -2135,10 +2193,13 @@ test_44a() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK2 0x146 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000146 mv $DIR1/$tfile-2 $DIR1/$tfile & - PID1=$! - sleep 1 - mkdir $DIR2/$tfile && error "mkdir must fail" - check_pdo_conflict $PID1 && { wait $PID1; error "mkdir isn't blocked"; } + PID1=$! ; pdo_sched + mkdir $DIR2/$tfile & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 + check_pdo_conflict $PID1 && { wait $PID1; date;error "mkdir isn't blocked"; } + wait $PID2 ; [ $? -ne 0 ] || error "mkdir must fail" + date rm -rf $DIR/$tfile* return 0 } @@ -2150,10 +2211,12 @@ test_44b() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK2 0x146 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000146 mv $DIR1/$tfile-2 $DIR1/$tfile & - PID1=$! - sleep 1 - $MULTIOP $DIR2/$tfile oO_CREAT:O_EXCL:c && error "create must fail" + PID1=$! ; pdo_sched + $MULTIOP $DIR2/$tfile oO_CREAT:O_EXCL:c & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "create isn't blocked"; } + wait $PID2 ; [ $? -ne 0 ] || error "create must fail" rm -rf $DIR/$tfile* return 0 } @@ -2166,10 +2229,12 @@ test_44c() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK2 0x146 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000146 mv $DIR1/$tfile-2 $DIR1/$tfile & - PID1=$! - sleep 1 - link $DIR2/$tfile-3 $DIR2/$tfile && error "link must fail" + PID1=$! ; pdo_sched + link $DIR2/$tfile-3 $DIR2/$tfile & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "link isn't blocked"; } + wait $PID2 ; [ $? -ne 0 ] || error "link must fail" rm -rf $DIR/$tfile* return 0 } @@ -2181,10 +2246,12 @@ test_44d() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK2 0x146 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000146 mv $DIR1/$tfile-2 $DIR1/$tfile & - PID1=$! - sleep 1 - rm $DIR2/$tfile || error "unlink must succeed" + PID1=$! ; pdo_sched + rm $DIR2/$tfile & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "unlink isn't blocked"; } + wait $PID2 ; [ $? -eq 0 ] || error "unlink must succeed" rm -rf $DIR/$tfile* return 0 } @@ -2198,10 +2265,12 @@ test_44e() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK2 0x146 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000146 mv $DIR1/$tfile-2 $DIR1/$tfile & - PID1=$! - sleep 1 - mv $DIR2/$tfile-3 $DIR2/$tfile || error "rename must succeed" + PID1=$! ; pdo_sched + mv $DIR2/$tfile-3 $DIR2/$tfile & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "rename isn't blocked"; } + wait $PID2 ; [ $? -eq 0 ] || error "rename must succeed" rm -rf $DIR/$tfile* return 0 } @@ -2214,10 +2283,12 @@ test_44f() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK2 0x146 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000146 mv $DIR1/$tfile-2 $DIR1/$tfile & - PID1=$! - sleep 1 - mv $DIR2/$tfile $DIR2/$tfile-3 || error "rename must succeed" + PID1=$! ; pdo_sched + mv $DIR2/$tfile $DIR2/$tfile-3 & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "rename isn't blocked"; } + wait $PID2 ; [ $? -eq 0 ] || error "rename must succeed" rm -rf $DIR/$tfile* return 0 } @@ -2229,10 +2300,12 @@ test_44g() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK2 0x146 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000146 mv $DIR1/$tfile-2 $DIR1/$tfile & - PID1=$! - sleep 1 - stat $DIR2/$tfile > /dev/null || error "stat must succeed" + PID1=$! ; pdo_sched + stat $DIR2/$tfile > /dev/null & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "getattr isn't blocked"; } + wait $PID2 ; [ $? -eq 0 ] || error "stat must succeed" rm -rf $DIR/$tfile* return 0 } @@ -2244,10 +2317,12 @@ test_44h() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK2 0x146 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000146 mv $DIR1/$tfile-2 $DIR1/$tfile & - PID1=$! - sleep 1 - ls -lia $DIR2/ > /dev/null + PID1=$! ; pdo_sched + ls -lia $DIR2/ > /dev/null & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "readdir isn't blocked"; } + wait $PID2 rm -rf $DIR/$tfile* return 0 } @@ -2261,11 +2336,13 @@ test_44i() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK2 0x146 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000146 mv $DIR1/$tfile-2 $DIR1/$tfile & - PID1=$! - sleep 1 - $LFS mkdir -i 1 $DIR2/$tfile && error "remote mkdir must fail" + PID1=$! ; pdo_sched + $LFS mkdir -i 1 $DIR2/$tfile & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "remote mkdir isn't blocked"; } + wait $PID2 ; [ $? -ne 0 ] || error "remote mkdir must fail" rm -rf $DIR/$tfile* return 0 } @@ -2278,10 +2355,12 @@ test_45a() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 mv $DIR1/$tfile $DIR1/$tfile-2 & - PID1=$! - sleep 1 - mkdir $DIR2/$tfile || error "mkdir must succeed" + PID1=$! ; pdo_sched + mkdir $DIR2/$tfile & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "mkdir isn't blocked"; } + wait $PID2 ; [ $? -eq 0 ] || error "mkdir must succeed" rm -rf $DIR/$tfile* return 0 } @@ -2293,10 +2372,12 @@ test_45b() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 mv $DIR1/$tfile $DIR1/$tfile-2 & - PID1=$! - sleep 1 - $MULTIOP $DIR2/$tfile oO_CREAT:O_EXCL:c || error "create must succeed" + PID1=$! ; pdo_sched + $MULTIOP $DIR2/$tfile oO_CREAT:O_EXCL:c & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "create isn't blocked"; } + wait $PID2 ; [ $? -eq 0 ] || error "create must succeed" rm -rf $DIR/$tfile* return 0 } @@ -2309,10 +2390,12 @@ test_45c() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 mv $DIR1/$tfile $DIR1/$tfile-2 & - PID1=$! - sleep 1 - link $DIR2/$tfile-3 $DIR2/$tfile || error "link must succeed" + PID1=$! ; pdo_sched + link $DIR2/$tfile-3 $DIR2/$tfile & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "link isn't blocked"; } + wait $PID2 ; [ $? -eq 0 ] || error "link must succeed" rm -rf $DIR/$tfile* return 0 } @@ -2324,10 +2407,12 @@ test_45d() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 mv $DIR1/$tfile $DIR1/$tfile-2 & - PID1=$! - sleep 1 - rm $DIR2/$tfile && error "unlink must fail" + PID1=$! ; pdo_sched + rm $DIR2/$tfile & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "unlink isn't blocked"; } + wait $PID2 ; [ $? -ne 0 ] || error "unlink must fail" rm -rf $DIR/$tfile* return 0 } @@ -2340,10 +2425,12 @@ test_45e() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 mv $DIR1/$tfile $DIR1/$tfile-2 & - PID1=$! - sleep 1 - mv $DIR2/$tfile-3 $DIR2/$tfile || error "rename must succeed" + PID1=$! ; pdo_sched + mv $DIR2/$tfile-3 $DIR2/$tfile & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "rename isn't blocked"; } + wait $PID2 ; [ $? -eq 0 ] || error "rename must succeed" rm -rf $DIR/$tfile* return 0 } @@ -2355,10 +2442,12 @@ test_45f() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 mv $DIR1/$tfile $DIR1/$tfile-2 & - PID1=$! - sleep 1 - mv $DIR2/$tfile $DIR2/$tfile-3 && error "rename must fail" + PID1=$! ; pdo_sched + mv $DIR2/$tfile $DIR2/$tfile-3 & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "rename isn't blocked"; } + wait $PID2 ; [ $? -ne 0 ] || error "rename must fail" rm -rf $DIR/$tfile* return 0 } @@ -2370,10 +2459,12 @@ test_45g() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 mv $DIR1/$tfile $DIR1/$tfile-2 & - PID1=$! - sleep 1 - stat $DIR2/$tfile > /dev/null && error "stat must fail" + PID1=$! ; pdo_sched + stat $DIR2/$tfile > /dev/null & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "getattr isn't blocked"; } + wait $PID2 ; [ $? -ne 0 ] || error "stat must fail" rm -rf $DIR/$tfile* return 0 } @@ -2385,10 +2476,11 @@ test_45h() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 mv $DIR1/$tfile $DIR1/$tfile-2 & - PID1=$! - sleep 1 - ls -lia $DIR2/ > /dev/null + PID1=$! ; pdo_sched + ls -lia $DIR2/ > /dev/null & + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "readdir isn't blocked"; } + wait $PID2 rm -rf $DIR/$tfile* return 0 } @@ -2401,11 +2493,13 @@ test_45i() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 mv $DIR1/$tfile $DIR1/$tfile-2 & - PID1=$! - sleep 1 - $LFS mkdir -i 1 $DIR2/$tfile || error "create remote dir must succeed" + PID1=$! ; pdo_sched + $LFS mkdir -i 1 $DIR2/$tfile & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "create remote dir isn't blocked"; } + wait $PID2 ; [ $? -eq 0 ] || error "create remote dir must succeed" rm -rf $DIR/$tfile* return 0 } @@ -2427,7 +2521,7 @@ sub_test_45j() { cat $DIR1/$tdir/$tfile >/dev/null & PID1=$! - sleep 0.5 + pdo_sched mrename $DIR2/$tdir/$tfile-2 $DIR2/$tdir/$tfile > /dev/null & PID2=$! @@ -2472,10 +2566,12 @@ test_46a() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 link $DIR1/$tfile-2 $DIR1/$tfile & - PID1=$! - sleep 1 - mkdir $DIR2/$tfile && error "mkdir must fail" + PID1=$! ; pdo_sched + mkdir $DIR2/$tfile & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "mkdir isn't blocked"; } + wait $PID2 ; [ $? -ne 0 ] || error "mkdir must fail" rm -rf $DIR/$tfile* return 0 } @@ -2487,10 +2583,12 @@ test_46b() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 link $DIR1/$tfile-2 $DIR1/$tfile & - PID1=$! - sleep 1 - $MULTIOP $DIR2/$tfile oO_CREAT:O_EXCL:c && error "create must fail" + PID1=$! ; pdo_sched + $MULTIOP $DIR2/$tfile oO_CREAT:O_EXCL:c & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "create isn't blocked"; } + wait $PID2 ; [ $? -ne 0 ] || error "create must fail" rm -rf $DIR/$tfile* return 0 } @@ -2502,10 +2600,12 @@ test_46c() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 link $DIR1/$tfile-2 $DIR1/$tfile & - PID1=$! - sleep 1 - link $DIR2/$tfile $DIR2/$tfile && error "link must fail" + PID1=$! ; pdo_sched + link $DIR2/$tfile $DIR2/$tfile & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "link isn't blocked"; } + wait $PID2 ; [ $? -ne 0 ] || error "link must fail" rm -rf $DIR/$tfile* return 0 } @@ -2517,10 +2617,12 @@ test_46d() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 link $DIR1/$tfile-2 $DIR1/$tfile & - PID1=$! - sleep 1 - rm $DIR2/$tfile || error "unlink must succeed" + PID1=$! ; pdo_sched + rm $DIR2/$tfile & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "unlink isn't blocked"; } + wait $PID2 ; [ $? -eq 0 ] || error "unlink must succeed" rm -rf $DIR/$tfile* return 0 } @@ -2533,10 +2635,12 @@ test_46e() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 link $DIR1/$tfile-2 $DIR1/$tfile & - PID1=$! - sleep 1 - mv $DIR2/$tfile-3 $DIR2/$tfile || error "rename must succeed" + PID1=$! ; pdo_sched + mv $DIR2/$tfile-3 $DIR2/$tfile & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "rename isn't blocked"; } + wait $PID2 ; [ $? -eq 0 ] || error "rename must succeed" rm -rf $DIR/$tfile* return 0 } @@ -2549,10 +2653,12 @@ test_46f() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 link $DIR1/$tfile-2 $DIR1/$tfile & - PID1=$! - sleep 1 - mv $DIR2/$tfile $DIR2/$tfile-3 || error "rename must succeed" + PID1=$! ; pdo_sched + mv $DIR2/$tfile $DIR2/$tfile-3 & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "rename isn't blocked"; } + wait $PID2 ; [ $? -eq 0 ] || error "rename must succeed" rm -rf $DIR/$tfile* return 0 } @@ -2564,10 +2670,12 @@ test_46g() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 link $DIR1/$tfile-2 $DIR1/$tfile & - PID1=$! - sleep 1 - stat $DIR2/$tfile > /dev/null || error "stat must succeed" + PID1=$! ; pdo_sched + stat $DIR2/$tfile > /dev/null & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "getattr isn't blocked"; } + wait $PID2 ; [ $? -eq 0 ] || error "stat must succeed" rm -rf $DIR/$tfile* return 0 } @@ -2579,11 +2687,12 @@ test_46h() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 link $DIR1/$tfile-2 $DIR1/$tfile & - PID1=$! - sleep 1 - ls -lia $DIR2/ > /dev/null - check_pdo_conflict $PID1 && { wait $PID1; - error "readdir isn't blocked"; } + PID1=$! ; pdo_sched + ls -lia $DIR2/ > /dev/null & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 + check_pdo_conflict $PID1 && { wait $PID1; error "readdir isn't blocked"; } + wait $PID2 rm -rf $DIR/$tfile* return 0 } @@ -2596,11 +2705,13 @@ test_46i() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 link $DIR1/$tfile-2 $DIR1/$tfile & - PID1=$! - sleep 1 - $LFS mkdir -i 1 $DIR2/$tfile && error "remote mkdir must fail" + PID1=$! ; pdo_sched + $LFS mkdir -i 1 $DIR2/$tfile & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "remote mkdir isn't blocked"; } + wait $PID2 ; [ $? -ne 0 ] || error "remote mkdir must fail" rm -rf $DIR/$tfile* return 0 } @@ -2613,10 +2724,12 @@ test_47a() { pdo_lru_clear do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 $LFS mkdir -i 1 $DIR1/$tfile & - PID1=$! - sleep 1 - mkdir $DIR2/$tfile && error "mkdir must fail" + PID1=$! ; pdo_sched + mkdir $DIR2/$tfile & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "mkdir isn't blocked"; } + wait $PID2 ; [ $? -ne 0 ] || error "mkdir must fail" rm -rf $DIR/$tfile* return 0 } @@ -2628,11 +2741,13 @@ test_47b() { pdo_lru_clear do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 $LFS mkdir -i 1 $DIR1/$tfile & - PID1=$! - sleep 1 - multiop $DIR2/$tfile oO_CREAT:O_EXCL:c && error "create must fail" + PID1=$! ; pdo_sched + multiop $DIR2/$tfile oO_CREAT:O_EXCL:c & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "create isn't blocked"; } + wait $PID2 ; [ $? -ne 0 ] || error "create must fail" rm -rf $DIR/$tfile* return 0 } @@ -2645,10 +2760,12 @@ test_47c() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 $LFS mkdir -i 1 $DIR1/$tfile & - PID1=$! - sleep 1 - link $DIR2/$tfile-2 $DIR2/$tfile && error "link must fail" + PID1=$! ; pdo_sched + link $DIR2/$tfile-2 $DIR2/$tfile & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "link isn't blocked"; } + wait $PID2 ; [ $? -ne 0 ] || error "link must fail" rm -rf $DIR/$tfile* return 0 } @@ -2660,11 +2777,13 @@ test_47d() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 $LFS mkdir -i 1 $DIR1/$tfile & - PID1=$! - sleep 1 - rmdir $DIR2/$tfile || error "unlink must succeed" + PID1=$! ; pdo_sched + rmdir $DIR2/$tfile & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "unlink isn't blocked"; } + wait $PID2 ; [ $? -eq 0 ] || error "rmdir must succeed" rm -rf $DIR/$tfile* return 0 } @@ -2677,11 +2796,13 @@ test_47e() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 $LFS mkdir -i 1 $DIR1/$tfile & - PID1=$! - sleep 1 - mv -T $DIR2/$tfile-2 $DIR2/$tfile && error "rename must fail" + PID1=$! ; pdo_sched + mv -T $DIR2/$tfile-2 $DIR2/$tfile & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "rename isn't blocked"; } + wait $PID2 ; [ $? -ne 0 ] || error "rename must fail" rm -rf $DIR/$tfile* return 0 } @@ -2693,11 +2814,13 @@ test_47f() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 $LFS mkdir -i 1 $DIR1/$tfile & - PID1=$! - sleep 1 - mv $DIR2/$tfile $DIR2/$tfile-2 || error "rename must succeed" + PID1=$! ; pdo_sched + mv $DIR2/$tfile $DIR2/$tfile-2 & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "rename isn't blocked"; } + wait $PID2 ; [ $? -eq 0 ] || error "rename must succeed" rm -rf $DIR/$tfile* return 0 } @@ -2711,11 +2834,13 @@ test_47g() { #define OBD_FAIL_ONCE|OBD_FAIL_MDS_PDO_LOCK 0x145 do_facet $SINGLEMDS lctl set_param fail_loc=0x80000145 $LFS mkdir -i 1 $DIR1/$tfile & - PID1=$! - sleep 2 - stat $DIR2/$tfile > /dev/null || error "stat must succeed" + PID1=$! ; pdo_sched + stat $DIR2/$tfile > /dev/null & + PID2=$! ; pdo_sched + do_facet $SINGLEMDS lctl set_param fail_loc=0 check_pdo_conflict $PID1 && { wait $PID1; error "getattr isn't blocked"; } + wait $PID2 ; [ $? -eq 0 ] || error "stat must succeed" rm -rf $DIR/$tfile* return 0 } -- 1.8.3.1