Re: [PATCH 2.6.30] xprtrdma: The frmr iova_start values are truncated by the nfs rdma client.

[Steve Wise <swise@opengridcomputing.com>]

Tom Talpey wrote:
> At 08:13 PM 5/11/2009, Steve Wise wrote:
>   
>> Trond Myklebust wrote:
>>     
>>> On Mon, 2009-05-11 at 17:25 -0500, Steve Wise wrote:
>>>   
>>>       
>>>> Hey Trond,
>>>>
>>>> Will this bug fix make 2.6.30?
>>>>
>>>> Thanks,
>>>>
>>>> Steve.
>>>>     
>>>>         
>>> Not in the form it is in now. As I've said earlier, I'm not happy about
>>> the sunrpc layer having to circumvent ordinary type checking on
>>> non-sunrpc structures.
>>>
>>> Cheers
>>>     Trond
>>>       
>> How is it circumventing?  It's currently incorrectly casting a pointer 
>> into a u64.  That seems just broken to me.
>>     
>
> The cast is definitely broken, and since your patch removes it I agree
> with the change, for the record. Especially since with the change, the
> code doesn't work on 32-bit CPU / 64-bit IOMMU platforms.
>
>   

That was my short-term concern.  Its just broke.


>>  Also, its really the sunrpc 
>> rdma transport layer.  It deals specifically with rdma.  It _should_ 
>> know about rdma interfaces and types.
>>     
>
> IOW, this is an issue for the OFA API to address? I agree with Trond that
> there are some hardware-specific warts in there. It might be a good idea
> for us to go through both the client and server code overall - you up for
> doing that? I'm in, if so.
>
>   

Ok.