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)
committerTrond Myklebust <Trond.Myklebust@netapp.com>
Tue, 18 Oct 2011 16:08:09 +0000 (09:08 -0700)
commit59b7c05fffba030e5d9e72324691e2f99aa69b79
tree20736d2dede2244fa76413302db0d89c43b8ee70
parent899e3ee404961a90b828ad527573aaaac39f0ab1
Revert "NFS: Ensure that writeback_single_inode() calls write_inode() when syncing"

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>
Tested-by: Simon Kirby <sim@hostway.ca>
Cc: stable@kernel.org [2.6.35+]
fs/nfs/write.c