search

linux-rdma / perftest

Infiniband Verbs Performance Tests
Other
592 stars 283 forks source link

Perftest: Fix rx_depth check for XRC #263

Closed hginjgerx closed 3 months ago

hginjgerx commented 4 months ago

When users manually specifies --use_srq in perftest command, the rx_depth will be checked. If rx_depth is less than the number of qps, the process will throw an error and exit.

For XRC SEND where SRQ is definitely used, users normally don't need to manually specifies --use_srq, since the use_srq flag will be set to on when parsing the XRC parameters. However, the XRC parameters parsing is after the SRQ rx_depth check. If rx_depth is less than the number of qps in this case, it will miss the check, size_per_qp in ctx_set_recv_wqes() will become 0 and ibv_post_srq_recv() won't be called.

Move the XRC parameters parsing ahead of SRQ rx_depth check and set --use_srq to on in advance so that the rx_depth error can be thrown.

Fixes: 4c774a951b3c ("Added XRC for ib_send_bw test")

sshaulnv commented 3 months ago

Thanks for your pull request, merged!