[ewg] Re: [PATCH] NFS-RDMA backport for RHEL 5.2

[Jeff Becker]

Steve Wise wrote:
> Jon Mason wrote:
>   
>> Hey Vlad,
>>
>> I wanted to get you the NFS-RDMA backport patches I have queued up prior
>> to rc1 being built.  I have this patch (RHEL5.2), as well as 2.6.22, and
>> 2.6.25 (which I will be sending in separate e-mails).
>>   
>>     
>
>
> Thanks Jon and Tom for doing this work!
>   

Yes. Thanks from me as well. Although I see much overlap between your 
RHEL5.2 patch and my SLES10SP2 patches, I will start with your work in 
order to unify the two backports as much as possible. Your backport also 
answers a few questions I've been having when reviewing my backport. 
Hopefully the SLES10SP2 backport is not too far off. Thanks again.

-jeff

>
>   
>> This patch contains the changes necessary to get NFS-RDMA 99% working on
>> RHEL5.2.  It 100% passes of the connectathon testcases when using RDMA.
>> When using TCP, 1 testcase hangs (for which I've found a code hack to
>> work around, but not a fix).  The testcase is the 30MB transfer in the
>> "special" tests, and is 100% reproducible.
>>
>> With the attached patch, you can now enable the nfs-rdma-mod by default
>> and run it through your nightly build tests.
>>
>>   
>>     
>
>
> Just to clarify:  Enable nfs-rdma-mod on rhel5.2, 2.6.22, and 2.6.18 
> only.  Also we'll have to address any hw platform issues that come up 
> during your builds.
>
>
>   
>> There is one problem with nfs-utils, which will necessitate including it
>> in the OFED distribution.  Currently, there is a check for the kernel
>> version you are running when running the mount.nfs command.  This check
>> is to verify if one is running a kernel greater than 2.6.22.  If not,
>> then it will not allow a mount with RDMA as an option.  In earlier
>> versions, there was an override option of "-i".  This has since been
>> removed for an unknown reason (but if you run `mount.nfs`, you can see
>> that they forgot to remove 'i' from "usage" options).  I will send them
>> a patch to fix this in the latest version, but this will be a problem
>> that will require it to be built locally until such a time as that is
>> pulled in.
>>
>>   
>>     
>
>
> We'll need to ship nfs-utils that works for nfsrdma on the older kernels 
> as part of ofed-1.4.1.  Stay tuned for this.
>
>
> Steve.
>