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

os/bluestore: improve mempool usage #15402

Merged
merged 4 commits into from Jun 2, 2017

Conversation

Projects
None yet
4 participants
@liewegas
Member

liewegas commented Jun 1, 2017

  • bluestore_cache_{onode,data,other} for cache
  • bluestore_writing[_deferred] for in-flight write buffers
  • bluestore_txc for TransContexts

liewegas added some commits May 31, 2017

os/bluestore: maybe_rebuild before adding to cache too
Signed-off-by: Sage Weil <sage@redhat.com>
os/bluestore: rename cache mempools to bluestore_cache_*
These map directly onto the bluestore_cache_size setting.

Signed-off-by: Sage Weil <sage@redhat.com>
os/bluestore: bluestore_txc mempool
Signed-off-by: Sage Weil <sage@redhat.com>
os/bluestore: mempools for writing and writing_deferred
Signed-off-by: Sage Weil <sage@redhat.com>

@liewegas liewegas requested a review from ifed01 Jun 1, 2017

@ifed01

ifed01 approved these changes Jun 1, 2017

@tchaikov

This comment has been minimized.

Show comment
Hide comment
@tchaikov

tchaikov Jun 1, 2017

Contributor

retest this please. see #15406

Contributor

tchaikov commented Jun 1, 2017

retest this please. see #15406

@liewegas

This comment has been minimized.

Show comment
Hide comment
@liewegas

liewegas Jun 1, 2017

Member

retest this please

Member

liewegas commented Jun 1, 2017

retest this please

@liewegas liewegas merged commit 773a0ac into ceph:master Jun 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

@liewegas liewegas deleted the liewegas:wip-bluestore-mempools branch Jun 2, 2017

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