On Jun 2, 2012, at 6:33 PM, Boaz Harrosh wrote:
> 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
Well, the meat of this patch is the LAYOUTRETURN for fencing.
-->Andy
>
> 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
