On Thu, 2013-07-18 at 15:35 -0400, andros@xxxxxxxxxx wrote:
> From: Andy Adamson <andros@xxxxxxxxxx>
>
> Signed-off-by: Andy Adamson <andros@xxxxxxxxxx>
> ---
> fs/nfs/write.c | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
> diff --git a/fs/nfs/write.c b/fs/nfs/write.c
> index f1bdb72..724c845 100644
> --- a/fs/nfs/write.c
> +++ b/fs/nfs/write.c
> @@ -1330,6 +1330,14 @@ void nfs_writeback_done(struct rpc_task *task, struct nfs_write_data *data)
> dprintk("NFS: %5u nfs_writeback_done (status %d)\n",
> task->tk_pid, task->tk_status);
>
> + if (task->tk_status == -EKEYEXPIRED)
> + pr_warn("NFS: write attempt with expired credential "
> + "req %s/%lld, %u bytes @ offset %llu\n",
> + inode->i_sb->s_id,
> + (long long)NFS_FILEID(inode),
> + data->args.count,
> + (unsigned long long)data->args.offset);
This needs to be ratelimited to avoid trouble with syslog.
> +
> /*
> * ->write_done will attempt to use post-op attributes to detect
> * conflicting writes by other clients. A strict interpretation
> @@ -1556,6 +1564,8 @@ static void nfs_commit_done(struct rpc_task *task, void *calldata)
>
> dprintk("NFS: %5u nfs_commit_done (status %d)\n",
> task->tk_pid, task->tk_status);
> + if (task->tk_status == -EKEYEXPIRED)
> + pr_warn("NFS: commit attempt with expired credential\n");
Again, this wants to be rate limited.
>
> /* Call the NFS version-specific code */
> NFS_PROTO(data->inode)->commit_done(task, data);
--
Trond Myklebust
Linux NFS client maintainer
NetApp
Trond.Myklebust@xxxxxxxxxx
www.netapp.com
��.n��������+%������w��{.n�����{��w��� jg��� �����ݢj����G���� ���j:+v���w�m������w�������h�����٥
