]> git.apps.os.sepia.ceph.com Git - xfstests-dev.git/commitdiff
common/attr: adbjust acl_max of f2fs
authorSun Ke <sunke32@huawei.com>
Tue, 8 Feb 2022 07:16:24 +0000 (15:16 +0800)
committerEryu Guan <guaneryu@gmail.com>
Sun, 20 Feb 2022 16:01:07 +0000 (00:01 +0800)
f2fs has set inline_xattr as a default option, and introduced a new
option named 'noinline_xattr' for disabling default inline_xattr
option.  So in _acl_get_max we need to check 'noinline_xattr' string
in fs option, otherwise we may select the wrong max acl number since
we always found the string 'inline_xattr' in fs option.

Additionally, f2fs has changed disk layout of xattr block a bit, so
will contain one more entry in both inline and noinline xattr inode,
this patch will modify the max acl number to adjust it.

Suggested-by: Chao Yu <chao@kernel.org>
Signed-off-by: Sun Ke <sunke32@huawei.com>
Reviewed-by: Eryu Guan <guaneryu@gmail.com>
Signed-off-by: Eryu Guan <guaneryu@gmail.com>
common/attr

index 35682d7c56cdf7af91b488fcf60853ee93d4e0de..dae8a1bb0824a153c2b4945621ccf6f2a69371a9 100644 (file)
@@ -26,11 +26,24 @@ _acl_get_max()
                echo 8191
                ;;
        f2fs)
-               _fs_options $TEST_DEV | grep "inline_xattr" >/dev/null 2>&1
+               # If noinline_xattr is enabled, max xattr size should be:
+               # (4096 - 24) - (24 + 4) = 4044
+               # then ACL_MAX_ENTRIES should be:
+               # (4044 - (4 + 4 * 4)) / 8 + 4 = 507
+               _fs_options $TEST_DEV | grep "noinline_xattr" >/dev/null 2>&1
                if [ $? -eq 0 ]; then
-                       echo 531
+                       echo 507
                else
-                       echo 506
+                       # If inline_xattr is enabled, max xattr size should be:
+                       # (4096 - 24 + 200) - (24 + 4) = 4244
+                       # then ACL_MAX_ENTRIES should be:
+                       # (4244 - (4 + 4 * 4)) / 8 + 4 = 532
+                       _fs_options $TEST_DEV | grep "inline_xattr" >/dev/null 2>&1
+                       if [ $? -eq 0 ]; then
+                               echo 532
+                       else
+                               echo 507
+                       fi
                fi
                ;;
        bcachefs)