From: Kefu Chai <kchai@redhat.com>
Date: Fri, 28 Apr 2017 06:15:54 +0000 (+0800)
Subject: bluestore: use std::atomic<uint8_t> for min_alloc_size_order
X-Git-Tag: v12.0.3~174^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=14fc705698aaeae25248f0f6ea95e0e4b38ffddc;p=ceph.git

bluestore: use std::atomic<uint8_t> for min_alloc_size_order

Signed-off-by: Kefu Chai <kchai@redhat.com>
---

diff --git a/src/os/bluestore/BlueStore.h b/src/os/bluestore/BlueStore.h
index f2a32b8b2df8..e96c41082571 100644
--- a/src/os/bluestore/BlueStore.h
+++ b/src/os/bluestore/BlueStore.h
@@ -1795,8 +1795,11 @@ private:
   int deferred_batch_ops = 0; ///< deferred batch size
 
   ///< bits for min_alloc_size
-  std::atomic<size_t> min_alloc_size_order = {0};
-  
+  std::atomic<uint8_t> min_alloc_size_order = {0};
+  static_assert(std::numeric_limits<uint8_t>::max() >
+		std::numeric_limits<decltype(min_alloc_size)>::digits,
+		"not enough bits for min_alloc_size");
+
   ///< size threshold for forced deferred writes
   std::atomic<uint64_t> prefer_deferred_size = {0};