-
Notifications
You must be signed in to change notification settings - Fork 368
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
prov/efa: Support unsolicited RDMA write with immediate receive #10036
Conversation
e8fbd11
to
dec5878
Compare
Need to rebase on #10042 |
This patch onboards Libfabric with the rdma-core feature that supports unsolicited RDMA write with immediate receive linux-rdma/rdma-core#1459 . When a rdma-write with imm is unsolicited, libfabric won't release the rx pkt and bump the rx pkt counters. Signed-off-by: Shi Jin <sjina@amazon.com>
dec5878
to
a1cac3e
Compare
@mrgolin can you review it, thanks |
if (!efa_rdm_cq_wc_is_unsolicited(ibv_cq_ex)) { | ||
/** | ||
* Recv with immediate will consume a pkt_entry, but the pkt is not | ||
* filled, so free the pkt_entry and record we have one less posted | ||
* packet now. | ||
*/ | ||
assert(pkt_entry); | ||
ep->efa_rx_pkts_posted--; | ||
efa_rdm_pke_release_rx(pkt_entry); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this block supposed to be guarded with a directive? E.g. #if HAVE_CAPS_UNSOLICITED_WRITE_RECV
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
efa_rdm_cq_wc_is_unsolicited
is defined for both HAVE_CAPS_UNSOLICITED_WRITE_RECV=1
and HAVE_CAPS_UNSOLICITED_WRITE_RECV=0
. For the latter one, it will just return false
This patch onboards EFA provider with the rdma-core feature that supports unsolicited RDMA write with immediate receive
linux-rdma/rdma-core#1459
. When a rdma-write with imm is unsolicited, libfabric won't release the rx pkt and bump the rx pkt counters.