[PATCH] xfs: do not discard page cache data on EAGAIN

Dave Chinner david at fromorbit.com
Mon Aug 23 07:11:22 CDT 2010


On Mon, Aug 23, 2010 at 05:10:15AM -0400, Christoph Hellwig wrote:
> If xfs_map_blocks returns EAGAIN because of lock contention we must redirty
> the page and not disard the pagecache content and return an error from
> writepage.  We used to do this correctly, but the logic got lost during
> the recent reshuffle of the writepage code.
> 
> Signed-off-by: Christoph Hellwig <hch at lst.de>
> Reported-by: Mike Gao <ygao.linux at gmail.com> 
> Tested-by: Mike Gao <ygao.linux at gmail.com> 

Looks good.

Reviewed-by: Dave Chinner <dchinner at redhat.com>

Cheers,

Dave.
-- 
Dave Chinner
david at fromorbit.com




More information about the xfs mailing list