xfs
[Top] [All Lists]

Re: xfs_iext_realloc_indirect and "XFS: possible memory allocation deadl

To: Dave Chinner <david@xxxxxxxxxxxxx>
Subject: Re: xfs_iext_realloc_indirect and "XFS: possible memory allocation deadlock"
From: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Date: Tue, 7 Jul 2015 02:05:11 -0700
Cc: Alex Lyakas <alex@xxxxxxxxxxxxxxxxx>, Danny Shavit <danny@xxxxxxxxxxxxxxxxx>, bfoster@xxxxxxxxxx, Yair Hershko <yair@xxxxxxxxxxxxxxxxx>, Shyam Kaushik <shyam@xxxxxxxxxxxxxxxxx>, xfs@xxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <20150707000911.GT7943@dastard>
References: <20150629222651.GG7943@dastard> <F1A27B6A9FB048488A388EC4D9FD71F7@alyakaslap> <20150707000911.GT7943@dastard>
User-agent: Mutt/1.5.23 (2014-03-12)
On Tue, Jul 07, 2015 at 10:09:11AM +1000, Dave Chinner wrote:
> server crash. i.e. the client side commit is an "fsync" to the
> server, and until the server responds with a success to the client
> commit RPC the client side will continue to retry sending the data
> to the server.
> 
> For the persepctive of metadata (i.e. directory entries) the use of
> the "dirsync" mount option is sufficient for HA failover servers to
> work correctly as it ensures that directory structure changes are always
> committed to disk before the RPC response is sent back to the
> client.
> 
> i.e. the "sync" mount option doesn't actually improve data integrity
> of an NFS server when you look at the end-to-end NFS protocol
> handling of async write data....


You don't need dirsync either.  NFS does the right sync usin the
commit_metadata export operation without using that big hammer.

<Prev in Thread] Current Thread [Next in Thread>