Re: TAKE 949916 - direct read vs delalloc

[Nathan Scott <nathans@xxxxxxx>]

On Fri, Mar 17, 2006 at 05:57:19PM +0000, Christoph Hellwig wrote:
> On Fri, Mar 17, 2006 at 04:30:19PM +1100, Nathan Scott wrote:
> > Flush and invalidate dirty pages at the start of a direct read also,
> > else we can hit a delalloc-extents-via-direct-io BUG.
> 
> If this is needed it should be done in the filemap.c code.

Hmm, maybe, maybe not.

filemap.c code doesn't know much about delayed allocation, so
this won't bite other filesystems.  We do the same thing on
write and always have, so having it inside XFS is OK by me.

cheers.

-- 
Nathan