This patchset is pretty much identical to the one that I sent last week. It has one minor fix -- I fixed it to use nfs_free_fattr instead of kfree in nfs_async_rename_release. I also cleaned up the comments a bit. We had a bug report recently where someone was complaining about silly-renamed files being left around: https://bugzilla.redhat.com/show_bug.cgi?id=511901 ...they attached a reproducer to the bug which involves a pthreads-based program killing a child thread that's unlinking and closing a file. The unlink triggers a sillyrename (since the file is still open). The parent kills the child thread and if timed just right, the child thread will be killed after the RENAME call is issued but before the reply is processed. The file will end up renamed on the server, but the client won't be aware of it and won't unlink it during dentry_iput. This patchset is intended to prevent this from happening by having nfs_sillyrename use an asynchronous rename call and simply wait for the call to complete in TASK_KILLABLE sleep before proceeding. If the task is killed while the rename RPC is on the wire, it'll still run to completion even though the thread that initiated it was killed off. To facilitate this, the set first changes all of the existing rename calls to use standardized argument and response containers. The nfs_sillyrename call is then moved to unlink.c (where the rest of the sillyrename code lives), and then is converted to use an asynchronous RPC call to handle the rename. I've tested this by running the reproducer in the above bug report in a loop for several hours and never ended up with a leftover .nfs* file. I'd like to see this in 2.6.37 if possible. Jeff Layton (8): nfs: eliminate nfs3_renameargs nfs: convert nfs_renameargs to use qstr structs nfs: eliminate nfs4_rename_arg nfs: standardize the rename response container nfs: move nfs_sillyrename to unlink.c nfs: add a rename_setup nfs_rpc_op for async renames nfs: add rename_done nfs_rpc_op nfs: make sillyrename an async operation fs/nfs/dir.c | 70 ------------- fs/nfs/nfs2xdr.c | 8 +- fs/nfs/nfs3proc.c | 62 +++++++++--- fs/nfs/nfs3xdr.c | 16 ++-- fs/nfs/nfs4proc.c | 43 ++++++++- fs/nfs/nfs4xdr.c | 4 +- fs/nfs/proc.c | 34 +++++- fs/nfs/unlink.c | 253 ++++++++++++++++++++++++++++++++++++++++++++++- include/linux/nfs_fs.h | 2 +- include/linux/nfs_xdr.h | 64 +++++-------- 10 files changed, 406 insertions(+), 150 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
