On Thu, Jun 04, 2009 at 07:45:12PM +1000, Dave Chinner wrote:
> On Mon, May 11, 2009 at 04:15:11PM -0400, Christoph Hellwig wrote:
> > On Sun, May 10, 2009 at 01:37:58PM -0500, Eric Sandeen wrote:
> > > I guess I too would like a reason for the filestream_flush move
> > > somewhere in the changelog...
> > Hehe, maybe Dave can chime in. Or I can test if it actually affects
> > anything and maybe move it out to another patch. The lack of
> > reliability of the filesystreams tests doesn't make this any easier to
> > test.
> (Just catching up on my mail backlog)
> I think that the filestream_flush() call should actually be after
> the data flush. filestream_flush() is used to clear the filestream
> association cache which holds references to the inodes.
> Where the flush is currently placed is destroying the association
> that defines the AG the data should be written to before the data is
> written. As a result it may not end up in the AG carefully
> associated with the inode during the write() syscall.
> This may be one of the reasons for the filestreams tests failing
Makes sense. I stil get reliable failures on 171 and 172, but the
others seem to pass with these changes.