]> git.apps.os.sepia.ceph.com Git - xfstests-dev.git/commitdiff
generic/233: use relative path to invoke fsstress
authorChristoph Hellwig <hch@lst.de>
Wed, 16 Apr 2025 04:51:12 +0000 (06:51 +0200)
committerZorro Lang <zlang@kernel.org>
Sat, 26 Apr 2025 12:53:52 +0000 (20:53 +0800)
This test uses su to run fsstress as $qa_user.  Because the shell calling
fsstress is run as $qa_user it might not be able to access the xfstests
or parent directory which might only be accessible to the user xfstests
is run as (e.g. root).  Switch back to calling fsstress using a relative
path, partially reverting the effects of commit c475ff6ff6d7
("common/config: add $here to FSSTRESS_PROG").

Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Darrick J. Wong <djwong@kernel.org>
Signed-off-by: Zorro Lang <zlang@kernel.org>
tests/generic/233

index df67b39092b7cb9858ba8b4215da41eb2b613447..3fc1b63abb2475fdff54f4a72d930ac0601ae75e 100755 (executable)
@@ -42,8 +42,11 @@ _fsstress()
        # io_uring_queue_init fail on ENOMEM, set max locked memory to unlimited
        # temporarily.
        ulimit -l unlimited
+
+       # The absolute path $here might not be traversable (+x) for $qa_user, so
+       # supply a relative path to fsstress here.
        echo "fsstress $args" >> $seqres.full
-       if ! _su $qa_user -c "$FSSTRESS_PROG $args" | tee -a $seqres.full | _filter_num
+       if ! _su $qa_user -c "ltp/fsstress $args" | tee -a $seqres.full | _filter_num
        then
                echo "    fsstress $args returned $?"
                cat $tmp.out | tee -a $seqres.full