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

common/config_opts: improve rdma buffer size to 128k #13510

Merged
merged 1 commit into from Mar 2, 2017

Conversation

Projects
None yet
2 participants
@yuyuyu101
Member

yuyuyu101 commented Feb 18, 2017

now default rdma memory pool is 128k * (1024 + 1024) = 256MB for each osd.
128k could avoid most of middle size ops fragments

Signed-off-by: Haomai Wang haomai@xsky.com

common/config_opts: improve rdma buffer size to 128k
now default rdma memory pool is 128k * (1024 + 1024) = 256MB.
128k could avoid most of middle size ops fragments

Signed-off-by: Haomai Wang <haomai@xsky.com>
@Adirl

This comment has been minimized.

Adirl commented Feb 19, 2017

we had so issues trying to set 128K
need to test it again

@yuyuyu101

This comment has been minimized.

Member

yuyuyu101 commented Mar 2, 2017

@Adirl do you have more make sense value?

@Adirl

This comment has been minimized.

Adirl commented Mar 2, 2017

@yuyuyu101
I don't have a good value to set right now
we need to test,
because we see more memory then 256MB per OSD

keep in mind that messages smaller then 128K will consume full chunk

@yuyuyu101

This comment has been minimized.

Member

yuyuyu101 commented Mar 2, 2017

sure, I think 256mb per osd is ok to me.

At least, the current 8k value is too small

@Adirl

This comment has been minimized.

Adirl commented Mar 2, 2017

Agree that 8K is too small

@yuyuyu101 yuyuyu101 merged commit 911fbf3 into ceph:master Mar 2, 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

@yuyuyu101 yuyuyu101 deleted the yuyuyu101:wip-rdma-buffer-size branch Mar 2, 2017

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