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

msg/async/rdma: check if fin message completed #15624

Merged
merged 1 commit into from Jun 12, 2017

Conversation

Projects
None yet
2 participants
@Adirl

Adirl commented Jun 12, 2017

Signed-off-by: Sarit Zubakov saritz@mellanox.com

@Adirl

This comment has been minimized.

Adirl commented Jun 12, 2017

@yuyuyu101
please see above commit to handle fin() message

// FIXME: why not tx?
if (global_infiniband->get_memory_manager()->is_tx_buffer(chunk->buffer))
//TX completion may come either from regular send message or from 'fin' message.
//In the case of 'fin' wr_id points to the QueuePair.

This comment has been minimized.

@yuyuyu101

yuyuyu101 Jun 12, 2017

Member

comment aligned

Adir Lev
msg/async/rdma: check if fin message completed
Change-Id: I9cb1e523be7b21127503a9252b09331a37bea204
Signed-off-by: Alexander Mikheev <alexm@mellanox.com>
Signed-off-by: Adir Lev <adirl@mellanox.com>
@Adirl

This comment has been minimized.

Adirl commented Jun 12, 2017

@yuyuyu101
comment text is now aligned

@yuyuyu101 yuyuyu101 merged commit 6d0b4ce into ceph:master Jun 12, 2017

2 of 3 checks passed

make check running make check
Details
Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment