Revert "NFS: Ensure that writeback_single_inode() calls write_inode() when syncing"
commitb04a57568d86586d704babc7584e5fff4343fc61
authorTrond Myklebust <Trond.Myklebust@netapp.com>
Tue, 18 Oct 2011 01:22:55 +0000 (17 18:22 -0700)
committerGreg Kroah-Hartman <gregkh@suse.de>
Fri, 11 Nov 2011 17:35:49 +0000 (11 09:35 -0800)
tree87731a45f3bc4e34cf8e4689a8ed022e9d5f6697
parent3cdf240310b4f3d94981d727d3660c78a50a40c6
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