Whamcloud - gitweb
LU-1187 tests: sanity fixes for multiple MDT
[fs/lustre-release.git] / lustre / tests / sanity.sh
index 5815b5f..48c4890 100644 (file)
@@ -1,4 +1,5 @@
 #!/bin/bash
+# -*- tab-width: 4; indent-tabs-mode: t; -*-
 #
 # Run select tests by setting ONLY, or as arguments to the script.
 # Skip specific tests by setting EXCEPT.
@@ -11,6 +12,11 @@ ONLY=${ONLY:-"$*"}
 ALWAYS_EXCEPT="                27u   42a  42b  42c  42d  45   51d   68b   $SANITY_EXCEPT"
 # UPDATE THE COMMENT ABOVE WITH BUG NUMBERS WHEN CHANGING ALWAYS_EXCEPT!
 
+# with LOD/OSP landing
+# bug number for skipped tests: LU-2036
+ALWAYS_EXCEPT="                 76     $ALWAYS_EXCEPT"
+
+
 # Tests that fail on uml
 CPU=`awk '/model/ {print $4}' /proc/cpuinfo`
 #                                    buffer i/o errs             sock spc runas
@@ -447,13 +453,29 @@ test_17f() {
 }
 run_test 17f "symlinks: long and very long symlink name ========================"
 
+# str_repeat(S, N) generate a string that is string S repeated N times
+str_repeat() {
+       local s=$1
+       local n=$2
+       local ret=''
+       while [ $((n -= 1)) -ge 0 ]; do
+               ret=$ret$s
+       done
+       echo $ret
+}
+
+# Long symlinks and LU-2241
 test_17g() {
-        mkdir -p $DIR/$tdir
-        LONGSYMLINK="$(dd if=/dev/zero bs=4095 count=1 | tr '\0' 'x')"
-        ln -s $LONGSYMLINK $DIR/$tdir/$tfile
-        ls -l $DIR/$tdir
+       mkdir -p $DIR/$tdir
+       local TESTS="59 60 61 4094 4095"
+
+       for i in $TESTS; do
+               local SYMNAME=$(str_repeat 'x' $i)
+               ln -s $SYMNAME $DIR/$tdir/f$i || error "failed $i-char symlink"
+               readlink $DIR/$tdir/f$i || error "failed $i-char readlink"
+       done
 }
