From 573cbb796e8ba2f433caa308925735101a8161a6 Mon Sep 17 00:00:00 2001 From: Kefu Chai Date: Wed, 2 Jun 2021 16:31:18 +0800 Subject: [PATCH] os/bluestore/AvlAllocator: use cbit for counting the order of alignment no need to calculate the alignment first, cbits() would suffice. as it counts the first set bit and the follow 0's in a number. the result is identical to the cbit(alignment of that number). Signed-off-by: Kefu Chai --- src/os/bluestore/AvlAllocator.cc | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/src/os/bluestore/AvlAllocator.cc b/src/os/bluestore/AvlAllocator.cc index 0390e6d6ce9..f3b721e8765 100644 --- a/src/os/bluestore/AvlAllocator.cc +++ b/src/os/bluestore/AvlAllocator.cc @@ -266,10 +266,7 @@ int AvlAllocator::_allocate( * not guarantee that other allocations sizes may exist in the same * region. */ - uint64_t align = size & -size; - ceph_assert(align != 0); - uint64_t* cursor = &lbas[cbits(align) - 1]; - + uint64_t* cursor = &lbas[cbits(size) - 1]; start = _pick_block_after(cursor, size, unit); dout(20) << __func__ << " first fit=" << start << " size=" << size << dendl; if (start != uint64_t(-1ULL)) {