Re: [PATCH 3/3] NFSv4.1 fence all layouts with file layout data server connection errors

[Boaz Harrosh <bharrosh@xxxxxxxxxxx>]

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