Revert "NFS: Ensure that writeback_single_inode() calls write_inode() when syncing"
authorTrond Myklebust <Trond.Myklebust@netapp.com>
Tue, 18 Oct 2011 01:22:55 +0000 (18:22 -0700)
committerGreg Kroah-Hartman <gregkh@suse.de>
Fri, 11 Nov 2011 17:43:02 +0000 (09:43 -0800)
commitddd5310cb95cac1f698d3f9e35c454e32699b81d
treeb00e34565b26279c579e2aa79be4857e73ef9255
parent112e8e4019fdf3fc987cf0f839a2bfbd6c169e60
Revert "NFS: Ensure that writeback_single_inode() calls write_inode() when syncing"

commit 59b7c05fffba030e5d9e72324691e2f99aa69b79 upstream.

This reverts commit b80c3cb628f0ebc241b02e38dd028969fb8026a2.

The reverted commit was rendered obsolete by a VFS fix: commit
5547e8aac6f71505d621a612de2fca0dd988b439 (writeback: Update dirty flags in
two steps). We now no longer need to worry about writeback_single_inode()
missing our marking the inode for COMMIT in 'do_writepages()' call.

Reverting this patch, fixes a performance regression in which the inode
would continuously get queued to the dirty list, causing the writeback
code to unnecessarily try to send a COMMIT.

Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Tested-by: Simon Kirby <sim@hostway.ca>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
fs/nfs/write.c