Andy, I've fixed this already in my tree, but with additionaly
checking on nfs4_sequence_done's status:
if (!nfs4_sequence_done(task, &res))
return 0;
Benny
On Jul. 16, 2010, 18:56 +0300, andros@xxxxxxxxxx wrote:
> From: Andy Adamson <andros@xxxxxxxxxx>
>
> Signed-off-by: Andy Adamson <andros@xxxxxxxxxx>
> ---
> fs/nfs/nfs4proc.c | 6 +++---
> 1 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/fs/nfs/nfs4proc.c b/fs/nfs/nfs4proc.c
> index f5779a1..17b0c31 100644
> --- a/fs/nfs/nfs4proc.c
> +++ b/fs/nfs/nfs4proc.c
> @@ -5448,7 +5448,7 @@ static void nfs4_pnfs_layoutget_done(struct rpc_task *task, void *calldata)
>
> dprintk("--> %s\n", __func__);
>
> - nfs4_sequence_done(server, &lgp->res.seq_res, task->tk_status);
> + nfs4_sequence_done(task, &lgp->res.seq_res);
>
> if (RPC_ASSASSINATED(task))
> return;
> @@ -5558,7 +5558,7 @@ pnfs_layoutcommit_done(struct rpc_task *task, void *calldata)
> (struct pnfs_layoutcommit_data *)calldata;
> struct nfs_server *server = NFS_SERVER(data->args.inode);
>
> - nfs4_sequence_done(server, &data->res.seq_res, task->tk_status);
> + nfs4_sequence_done(task, &data->res.seq_res);
> if (RPC_ASSASSINATED(task))
> return;
>
> @@ -5666,7 +5666,7 @@ static void nfs4_pnfs_layoutreturn_done(struct rpc_task *task, void *calldata)
>
> dprintk("--> %s\n", __func__);
>
> - nfs4_sequence_done(server, &lrp->res.seq_res, task->tk_status);
> + nfs4_sequence_done(task, &lrp->res.seq_res);
> if (RPC_ASSASSINATED(task))
> return;
>
--
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
