From: André Almeida <andrealmeid@igalia.com>
Date: Wed, 21 Aug 2024 15:57:46 +0000 (-0300)
Subject: common/config: Correctly ignore {TEST|SCRATCH}_DEV for tmpfs
X-Git-Tag: v2024.08.25~2
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=5381898ebaef1fef801dc83770521bb1c28eaf78;p=xfstests-dev.git

common/config: Correctly ignore {TEST|SCRATCH}_DEV for tmpfs

As per commit 264e5358e2c2 ("tmpfs: don't require {TEST|SCRATCH}_DEV"),
tmpfs doesn't need TEST or SCRATCH devices due to being a RAM-based
filesystem.

Fix the check by comparing the content of the variable TEST_DEV, instead
of comparing with the string TEST_DEV. Same for SCRATCH_DEV.

Fixes: 264e5358e2c2 ("tmpfs: don't require {TEST|SCRATCH}_DEV")
Signed-off-by: André Almeida <andrealmeid@igalia.com>
Reviewed-by: Darrick J. Wong <djwong@kernel.org>
Signed-off-by: Zorro Lang <zlang@kernel.org>
---

diff --git a/common/config b/common/config
index 22740c0a..307f93fb 100644
--- a/common/config
+++ b/common/config
@@ -829,10 +829,10 @@ get_next_config() {
 	fi
 
 	if [ "$FSTYP" == "tmpfs" ]; then
-		if [ -z "TEST_DEV" ]; then
+		if [ -z "$TEST_DEV" ]; then
 			export TEST_DEV=tmpfs_test
 		fi
-		if [ -z "SCRATCH_DEV" ]; then
+		if [ -z "$SCRATCH_DEV" ]; then
 			export TEST_DEV=tmpfs_scratch
 		fi
 	fi