[ewg] [GIT PULL ofed-1.4.1] xprtrdma: The frmr iova_start values are truncated by the nfs rdma client.

[Steve Wise]

I'm seeing a server crash even with this frmr iova_start fix.  I'm 
debugging now...

Tziporet, I request we hold off on the new RC build until i resolve 
these i386 issues.

Thanks,

Steve.


Vu Pham wrote:
> Steve,
>
> You should have the same fix for the server side as well
> Here is the patch
>
> -vu
>
>>
>>
>>
>> A bad cast causes the iova_start, which in this case is a DMA bus 
>> address,
>> to be truncated on 32b systems.  No cast is needed.
>>
>> Signed-off-by: Steve Wise <swise at opengridcomputing.com>
>> ---
>>
>> net/sunrpc/xprtrdma/verbs.c |    2 +-
>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/net/sunrpc/xprtrdma/verbs.c b/net/sunrpc/xprtrdma/verbs.c
>> index 3b21e0c..3a374f5 100644
>> --- a/net/sunrpc/xprtrdma/verbs.c
>> +++ b/net/sunrpc/xprtrdma/verbs.c
>> @@ -1489,7 +1489,7 @@ rpcrdma_register_frmr_external(struct 
>> rpcrdma_mr_seg *seg,
>>     memset(&frmr_wr, 0, sizeof frmr_wr);
>>     frmr_wr.opcode = IB_WR_FAST_REG_MR;
>>     frmr_wr.send_flags = 0;            /* unsignaled */
>> -    frmr_wr.wr.fast_reg.iova_start = (unsigned long)seg1->mr_dma;
>> +    frmr_wr.wr.fast_reg.iova_start = seg1->mr_dma;
>>     frmr_wr.wr.fast_reg.page_list = seg1->mr_chunk.rl_mw->r.frmr.fr_pgl;
>>     frmr_wr.wr.fast_reg.page_list_len = i;
>>     frmr_wr.wr.fast_reg.page_shift = PAGE_SHIFT;
>>
>>
>>
>> _______________________________________________
>> ewg mailing list
>> ewg at lists.openfabrics.org
>> http://lists.openfabrics.org/cgi-bin/mailman/listinfo/ewg
>