There is a need to change the test itself.

On the file ndrping.cpp lines 616 you need to change:

    hr = pConnector->CreateEndpoint(
        pCq,
        pCq,
        1,
        QueueDepth,
        1,
        nSge,
        16, // was 0 InboundReadLimit
        16, // was 0 OutboundReadLimit
        NULL,
        &pEndpoint
        );

By the way, I have tried your provider and it did work, so I don't think you 
should change anything there (you only use inline for rdma read if the size is 
0).

Thanks
Tzachi 

-----Original Message-----
From: Sean Hefty [mailto:[email protected]]
Sent: Tuesday, March 09, 2010 10:07 PM
To: Tzachi Dar; [email protected]
Subject: RE: Patch: Don't use inline for RDMA reads

>In order to allow the ndrping to work also with RDMA reads, the 
>following fix is needed.

Was this the only fix that was needed?  Did the available ndpring.exe work as 
is?

I can see that the WV ND provider has a similar problem.  But applying the same 
change there didn't fix the issue.  I just want to verify that this was the 
only change before investigating the problem further.

_______________________________________________
ofw mailing list
[email protected]
http://lists.openfabrics.org/cgi-bin/mailman/listinfo/ofw

Reply via email to