[U-Boot] [PATCH V2] net: NFS: Add NFSv3 support

[Guillaume Gardet]

Le 25/06/2016 à 21:26, Joe Hershberger a écrit :
> On Fri, Jun 24, 2016 at 10:03 AM, Tom Rini <trini at konsulko.com> wrote:
>> On Fri, Jun 24, 2016 at 09:45:38AM +0200, Guillaume Gardet wrote:
>>>
>>> Le 23/06/2016 21:08, Tom Rini a écrit :
>>>> On Thu, Jun 23, 2016 at 11:10:26AM +0200, Guillaume GARDET wrote:
>>>>
>>>>> This patch enables NFSv3 support.
>>>>> If NFSv2 is available use it as usual.
>>>>> If NFSv2 is not available, but NFSv3 is available, use NFSv3.
>>>>> If NFSv2 and NFSv3 are not available, print an error message since NFSv4 is not supported.
>>>>>
>>>>> Tested on iMX6 sabrelite with 4 Linux NFS servers:
>>>>>    * NFSv2 + NFSv3 + NFSv4 server: use NFSv2 protocol
>>>>>    * NFSv2 + NFSv3 server: use NFSv3 protocol
>>>>>    * NFSv3 + NFSv4 server: use NFSv3 protocol
>>>>>    * NFSv3 server: use NFSv3 protocol
>>>> So, why do we have v2+v3+v4 -> v2 and not v2+v3+v4 -> v3, when we do
>>>> v2+v3 -> v3 and v3+v4 -> v3 ?  We should be consistent in preferring
>>>> either v2 over v3 or v3 over v2.  Thanks!
>>>>
>>> Sorry, it is a typo error. Please read: "NFSv2 + NFSv3 server: use NFSv2 protocol".
>>>
>>> As long as NFSv2 is available, we use it. Otherwise, we use v3 if
>>> available. As explained above.
>> OK, that makes sense, thanks!  Joe, I assume you can just fix that in
>> the commit message when you pick this up.
> Yup, no problem.
>

Just a friendly reminder for this patch. :)


Guillaume