diff --git a/lustre/tests/sanity.sh b/lustre/tests/sanity.sh index 7e6fc3e5c986bd808e4fce1a0d6ed0f7302cf7b4..223389bfec27ed6d7cf8347f536a97116c6a8e7b 100644 --- a/lustre/tests/sanity.sh +++ b/lustre/tests/sanity.sh @@ -5269,10 +5269,18 @@ test_129() { } run_test 129 "test directory size limit ========================" +OLDIFS="$IFS" +cleanup_130() { + trap 0 + IFS="$OLDIFS" +} + test_130a() { filefrag_op=$(filefrag -e 2>&1 | grep "invalid option") [ -n "$filefrag_op" ] && skip "filefrag does not support FIEMAP" && return + trap cleanup_130 EXIT RETURN + local fm_file=$DIR/$tfile lfs setstripe -s 65536 -c 1 $fm_file || error "setstripe failed on $fm_file" dd if=/dev/zero of=$fm_file bs=65536 count=1 || error "dd failed for $fm_file" @@ -5290,6 +5298,7 @@ test_130a() { frag_lun=`echo $line | cut -d: -f5` ext_len=`echo $line | cut -d: -f4` if (( $frag_lun != $lun )); then + cleanup_130 error "FIEMAP on 1-stripe file($fm_file) failed" return fi @@ -5297,9 +5306,13 @@ test_130a() { done if (( lun != frag_lun || start_blk != 0 || tot_len != 64 )); then + cleanup_130 error "FIEMAP on 1-stripe file($fm_file) failed;" return fi + + cleanup_130 + echo "FIEMAP on single striped file succeeded" } run_test 130a "FIEMAP (1-stripe file)" @@ -5310,6 +5323,8 @@ test_130b() { filefrag_op=$(filefrag -e 2>&1 | grep "invalid option") [ -n "$filefrag_op" ] && skip "filefrag does not support FIEMAP" && return + trap cleanup_130 EXIT RETURN + local fm_file=$DIR/$tfile lfs setstripe -s 65536 -c 2 $fm_file || error "setstripe failed on $fm_file" dd if=/dev/zero of=$fm_file bs=1M count=2 || error "dd failed on $fm_file" @@ -5328,6 +5343,7 @@ test_130b() { ext_len=`echo $line | cut -d: -f4` if (( $frag_lun != $last_lun )); then if (( tot_len != 1024 )); then + cleanup_130 error "FIEMAP on $fm_file failed; returned len $tot_len for OST $last_lun instead of 256" return else @@ -5339,10 +5355,13 @@ test_130b() { last_lun=$frag_lun done if (( num_luns != 2 || tot_len != 1024 )); then + cleanup_130 error "FIEMAP on $fm_file failed; returned wrong number of luns or wrong len for OST $last_lun" return fi + cleanup_130 + echo "FIEMAP on 2-stripe file succeeded" } run_test 130b "FIEMAP (2-stripe file)" @@ -5353,6 +5372,8 @@ test_130c() { filefrag_op=$(filefrag -e 2>&1 | grep "invalid option") [ -n "$filefrag_op" ] && skip "filefrag does not support FIEMAP" && return + trap cleanup_130 EXIT RETURN + local fm_file=$DIR/$tfile lfs setstripe -s 65536 -c 2 $fm_file || error "setstripe failed on $fm_file" dd if=/dev/zero of=$fm_file seek=1 bs=1M count=1 || error "dd failed on $fm_file" @@ -5372,10 +5393,12 @@ test_130c() { if (( $frag_lun != $last_lun )); then logical=`echo $line | cut -d: -f2 | cut -d. -f1` if (( logical != 512 )); then + cleanup_130 error "FIEMAP on $fm_file failed; returned logical start for lun $logical instead of 512" return fi if (( tot_len != 512 )); then + cleanup_130 error "FIEMAP on $fm_file failed; returned len $tot_len for OST $last_lun instead of 1024" return else @@ -5387,10 +5410,13 @@ test_130c() { last_lun=$frag_lun done if (( num_luns != 2 || tot_len != 512 )); then + cleanup_130 error "FIEMAP on $fm_file failed; returned wrong number of luns or wrong len for OST $last_lun" return fi + cleanup_130 + echo "FIEMAP on 2-stripe file with hole succeeded" } run_test 130c "FIEMAP (2-stripe file with hole)" @@ -5401,6 +5427,8 @@ test_130d() { filefrag_op=$(filefrag -e 2>&1 | grep "invalid option") [ -n "$filefrag_op" ] && skip "filefrag does not support FIEMAP" && return + trap cleanup_130 EXIT RETURN + local fm_file=$DIR/$tfile lfs setstripe -s 65536 -c $OSTCOUNT $fm_file || error "setstripe failed on $fm_file" dd if=/dev/zero of=$fm_file bs=1M count=$OSTCOUNT || error "dd failed on $fm_file" @@ -5419,6 +5447,7 @@ test_130d() { ext_len=`echo $line | cut -d: -f4` if (( $frag_lun != $last_lun )); then if (( tot_len != 1024 )); then + cleanup_130 error "FIEMAP on $fm_file failed; returned len $tot_len for OST $last_lun instead of 1024" return else @@ -5430,10 +5459,13 @@ test_130d() { last_lun=$frag_lun done if (( num_luns != OSTCOUNT || tot_len != 1024 )); then + cleanup_130 error "FIEMAP on $fm_file failed; returned wrong number of luns or wrong len for OST $last_lun" return fi + cleanup_130 + echo "FIEMAP on N-stripe file succeeded" } run_test 130d "FIEMAP (N-stripe file)" @@ -5444,6 +5476,8 @@ test_130e() { filefrag_op=$(filefrag -e 2>&1 | grep "invalid option") [ -n "$filefrag_op" ] && skip "filefrag does not support FIEMAP" && return + trap cleanup_130 EXIT RETURN + local fm_file=$DIR/$tfile lfs setstripe -s 65536 -c 2 $fm_file || error "setstripe failed on $fm_file" NUM_BLKS=512 @@ -5467,6 +5501,7 @@ test_130e() { ext_len=`echo $line | cut -d: -f4` if (( $frag_lun != $last_lun )); then if (( tot_len != $EXPECTED_LEN )); then + cleanup_130 error "FIEMAP on $fm_file failed; returned len $tot_len for OST $last_lun instead of $EXPECTED_LEN" return else @@ -5478,11 +5513,13 @@ test_130e() { last_lun=$frag_lun done if (( num_luns != 2 || tot_len != $EXPECTED_LEN )); then - echo "$num_luns $tot_len" + cleanup_130 error "FIEMAP on $fm_file failed; returned wrong number of luns or wrong len for OST $last_lun" return fi + cleanup_130 + echo "FIEMAP with continuation calls succeeded" } run_test 130e "FIEMAP (test continuation FIEMAP calls)"