[PATCH 4/4] [PATCH 5/5] xfs: fix xfs_quiesce_data
Dave Chinner
david at fromorbit.com
Sun Sep 13 19:37:39 CDT 2009
On Fri, Aug 28, 2009 at 06:18:45PM -0500, Alex Elder wrote:
> Christoph Hellwig wrote:
> > From: Dave Chinner <david at fromorbit.com>
> >
> > We need to do a synchronous xfs_sync_fsdata to make sure the superblock
> > actually is on disk when we return.
> >
> > Also remove SYNC_BDFLUSH flag to xfs_sync_inodes because that particular
> > flag is never checked.
> >
> > Move xfs_filestream_flush call later [hch: why? seems unrelated].
>
> I concur with your question. Why not release the inode references early?
Because if you drop the filestreams references before you write the
data, the data doesn't get put where the filestreams allocator
decided it should go when it created the reference....
Cheers,
Dave.
--
Dave Chinner
david at fromorbit.com
More information about the xfs
mailing list