Re: [PATCH v12 2/6] generic/632: add fstests for idmapped mounts

Eryu Guan
2021-04-11T14:30:24Z

On Mon, Mar 29, 2021 at 12:33:56AM +0200, Christian Brauner wrote:
> From: Christian Brauner <christian.brauner@ubuntu.com>
>
> Add a test suite to verify the behavior of idmapped mounts. The test
> suite also includes a range of vfs tests to verify that no regressions
> are introduced by idmapped mounts. The following tests are currently
> available with more to come in the future:
>
> 01. posix acls on regular and idmapped mounts

I'm getting failures like below when testing on btrfs and overlayfs

[root@fedoravm xfstests]# diff -u
/root/workspace/xfstests/tests/generic/633.out /root/workspace/xfstests/results//btrfs/generic/633.out.bad
--- /root/workspace/xfstests/tests/generic/633.out 2021-04-11 22:18: 24.458518716 +0800
+++ /root/workspace/xfstests/results//btrfs/generic/633.out.bad 2021-04-11 22:19:58.887980770 +0800
@@ -1,2 +1,4 @@
QA output created by 633
Silence is golden
+idmapped-mounts.c: 6622: acls - Invalid argument - failure: sys_mount_setattr
+failure: posix acls on regular mounts

Is this a known issue or test needs some fix?

[snip]

> create mode 100644 tests/generic/632

Please change mode of new test file to 755 if you end up with sending
new version, otherwise I can fix it on commit.

Thanks,
Eryu