Whamcloud - gitweb
LU-13706 tests: remove test 119d 31/50731/2
authorPatrick Farrell <pfarrell@whamcloud.com>
Mon, 24 Apr 2023 21:49:18 +0000 (17:49 -0400)
committerOleg Drokin <green@whamcloud.com>
Mon, 1 May 2023 04:12:00 +0000 (04:12 +0000)
The fail_loc used by test 119d was removed in Lustre 2.0.
The fail_loc tests for a bug which should be obvious - a
serious delay when doing DIO writes - and is definitely
fixed in current versions.  (Bugzilla 15950)

And without the fail_loc, the test isn't doing anything
interesting.  But the timer based aspect of it fails
occasionally due to hardware delays.  So let's just remove
the test.

test-parameters: trivial

Signed-off-by: Patrick Farrell <pfarrell@whamcloud.com>
Change-Id: I2bc18869258e26dad99c72006f55f31315e67bdd
Reviewed-on: https://review.whamcloud.com/c/fs/lustre-release/+/50731
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-by: Alex Zhuravlev <bzzz@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
lustre/include/obd_support.h
lustre/tests/sanity.sh

index 0cdde3b..d190c1a 100644 (file)
@@ -428,7 +428,7 @@ extern char obd_jobid_var[];
 #define OBD_FAIL_OSC_BRW_PREP_REQ2       0x40a
 /* #define OBD_FAIL_OSC_CONNECT_CKSUM       0x40b Obsolete since 2.9 */
 #define OBD_FAIL_OSC_CKSUM_ADLER_ONLY    0x40c
-#define OBD_FAIL_OSC_DIO_PAUSE           0x40d
+/*#define OBD_FAIL_OSC_DIO_PAUSE           0x40d removed in 2.0 */
 #define OBD_FAIL_OSC_OBJECT_CONTENTION   0x40e
 #define OBD_FAIL_OSC_CP_CANCEL_RACE      0x40f
 #define OBD_FAIL_OSC_CP_ENQ_RACE         0x410
index d0fc80a..f58f69f 100755 (executable)
@@ -13269,33 +13269,6 @@ test_119c() # bug 13099
 }
 run_test 119c "Testing for direct read hitting hole"
 
-test_119d() # bug 15950
-{
-       [ $PARALLEL == "yes" ] && skip "skip parallel run"
-
-        MAX_RPCS_IN_FLIGHT=`$LCTL get_param -n osc.*OST0000-osc-[^mM]*.max_rpcs_in_flight`
-        $LCTL set_param -n osc.*OST0000-osc-[^mM]*.max_rpcs_in_flight 1
-        BSIZE=1048576
-        $LFS setstripe $DIR/$tfile -i 0 -c 1 || error "setstripe failed"
-        $DIRECTIO write $DIR/$tfile 0 1 $BSIZE || error "first directio failed"
-        #define OBD_FAIL_OSC_DIO_PAUSE           0x40d
-        lctl set_param fail_loc=0x40d
-        $DIRECTIO write $DIR/$tfile 1 4 $BSIZE &
-        pid_dio=$!
-        sleep 1
-        cat $DIR/$tfile > /dev/null &
-        lctl set_param fail_loc=0
-        pid_reads=$!
-        wait $pid_dio
-        log "the DIO writes have completed, now wait for the reads (should not block very long)"
-        sleep 2
-        [ -n "`ps h -p $pid_reads -o comm`" ] && \
-        error "the read rpcs have not completed in 2s"
-        rm -f $DIR/$tfile
-        $LCTL set_param -n osc.*OST0000-osc-[^mM]*.max_rpcs_in_flight $MAX_RPCS_IN_FLIGHT
-}
-run_test 119d "The DIO path should try to send a new rpc once one is completed"
-
 test_120a() {
        [ $PARALLEL == "yes" ] && skip "skip parallel run"
        remote_mds_nodsh && skip "remote MDS with nodsh"