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: reduce number of rdma rx/tx buffers #13190

Merged
merged 1 commit into from Jan 31, 2017

Conversation

Projects
None yet
3 participants
@Adirl

Adirl commented Jan 30, 2017

Signed-off-by: Adir Lev adirl@mellanox.com

Adir Lev
msg/async/rdma: reduce number of rdma rx/tx buffers
issue: none

Change-Id: Ied523fec3210af152508b102c48aff5633343ab4
Signed-off-by: Adir Lev <adirl@mellanox.com>
@Adirl

This comment has been minimized.

Adirl commented Jan 30, 2017

@yuyuyu101

This comment has been minimized.

Member

yuyuyu101 commented Jan 31, 2017

is this will be too small? each buffer is 8k and 1024 means we only have 8M inflight buffer..

@Adirl

This comment has been minimized.

Adirl commented Jan 31, 2017

Testing on all our ceph clusters (large and small) with a lot of traffic this seems to be more than enough
past experience with other applications using SRQ indicates the same.
so... i think we are OK

@yuyuyu101 yuyuyu101 merged commit ddd52eb into ceph:master Jan 31, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details

@Adirl Adirl deleted the Adirl:reduce_num_buffers branch Apr 18, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment