]> git.apps.os.sepia.ceph.com Git - xfstests-dev.git/commitdiff
generic/231: use relative path to invoke fsx
authorChristoph Hellwig <hch@lst.de>
Wed, 16 Apr 2025 04:51:11 +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 fsx as $qa_user.  Because the shell calling
fsx 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 fsx using a relative
path, partially reverting commit 1c67e8b191fe
("config: add FSX_PROG variable").

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/231

index b598a5d568bdf60c54a6959fe363e23e046f1372..ce7e62ea1886938d37cd2eb7df0825585adb5eb0 100755 (executable)
@@ -23,8 +23,10 @@ _fsx()
        echo "=== FSX Standard Mode, Memory Mapping, $tasks Tasks ==="
        for (( i = 1; i <= $tasks; i++ )); do
                SEED=$RANDOM
-               echo "$FSX_PROG $FSX_ARGS -S $SEED $SCRATCH_MNT/fsx_file$i" >>$seqres.full
-               _su $qa_user -c "$FSX_PROG $FSX_ARGS -S $SEED \
+               # The absolute path $here might not be traversable (+x) for
+               # $qa_user, so supply a relative path to fsx here.
+               echo "ltp/fsx $FSX_ARGS -S $SEED $SCRATCH_MNT/fsx_file$i" >>$seqres.full
+               _su $qa_user -c "ltp/fsx $FSX_ARGS -S $SEED \
                        $FSX_AVOID $SCRATCH_MNT/fsx_file$i" >$tmp.output$i 2>&1 &
        done