-run_test 17g "symlinks: really long symlink name ==============================="
+run_test 17g "symlinks: really long symlink name and inode boundaries"
 
 test_17h() { #bug 17378
         remote_mds_nodsh && skip "remote MDS with nodsh" && return
@@ -488,6 +510,82 @@ test_17k() { #bug 22301
 }
 run_test 17k "symlinks: rsync with xattrs enabled ========================="
 
+test_17l() { # LU-279
+       mkdir -p $DIR/$tdir
+       touch $DIR/$tdir/$tfile
+       ln -s $DIR/$tdir/$tfile $DIR/$tdir/$tfile.lnk
+       for path in "$DIR/$tdir" "$DIR/$tdir/$tfile" "$DIR/$tdir/$tfile.lnk"; do
+               # -h to not follow symlinks. -m '' to list all the xattrs.
+               # grep to remove first line: '# file: $path'.
+               for xattr in `getfattr -hm '' $path 2>/dev/null | grep -v '^#'`;
+               do
+                       lgetxattr_size_check $path $xattr ||
+                               error "lgetxattr_size_check $path $xattr failed"
+               done
+       done
+}
+run_test 17l "Ensure lgetxattr's returned xattr size is consistent ========"
+
+# LU-1540
+test_17m() {
+       local short_sym="0123456789"
+       local WDIR=$DIR/${tdir}m
+       local mds_index
+       local devname
+       local cmd
+       local i
+       local rc=0
+
+       [ $(lustre_version_code $SINGLEMDS) -ge $(version_code 2.2.0) ] &&
+       [ $(lustre_version_code $SINGLEMDS) -le $(version_code 2.2.93) ] &&
+               skip "MDS 2.2.0-2.2.93 do not NUL-terminate symlinks" && return
+
+       [ "$(facet_fstype $SINGLEMDS)" != "ldiskfs" ] &&
+               skip "only for ldiskfs MDT" && return 0
+
+       mkdir -p $WDIR
+       long_sym=$short_sym
+       # create a long symlink file
+       for ((i = 0; i < 4; ++i)); do
+               long_sym=${long_sym}${long_sym}
+       done
+
+       echo "create 512 short and long symlink files under $WDIR"
+       for ((i = 0; i < 256; ++i)); do
+               ln -sf ${long_sym}"a5a5" $WDIR/long-$i
+               ln -sf ${short_sym}"a5a5" $WDIR/short-$i
+       done
+
+       echo "erase them"
+       rm -f $WDIR/*
+       sync
+       wait_delete_completed
+
+       echo "recreate the 512 symlink files with a shorter string"
+       for ((i = 0; i < 512; ++i)); do
+               # rewrite the symlink file with a shorter string
+               ln -sf ${long_sym} $WDIR/long-$i
+               ln -sf ${short_sym} $WDIR/short-$i
+       done
+
+       mds_index=$($LFS getstripe -M $WDIR)
+       mds_index=$((mds_index+1))
+       devname=$(mdsdevname $mds_index)
+       cmd="$E2FSCK -fnvd $devname"
+
+       echo "stop and checking mds${mds_index}: $cmd"
+       # e2fsck should not return error
+       stop mds${mds_index} -f
+       do_facet mds${mds_index} $cmd || rc=$?
+
+       start mds${mds_index} $devname $MDS_MOUNT_OPTS
+       df $MOUNT > /dev/null 2>&1
+       [ $rc -ne 0 ] && error "e2fsck should not report error upon "\
+               "short/long symlink MDT: rc=$rc"
+       return $rc
+}
+run_test 17m "run e2fsck against MDT which contains short/long symlink"
+
 test_18() {
        touch $DIR/f
        ls $DIR || error
@@ -778,20 +876,28 @@ page_size() {
        getconf PAGE_SIZE
 }
 
+simple_cleanup_common() {
+       trap 0
+       rm -rf $DIR/$tdir
+       wait_delete_completed
+}
+
 test_24v() {
        local NRFILES=100000
-       local FREE_INODES=`lfs df -i|grep "filesystem summary" | awk '{print $5}'`
+       local FREE_INODES=$(lfs_df -i | grep "summary" | awk '{print $4}')
        [ $FREE_INODES -lt $NRFILES ] && \
-               skip "not enough free inodes $FREE_INODES required $NRFILES" && \
+               skip "not enough free inodes $FREE_INODES required $NRFILES" &&
                return
 
-       mkdir -p $DIR/d24v
-       createmany -m $DIR/d24v/$tfile $NRFILES
+       trap simple_cleanup_common EXIT
+
+       mkdir -p $DIR/$tdir
+       createmany -m $DIR/$tdir/$tfile $NRFILES
 
        cancel_lru_locks mdc
        lctl set_param mdc.*.stats clear
 
-       ls $DIR/d24v >/dev/null || error "error in listing large dir"
+       ls $DIR/$tdir >/dev/null || error "error in listing large dir"
 
        # LU-5 large readdir
        # DIRENT_SIZE = 32 bytes for sizeof(struct lu_dirent) +
@@ -807,7 +913,7 @@ test_24v() {
        [ $mds_readpage -gt $RPC_NUM ] && \
                error "large readdir doesn't take effect"
 
-       rm $DIR/d24v -rf
+       simple_cleanup_common
 }
 run_test 24v "list directory with large files (handle hash collision, bug: 17560)"
 
@@ -988,30 +1094,31 @@ test_27l() {
 run_test 27l "check setstripe permissions (should return error)"
 
 test_27m() {
-       [ "$OSTCOUNT" -lt "2" ] && skip_env "$OSTCOUNT < 2 OSTs -- skipping" && return
+       [ "$OSTCOUNT" -lt "2" ] && skip_env "$OSTCOUNT < 2 OSTs -- skipping" &&
+               return
        if [ $ORIGFREE -gt $MAXFREE ]; then
                skip "$ORIGFREE > $MAXFREE skipping out-of-space test on OST0"
                return
        fi
-       mkdir -p $DIR/d27
-       $SETSTRIPE -i 0 -c 1 $DIR/d27/f27m_1
-       dd if=/dev/zero of=$DIR/d27/f27m_1 bs=1024 count=$MAXFREE &&
+       trap simple_cleanup_common EXIT
+       mkdir -p $DIR/$tdir
+       $SETSTRIPE -i 0 -c 1 $DIR/$tdir/f27m_1
+       dd if=/dev/zero of=$DIR/$tdir/f27m_1 bs=1024 count=$MAXFREE &&
                error "dd should fill OST0"
        i=2
-       while $SETSTRIPE -i 0 -c 1 $DIR/d27/f27m_$i; do
+       while $SETSTRIPE -i 0 -c 1 $DIR/$tdir/f27m_$i; do
                i=`expr $i + 1`
                [ $i -gt 256 ] && break
        done
        i=`expr $i + 1`
-       touch $DIR/d27/f27m_$i
-       [ `$GETSTRIPE $DIR/d27/f27m_$i | grep -A 10 obdidx | awk '{print $1}'| grep -w "0"` ] && \
+       touch $DIR/$tdir/f27m_$i
+       [ `$GETSTRIPE $DIR/$tdir/f27m_$i | grep -A 10 obdidx | awk '{print $1}'| grep -w "0"` ] &&
                error "OST0 was full but new created file still use it"
        i=`expr $i + 1`
-       touch $DIR/d27/f27m_$i
-       [ `$GETSTRIPE $DIR/d27/f27m_$i | grep -A 10 obdidx | awk '{print $1}'| grep -w "0"` ] && \
+       touch $DIR/$tdir/f27m_$i
+       [ `$GETSTRIPE $DIR/$tdir/f27m_$i | grep -A 10 obdidx | awk '{print $1}'| grep -w "0"` ] &&
                error "OST0 was full but new created file still use it"
-       rm -r $DIR/d27
-       sleep 15
+       simple_cleanup_common
 }
 run_test 27m "create file while OST0 was full =================="
 
@@ -1029,6 +1136,7 @@ reset_enospc() {
        [ "$OSTIDX" ] && list=$(facet_host ost$((OSTIDX + 1)))
 
        do_nodes $list lctl set_param fail_loc=0
+       sync    # initiate all OST_DESTROYs from MDS to OST
        sleep_maxage
 }
 
@@ -1296,37 +1404,53 @@ test_27y() {
         [ $fcount -eq 0 ] && skip "not enough space on OST0" && return
         [ $fcount -gt $OSTCOUNT ] && fcount=$OSTCOUNT
 
-        MDS_OSCS=`do_facet $SINGLEMDS lctl dl | awk '/[oO][sS][cC].*md[ts]/ { print $4 }'`
-        OFFSET=$(($OSTCOUNT-1))
-        OST=-1
-        for OSC in $MDS_OSCS; do
-                if [ $OST == -1 ]; then {
-                        OST=`osc_to_ost $OSC`
-                } else {
-                        echo $OSC "is Deactivate:"
-                        do_facet $SINGLEMDS lctl --device  %$OSC deactivate
-                } fi
-        done
-
-        OSTIDX=$(index_from_ostuuid $OST)
-        mkdir -p $DIR/$tdir
-        $SETSTRIPE -c 1 $DIR/$tdir      # 1 stripe / file
+       local MDS_OSCS=$(do_facet $SINGLEMDS lctl dl |
+                        awk '/[oO][sS][cC].*md[ts]/ { print $4 }')
+       local OST_DEACTIVE_IDX=-1
+       local OSC
+       local OSTIDX
+       local OST
+
+       for OSC in $MDS_OSCS; do
+               OST=$(osc_to_ost $OSC)
+               OSTIDX=$(index_from_ostuuid $OST)
+               if [ $OST_DEACTIVE_IDX == -1 ]; then
+                       OST_DEACTIVE_IDX=$OSTIDX
+               fi
+               if [ $OSTIDX != $OST_DEACTIVE_IDX ]; then
+                       echo $OSC "is Deactivated:"
+                       do_facet $SINGLEMDS lctl --device  %$OSC deactivate
+               fi
+       done
 
-        do_facet ost$((OSTIDX+1)) lctl set_param -n obdfilter.$OST.degraded 1
-        sleep_maxage
-        createmany -o $DIR/$tdir/$tfile $fcount
-        do_facet ost$((OSTIDX+1)) lctl set_param -n obdfilter.$OST.degraded 0
+       OSTIDX=$(index_from_ostuuid $OST)
+       mkdir -p $DIR/$tdir
+       $SETSTRIPE -c 1 $DIR/$tdir      # 1 stripe / file
+
+       for OSC in $MDS_OSCS; do
+               OST=$(osc_to_ost $OSC)
+               OSTIDX=$(index_from_ostuuid $OST)
+               if [ $OSTIDX == $OST_DEACTIVE_IDX ]; then
+                       echo $OST "is degraded:"
+                       do_facet ost$((OSTIDX+1)) lctl set_param -n \
+                                               obdfilter.$OST.degraded=1
+               fi
+       done
 
-        for i in `seq 0 $OFFSET`; do
-                [ `$GETSTRIPE $DIR/$tdir/$tfile$i | grep -A 10 obdidx | awk '{print $1}'| grep -w "$OSTIDX"` ] || \
-                      error "files created on deactivated OSTs instead of degraded OST"
-        done
-        for OSC in $MDS_OSCS; do
-                [ `osc_to_ost $OSC` != $OST  ] && {
-                        echo $OSC "is activate"
-                        do_facet $SINGLEMDS lctl --device %$OSC activate
-                }
-        done
+       sleep_maxage
+       createmany -o $DIR/$tdir/$tfile $fcount
+
+       for OSC in $MDS_OSCS; do
+               OST=$(osc_to_ost $OSC)
+               OSTIDX=$(index_from_ostuuid $OST)
+               if [ $OSTIDX == $OST_DEACTIVE_IDX ]; then
+                       echo $OST "is recovered from degraded:"
+                       do_facet ost$((OSTIDX+1)) lctl set_param -n \
+                                               obdfilter.$OST.degraded=0
+               else
+                       do_facet $SINGLEMDS lctl --device %$OSC activate
+               fi
+       done
 }
 run_test 27y "create files while OST0 is degraded and the rest inactive"
 
@@ -1679,42 +1803,51 @@ test_31m() {
 }
 run_test 31m "link to file: the same, non-existing, dir==============="
 
+cleanup_test32_mount() {
+       trap 0
+       $UMOUNT $DIR/$tdir/ext2-mountpoint
+}
+
 test_32a() {
        echo "== more mountpoints and symlinks ================="
-       [ -e $DIR/d32a ] && rm -fr $DIR/d32a
-       mkdir -p $DIR/d32a/ext2-mountpoint
-       mount -t ext2 -o loop $EXT2_DEV $DIR/d32a/ext2-mountpoint || error
-       $CHECKSTAT -t dir $DIR/d32a/ext2-mountpoint/.. || error
-       $UMOUNT $DIR/d32a/ext2-mountpoint || error
+       [ -e $DIR/$tdir ] && rm -fr $DIR/$tdir
+       trap cleanup_test32_mount EXIT
+       mkdir -p $DIR/$tdir/ext2-mountpoint
+       mount -t ext2 -o loop $EXT2_DEV $DIR/$tdir/ext2-mountpoint || error
+       $CHECKSTAT -t dir $DIR/$tdir/ext2-mountpoint/.. || error
+       cleanup_test32_mount
 }
 run_test 32a "stat d32a/ext2-mountpoint/.. ====================="
 
 test_32b() {
-       [ -e $DIR/d32b ] && rm -fr $DIR/d32b
-       mkdir -p $DIR/d32b/ext2-mountpoint
-       mount -t ext2 -o loop $EXT2_DEV $DIR/d32b/ext2-mountpoint || error
-       ls -al $DIR/d32b/ext2-mountpoint/.. || error
-       $UMOUNT $DIR/d32b/ext2-mountpoint || error
+       [ -e $DIR/$tdir ] && rm -fr $DIR/$tdir
+       trap cleanup_test32_mount EXIT
+       mkdir -p $DIR/$tdir/ext2-mountpoint
+       mount -t ext2 -o loop $EXT2_DEV $DIR/$tdir/ext2-mountpoint || error
+       ls -al $DIR/$tdir/ext2-mountpoint/.. || error
+       cleanup_test32_mount
 }
 run_test 32b "open d32b/ext2-mountpoint/.. ====================="
 
 test_32c() {
-       [ -e $DIR/d32c ] && rm -fr $DIR/d32c
-       mkdir -p $DIR/d32c/ext2-mountpoint
-       mount -t ext2 -o loop $EXT2_DEV $DIR/d32c/ext2-mountpoint || error
-       mkdir -p $DIR/d32c/d2/test_dir
-       $CHECKSTAT -t dir $DIR/d32c/ext2-mountpoint/../d2/test_dir || error
-       $UMOUNT $DIR/d32c/ext2-mountpoint || error
+       [ -e $DIR/$tdir ] && rm -fr $DIR/$tdir
+       trap cleanup_test32_mount EXIT
+       mkdir -p $DIR/$tdir/ext2-mountpoint
+       mount -t ext2 -o loop $EXT2_DEV $DIR/$tdir/ext2-mountpoint || error
+       mkdir -p $DIR/$tdir/d2/test_dir
+       $CHECKSTAT -t dir $DIR/$tdir/ext2-mountpoint/../d2/test_dir || error
+       cleanup_test32_mount
 }
 run_test 32c "stat d32c/ext2-mountpoint/../d2/test_dir ========="
 
 test_32d() {
-       [ -e $DIR/d32d ] && rm -fr $DIR/d32d
-       mkdir -p $DIR/d32d/ext2-mountpoint
-       mount -t ext2 -o loop $EXT2_DEV $DIR/d32d/ext2-mountpoint || error
-       mkdir -p $DIR/d32d/d2/test_dir
-       ls -al $DIR/d32d/ext2-mountpoint/../d2/test_dir || error
-       $UMOUNT $DIR/d32d/ext2-mountpoint || error
+       [ -e $DIR/$tdir ] && rm -fr $DIR/$tdir
+       trap cleanup_test32_mount EXIT
+       mkdir -p $DIR/$tdir/ext2-mountpoint
+       mount -t ext2 -o loop $EXT2_DEV $DIR/$tdir/ext2-mountpoint || error
+       mkdir -p $DIR/$tdir/d2/test_dir
+       ls -al $DIR/$tdir/ext2-mountpoint/../d2/test_dir || error
+       cleanup_test32_mount
 }
 run_test 32d "open d32d/ext2-mountpoint/../d2/test_dir ========="
 
@@ -1764,44 +1897,48 @@ test_32h() {
 run_test 32h "open d32h/symlink->tmp/symlink->lustre-subdir/${tdir}2"
 
 test_32i() {
-       [ -e $DIR/d32i ] && rm -fr $DIR/d32i
-       mkdir -p $DIR/d32i/ext2-mountpoint
-       mount -t ext2 -o loop $EXT2_DEV $DIR/d32i/ext2-mountpoint || error
-       touch $DIR/d32i/test_file
-       $CHECKSTAT -t file $DIR/d32i/ext2-mountpoint/../test_file || error
-       $UMOUNT $DIR/d32i/ext2-mountpoint || error
+       [ -e $DIR/$tdir ] && rm -fr $DIR/$tdir
+       trap cleanup_test32_mount EXIT
+       mkdir -p $DIR/$tdir/ext2-mountpoint
+       mount -t ext2 -o loop $EXT2_DEV $DIR/$tdir/ext2-mountpoint || error
+       touch $DIR/$tdir/test_file
+       $CHECKSTAT -t file $DIR/$tdir/ext2-mountpoint/../test_file || error
+       cleanup_test32_mount
 }
 run_test 32i "stat d32i/ext2-mountpoint/../test_file ==========="
 
 test_32j() {
-       [ -e $DIR/d32j ] && rm -fr $DIR/d32j
-       mkdir -p $DIR/d32j/ext2-mountpoint
-       mount -t ext2 -o loop $EXT2_DEV $DIR/d32j/ext2-mountpoint || error
-       touch $DIR/d32j/test_file
-       cat $DIR/d32j/ext2-mountpoint/../test_file || error
-       $UMOUNT $DIR/d32j/ext2-mountpoint || error
+       [ -e $DIR/$tdir ] && rm -fr $DIR/$tdir
+       trap cleanup_test32_mount EXIT
+       mkdir -p $DIR/$tdir/ext2-mountpoint
+       mount -t ext2 -o loop $EXT2_DEV $DIR/$tdir/ext2-mountpoint || error
+       touch $DIR/$tdir/test_file
+       cat $DIR/$tdir/ext2-mountpoint/../test_file || error
+       cleanup_test32_mount
 }
 run_test 32j "open d32j/ext2-mountpoint/../test_file ==========="
 
 test_32k() {
-       rm -fr $DIR/d32k
-       mkdir -p $DIR/d32k/ext2-mountpoint
-       mount -t ext2 -o loop $EXT2_DEV $DIR/d32k/ext2-mountpoint
-       mkdir -p $DIR/d32k/d2
-       touch $DIR/d32k/d2/test_file || error
-       $CHECKSTAT -t file $DIR/d32k/ext2-mountpoint/../d2/test_file || error
-       $UMOUNT $DIR/d32k/ext2-mountpoint || error
+       rm -fr $DIR/$tdir
+       trap cleanup_test32_mount EXIT
+       mkdir -p $DIR/$tdir/ext2-mountpoint
+       mount -t ext2 -o loop $EXT2_DEV $DIR/$tdir/ext2-mountpoint
+       mkdir -p $DIR/$tdir/d2
+       touch $DIR/$tdir/d2/test_file || error
+       $CHECKSTAT -t file $DIR/$tdir/ext2-mountpoint/../d2/test_file || error
+       cleanup_test32_mount
 }
 run_test 32k "stat d32k/ext2-mountpoint/../d2/test_file ========"
 
 test_32l() {
-       rm -fr $DIR/d32l
-       mkdir -p $DIR/d32l/ext2-mountpoint
-       mount -t ext2 -o loop $EXT2_DEV $DIR/d32l/ext2-mountpoint || error
-       mkdir -p $DIR/d32l/d2
-       touch $DIR/d32l/d2/test_file
-       cat  $DIR/d32l/ext2-mountpoint/../d2/test_file || error
-       $UMOUNT $DIR/d32l/ext2-mountpoint || error
+       rm -fr $DIR/$tdir
+       trap cleanup_test32_mount EXIT
+       mkdir -p $DIR/$tdir/ext2-mountpoint
+       mount -t ext2 -o loop $EXT2_DEV $DIR/$tdir/ext2-mountpoint || error
+       mkdir -p $DIR/$tdir/d2
+       touch $DIR/$tdir/d2/test_file
+       cat  $DIR/$tdir/ext2-mountpoint/../d2/test_file || error
+       cleanup_test32_mount
 }
 run_test 32l "open d32l/ext2-mountpoint/../d2/test_file ========"
 
@@ -1864,23 +2001,30 @@ test_32p() {
 }
 run_test 32p "open d32p/symlink->tmp/symlink->lustre-root/$tfile"
 
+cleanup_testdir_mount() {
+       trap 0
+       $UMOUNT $DIR/$tdir
+}
+
 test_32q() {
-       [ -e $DIR/d32q ] && rm -fr $DIR/d32q
-       mkdir -p $DIR/d32q
-        touch $DIR/d32q/under_the_mount
-       mount -t ext2 -o loop $EXT2_DEV $DIR/d32q
-       ls $DIR/d32q/under_the_mount && error || true
-       $UMOUNT $DIR/d32q || error
+       [ -e $DIR/$tdir ] && rm -fr $DIR/$tdir
+       trap cleanup_testdir_mount EXIT
+       mkdir -p $DIR/$tdir
+        touch $DIR/$tdir/under_the_mount
+       mount -t ext2 -o loop $EXT2_DEV $DIR/$tdir
+       ls $DIR/$tdir | grep "\<under_the_mount\>" && error
+       cleanup_testdir_mount
 }
 run_test 32q "stat follows mountpoints in Lustre (should return error)"
 
 test_32r() {
-       [ -e $DIR/d32r ] && rm -fr $DIR/d32r
-       mkdir -p $DIR/d32r
-        touch $DIR/d32r/under_the_mount
-       mount -t ext2 -o loop $EXT2_DEV $DIR/d32r
-       ls $DIR/d32r | grep -q under_the_mount && error || true
-       $UMOUNT $DIR/d32r || error
+       [ -e $DIR/$tdir ] && rm -fr $DIR/$tdir
+       trap cleanup_testdir_mount EXIT
+       mkdir -p $DIR/$tdir
+        touch $DIR/$tdir/under_the_mount
+       mount -t ext2 -o loop $EXT2_DEV $DIR/$tdir
+       ls $DIR/$tdir | grep -q under_the_mount && error || true
+       cleanup_testdir_mount
 }
 run_test 32r "opendir follows mountpoints in Lustre (should return error)"
 
@@ -2185,15 +2329,7 @@ test_36h() {
 }
 run_test 36h "utime on file racing with OST BRW write =========="
 
-test_37() {
-       mkdir -p $DIR/$tdir
-       echo f > $DIR/$tdir/fbugfile
-       mount -t ext2 -o loop $EXT2_DEV $DIR/$tdir
-       ls $DIR/$tdir | grep "\<fbugfile\>" && error
-       $UMOUNT $DIR/$tdir || error
-       rm -f $DIR/$tdir/fbugfile || error
-}
-run_test 37 "ls a mounted file system to check old content ====="
+# test_37 - duplicate with tests 32q 32r
 
 test_38() {
        local file=$DIR/$tfile
@@ -2426,7 +2562,10 @@ test_39j() {
        touch $DIR1/$tfile
        sleep 1
 
-       multiop_bg_pause $DIR1/$tfile oO_RDWR:w2097152_c || error "multiop failed"
+       #define OBD_FAIL_OSC_DELAY_SETTIME       0x412
+       lctl set_param fail_loc=0x80000412
+       multiop_bg_pause $DIR1/$tfile oO_RDWR:w2097152_c ||
+               error "multiop failed"
        local multipid=$!
        local mtime1=`stat -c %Y $DIR1/$tfile`
 
@@ -2437,12 +2576,14 @@ test_39j() {
 
        for (( i=0; i < 2; i++ )) ; do
                local mtime2=`stat -c %Y $DIR1/$tfile-1`
-               [ "$mtime1" = "$mtime2" ] || \
-                       error "mtime is lost on close: $mtime2, should be $mtime1"
+               [ "$mtime1" = "$mtime2" ] ||
+                       error "mtime is lost on close: $mtime2, " \
+                             "should be $mtime1"
 
                cancel_lru_locks osc
                if [ $i = 0 ] ; then echo "repeat after cancel_lru_locks"; fi
        done
+       lctl set_param fail_loc=0
        stop_full_debug_logging
 }
 run_test 39j "write, rename, close, stat ======================="
@@ -2477,15 +2618,15 @@ TEST_39_ATIME=`date -d "1 year" +%s`
 
 test_39l() {
        remote_mds_nodsh && skip "remote MDS with nodsh" && return
-       local atime_diff=$(do_facet $SINGLEMDS lctl get_param -n mdd.*.atime_diff)
-
+       local atime_diff=$(do_facet $SINGLEMDS \
+                               lctl get_param -n mdd.*MDT0000*.atime_diff)
        mkdir -p $DIR/$tdir
 
        # test setting directory atime to future
        touch -a -d @$TEST_39_ATIME $DIR/$tdir
        local atime=$(stat -c %X $DIR/$tdir)
        [ "$atime" = $TEST_39_ATIME ] || \
-               error "atime is not set to future: $atime, should be $TEST_39_ATIME"
+               error "atime is not set to future: $atime, $TEST_39_ATIME"
 
        # test setting directory atime from future to now
        local d1=$(date +%s)
@@ -2495,9 +2636,9 @@ test_39l() {
        cancel_lru_locks mdc
        atime=$(stat -c %X $DIR/$tdir)
        [ "$atime" -ge "$d1" -a "$atime" -le "$d2" ] || \
-               error "atime is not updated from future: $atime, should be $d1<atime<$d2"
+               error "atime is not updated from future: $atime, $d1<atime<$d2"
 
-       do_facet $SINGLEMDS lctl set_param -n mdd.*.atime_diff=2
+       do_facet $SINGLEMDS lctl set_param -n mdd.*MDT0000*.atime_diff=2
        sleep 3
 
        # test setting directory atime when now > dir atime + atime_diff
@@ -2509,7 +2650,7 @@ test_39l() {
        [ "$atime" -ge "$d1" -a "$atime" -le "$d2" ] || \
                error "atime is not updated  : $atime, should be $d2"
 
-       do_facet $SINGLEMDS lctl set_param -n mdd.*.atime_diff=60
+       do_facet $SINGLEMDS lctl set_param -n mdd.*MDT0000*.atime_diff=60
        sleep 3
 
        # test not setting directory atime when now < dir atime + atime_diff
@@ -2519,7 +2660,8 @@ test_39l() {
        [ "$atime" -ge "$d1" -a "$atime" -le "$d2" ] || \
                error "atime is updated to $atime, should remain $d1<atime<$d2"
 
-       do_facet $SINGLEMDS lctl set_param -n mdd.*.atime_diff=$atime_diff
+       do_facet $SINGLEMDS \
+               lctl set_param -n mdd.*MDT0000*.atime_diff=$atime_diff
 }
 run_test 39l "directory atime update ==========================="
 
@@ -3041,7 +3183,7 @@ test_49() { # LU-1030
        local osc1_mppc=osc.$(get_osc_import_name client ost1).max_pages_per_rpc
        local orig_mppc=`$LCTL get_param -n $osc1_mppc`
        # loop until dd process exits
-       while ps ax -opid | grep -q $dd_pid; do
+       while ps ax -opid | grep -wq $dd_pid; do
                $LCTL set_param $osc1_mppc=$((RANDOM % 256 + 1))
                sleep $((RANDOM % 5 + 1))
        done
@@ -3079,93 +3221,72 @@ run_test 51a "special situations: split htree with empty entry =="
 
 export NUMTEST=70000
 test_51b() {
-       NUMFREE=`df -i -P $DIR | tail -n 1 | awk '{ print $4 }'`
-       [ $NUMFREE -lt 21000 ] && \
-               skip "not enough free inodes ($NUMFREE)" && \
+       local BASE=$DIR/$tdir
+       mkdir -p $BASE
+
+       local mdtidx=$(printf "%04x" $($LFS getstripe -M $BASE))
+       local numfree=$(lctl get_param -n mdc.$FSNAME-MDT$mdtidx*.filesfree)
+       [ $numfree -lt 21000 ] && skip "not enough free inodes ($numfree)" &&
                return
 
-       [ $NUMFREE -lt $NUMTEST ] && NUMTEST=$(($NUMFREE - 50))
+       [ $numfree -lt $NUMTEST ] && NUMTEST=$(($numfree - 50)) &&
+               echo "reduced count to $NUMTEST due to inodes"
 
-       mkdir -p $DIR/d51b
-       createmany -d $DIR/d51b/t- $NUMTEST
+       # need to check free space for the directories as well
+       local blkfree=$(lctl get_param -n mdc.$FSNAME-MDT$mdtidx*.kbytesavail)
+       numfree=$((blkfree / 4))
+       [ $numfree -lt $NUMTEST ] && NUMTEST=$(($numfree - 50)) &&
+               echo "reduced count to $NUMTEST due to blocks"
+
+       createmany -d $BASE/d $NUMTEST && echo $NUMTEST > $BASE/fnum ||
+               echo "failed" > $BASE/fnum
 }
-run_test 51b "mkdir .../t-0 --- .../t-$NUMTEST ===================="
+run_test 51b "exceed 64k subdirectory nlink limit"
 
 test_51ba() { # LU-993
-        local BASE=$DIR/d51b
-        # unlink all but 100 subdirectories, then check it still works
-        local LEFT=100
-        local DELETE=$((NUMTEST - LEFT))
-
-        ! [ -d "${BASE}/t-$DELETE" ] && skip "test_51b() not run" && return 0
-
-        # for ldiskfs the nlink count should be 1, but this is OSD specific
-        # and so this is listed for informational purposes only
-        log "nlink before: $(stat -c %h $BASE)"
-        unlinkmany -d $BASE/t- $DELETE ||
-                error "unlink of first $DELETE subdirs failed"
-
-        log "nlink between: $(stat -c %h $BASE)"
-        local FOUND=$(ls -l ${BASE} | wc -l)
-        FOUND=$((FOUND - 1))  # trim the first line of ls output
-        [ $FOUND -ne $LEFT ] &&
-                error "can't find subdirs: found only $FOUND/$LEFT"
-
-        unlinkmany -d $BASE/t- $DELETE $LEFT ||
-                error "unlink of second $LEFT subdirs failed"
-        log "nlink after: $(stat -c %h $BASE)"
-}
-run_test 51ba "rmdir .../t-0 --- .../t-$NUMTEST"
-
-test_51bb() {
-       [ $MDSCOUNT -lt 2 ] && skip "needs >= 2 MDTs" && return
-
-       local ndirs=${TEST51BB_NDIRS:-10}
-       local nfiles=${TEST51BB_NFILES:-100}
-
-       local numfree=`df -i -P $DIR | tail -n 1 | awk '{ print $4 }'`
-
-       [ $numfree -lt $(( ndirs * nfiles)) ] && \
-               nfiles=$(( numfree / ndirs - 10 ))
-
-       local dir=$DIR/d51bb
-       mkdir -p $dir
-       local savePOLICY=$(lctl get_param -n lmv.*.placement)
-       lctl set_param -n lmv.*.placement=CHAR
-
-       lfs df -i $dir
-       local IUSED=$(lfs df -i $dir | grep MDT | awk '{print $3}')
-       OLDUSED=($IUSED)
-
-       declare -a dirs
-       for ((i=0; i < $ndirs; i++)); do
-               dirs[i]=$dir/$RANDOM
-               echo Creating directory ${dirs[i]}
-               mkdir -p ${dirs[i]}
-               ls $dir
-               echo Creating $nfiles in dir ${dirs[i]} ...
-               echo "createmany -o ${dirs[i]}/$tfile- $nfiles"
-               createmany -o ${dirs[i]}/$tfile- $nfiles
-       done
-       ls $dir
-
-       sleep 1
-
-       IUSED=$(lfs df -i $dir | grep MDT | awk '{print $3}')
-       NEWUSED=($IUSED)
+       local BASE=$DIR/$tdir
+       # unlink all but 100 subdirectories, then check it still works
+       local LEFT=100
+       [ -f $BASE/fnum ] && local NUMPREV=$(cat $BASE/fnum) && rm $BASE/fnum
+
+       [ "$NUMPREV" != "failed" ] && NUMTEST=$NUMPREV
+       local DELETE=$((NUMTEST - LEFT))
+
+       # continue on to run this test even if 51b didn't finish,
+       # just to delete the many subdirectories created.
+       [ ! -d "${BASE}/d1" ] && skip "test_51b() not run" && return 0
+
+       # for ldiskfs the nlink count should be 1, but this is OSD specific
+       # and so this is listed for informational purposes only
+       echo "nlink before: $(stat -c %h $BASE), created before: $NUMTEST"
+       unlinkmany -d $BASE/d $DELETE
+       RC=$?
 
-       local rc=0
-       for ((i=0; i<${#NEWUSED[@]}; i++)); do
-               echo "mds $i: inodes count OLD ${OLDUSED[$i]} NEW ${NEWUSED[$i]}"
-               [ ${OLDUSED[$i]} -lt ${NEWUSED[$i]} ] || rc=$((rc + 1))
-       done
+       if [ $RC -ne 0 ]; then
+               if [ "$NUMPREV" == "failed" ]; then
+                       skip "previous setup failed"
+                       return 0
+               else
+                       error "unlink of first $DELETE subdirs failed"
+                       return $RC
+               fi
+       fi
 
-       lctl set_param -n lmv.*.placement=$savePOLICY
+       echo "nlink between: $(stat -c %h $BASE)"
+       # trim the first line of ls output
+       local FOUND=$(($(ls -l ${BASE} | wc -l) - 1))
+       [ $FOUND -ne $LEFT ] &&
+               error "can't find subdirs: found only $FOUND/$LEFT"
 
-       [ $rc -ne $MDSCOUNT ] || \
-               error "Objects/inodes are not distributed over all mds servers"
+       unlinkmany -d $BASE/d $DELETE $LEFT ||
+               error "unlink of second $LEFT subdirs failed"
+       # regardless of whether the backing filesystem tracks nlink accurately
+       # or not, the nlink count shouldn't be more than "." and ".." here
+       local AFTER=$(stat -c %h $BASE)
+       [ $AFTER -gt 2 ] && error "nlink after: $AFTER > 2" ||
+               echo "nlink after: $AFTER"
 }
-run_test 51bb "mkdir createmany CMD $MDSCOUNT  ===================="
+run_test 51ba "verify nlink for many subdirectory cleanup"
 
 test_51d() {
         [  "$OSTCOUNT" -lt "3" ] && skip_env "skipping test with few OSTs" && return
@@ -3896,6 +4017,11 @@ run_test 56w "check lfs_migrate -c stripe_count works"
 
 test_57a() {
        # note test will not do anything if MDS is not local
+       if [ "$(facet_fstype $SINGLEMDS)" != ldiskfs ]; then
+               skip "Only applicable to ldiskfs-based MDTs"
+               return
+       fi
+
        remote_mds_nodsh && skip "remote MDS with nodsh" && return
        local MNTDEV="osd*.*MDT*.mntdev"
        DEV=$(do_facet $SINGLEMDS lctl get_param -n $MNTDEV)
@@ -3911,6 +4037,11 @@ test_57a() {
 run_test 57a "verify MDS filesystem created with large inodes =="
 
 test_57b() {
+       if [ "$(facet_fstype $SINGLEMDS)" != ldiskfs ]; then
+               skip "Only applicable to ldiskfs-based MDTs"
+               return
+       fi
+
        remote_mds_nodsh && skip "remote MDS with nodsh" && return
        local dir=$DIR/d57b
 
@@ -3976,8 +4107,8 @@ test_59() {
        echo "rm 130 files"
        unlinkmany $DIR/f59- 130
        sync
-       sleep 2
-        # wait for commitment of removal
+       # wait for commitment of removal
+       wait_delete_completed
 }
 run_test 59 "verify cancellation of llog records async ========="
 
@@ -4108,6 +4239,11 @@ test_64b () {
 }
 run_test 64b "check out-of-space detection on client ==========="
 
+test_64c() {
+       $LCTL set_param osc.*OST0000-osc-[^mM]*.cur_grant_bytes=0
+}
+run_test 64c "verify grant shrink ========================------"
+
 # bug 1414 - set/get directories' stripe info
 test_65a() {
        mkdir -p $DIR/d65
@@ -4201,7 +4337,7 @@ test_65ib() { # bug12836
 run_test 65ib "getstripe -v on -1 default directory striping"
 
 test_65ic() { # bug12836
-       $LFS find -mtime -1 $MOUNT || error "find $MOUNT failed"
+       $LFS find -mtime -1 $MOUNT > /dev/null || error "find $MOUNT failed"
 }
 run_test 65ic "new find on -1 default directory striping"
 
@@ -4262,7 +4398,8 @@ run_test 65l "lfs find on -1 stripe dir ========================"
 test_66() {
        COUNT=${COUNT:-8}
        dd if=/dev/zero of=$DIR/f66 bs=1k count=$COUNT
-       sync; sleep 1; sync
+       sync; sync_all_data; sync; sync_all_data
+       cancel_lru_locks osc
        BLOCKS=`ls -s $DIR/f66 | awk '{ print $1 }'`
        [ $BLOCKS -ge $COUNT ] || error "$DIR/f66 blocks $BLOCKS < $COUNT"
 }
@@ -4416,7 +4553,6 @@ test_72a() { # bug 5695 - Test that on 2.6 remove_suid works properly
        # Now test that MDS is updated too
        cancel_lru_locks mdc
        test -u $DIR/f72 -o -g $DIR/f72 && error "S/gid is not dropped on MDS"
-       true
        rm -f $DIR/f72
 }
 run_test 72a "Test that remove suid works properly (bug5695) ===="
@@ -4716,7 +4852,7 @@ test_77i() { # bug 13805
        for VALUE in `lctl get_param osc.*osc-[^mM]*.checksum_type`; do
                PARAM=`echo ${VALUE[0]} | cut -d "=" -f1`
                algo=`lctl get_param -n $PARAM | sed 's/.*\[\(.*\)\].*/\1/g'`
-               [ "$algo" = "crc32" ] || error "algo set to $algo instead of crc32"
+               [ "$algo" = "adler" ] || error "algo set to $algo instead of adler"
        done
        remount_client $MOUNT
 }
@@ -4815,6 +4951,16 @@ test_79() { # bug 12743
 run_test 79 "df report consistency check ======================="
 
 test_80() { # bug 10718
+        # relax strong synchronous semantics for slow backends like ZFS
+        local soc="obdfilter.*.sync_on_lock_cancel"
+        local soc_old=$(do_facet ost1 lctl get_param -n $soc | head -n1)
+        local hosts=
+        if [ "$soc_old" != "never" -a "$(facet_fstype ost1)" != "ldiskfs" ]; then
+                hosts=$(for host in $(seq -f "ost%g" 1 $OSTCOUNT); do
+                          facet_active_host $host; done | sort -u)
+                do_nodes $hosts lctl set_param $soc=never
+        fi
+
         dd if=/dev/zero of=$DIR/$tfile bs=1M count=1 seek=1M
         sync; sleep 1; sync
         local BEFORE=`date +%s`
@@ -4824,7 +4970,9 @@ test_80() { # bug 10718
         if [ $DIFF -gt 1 ] ; then
                 error "elapsed for 1M@1T = $DIFF"
         fi
-        true
+
+        [ -n "$hosts" ] && do_nodes $hosts lctl set_param $soc=$soc_old
+
         rm -f $DIR/$tfile
 }
 run_test 80 "Page eviction is equally fast at high offsets too  ===="
@@ -4833,7 +4981,7 @@ test_81a() { # LU-456
         remote_ost_nodsh && skip "remote OST with nodsh" && return
         # define OBD_FAIL_OST_MAPBLK_ENOSPC    0x228
         # MUST OR with the OBD_FAIL_ONCE (0x80000000)
-        do_facet ost0 lctl set_param fail_loc=0x80000228
+        do_facet ost1 lctl set_param fail_loc=0x80000228
 
         # write should trigger a retry and success
         $SETSTRIPE -i 0 -c 1 $DIR/$tfile
@@ -4849,7 +4997,7 @@ test_81b() { # LU-456
         remote_ost_nodsh && skip "remote OST with nodsh" && return
         # define OBD_FAIL_OST_MAPBLK_ENOSPC    0x228
         # Don't OR with the OBD_FAIL_ONCE (0x80000000)
-        do_facet ost0 lctl set_param fail_loc=0x228
+        do_facet ost1 lctl set_param fail_loc=0x228
 
         # write should retry several times and return -ENOSPC finally
         $SETSTRIPE -i 0 -c 1 $DIR/$tfile
@@ -5161,7 +5309,7 @@ test_101d() {
         { skip "Need free space ${size}M, have $space" && return; }
 
     echo Creating ${size}M test file $file
-    dd if=/dev/zero of=$file bs=1M count=$size
+    dd if=/dev/zero of=$file bs=1M count=$size || error "dd failed"
     echo Cancel LRU locks on lustre client to flush the client cache
     cancel_lru_locks osc
 
@@ -5182,8 +5330,9 @@ test_101d() {
     echo read-ahead disabled time read $time_ra_OFF
     echo read-ahead enabled  time read $time_ra_ON
 
-    set_read_ahead $old_READAHEAD
-    rm -f $file
+       set_read_ahead $old_READAHEAD
+       rm -f $file
+       wait_delete_completed
 
     [ $time_ra_ON -lt $time_ra_OFF ] ||
         error "read-ahead enabled  time read (${time_ra_ON}s) is more than
@@ -5599,9 +5748,9 @@ test_103 () {
 
     declare -a identity_old
 
-    for num in `seq $MDSCOUNT`; do
-        switch_identity $num true || identity_old[$num]=$?
-    done
+       for num in $(seq $MDSCOUNT); do
+               switch_identity $num true || identity_old[$num]=$?
+       done
 
     SAVE_UMASK=`umask`
     umask 0022
@@ -5625,14 +5774,20 @@ test_103 () {
     run_acl_subtest inheritance || error "inheritance test failed"
     rm -f make-tree
 
+    echo "LU-974 ignore umask when acl is enabled..."
+    mkdir $DIR/974
+    cd $DIR/974
+    run_acl_subtest 974 || error "LU-974 test failed"
+    rm -rf $DIR/974
+
     cd $SAVE_PWD
     umask $SAVE_UMASK
 
-    for num in `seq $MDSCOUNT`; do
-       if [ "${identity_old[$num]}" = 1 ]; then
-            switch_identity $num false || identity_old[$num]=$?
-       fi
-    done
+       for num in $(seq $MDSCOUNT); do
+               if [ "${identity_old[$num]}" = 1 ]; then
+                       switch_identity $num false || identity_old[$num]=$?
+               fi
+       done
 }
 run_test 103 "acl test ========================================="
 
@@ -5764,14 +5919,18 @@ test_107() {
 run_test 107 "Coredump on SIG"
 
 test_110() {
-       mkdir -p $DIR/d110
-       mkdir $DIR/d110/aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa || error "mkdir with 255 char fail"
-       mkdir $DIR/d110/bbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbb && error "mkdir with 256 char should fail, but not"
-       touch $DIR/d110/xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx || error "create with 255 char fail"
-       touch $DIR/d110/yyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy && error ""create with 256 char should fail, but not
-
-       ls -l $DIR/d110
-    rm -fr $DIR/d110
+       mkdir -p $DIR/$tdir
+       mkdir $DIR/$tdir/$(str_repeat 'a' 255) ||
+               error "mkdir with 255 char failed"
+       mkdir $DIR/$tdir/$(str_repeat 'b' 256) &&
+               error "mkdir with 256 char should fail, but did not"
+       touch $DIR/$tdir/$(str_repeat 'x' 255) ||
+               error "create with 255 char failed"
+       touch $DIR/$tdir/$(str_repeat 'y' 256) &&
+               error "create with 256 char should fail, but did not"
+
+       ls -l $DIR/$tdir
+       rm -rf $DIR/$tdir
 }
 run_test 110 "filename length checking"
 
@@ -5794,9 +5953,9 @@ test_115() {
            cut -c11-20)
 
        # don't return an error
-        [ $OSTIO_post -eq $OSTIO_pre ] && echo \
-           "WARNING: No new ll_ost_io threads were created ($OSTIO_pre)" &&\
-           echo "This may be fine, depending on what ran before this test" &&\
+        [ $OSTIO_post == $OSTIO_pre ] && echo \
+           "WARNING: No new ll_ost_io threads were created ($OSTIO_pre)" &&
+           echo "This may be fine, depending on what ran before this test" &&
            echo "and how fast this system is." && return
 
         echo "Started with $OSTIO_pre threads, ended with $OSTIO_post"
@@ -5822,7 +5981,7 @@ free_min_max () {
        echo Max free space: OST $MAXI: $MAXV
 }
 
-test_116() {
+test_116a() { # was previously test_116()
        [ "$OSTCOUNT" -lt "2" ] && skip_env "$OSTCOUNT < 2 OSTs" && return
 
        echo -n "Free space priority "
@@ -5905,7 +6064,21 @@ test_116() {
 
        rm -rf $DIR/$tdir
 }
-run_test 116 "stripe QOS: free space balance ==================="
+run_test 116a "stripe QOS: free space balance ==================="
+
+test_116b() { # LU-2093
+#define OBD_FAIL_MDS_OSC_CREATE_FAIL     0x147
+       local old_rr
+       old_rr=$(do_facet $SINGLEMDS lctl get_param -n lov.*mdtlov*.qos_threshold_rr)
+       do_facet $SINGLEMDS lctl set_param lov.*mdtlov*.qos_threshold_rr 0
+       mkdir -p $DIR/$tdir
+       do_facet $SINGLEMDS lctl set_param fail_loc=0x147
+       createmany -o $DIR/$tdir/f- 20 || error "can't create"
+       do_facet $SINGLEMDS lctl set_param fail_loc=0
+       rm -rf $DIR/$tdir
+       do_facet $SINGLEMDS lctl set_param lov.*mdtlov*.qos_threshold_rr $old_rr
+}
+run_test 116b "QoS shouldn't LBUG if not enough OSTs found on the 2nd pass"
 
 test_117() # bug 10891
 {
@@ -6180,6 +6353,8 @@ test_118h() {
 }
 run_test 118h "Verify timeout in handling recoverables errors  =========="
 
+[ "$SLOW" = "no" ] && [ -n "$OLD_RESENDCOUNT" ] && set_resend_count $OLD_RESENDCOUNT
+
 test_118i() {
        remote_ost_nodsh && skip "remote OST with nodsh" && return
 
@@ -6218,6 +6393,8 @@ test_118i() {
 }
 run_test 118i "Fix error before timeout in recoverable error  =========="
 
+[ "$SLOW" = "no" ] && set_resend_count 4
+
 test_118j() {
        remote_ost_nodsh && skip "remote OST with nodsh" && return
 
@@ -6926,15 +7103,18 @@ set_dir_limits () {
        local node
 
        local LDPROC=/proc/fs/ldiskfs
+       local facets=$(get_facets MDS)
 
-       for facet in $(get_facets MDS); do
-               canondev=$(ldiskfs_canon *.$(convert_facet2label $facet).mntdev $facet)
-               do_facet $facet "test -e $LDPROC/$canondev/max_dir_size" || LDPROC=/sys/fs/ldiskfs
+       for facet in ${facets//,/ }; do
+               canondev=$(ldiskfs_canon \
+                          *.$(convert_facet2label $facet).mntdev $facet)
+               do_facet $facet "test -e $LDPROC/$canondev/max_dir_size" ||
+                                               LDPROC=/sys/fs/ldiskfs
                do_facet $facet "echo $1 >$LDPROC/$canondev/max_dir_size"
        done
 }
 test_129() {
-       if [ "$(facet_type_fstype MDS)" != ldiskfs ]; then
+       if [ "$(facet_fstype $SINGLEMDS)" != ldiskfs ]; then
                skip "Only applicable to ldiskfs-based MDTs"
                return
        fi
@@ -6944,11 +7124,10 @@ test_129() {
        MAX=16384
 
        set_dir_limits $MAX
-
        mkdir -p $DIR/$tdir
 
-       I=0
-       J=0
+       local I=0
+       local J=0
        while [ ! $I -gt $((MAX * MDSCOUNT)) ]; do
                $MULTIOP $DIR/$tdir/$J Oc
                rc=$?
@@ -6976,18 +7155,26 @@ cleanup_130() {
 }
 
 test_130a() {
-       filefrag_op=$(filefrag -e 2>&1 | grep "invalid option")
-       [ -n "$filefrag_op" ] && skip "filefrag does not support FIEMAP" && return
+       local filefrag_op=$(filefrag -e 2>&1 | grep "invalid option")
+       [ -n "$filefrag_op" ] && skip_env "filefrag does not support FIEMAP" &&
+               return
 
        trap cleanup_130 EXIT RETURN
 
        local fm_file=$DIR/$tfile
        $SETSTRIPE -S 65536 -c 1 $fm_file || error "setstripe on $fm_file"
-       dd if=/dev/zero of=$fm_file bs=65536 count=1 || error "dd failed for $fm_file"
+       dd if=/dev/zero of=$fm_file bs=65536 count=1 ||
+               error "dd failed for $fm_file"
 
-       filefrag -ves $fm_file || error "filefrag $fm_file failed"
-       filefrag_op=`filefrag -ve $fm_file | grep -A 100 "ext:" | grep -v "ext:" | grep -v "found"`
+       # LU-1795: test filefrag/FIEMAP once, even if unsupported
+       filefrag -ves $fm_file
+       RC=$?
+       [ "$(facet_fstype ost$(($($GETSTRIPE -i $fm_file) + 1)))" = "zfs" ] &&
+               skip "ORI-366/LU-1941: FIEMAP unimplemented on ZFS" && return
+       [ $RC != 0 ] && error "filefrag $fm_file failed"
 
+       filefrag_op=$(filefrag -ve $fm_file | grep -A 100 "ext:" |
+                     grep -v "ext:" | grep -v "found")
        lun=$($GETSTRIPE -i $fm_file)
 
        start_blk=`echo $filefrag_op | cut -d: -f2 | cut -d. -f1`
@@ -7018,21 +7205,28 @@ test_130a() {
 run_test 130a "FIEMAP (1-stripe file)"
 
 test_130b() {
-       [ "$OSTCOUNT" -lt "2" ] && skip_env "skipping FIEMAP on 2-stripe file test" && return
+       [ "$OSTCOUNT" -lt "2" ] &&
+               skip_env "skipping FIEMAP on 2-stripe file test" && return
 
-       filefrag_op=$(filefrag -e 2>&1 | grep "invalid option")
-       [ -n "$filefrag_op" ] && skip "filefrag does not support FIEMAP" && return
+       local filefrag_op=$(filefrag -e 2>&1 | grep "invalid option")
+       [ -n "$filefrag_op" ] && skip_env "filefrag does not support FIEMAP" &&
+               return
 
        trap cleanup_130 EXIT RETURN
 
        local fm_file=$DIR/$tfile
        $SETSTRIPE -S 65536 -c 2 $fm_file || error "setstripe on $fm_file"
-       dd if=/dev/zero of=$fm_file bs=1M count=2 || error "dd failed on $fm_file"
+       [ "$(facet_fstype ost$(($($GETSTRIPE -i $fm_file) + 1)))" = "zfs" ] &&
+               skip "ORI-366/LU-1941: FIEMAP unimplemented on ZFS" && return
+
+       dd if=/dev/zero of=$fm_file bs=1M count=2 ||
+               error "dd failed on $fm_file"
 
        filefrag -ves $fm_file || error "filefrag $fm_file failed"
-       filefrag_op=`filefrag -ve $fm_file | grep -A 100 "ext:" | grep -v "ext:" | grep -v "found"`
+       filefrag_op=$(filefrag -ve $fm_file | grep -A 100 "ext:" |
+                     grep -v "ext:" | grep -v "found")
 
-       last_lun=`echo $filefrag_op | cut -d: -f5`
+       last_lun=$(echo $filefrag_op | cut -d: -f5)
 
        IFS=$'\n'
        tot_len=0
@@ -7067,15 +7261,20 @@ test_130b() {
 run_test 130b "FIEMAP (2-stripe file)"
 
 test_130c() {
-       [ "$OSTCOUNT" -lt "2" ] && skip_env "skipping FIEMAP on 2-stripe file with hole test" && return
+       [ "$OSTCOUNT" -lt "2" ] &&
+               skip_env "skipping FIEMAP on 2-stripe file" && return
 
        filefrag_op=$(filefrag -e 2>&1 | grep "invalid option")
-       [ -n "$filefrag_op" ] && skip "filefrag does not support FIEMAP" && return
+       [ -n "$filefrag_op" ] && skip "filefrag does not support FIEMAP" &&
+               return
 
        trap cleanup_130 EXIT RETURN
 
        local fm_file=$DIR/$tfile
        $SETSTRIPE -S 65536 -c 2 $fm_file || error "setstripe on $fm_file"
+       [ "$(facet_fstype ost$(($($GETSTRIPE -i $fm_file) + 1)))" = "zfs" ] &&
+               skip "ORI-366/LU-1941: FIEMAP unimplemented on ZFS" && return
+
        dd if=/dev/zero of=$fm_file seek=1 bs=1M count=1 || error "dd failed on $fm_file"
 
        filefrag -ves $fm_file || error "filefrag $fm_file failed"
@@ -7131,6 +7330,8 @@ test_130d() {
 
        local fm_file=$DIR/$tfile
        $SETSTRIPE -S 65536 -c $OSTCOUNT $fm_file||error "setstripe on $fm_file"
+       [ "$(facet_fstype ost$(($($GETSTRIPE -i $fm_file) + 1)))" = "zfs" ] &&
+               skip "ORI-366/LU-1941: FIEMAP unimplemented on ZFS" && return
        dd if=/dev/zero of=$fm_file bs=1M count=$OSTCOUNT || error "dd failed on $fm_file"
 
        filefrag -ves $fm_file || error "filefrag $fm_file failed"
@@ -7180,6 +7381,9 @@ test_130e() {
 
        local fm_file=$DIR/$tfile
        $SETSTRIPE -S 131072 -c 2 $fm_file || error "setstripe on $fm_file"
+       [ "$(facet_fstype ost$(($($GETSTRIPE -i $fm_file) + 1)))" = "zfs" ] &&
+               skip "ORI-366/LU-1941: FIEMAP unimplemented on ZFS" && return
+
        NUM_BLKS=512
        EXPECTED_LEN=$(( (NUM_BLKS / 2) * 64 ))
        for ((i = 0; i < $NUM_BLKS; i++))
@@ -7419,8 +7623,11 @@ test_133b() {
        chmod 444 ${testdir}/${tfile} || error "chmod failed"
        check_stats $SINGLEMDS "setattr" 1
        do_facet $SINGLEMDS $LCTL set_param mdt.*.md_stats=clear
-       ls -l ${testdir}/${tfile} > /dev/null|| error "ls failed"
-       check_stats $SINGLEMDS "getattr" 1
+       if [ $(lustre_version_code $SINGLEMDS) -ne $(version_code 2.2.0) ]
+       then            # LU-1740
+               ls -l ${testdir}/${tfile} > /dev/null|| error "ls failed"
+               check_stats $SINGLEMDS "getattr" 1
+       fi
        $LFS df || error "lfs failed"
        check_stats $SINGLEMDS "statfs" 1
 
@@ -7438,6 +7645,7 @@ test_133c() {
        $SETSTRIPE -c 1 -i 0 ${testdir}/${tfile}
        sync
        cancel_lru_locks osc
+       wait_delete_completed
 
        # clear stats.
        do_facet $SINGLEMDS $LCTL set_param mdt.*.md_stats=clear
@@ -7455,6 +7663,7 @@ test_133c() {
        check_stats ost "punch" 1
 
        rm -f ${testdir}/${tfile} || error "file remove failed"
+       wait_delete_completed
        check_stats ost "destroy" 1
 
        rm -rf $DIR/${tdir}
@@ -7503,8 +7712,10 @@ size_in_KMGT() {
 
 get_rename_size() {
     local size=$1
+    local context=${2:-.}
     local sample=$(do_facet $SINGLEMDS $LCTL get_param mdt.*.rename_stats |
-                   awk '/ '${size}'/ {print $4}' | sed -e "s/,//g")
+               grep -A1 $context |
+               awk '/ '${size}'/ {print $4}' | sed -e "s/,//g")
     echo $sample
 }
 
@@ -7523,22 +7734,24 @@ test_133d() {
     mkdir -p ${testdir2} || error "mkdir failed"
 
     createmany -o $testdir1/test 512 || error "createmany failed"
-    local testdir1_size=$(ls -l $DIR/${tdir} |
-                          awk '/stats_testdir1/ {print $5}')
-    local testdir2_size=$(ls -l $DIR/${tdir} |
-                          awk '/stats_testdir2/ {print $5}')
 
-    testdir1_size=$(order_2 $testdir1_size)
-    testdir2_size=$(order_2 $testdir2_size)
+       # check samedir rename size
+       mv ${testdir1}/test0 ${testdir1}/test_0
+
+       local testdir1_size=$(ls -l $DIR/${tdir} |
+               awk '/stats_testdir1/ {print $5}')
+       local testdir2_size=$(ls -l $DIR/${tdir} |
+               awk '/stats_testdir2/ {print $5}')
+
+       testdir1_size=$(order_2 $testdir1_size)
+       testdir2_size=$(order_2 $testdir2_size)
 
-    testdir1_size=$(size_in_KMGT $testdir1_size)
-    testdir2_size=$(size_in_KMGT $testdir2_size)
+       testdir1_size=$(size_in_KMGT $testdir1_size)
+       testdir2_size=$(size_in_KMGT $testdir2_size)
 
-    echo "source rename dir size: ${testdir1_size}"
-    echo "target rename dir size: ${testdir2_size}"
+       echo "source rename dir size: ${testdir1_size}"
+       echo "target rename dir size: ${testdir2_size}"
 
-    # check samedir rename size
-    mv ${testdir1}/test0 ${testdir1}/test_0
     local cmd="do_facet $SINGLEMDS $LCTL get_param mdt.*.rename_stats"
     eval $cmd || error "$cmd failed"
     local samedir=$($cmd | grep 'same_dir')
@@ -7551,10 +7764,25 @@ test_133d() {
 
     # check crossdir rename size
     mv ${testdir1}/test_0 ${testdir2}/test_0
+
+       testdir1_size=$(ls -l $DIR/${tdir} |
+               awk '/stats_testdir1/ {print $5}')
+       testdir2_size=$(ls -l $DIR/${tdir} |
+               awk '/stats_testdir2/ {print $5}')
+
+       testdir1_size=$(order_2 $testdir1_size)
+       testdir2_size=$(order_2 $testdir2_size)
+
+       testdir1_size=$(size_in_KMGT $testdir1_size)
+       testdir2_size=$(size_in_KMGT $testdir2_size)
+
+       echo "source rename dir size: ${testdir1_size}"
+       echo "target rename dir size: ${testdir2_size}"
+
     eval $cmd || error "$cmd failed"
     local crossdir=$($cmd | grep 'crossdir')
-    local src_sample=$(get_rename_size $testdir1_size)
-    local tgt_sample=$(get_rename_size $testdir2_size)
+    local src_sample=$(get_rename_size $testdir1_size crossdir_src)
+    local tgt_sample=$(get_rename_size $testdir2_size crossdir_tgt)
     [ -z "$crossdir" ] && error "crossdir_rename_size count error"
     [ "$src_sample" -eq 1 ] || error "crossdir_rename_size error $src_sample"
     [ "$tgt_sample" -eq 1 ] || error "crossdir_rename_size error $tgt_sample"
@@ -7568,8 +7796,9 @@ test_140() { #bug-17379
         cd $DIR/$tdir || error "Changing to $DIR/$tdir"
         cp /usr/bin/stat . || error "Copying stat to $DIR/$tdir"
 
-        # VFS limits max symlink depth to 5(4KSTACK) or 7(8KSTACK) or 8
-        local i=0
+       # VFS limits max symlink depth to 5(4KSTACK) or 7(8KSTACK) or 8
+       # For kernel > 3.5, bellow only tests consecutive symlink (MAX 40)
+       local i=0
         while i=`expr $i + 1`; do
                 mkdir -p $i || error "Creating dir $i"
                 cd $i || error "Changing to $i"
@@ -7589,7 +7818,14 @@ test_140() { #bug-17379
         done
         i=`expr $i - 1`
         echo "The symlink depth = $i"
-        [ $i -eq 5 -o $i -eq 7 -o $i -eq 8 ] || error "Invalid symlink depth"
+       [ $i -eq 5 -o $i -eq 7 -o $i -eq 8 -o $i -eq 40 ] ||
+                                       error "Invalid symlink depth"
+
+       # Test recursive symlink
+       ln -s symlink_self symlink_self
+       $OPENFILE -f O_RDONLY symlink_self >/dev/null 2>&1; ret=$?
+       echo "open symlink_self returns $ret"
+       [ $ret -eq 40 ] || error "recursive symlink doesn't return -ELOOP"
 }
 run_test 140 "Check reasonable stack depth (shouldn't LBUG) ===="
 
@@ -7625,44 +7861,43 @@ test_150() {
 run_test 150 "truncate/append tests"
 
 function roc_hit() {
-    local list=$(comma_list $(osts_nodes))
+       local list=$(comma_list $(osts_nodes))
 
-    ACCNUM=$(do_nodes $list $LCTL get_param -n obdfilter.*.stats | \
-        awk '/'cache_hit'/ {sum+=$2} END {print sum}')
-    echo $ACCNUM
+       echo $(get_obdfilter_param $list '' stats |
+              awk '/'cache_hit'/ {sum+=$2} END {print sum}')
 }
 
 function set_cache() {
-    local on=1
+       local on=1
 
-    if [ "$2" == "off" ]; then
-        on=0;
-    fi
-    local list=$(comma_list $(osts_nodes))
-    do_nodes $list lctl set_param obdfilter.*.${1}_cache_enable $on
+       if [ "$2" == "off" ]; then
+               on=0;
+       fi
+       local list=$(comma_list $(osts_nodes))
+       set_obdfilter_param $list '' $1_cache_enable $on
 
-    cancel_lru_locks osc
+       cancel_lru_locks osc
 }
 
 test_151() {
-        remote_ost_nodsh && skip "remote OST with nodsh" && return
+       remote_ost_nodsh && skip "remote OST with nodsh" && return
 
-        local CPAGES=3
-        local list=$(comma_list $(osts_nodes))
+       local CPAGES=3
+       local list=$(comma_list $(osts_nodes))
 
-        # check whether obdfilter is cache capable at all
-        if ! do_nodes $list $LCTL get_param -n obdfilter.*.read_cache_enable > /dev/null; then
-                echo "not cache-capable obdfilter"
-                return 0
-        fi
+       # check whether obdfilter is cache capable at all
+       if ! get_obdfilter_param $list '' read_cache_enable >/dev/null; then
+               echo "not cache-capable obdfilter"
+               return 0
+       fi
 
-        # check cache is enabled on all obdfilters
-        if do_nodes $list $LCTL get_param -n obdfilter.*.read_cache_enable | grep 0 >&/dev/null; then
-                echo "oss cache is disabled"
-                return 0
-        fi
+       # check cache is enabled on all obdfilters
+       if get_obdfilter_param $list '' read_cache_enable | grep 0; then
+               echo "oss cache is disabled"
+               return 0
+       fi
 
-        do_nodes $list $LCTL set_param -n obdfilter.*.writethrough_cache_enable 1
+       set_obdfilter_param $list '' writethrough_cache_enable 1
 
         # pages should be in the case right after write
         dd if=/dev/urandom of=$DIR/$tfile bs=4k count=$CPAGES || error "dd failed"
@@ -7676,7 +7911,7 @@ test_151() {
 
         # the following read invalidates the cache
         cancel_lru_locks osc
-        do_nodes $list $LCTL set_param -n obdfilter.*.read_cache_enable 0
+       set_obdfilter_param $list '' read_cache_enable 0
         cat $DIR/$tfile >/dev/null
 
         # now data shouldn't be found in the cache
@@ -7688,7 +7923,7 @@ test_151() {
                 error "IN CACHE: before: $BEFORE, after: $AFTER"
         fi
 
-        do_nodes $list $LCTL set_param -n obdfilter.*.read_cache_enable 1
+       set_obdfilter_param $list '' read_cache_enable 1
         rm -f $DIR/$tfile
 }
 run_test 151 "test cache on oss and controls ==============================="
@@ -7722,6 +7957,9 @@ test_153() {
 run_test 153 "test if fdatasync does not crash ======================="
 
 test_154() {
+       [[ $(lustre_version_code $SINGLEMDS) -ge $(version_code 2.2.51) ]] ||
+               { skip "Need MDS version at least 2.2.51"; return 0; }
+
        cp /etc/hosts $DIR/$tfile
 
        fid=$($LFS path2fid $DIR/$tfile)
@@ -7787,6 +8025,46 @@ test_154() {
        touch $DIR/.lustre/fid/$tfile && \
                error "touch $DIR/.lustre/fid/$tfile should fail."
 
+       echo "setxattr to $DIR/.lustre/fid"
+       setfattr -n trusted.name1 -v value1 $DIR/.lustre/fid &&
+               error "setxattr should fail."
+
+       echo "listxattr for $DIR/.lustre/fid"
+       getfattr -d -m "^trusted" $DIR/.lustre/fid &&
+               error "listxattr should fail."
+
+       echo "delxattr from $DIR/.lustre/fid"
+       setfattr -x trusted.name1 $DIR/.lustre/fid &&
+               error "delxattr should fail."
+
+       echo "touch invalid fid: $DIR/.lustre/fid/[0x200000400:0x2:0x3]"
+       touch $DIR/.lustre/fid/[0x200000400:0x2:0x3] &&
+               error "touch invalid fid should fail."
+
+       echo "touch non-normal fid: $DIR/.lustre/fid/[0x1:0x2:0x0]"
+       touch $DIR/.lustre/fid/[0x1:0x2:0x0] &&
+               error "touch non-normal fid should fail."
+
+       echo "rename $tdir to $DIR/.lustre/fid"
+       mrename $DIR/$tdir $DIR/.lustre/fid &&
+               error "rename to $DIR/.lustre/fid should fail."
+
+       echo "rename .lustre to itself"
+       fid=$($LFS path2fid $DIR)
+       mrename $DIR/.lustre $DIR/.lustre/fid/$fid/.lustre &&
+               error "rename .lustre to itself should fail."
+
+       $OPENFILE -f O_LOV_DELAY_CREATE:O_CREAT $DIR/$tfile-2
+       fid=$($LFS path2fid $DIR/$tfile-2)
+       echo "cp /etc/passwd $DIR/.lustre/fid/$fid"
+       cp /etc/passwd $DIR/.lustre/fid/$fid &&
+               error "create lov data thru .lustre should fail."
+       echo "cp /etc/passwd $DIR/$tfile-2"
+       cp /etc/passwd $DIR/$tfile-2 || error "copy to $DIR/$tfile-2 failed."
+       echo "diff /etc/passwd $DIR/.lustre/fid/$fid"
+       diff /etc/passwd $DIR/.lustre/fid/$fid ||
+               error "diff /etc/passwd $DIR/.lustre/fid/$fid failed."
+
        echo "Open-by-FID succeeded"
 }
 run_test 154 "Open-by-FID"
@@ -7909,6 +8187,10 @@ test_156() {
     local AFTER
     local file="$DIR/$tfile"
 
+       [ "$(facet_fstype ost1)" = "zfs" ] &&
+               skip "LU-1956/LU-2261: stats unimplemented on OSD ZFS" &&
+               return
+
     log "Turn on read and write cache"
     set_cache read on
     set_cache writethrough on
@@ -8486,190 +8768,321 @@ test_182() {
 }
 run_test 182 "Disable MDC RPCs semaphore wouldn't crash client ================"
 
-# OST pools tests
-POOL=${POOL:-cea1}
-TGT_COUNT=$OSTCOUNT
-TGTPOOL_FIRST=1
-TGTPOOL_MAX=$(($TGT_COUNT - 1))
-TGTPOOL_STEP=2
-TGTPOOL_LIST=`seq $TGTPOOL_FIRST $TGTPOOL_STEP $TGTPOOL_MAX`
-POOL_ROOT=${POOL_ROOT:-$DIR/d200.pools}
-POOL_DIR_NAME=dir_tst
-POOL_DIR=$POOL_ROOT/$POOL_DIR_NAME
-POOL_FILE=$POOL_ROOT/file_tst
+test_183() { # LU-2275
+       mkdir -p $DIR/$tdir || error "creating dir $DIR/$tdir"
+       echo aaa > $DIR/$tdir/$tfile
+
+#define OBD_FAIL_MDS_NEGATIVE_POSITIVE  0x148
+       do_facet $SINGLEMDS $LCTL set_param fail_loc=0x148
+
+       ls -l $DIR/$tdir && error "ls succeeded, should have failed"
+       cat $DIR/$tdir/$tfile && error "cat succeeded, should have failed"
+
+       do_facet $SINGLEMDS $LCTL set_param fail_loc=0
+
+       # Flush negative dentry cache
+       touch $DIR/$tdir/$tfile
 
+       # We are not checking for any leaked references here, they'll
+       # become evident next time we do cleanup with module unload.
+       rm -rf $DIR/$tdir
+}
+run_test 183 "No crash or request leak in case of strange dispositions ========"
+
+# OST pools tests
 check_file_in_pool()
 {
-       file=$1
-       res=$($GETSTRIPE $file | grep 0x | cut -f2)
+       local file=$1
+       local pool=$2
+       local tlist="$3"
+       local res=$($GETSTRIPE $file | grep 0x | cut -f2)
        for i in $res
        do
-               found=$(echo :$TGTPOOL_LIST: | tr " " ":"  | grep :$i:)
-               if [[ "$found" == "" ]]
-               then
-                       echo "pool list: $TGTPOOL_LIST"
-                       echo "striping: $res"
-                       error "$file not allocated in $POOL"
-                       return 1
-               fi
+               for t in $tlist ; do
+                       [ "$i" -eq "$t" ] && continue 2
+               done
+
+               echo "pool list: $tlist"
+               echo "striping: $res"
+               error_noexit "$file not allocated in $pool"
+               return 1
        done
        return 0
 }
 
-trap "cleanup_pools $FSNAME" EXIT
+pool_add() {
+       echo "Creating new pool"
+       local pool=$1
 
-test_200a() {
-       remote_mgs_nodsh && skip "remote MGS with nodsh" && return
-    create_pool $FSNAME.$POOL || return $?
-       [ $($LFS pool_list $FSNAME | grep -c $POOL) -eq 1 ] ||
-               error "$POOL not in lfs pool_list"
+       create_pool $FSNAME.$pool ||
+               { error_noexit "No pool created, result code $?"; return 1; }
+       [ $($LFS pool_list $FSNAME | grep -c $pool) -eq 1 ] ||
+               { error_noexit "$pool not in lfs pool_list"; return 2; }
 }
-run_test 200a "Create new pool =========================================="
 
-test_200b() {
-       remote_mgs_nodsh && skip "remote MGS with nodsh" && return
-       TGT=$(for i in $TGTPOOL_LIST; do printf "$FSNAME-OST%04x_UUID " $i; done)
-       do_facet mgs $LCTL pool_add $FSNAME.$POOL \
-               $FSNAME-OST[$TGTPOOL_FIRST-$TGTPOOL_MAX/$TGTPOOL_STEP]
-       wait_update $HOSTNAME "lctl get_param -n lov.$FSNAME-*.pools.$POOL | sort -u | tr '\n' ' ' " "$TGT" ||
-               error "Add to pool failed"
-       local lfscount=$($LFS pool_list $FSNAME.$POOL | grep -c "\-OST")
-       local addcount=$((($TGTPOOL_MAX - $TGTPOOL_FIRST) / $TGTPOOL_STEP + 1))
-       [ $lfscount -eq $addcount ] ||
-               error "lfs pool_list bad ost count $lfscount != $addcount"
-}
-run_test 200b "Add targets to a pool ===================================="
-
-test_200c() {
-       remote_mgs_nodsh && skip "remote MGS with nodsh" && return
-       mkdir -p $POOL_DIR
-       $SETSTRIPE -c 2 -p $POOL $POOL_DIR
-       [ $? = 0 ] || error "Cannot set pool $POOL to $POOL_DIR"
-       # b-19919 test relative path works well
-       mkdir -p $POOL_DIR/$POOL_DIR_NAME
-       cd $POOL_DIR
-       $SETSTRIPE -c 2 -p $POOL $POOL_DIR_NAME
-       [ $? = 0 ] || error "Cannot set pool $POOL to $POOL_DIR/$POOL_DIR_NAME"
-       $SETSTRIPE -c 2 -p $POOL ./$POOL_DIR_NAME
-       [ $? = 0 ] || error "Cannot set pool $POOL to $POOL_DIR/./$POOL_DIR_NAME"
-       $SETSTRIPE -c 2 -p $POOL ../$POOL_DIR_NAME
-       [ $? = 0 ] || error "Cannot set pool $POOL to $POOL_DIR/../$POOL_DIR_NAME"
-       $SETSTRIPE -c 2 -p $POOL ../$POOL_DIR_NAME/$POOL_DIR_NAME
-       [ $? = 0 ] || error "Cannot set pool $POOL to $POOL_DIR/../$POOL_DIR_NAME/$POOL_DIR_NAME"
-       rm -rf $POOL_DIR_NAME; cd -
-}
-run_test 200c "Set pool on a directory ================================="
-
-test_200d() {
-       remote_mgs_nodsh && skip "remote MGS with nodsh" && return
-       res=$($GETSTRIPE --pool $POOL_DIR)
-       [ $res = $POOL ] || error "Pool on $POOL_DIR is $res, not $POOL"
+pool_add_targets() {
+       echo "Adding targets to pool"
+       local pool=$1
+       local first=$2
+       local last=$3
+       local step=${4:-1}
+
+       local list=$(seq $first $step $last)
+
+       local t=$(for i in $list; do printf "$FSNAME-OST%04x_UUID " $i; done)
+       do_facet mgs $LCTL pool_add \
+                       $FSNAME.$pool $FSNAME-OST[$first-$last/$step]
+       wait_update $HOSTNAME "lctl get_param -n lov.$FSNAME-*.pools.$pool \
+                       | sort -u | tr '\n' ' ' " "$t" || { 
+               error_noexit "Add to pool failed"
+               return 1
+       }
+       local lfscount=$($LFS pool_list $FSNAME.$pool | grep -c "\-OST")
+       local addcount=$(((last - first) / step + 1))
+       [ $lfscount -eq $addcount ] || {
+               error_noexit "lfs pool_list bad ost count" \
+                                               "$lfscount != $addcount"
+               return 2
+       }
 }
-run_test 200d "Check pool on a directory ==============================="
 
-test_200e() {
-       remote_mgs_nodsh && skip "remote MGS with nodsh" && return
-       failed=0
-       for i in $(seq -w 1 $(($TGT_COUNT * 3)))
+pool_set_dir() {
+       local pool=$1
+       local tdir=$2
+       echo "Setting pool on directory $tdir"
+
+       $SETSTRIPE -c 2 -p $pool $tdir && return 0
+
+       error_noexit "Cannot set pool $pool to $tdir"
+       return 1
+}
+
+pool_check_dir() {
+       local pool=$1
+       local tdir=$2
+       echo "Checking pool on directory $tdir"
+
+       local res=$($GETSTRIPE --pool $tdir | sed "s/\s*$//")
+       [ "$res" = "$pool" ] && return 0
+
+       error_noexit "Pool on '$tdir' is '$res', not '$pool'"
+       return 1
+}
+
+pool_dir_rel_path() {
+       echo "Testing relative path works well"
+       local pool=$1
+       local tdir=$2
+       local root=$3
+
+       mkdir -p $root/$tdir/$tdir
+       cd $root/$tdir
+       pool_set_dir $pool $tdir          || return 1
+       pool_set_dir $pool ./$tdir        || return 2
+       pool_set_dir $pool ../$tdir       || return 3
+       pool_set_dir $pool ../$tdir/$tdir || return 4
+       rm -rf $tdir; cd - > /dev/null
+}
+
+pool_alloc_files() {
+       echo "Checking files allocation from directory pool"
+       local pool=$1
+       local tdir=$2
+       local count=$3
+       local tlist="$4"
+
+       local failed=0
+       for i in $(seq -w 1 $count)
        do
-               file=$POOL_DIR/file-$i
+               local file=$tdir/file-$i
                touch $file
-               check_file_in_pool $file
-               if [[ $? != 0 ]]
-               then
-                       failed=$(($failed + 1))
-               fi
+               check_file_in_pool $file $pool "$tlist" || \
+                       failed=$((failed + 1))
        done
-       [ "$failed" = 0 ] || error "$failed files not allocated in $POOL"
+       [ "$failed" = 0 ] && return 0
+
+       error_noexit "$failed files not allocated in $pool"
+       return 1
 }
-run_test 200e "Check files allocation from directory pool =============="
 
-test_200f() {
-       remote_mgs_nodsh && skip "remote MGS with nodsh" && return
-       mkdir -p $POOL_FILE
-       failed=0
-       for i in $(seq -w 1 $(($TGT_COUNT * 3)))
+pool_create_files() {
+       echo "Creating files in pool"
+       local pool=$1
+       local tdir=$2
+       local count=$3
+       local tlist="$4"
+
+       mkdir -p $tdir
+       local failed=0
+       for i in $(seq -w 1 $count)
        do
-               file=$POOL_FILE/spoo-$i
-               $SETSTRIPE -p $POOL $file
-               check_file_in_pool $file
-               if [[ $? != 0 ]]
-               then
-                       failed=$(($failed + 1))
-               fi
+               local file=$tdir/spoo-$i
+               $SETSTRIPE -p $pool $file
+               check_file_in_pool $file $pool "$tlist" || \
+                       failed=$((failed + 1))
        done
-       [ "$failed" = 0 ] || error "$failed files not allocated in $POOL"
+       [ "$failed" = 0 ] && return 0
+
+       error_noexit "$failed files not allocated in $pool"
+       return 1
 }
-run_test 200f "Create files in a pool ==================================="
 
-test_200g() {
-       remote_mgs_nodsh && skip "remote MGS with nodsh" && return
-       TGT=$($LCTL get_param -n lov.$FSNAME-clilov-*.pools.$POOL | tr '\n' ' ')
-       res=$($LFS df --pool $FSNAME.$POOL | awk '{print $1}' | grep "$FSNAME-OST" | tr '\n' ' ')
-       [ "$res" = "$TGT" ] || error "Pools OSTs '$TGT' is not '$res' that lfs df reports"
+pool_lfs_df() {
+       echo "Checking 'lfs df' output"
+       local pool=$1
+
+       local t=$($LCTL get_param -n lov.$FSNAME-clilov-*.pools.$pool |
+                       tr '\n' ' ')
+       local res=$($LFS df --pool $FSNAME.$pool |
+                       awk '{print $1}' |
+                       grep "$FSNAME-OST" |
+                       tr '\n' ' ')
+       [ "$res" = "$t" ] && return 0
+
+       error_noexit "Pools OSTs '$t' is not '$res' that lfs df reports"
+       return 1
 }
-run_test 200g "lfs df a pool ============================================"
 
-test_200h() { # b=24039
-       mkdir -p $POOL_DIR || error "unable to create $POOL_DIR"
+pool_file_rel_path() {
+       echo "Creating files in a pool with relative pathname"
+       local pool=$1
+       local tdir=$2
 
-       local file="/..$POOL_DIR/$tfile-1"
-       $SETSTRIPE -p $POOL $file || error "unable to create $file"
+       mkdir -p $tdir ||
+               { error_noexit "unable to create $tdir"; return 1 ; }
+       local file="/..$tdir/$tfile-1"
+       $SETSTRIPE -p $pool $file ||
+               { error_noexit "unable to create $file" ; return 2 ; }
 
-       cd $POOL_DIR
-       $SETSTRIPE -p $POOL $tfile-2 || \
-               error "unable to create $tfile-2 in $POOL_DIR"
+       cd $tdir
+       $SETSTRIPE -p $pool $tfile-2 || {
+               error_noexit "unable to create $tfile-2 in $tdir"
+               return 3
+       }
 }
-run_test 200h "Create files in a pool with relative pathname ============"
 
-test_201a() {
-       remote_mgs_nodsh && skip "remote MGS with nodsh" && return
-       TGT=$($LCTL get_param -n lov.$FSNAME-*.pools.$POOL | head -1)
-       do_facet mgs $LCTL pool_remove $FSNAME.$POOL $TGT
-       wait_update $HOSTNAME "lctl get_param -n lov.$FSNAME-*.pools.$POOL | grep $TGT" "" ||
-               error "$TGT not removed from $FSNAME.$POOL"
+pool_remove_first_target() {
+       echo "Removing first target from a pool"
+       local pool=$1
+
+       local pname="lov.$FSNAME-*.pools.$pool"
+       local t=$($LCTL get_param -n $pname | head -1)
+       do_facet mgs $LCTL pool_remove $FSNAME.$pool $t
+       wait_update $HOSTNAME "lctl get_param -n $pname | grep $t" "" || {
+               error_noexit "$t not removed from $FSNAME.$pool"
+               return 1
+       }
 }
-run_test 201a "Remove a target from a pool ============================="
 
-test_201b() {
-       remote_mgs_nodsh && skip "remote MGS with nodsh" && return
-       for TGT in $($LCTL get_param -n lov.$FSNAME-*.pools.$POOL | sort -u)
+pool_remove_all_targets() {
+       echo "Removing all targets from pool"
+       local pool=$1
+       local file=$2
+       local pname="lov.$FSNAME-*.pools.$pool"
+       for t in $($LCTL get_param -n $pname | sort -u)
        do
-               do_facet mgs $LCTL pool_remove $FSNAME.$POOL $TGT
-       done
-       wait_update $HOSTNAME "lctl get_param -n lov.$FSNAME-*.pools.$POOL" "" ||
-               error "Pool $FSNAME.$POOL cannot be drained"
-       # striping on an empty/nonexistant pool should fall back to "pool of everything"
-       touch ${POOL_DIR}/$tfile || error "failed to use fallback striping for empty pool"
+               do_facet mgs $LCTL pool_remove $FSNAME.$pool $t
+       done
+       wait_update $HOSTNAME "lctl get_param -n $pname" "" || {
+               error_noexit "Pool $FSNAME.$pool cannot be drained"
+               return 1
+       }
+       # striping on an empty/nonexistant pool should fall back 
+       # to "pool of everything"
+       touch $file || {
+               error_noexit "failed to use fallback striping for empty pool"
+               return 2
+       }
        # setstripe on an empty pool should fail
-       $SETSTRIPE -p $POOL ${POOL_FILE}/$tfile 2>/dev/null && \
-               error "expected failure when creating file with empty pool"
+       $SETSTRIPE -p $pool $file 2>/dev/null && {
+               error_noexit "expected failure when creating file" \
+                                                       "with empty pool"
+               return 3
+       }
        return 0
 }
-run_test 201b "Remove all targets from a pool =========================="
 
-test_201c() {
-       remote_mgs_nodsh && skip "remote MGS with nodsh" && return
-       do_facet mgs $LCTL pool_destroy $FSNAME.$POOL
+pool_remove() {
+       echo "Destroying pool"
+       local pool=$1
+       local file=$2
+
+       do_facet mgs $LCTL pool_destroy $FSNAME.$pool
 
        sleep 2
-    # striping on an empty/nonexistant pool should fall back to "pool of everything"
-       touch ${POOL_DIR}/$tfile || error "failed to use fallback striping for missing pool"
+       # striping on an empty/nonexistant pool should fall back 
+       # to "pool of everything"
+       touch $file || {
+               error_noexit "failed to use fallback striping for missing pool"
+               return 1
+       }
        # setstripe on an empty pool should fail
-       $SETSTRIPE -p $POOL ${POOL_FILE}/$tfile 2>/dev/null && \
-               error "expected failure when creating file with missing pool"
+       $SETSTRIPE -p $pool $file 2>/dev/null && {
+               error_noexit "expected failure when creating file" \
+                                                       "with missing pool"
+               return 2
+       }
 
        # get param should return err once pool is gone
-       if wait_update $HOSTNAME "lctl get_param -n lov.$FSNAME-*.pools.$POOL 2>/dev/null ||
-                       echo foo" "foo"; then
-               remove_pool_from_list $FSNAME.$POOL
+       if wait_update $HOSTNAME "lctl get_param -n \
+               lov.$FSNAME-*.pools.$pool 2>/dev/null || echo foo" "foo"
+       then
+               remove_pool_from_list $FSNAME.$pool
                return 0
        fi
-       error "Pool $FSNAME.$POOL is not destroyed"
+       error_noexit "Pool $FSNAME.$pool is not destroyed"
+       return 3
 }
-run_test 201c "Remove a pool ============================================"
 
-cleanup_pools $FSNAME
+test_200() {
+       remote_mgs_nodsh && skip "remote MGS with nodsh" && return
+
+       local POOL=${POOL:-cea1}
+       local POOL_ROOT=${POOL_ROOT:-$DIR/d200.pools}
+       local POOL_DIR_NAME=${POOL_DIR_NAME:-dir_tst}
+       # Pool OST targets
+       local first_ost=0
+       local last_ost=$(($OSTCOUNT - 1))
+       local ost_step=2
+       local ost_list=$(seq $first_ost $ost_step $last_ost)
+       local ost_range="$first_ost $last_ost $ost_step"
+       local test_path=$POOL_ROOT/$POOL_DIR_NAME
+       local file_dir=$POOL_ROOT/file_tst
+
+       local rc=0
+       while : ; do
+               # former test_200a test_200b
+               pool_add $POOL                          || { rc=$? ; break; }
+               pool_add_targets  $POOL $ost_range      || { rc=$? ; break; }
+               # former test_200c test_200d
+               mkdir -p $test_path
+               pool_set_dir      $POOL $test_path      || { rc=$? ; break; }
+               pool_check_dir    $POOL $test_path      || { rc=$? ; break; }
+               pool_dir_rel_path $POOL $POOL_DIR_NAME $POOL_ROOT \
+                                                       || { rc=$? ; break; }
+               # former test_200e test_200f
+               local files=$((OSTCOUNT*3))
+               pool_alloc_files  $POOL $test_path $files "$ost_list" \
+                                                       || { rc=$? ; break; }
+               pool_create_files $POOL $file_dir $files "$ost_list" \
+                                                       || { rc=$? ; break; }
+               # former test_200g test_200h
+               pool_lfs_df $POOL                       || { rc=$? ; break; }
+               pool_file_rel_path $POOL $test_path     || { rc=$? ; break; }
+
+               # former test_201a test_201b test_201c
+               pool_remove_first_target $POOL          || { rc=$? ; break; }
+
+               local f=$test_path/$tfile
+               pool_remove_all_targets $POOL $f        || { rc=$? ; break; }
+               pool_remove $POOL $f                    || { rc=$? ; break; }
+               break
+       done
+
+       cleanup_pools
+       return $rc
+}
+run_test 200 "OST pools"
 
 # usage: default_attr <count | size | offset>
 default_attr() {
@@ -8806,7 +9219,7 @@ verify_jobstats() {
 
        # clear old jobstats
        do_facet $SINGLEMDS lctl set_param mdt.*.job_stats="clear"
-       do_facet ost0 lctl set_param obdfilter.*.job_stats="clear"
+       do_facet ost1 lctl set_param obdfilter.*.job_stats="clear"
 
        # use a new JobID for this test, or we might see an old one
        [ "$JOBENV" = "FAKE_JOBID" ] && FAKE_JOBID=test_id.$testnum.$RANDOM
@@ -8827,13 +9240,16 @@ verify_jobstats() {
                        grep $JOBVAL || error "No job stats found on MDT $FACET"
        fi
        if [ "$target" = "ost" -o "$target" = "both" ]; then
-               FACET=ost0
+               FACET=ost1
                do_facet $FACET lctl get_param obdfilter.*.job_stats |
                        grep $JOBVAL || error "No job stats found on OST $FACET"
        fi
 }
 
 test_205() { # Job stats
+       [ -z "$(lctl get_param -n mdc.*.connect_flags | grep jobstats)" ] &&
+               skip "Server doesn't support jobstats" && return 0
+
        local cmd
        OLD_JOBENV=`$LCTL get_param -n jobid_var`
        if [ $OLD_JOBENV != $JOBENV ]; then
@@ -8884,6 +9300,17 @@ test_205() { # Job stats
 }
 run_test 205 "Verify job stats"
 
+# LU-1480, LU-1773 and LU-1657
+test_206() {
+       mkdir -p $DIR/$tdir
+       lfs setstripe -c -1 $DIR/$tdir
+#define OBD_FAIL_LOV_INIT 0x1403
+       $LCTL set_param fail_loc=0xa0001403
+       $LCTL set_param fail_val=1
+       touch $DIR/$tdir/$tfile || true
+}
+run_test 206 "fail lov_init_raid0() doesn't lbug"
+
 test_212() {
        size=`date +%s`
        size=$((size % 8192 + 1))
@@ -9031,10 +9458,10 @@ test_215() { # for bugs 18102, 21079, 21517
        # /proc/sys/lnet/nis should look like this:
        # nid status alive refs peer rtr max tx min
        # where nid is a string like 192.168.1.1@tcp2, status is up/down,
-       # alive is numeric (0 or >0 or <0), refs > 0, peer >= 0,
+       # alive is numeric (0 or >0 or <0), refs >= 0, peer >= 0,
        # rtr >= 0, max >=0, tx and min are numeric (0 or >0 or <0).
        L1="^nid +status +alive +refs +peer +rtr +max +tx +min$"
-       BR="^$NID +(up|down) +$I +$P +$N +$N +$N +$I +$I$"
+       BR="^$NID +(up|down) +$I +$N +$N +$N +$N +$I +$I$"
        create_lnet_proc_files "nis"
        check_lnet_proc_entry "nis.out" "/proc/sys/lnet/nis" "$BR" "$L1"
        check_lnet_proc_entry "nis.sys" "lnet.nis" "$BR" "$L1"
@@ -9367,6 +9794,139 @@ test_227() {
 }
 run_test 227 "running truncated executable does not cause OOM"
 
+# LU-1512 try to reuse idle OI blocks
+test_228a() {
+       [ "$(facet_fstype $SINGLEMDS)" != "ldiskfs" ] &&
+               skip "non-ldiskfs backend" && return
+
+       local MDT_DEV=$(mdsdevname ${SINGLEMDS//mds/})
+       local myDIR=$DIR/$tdir
+
+       mkdir -p $myDIR
+       #define OBD_FAIL_SEQ_EXHAUST             0x1002
+       $LCTL set_param fail_loc=0x80001002
+       createmany -o $myDIR/t- 10000
+       $LCTL set_param fail_loc=0
+       # The guard is current the largest FID holder
+       touch $myDIR/guard
+       local SEQ=$($LFS path2fid $myDIR/guard | awk -F ':' '{print $1}' |
+                   tr -d '[')
+       local IDX=$(($SEQ % 64))
+
+       do_facet $SINGLEMDS sync
+       # Make sure journal flushed.
+       sleep 6
+       local blk1=$(do_facet $SINGLEMDS \
+                    "$DEBUGFS -c -R \\\"stat oi.16.${IDX}\\\" $MDT_DEV" |
+                    grep Blockcount | awk '{print $4}')
+
+       # Remove old files, some OI blocks will become idle.
+       unlinkmany $myDIR/t- 10000
+       # Create new files, idle OI blocks should be reused.
+       createmany -o $myDIR/t- 2000
+       do_facet $SINGLEMDS sync
+       # Make sure journal flushed.
+       sleep 6
+       local blk2=$(do_facet $SINGLEMDS \
+                    "$DEBUGFS -c -R \\\"stat oi.16.${IDX}\\\" $MDT_DEV" |
+                    grep Blockcount | awk '{print $4}')
+
+       [ $blk1 == $blk2 ] || error "old blk1=$blk1, new blk2=$blk2, unmatched!"
+}
+run_test 228a "try to reuse idle OI blocks"
+
+test_228b() {
+       [ "$(facet_fstype $SINGLEMDS)" != "ldiskfs" ] &&
+               skip "non-ldiskfs backend" && return
+
+       local MDT_DEV=$(mdsdevname ${SINGLEMDS//mds/})
+       local myDIR=$DIR/$tdir
+
+       mkdir -p $myDIR
+       #define OBD_FAIL_SEQ_EXHAUST             0x1002
+       $LCTL set_param fail_loc=0x80001002
+       createmany -o $myDIR/t- 10000
+       $LCTL set_param fail_loc=0
+       # The guard is current the largest FID holder
+       touch $myDIR/guard
+       local SEQ=$($LFS path2fid $myDIR/guard | awk -F ':' '{print $1}' |
+                   tr -d '[')
+       local IDX=$(($SEQ % 64))
+
+       do_facet $SINGLEMDS sync
+       # Make sure journal flushed.
+       sleep 6
+       local blk1=$(do_facet $SINGLEMDS \
+                    "$DEBUGFS -c -R \\\"stat oi.16.${IDX}\\\" $MDT_DEV" |
+                    grep Blockcount | awk '{print $4}')
+
+       # Remove old files, some OI blocks will become idle.
+       unlinkmany $myDIR/t- 10000
+
+       # stop the MDT
+       stop $SINGLEMDS || error "Fail to stop MDT."
+       # remount the MDT
+       start $SINGLEMDS $MDT_DEV $MDS_MOUNT_OPTS || error "Fail to start MDT."
+
+       df $MOUNT || error "Fail to df."
+       # Create new files, idle OI blocks should be reused.
+       createmany -o $myDIR/t- 2000
+       do_facet $SINGLEMDS sync
+       # Make sure journal flushed.
+       sleep 6
+       local blk2=$(do_facet $SINGLEMDS \
+                    "$DEBUGFS -c -R \\\"stat oi.16.${IDX}\\\" $MDT_DEV" |
+                    grep Blockcount | awk '{print $4}')
+
+       [ $blk1 == $blk2 ] || error "old blk1=$blk1, new blk2=$blk2, unmatched!"
+}
+run_test 228b "idle OI blocks can be reused after MDT restart"
+
+#LU-1881
+test_228c() {
+       [ "$(facet_fstype $SINGLEMDS)" != "ldiskfs" ] &&
+               skip "non-ldiskfs backend" && return
+
+       local MDT_DEV=$(mdsdevname ${SINGLEMDS//mds/})
+       local myDIR=$DIR/$tdir
+
+       mkdir -p $myDIR
+       #define OBD_FAIL_SEQ_EXHAUST             0x1002
+       $LCTL set_param fail_loc=0x80001002
+       # 20000 files can guarantee there are index nodes in the OI file
+       createmany -o $myDIR/t- 20000
+       $LCTL set_param fail_loc=0
+       # The guard is current the largest FID holder
+       touch $myDIR/guard
+       local SEQ=$($LFS path2fid $myDIR/guard | awk -F ':' '{print $1}' |
+                   tr -d '[')
+       local IDX=$(($SEQ % 64))
+
+       do_facet $SINGLEMDS sync
+       # Make sure journal flushed.
+       sleep 6
+       local blk1=$(do_facet $SINGLEMDS \
+                    "$DEBUGFS -c -R \\\"stat oi.16.${IDX}\\\" $MDT_DEV" |
+                    grep Blockcount | awk '{print $4}')
+
+       # Remove old files, some OI blocks will become idle.
+       unlinkmany $myDIR/t- 20000
+       rm -f $myDIR/guard
+       # The OI file should become empty now
+
+       # Create new files, idle OI blocks should be reused.
+       createmany -o $myDIR/t- 2000
+       do_facet $SINGLEMDS sync
+       # Make sure journal flushed.
+       sleep 6
+       local blk2=$(do_facet $SINGLEMDS \
+                    "$DEBUGFS -c -R \\\"stat oi.16.${IDX}\\\" $MDT_DEV" |
+                    grep Blockcount | awk '{print $4}')
+
+       [ $blk1 == $blk2 ] || error "old blk1=$blk1, new blk2=$blk2, unmatched!"
+}
+run_test 228c "NOT shrink the last entry in OI index node to recycle idle leaf"
+
 #
 # tests that do cleanup/setup should be run at the end
 #
@@ -9379,12 +9939,12 @@ test_900() {
         for ls in /proc/fs/lustre/ldlm/namespaces/MGC*/lru_size; do
                 echo "clear" > $ls
         done
-        FAIL_ON_ERROR=true cleanup
-        FAIL_ON_ERROR=true setup
+       FAIL_ON_ERROR=true cleanup
+       FAIL_ON_ERROR=true setup
 }
 run_test 900 "umount should not race with any mgc requeue thread"
 
-complete $(basename $0) $SECONDS
+complete $SECONDS
 check_and_cleanup_lustre
 if [ "$I_MOUNTED" != "yes" ]; then
        lctl set_param debug="$OLDDEBUG" 2> /dev/null || true