NFSv4.1 skip rpc_call_done only on disconnected DS slot_table_waitq tasks
authorAndy Adamson <andros@netapp.com>
Tue, 22 May 2012 12:09:27 +0000 (08:09 -0400)
committerTrond Myklebust <Trond.Myklebust@netapp.com>
Tue, 22 May 2012 20:42:42 +0000 (16:42 -0400)
We reset all I/O on a disconnected data server through the pgio layer indicated
by the NFS_IOHDR_REDO flag.

Differentiate between on-the-wire tasks returning with an error which must
call rpc_call_done and tasks woken from the data server slot_table_waitq
waiting for a session slot with a status of zero which call rpc_exit in
rpc_prepare and need to skip rpc_call_done.

Signed-off-by: Andy Adamson <andros@netapp.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
fs/nfs/nfs4filelayout.c

index 466284793119e963b831477260cf63f2b179dac3..56aa0ec1b16944059841e13e7a893498fdb773de 100644 (file)
@@ -302,7 +302,8 @@ static void filelayout_read_call_done(struct rpc_task *task, void *data)
 
        dprintk("--> %s task->tk_status %d\n", __func__, task->tk_status);
 
-       if (test_bit(NFS_IOHDR_REDO, &rdata->header->flags))
+       if (test_bit(NFS_IOHDR_REDO, &rdata->header->flags) &&
+           task->tk_status == 0)
                return;
 
        /* Note this may cause RPC to be resent */
@@ -399,7 +400,8 @@ static void filelayout_write_call_done(struct rpc_task *task, void *data)
 {
        struct nfs_write_data *wdata = data;
 
-       if (test_bit(NFS_IOHDR_REDO, &wdata->header->flags))
+       if (test_bit(NFS_IOHDR_REDO, &wdata->header->flags) &&
+           task->tk_status == 0)
                return;
 
        /* Note this may cause RPC to be resent */