On Sat, Oct 06, 2018 at 07:34:34PM +1000, Dave Chinner wrote: > I bisected the generic/127 rmap corruption down to this patch: Heh, I just did so too this morning. > The corruption check that is failing is this: > > /* Make sure the unwritten flag matches. */ > XFS_WANT_CORRUPTED_GOTO(mp, (flags & XFS_RMAP_UNWRITTEN) == > (rec->rm_flags & XFS_RMAP_UNWRITTEN), out); > > So this patch does not appear to be doing the right thing with > unwritten extent flagging in some case. I suspect the asserts actually are what is incorrect. But given how late we are in the cycle I've just dropped the patch and kicked off xfstests runs (now including rmap, sigh..).