On 06/01/2012 08:19 PM, andros@xxxxxxxxxx wrote:
> From: Andy Adamson <andros@xxxxxxxxxx>
>
The tittle does not match at all the message text nor the patch.
More like: Don't send LAYOUTCOMMIT if data is rewritten through MDS
I think.
Boaz
> Clear the inode LAYOUTCOMMIT flag as we will resend writes through the MDS.
>
> Signed-off-by: Andy Adamson <andros@xxxxxxxxxx>
> ---
> fs/nfs/nfs4filelayout.c | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/fs/nfs/nfs4filelayout.c b/fs/nfs/nfs4filelayout.c
> index e134029..96a75d1 100644
> --- a/fs/nfs/nfs4filelayout.c
> +++ b/fs/nfs/nfs4filelayout.c
> @@ -205,8 +205,8 @@ static int filelayout_async_handle_error(struct rpc_task *task,
> case -EPIPE:
> dprintk("%s DS connection error %d\n", __func__,
> task->tk_status);
> - if (!filelayout_test_devid_invalid(devid))
> - _pnfs_return_layout(inode);
> + clear_bit(NFS_INO_LAYOUTCOMMIT, &NFS_I(inode)->flags);
> + _pnfs_return_layout(inode);
> filelayout_mark_devid_invalid(devid);
> rpc_wake_up(&tbl->slot_tbl_waitq);
> nfs4_ds_disconnect(clp);
--
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
