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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 01/26/2012 09:56 AM, Srinivas KANDAGATLA wrote:
> Jim Rees wrote:
>> Srinivas KANDAGATLA wrote:
>>
>>   From: Srinivas Kandagatla <srinivas.kandagatla@xxxxxx>
>>   
>>   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?

> Thanks,
> srini

Thanks
Boaz

> --
> 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

--
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


[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux