On Tue, Jul 27, 2021 at 4:05 PM Bob Peterson <rpete...@redhat.com> wrote: > On 7/27/21 2:05 AM, Christoph Hellwig wrote: > > On Mon, Jul 26, 2021 at 01:50:11PM -0500, Bob Peterson wrote: > >> FYI: I just pushed a replacement patch to linux-gfs2/for-next.next4. > >> The patch I mentioned last time had problems, so this is a simplified > >> middle-ground between that patch and the one I suggested earlier. > >> Patch is a89a427e0afe. > > > > With just that patch cherry picked the test still hangs. > > > > The entire linux-gfs2/for-next.next4 completes a quick group xfstests > > run, but with a lot of failures: > > > > Failures: generic/079 generic/082 generic/092 generic/094 generic/103 > > generic/219 generic/230 generic/235 generic/244 generic/294 generic/306 > > generic/347 generic/379 generic/380 generic/382 generic/383 generic/384 > > generic/385 generic/386 generic/400 generic/441 generic/452 generic/488 > > generic/545 generic/566 generic/587 generic/594 generic/600 generic/601 > > generic/603 > > Failed 30 of 514 tests > > > > Hi Christoph, > > Thanks for testing gfs2. > Andreas and I are fully aware of these failures. When I do a full run > (which I do almost every day), I get these same 9 failures that you also > reported: > > generic/294 > generic/306 > generic/441 > generic/452 > generic/488 > generic/545 > generic/594 > generic/619 > generic/631 > > I've not done a detailed analysis of these 9 lately, but in the past > I've looked at individual failures and they're usually because gfs2 does > a "file system withdraw" due to IO errors writing to its journal (and > such--as we want and expect) whereas most file systems don't do that. So > these are kind of expected failures. Perhaps I should go back check them > again. > > The others you reported are are exempted from my testing (I've got them > in my excludes file) because in most cases the problem is gfs2's lack of > support for the usrquota / grpquota / prjquota mount options: > > generic/082 - Lack of support for -o usrquota / grpquota / prjquota > generic/219 > generic/230 > generic/235 > generic/244 > generic/379 > generic/380 > generic/382 > generic/383 > generic/384 > generic/385 > generic/386 > generic/400 > generic/566 > generic/587 > generic/600 > generic/601 > generic/603 > > That leaves us: > > generic/079 [expunged] - Incorrect permission check (doesn't match the > one in notify_change)
I've tried fixing that one a long time ago: https://listman.redhat.com/archives/cluster-devel/2018-February/msg00042.html > generic/092 [expunged] - Lack of support for unwritten extents > generic/094 [expunged] - Lack of support for unwritten extents > generic/103 [expunged] - Command having problems: xfs_io -i -fc > falloc 0 66574524k /mnt/scratch/spc > (I haven't taken the time to analyze this yet) > Inconsistent filesystem when running out of space on dm-thin > (I haven't taken the time to analyze this yet either) > > So we know we've still got some work to do here, but we've got higher > priorities at the moment. They're still on our to-do list. > > Regards, > > Bob Peterson Andreas
