Skip to contentSkip to conversation

overlay/061: remove from auto and quick groups

PATCH In review Amir Goldstein 2 participants 3 messages Jun 18, 2019 – Jun 23, 2019

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 overlay/061: remove from auto and quick groups

In late 2016, tests overlay/01[6-8] where merged to track the state of several ilong standing posix compliance issues on overlayfs.

This practice was somewhat of an exception for xfstests project, which more often merges tests for issues that are expected to be resolved in the short term.

Over the years, some test cases have been fixed and more tests where created to cover the remaining issues (e.g. overlay/04[34]).

Currently, the only failing test from this category is overlay/061 which covers item b) in the "Non-standard behavior" section of Documentation/filesystems/overlayfs.txt.

Since there is no clear design, roadmap nor allocated resources to resolve this remaining issue, the test is removed from the "auto" and "quick" groups, following a suggestion that Darrick has made for a similarly long failing generic test.

The test was added to the new "posix" group, to allow testing for posix compliance.

---

Eryu,

With this change, check -overlay -g overlay/quick run is expdected to be all green.

I did not observe any regressing with check -overlay -g generic/quick compared to check -g generic/quick on xfs with recent kernel. I did not test all filesystems and -g generic/auto with recent kernel, but I am not aware of any expected failures specific for -overlay run on generic tests.

check -overlay -g overlay/auto has one non-quick stress test (overlay/019) which is failing on lockdep circular locking dependency warning (if you have lockdep enabled).
This is a known issue that has also been reported by syzbot.

Thanks,
Amir.

 tests/overlay/group | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
tests/overlay/group +1 -1
index 8bde6ea1..ef8517a1 100644
@@ -63,7 +63,7 @@
 058 auto quick exportfs
 059 auto quick copyup
 060 auto quick metacopy
-061 auto quick copyup
+061 posix copyup
 062 auto quick exportfs
 063 auto quick whiteout
 064 auto quick copyup
Signature · 1 line
2.17.1
#1 lore raw
Quoted 7 lines

I did not observe any regressing with check -overlay -g generic/quick compared to check -g generic/quick on xfs with recent kernel. I did not test all filesystems and -g generic/auto with recent kernel, but I am not aware of any expected failures specific for -overlay run on generic tests.

Oh! I was lying. I *did* notice two regressions with check -overlay -g generic/quick (compared to xfs), so I posted fixes for them:

1) generic/504 is failing with -overlay on master.

This kernel fix commit is on linux-next:
6ef048fd5955 locks: eliminate false positive conflicts for write lease

2) generic/555 (was just merged this week) is failing with -overlay on master.

This kernel fix commit is on Miklos' overlayfs-next:
941d935ac763 ovl: fix wrong flags check in FS_IOC_FS[SG]ETXATTR ioctls

Thanks,
Amir.

#2 lore raw
Quoted 14 lines · Amir Goldstein
Quoted 7 lines

I did not observe any regressing with check -overlay -g generic/quick compared to check -g generic/quick on xfs with recent kernel. I did not test all filesystems and -g generic/auto with recent kernel, but I am not aware of any expected failures specific for -overlay run on generic tests.

Oh! I was lying. I *did* notice two regressions with check -overlay -g generic/quick (compared to xfs), so I posted fixes for them:

1) generic/504 is failing with -overlay on master.

Quoted 8 lines

This kernel fix commit is on linux-next:
6ef048fd5955 locks: eliminate false positive conflicts for write lease

2) generic/555 (was just merged this week) is failing with -overlay on master.

This kernel fix commit is on Miklos' overlayfs-next:
941d935ac763 ovl: fix wrong flags check in FS_IOC_FS[SG]ETXATTR ioctls

Thanks a lot for the detailed information!

Eryu