Re: [PATCH] jfs: flush journal completely before releasing metadata inodes

[Christoph Hellwig <hch@xxxxxxxxxxxxx>]

On Mon, Aug 01, 2011 at 01:29:26PM -0500, Dave Kleikamp wrote:
> On 07/29/2011 11:03 AM, Dave Kleikamp wrote:
> > On 07/29/2011 10:50 AM, Christoph Hellwig wrote:
> >> The patch below adds basic jfs support to xfstests.  Unfortunately it
> >> doesn't get very far, as test 001 already blows up with a kernel crash,
> >> as do 011 and 013 as other random picks.
> > 
> > Great :-)
> > 
> > Seems to be a good idea to support jfs. I'll carve out some time to
> > reproduce and debug the crashes.
> 
> This fixes a race during unmount. We need to not only make sure that
> the journal is completely written, but that the metadata changes make
> it to disk before releasing ipimap and ipbmap.

Looks sensible.  How far does a ./check -g auto get now?  Any other
failures?