[PATCH 3/1] xfstests: generic/062: Do not run on newer kernels

From: Vivek Goyal
Date: Thu Sep 02 2021 - 11:47:41 EST



xfstests: generic/062: Do not run on newer kernels

This test has been written with assumption that setting user.* xattrs will
fail on symlink and special files. When newer kernels support setting
user.* xattrs on symlink and special files, this test starts failing.

Found it hard to change test in such a way that it works on both type of
kernels. Primary problem is 062.out file which hardcodes the output and
output will be different on old and new kernels.

So instead, do not run this test if kernel is new and is expected to
exhibit new behavior. Next patch will create a new test and run that
test on new kernel.

IOW, on old kernels run 062 and on new kernels run new test.

This is a proposed patch. Will need to be fixed if corresponding
kernel changes are merged upstream.

Signed-off-by: Vivek Goyal <vgoyal@xxxxxxxxxx>
---
tests/generic/062 | 20 ++++++++++++++++++++
1 file changed, 20 insertions(+)

Index: xfstests-dev/tests/generic/062
===================================================================
--- xfstests-dev.orig/tests/generic/062 2021-08-31 15:51:08.160307982 -0400
+++ xfstests-dev/tests/generic/062 2021-08-31 16:27:41.678307982 -0400
@@ -55,6 +55,26 @@ _require_attrs
_require_symlinks
_require_mknod

+user_xattr_allowed()
+{
+ local kernel_version kernel_patchlevel
+
+ kernel_version=`uname -r | awk -F. '{print $1}'`
+ kernel_patchlevel=`uname -r | awk -F. '{print $2}'`
+
+ # Kernel version 5.14 onwards allow user xattr on symlink/special files.
+ [ $kernel_version -lt 5 ] && return 1
+ [ $kernel_patchlevel -lt 14 ] && return 1
+ return 0;
+}
+
+
+# Kernel version 5.14 onwards allow user xattr on symlink/special files.
+# Do not run this test on newer kernels. Instead run the new test
+# which has been written with the assumption that user.* xattr
+# will succeed on symlink and special files.
+user_xattr_allowed && _notrun "Kernel allows user.* xattrs on symlinks and special files. Skipping this test. Run newer test instead."
+
rm -f $tmp.backup1 $tmp.backup2 $seqres.full

# real QA test starts here