Re: [RFC:PATCH 3.1.0] do_mount: Add mount retry option for nfs root mount.

[Srinivas KANDAGATLA <srinivas.kandagatla@st.com>]

Boaz Harrosh wrote:
> On 01/26/2012 09:56 AM, Srinivas KANDAGATLA wrote:
>   
>> Jim Rees wrote:
>>     
>>> Srinivas KANDAGATLA wrote:
>>>
>>>   From: Srinivas Kandagatla <srinivas.kandagatla@st.com>
>>>   
>>>   This patch adds mountretry kernel parameter for nfs root mount.
>>>   mount retry indicates the number of times nfs root mount attempts to be
>>>   made before giving up. If this option is not specified, the default
>>>   value of 3 retries is used.
>>>
>>> This has already been fixed in commit 43717c7d, "NFS: Retry mounting
>>> NFSROOT".  The retry count is fixed at 5.  Is there some reason you think
>>> the retry count should be configurable?
>>>   
>>>       
>> Thanks for pointing to the fixed commit. I did realize soon after
>> sending this patch to mailing list.
>>
>> Only reason I can think of having retry count configurable is, an
>> arbitrary number(3 or 5 or n) of retry count may not be enough to
>> address all the use cases.
>> So, I think the user should be given a chance to configure the retry
>> count if he wish to, either timeout quickly and try the succession
>> root-mount or If he wish to keep kernel trying very harder till he gets
>> the NFS root mount successful.
>>
>>     
>
> I agree. Specially if there is no "succession root-mount" setup then user
> can specify a very large count.
>
> Is there some delay between retries. So not to burn 100% power?
>
>   
An nfs root mount attempt timeouts in about 3 sec's and in commit
43717c7d, "NFS: Retry mounting NFSROOT" each retry has additional sleep
which gives sufficient delay between retries.
>> Thanks,
>> srini
>>     
>
> Thanks
> Boaz
>
>   
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>>     
>
>