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: leverage the type knowledge in BitMapAreaLeaf. #13736

Merged
merged 3 commits into from Mar 6, 2017

Conversation

Projects
None yet
2 participants
@rzarzynski
Contributor

rzarzynski commented Mar 2, 2017

We're sure the only element type BitMapAreaLeaf aggregates is BitMapZone. There is no business to go through vptrs and thus prohibit compiler to inline in a method that is crucial for overall performance of the BitMap allocator.

Signed-off-by: Radoslaw Zarzynski rzarzynski@mirantis.com

os/bluestore: leverage the type knowledge in BitMapAreaLeaf.
We're sure the only element type BitMapAreaLeaf aggregates is
BitMapZone. There is no business to go through vptrs and thus
prohibit compiler to inline in a method that is crucial for
overall performance of the BitMap allocator.

Signed-off-by: Radoslaw Zarzynski <rzarzynski@mirantis.com>

@rzarzynski rzarzynski added the bluestore label Mar 2, 2017

rzarzynski added some commits Mar 2, 2017

os/bluestore: inline for the sake of BitMapAreaLeaf::child_check_n_lock.
Signed-off-by: Radoslaw Zarzynski <rzarzynski@mirantis.com>
os/bluestore: switch to alloc_dbg_assert in BmapEntityListIter::next().
Signed-off-by: Radoslaw Zarzynski <rzarzynski@mirantis.com>

@liewegas liewegas merged commit 63a903f into ceph:master Mar 6, 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment