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

osd: put osdmap in mempool #14780

Merged
merged 6 commits into from Apr 26, 2017

Conversation

Projects
None yet
3 participants
@liewegas
Member

liewegas commented Apr 25, 2017

No description provided.

liewegas added some commits Apr 24, 2017

osd/OSDMap: use auto where possible
Signed-off-by: Sage Weil <sage@redhat.com>
include/encoding: use more auto
Signed-off-by: Sage Weil <sage@redhat.com>
include/mempool: add osdmap mempool
Signed-off-by: Sage Weil <sage@redhat.com>
crush: templatize weights arg for do_rule
Signed-off-by: Sage Weil <sage@redhat.com>
osd/OSDMap: move to 'osdmap' mempool
We leave a few things out:

- strings (to annoying, and they are small)
- erasure code profiles (ditto)

Signed-off-by: Sage Weil <sage@redhat.com>
@liewegas

This comment has been minimized.

Member

liewegas commented Apr 25, 2017

retest this please

@Liuchang0812

This comment has been minimized.

Contributor

Liuchang0812 commented Apr 25, 2017

I know how to put something in ceph's mempool now. 😃

@tchaikov tchaikov self-requested a review Apr 25, 2017

@liewegas

This comment has been minimized.

Member

liewegas commented Apr 25, 2017

retest this please

osd/OSDMap: put OSDMap[::Incremental] itself in mempool too
Signed-off-by: Sage Weil <sage@redhat.com>

@liewegas liewegas merged commit 3608a00 into ceph:master Apr 26, 2017

3 of 4 checks passed

arm build Build triggered. sha1 is merged.
Details
Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details

@liewegas liewegas deleted the liewegas:wip-osdmap-mempool branch Apr 26, 2017

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