From 6788ea0e9a51217cd56b282c40533a8971a36530 Mon Sep 17 00:00:00 2001
From: Kefu Chai <kchai@redhat.com>
Date: Fri, 29 Sep 2017 10:51:54 +0800
Subject: [PATCH] common/options: use user-defined literals for sizes

Signed-off-by: Kefu Chai <kchai@redhat.com>
---
 src/common/options.cc | 44 +++++++++++++++++++++----------------------
 1 file changed, 22 insertions(+), 22 deletions(-)

diff --git a/src/common/options.cc b/src/common/options.cc
index d8dc168124cd2..d30c3e54f79c5 100644
--- a/src/common/options.cc
+++ b/src/common/options.cc
@@ -599,7 +599,7 @@ std::vector<Option> get_global_options() {
     .set_description(""),
 
     Option("ms_tcp_prefetch_max_size", Option::TYPE_INT, Option::LEVEL_ADVANCED)
-    .set_default(4096)
+    .set_default(4_K)
     .set_description(""),
 
     Option("ms_initial_backoff", Option::TYPE_FLOAT, Option::LEVEL_ADVANCED)
@@ -1253,7 +1253,7 @@ std::vector<Option> get_global_options() {
     .set_description(""),
 
     Option("mon_config_key_max_entry_size", Option::TYPE_INT, Option::LEVEL_ADVANCED)
-    .set_default(4096)
+    .set_default(4_K)
     .set_description(""),
 
     Option("mon_sync_timeout", Option::TYPE_FLOAT, Option::LEVEL_ADVANCED)
@@ -1261,7 +1261,7 @@ std::vector<Option> get_global_options() {
     .set_description(""),
 
     Option("mon_sync_max_payload_size", Option::TYPE_UINT, Option::LEVEL_ADVANCED)
-    .set_default(1048576)
+    .set_default(1_M)
     .set_description(""),
 
     Option("mon_sync_debug", Option::TYPE_BOOL, Option::LEVEL_ADVANCED)
@@ -1687,7 +1687,7 @@ std::vector<Option> get_global_options() {
     .set_description(""),
 
     Option("osd_pool_erasure_code_stripe_unit", Option::TYPE_UINT, Option::LEVEL_ADVANCED)
-    .set_default(4096)
+    .set_default(4_K)
     .set_description(""),
 
     Option("osd_pool_default_size", Option::TYPE_INT, Option::LEVEL_ADVANCED)
@@ -2309,11 +2309,11 @@ std::vector<Option> get_global_options() {
     .set_description(""),
 
     Option("osd_command_thread_timeout", Option::TYPE_INT, Option::LEVEL_ADVANCED)
-    .set_default(10*60)
+    .set_default(10_min)
     .set_description(""),
 
     Option("osd_command_thread_suicide_timeout", Option::TYPE_INT, Option::LEVEL_ADVANCED)
-    .set_default(15*60)
+    .set_default(15_min)
     .set_description(""),
 
     Option("osd_heartbeat_addr", Option::TYPE_ADDR, Option::LEVEL_ADVANCED)
@@ -2897,7 +2897,7 @@ std::vector<Option> get_global_options() {
     .set_long_description("This is an experimental option for rocksdb that works in conjunction with two_level indices to avoid having to keep the entire filter/index in cache when cache_index_and_filter_blocks is true.  The idea is to keep a much smaller top-level index in heap/cache and then opportunistically cache the lower level indices.  See: https://github.com/facebook/rocksdb/wiki/Partitioned-Index-Filters"),
 
     Option("rocksdb_metadata_block_size", Option::TYPE_UINT, Option::LEVEL_DEV)
-    .set_default(4096)
+    .set_default(4_K)
     .set_description("The block size for index partitions. (0 = rocksdb default)"),
 
     Option("mon_rocksdb_options", Option::TYPE_STR, Option::LEVEL_ADVANCED)
@@ -3029,7 +3029,7 @@ std::vector<Option> get_global_options() {
     .set_description(""),
 
     Option("memstore_page_size", Option::TYPE_UINT, Option::LEVEL_ADVANCED)
-    .set_default(64 << 10)
+    .set_default(64_K)
     .set_description(""),
 
     Option("objectstore_blackhole", Option::TYPE_BOOL, Option::LEVEL_ADVANCED)
@@ -3068,7 +3068,7 @@ std::vector<Option> get_global_options() {
     .set_description(""),
 
     Option("bdev_block_size", Option::TYPE_INT, Option::LEVEL_ADVANCED)
-    .set_default(4096)
+    .set_default(4_K)
     .set_description(""),
 
     Option("bdev_debug_aio", Option::TYPE_BOOL, Option::LEVEL_DEV)
@@ -3088,15 +3088,15 @@ std::vector<Option> get_global_options() {
     .set_description(""),
 
     Option("bluefs_alloc_size", Option::TYPE_UINT, Option::LEVEL_ADVANCED)
-    .set_default(1048576)
+    .set_default(1_M)
     .set_description(""),
 
     Option("bluefs_max_prefetch", Option::TYPE_UINT, Option::LEVEL_ADVANCED)
-    .set_default(1048576)
+    .set_default(1_M)
     .set_description(""),
 
     Option("bluefs_min_log_runway", Option::TYPE_UINT, Option::LEVEL_ADVANCED)
-    .set_default(1048576)
+    .set_default(1_M)
     .set_description(""),
 
     Option("bluefs_max_log_runway", Option::TYPE_UINT, Option::LEVEL_ADVANCED)
@@ -3108,11 +3108,11 @@ std::vector<Option> get_global_options() {
     .set_description(""),
 
     Option("bluefs_log_compact_min_size", Option::TYPE_UINT, Option::LEVEL_ADVANCED)
-    .set_default(16*1048576)
+    .set_default(16_M)
     .set_description(""),
 
     Option("bluefs_min_flush_size", Option::TYPE_UINT, Option::LEVEL_ADVANCED)
-    .set_default(524288)
+    .set_default(512_K)
     .set_description(""),
 
     Option("bluefs_compact_log_sync", Option::TYPE_BOOL, Option::LEVEL_ADVANCED)
@@ -3564,7 +3564,7 @@ std::vector<Option> get_global_options() {
     .set_description("simulate fragmentation"),
 
     Option("bluestore_debug_prefragment_max", Option::TYPE_INT, Option::LEVEL_DEV)
-    .set_default(1048576)
+    .set_default(1_M)
     .set_description(""),
 
     Option("bluestore_debug_inject_read_err", Option::TYPE_BOOL, Option::LEVEL_DEV)
@@ -3914,7 +3914,7 @@ std::vector<Option> get_global_options() {
     .set_description("Seconds before backing file system is considered hung"),
 
     Option("filestore_fiemap_threshold", Option::TYPE_INT, Option::LEVEL_DEV)
-    .set_default(4096)
+    .set_default(4_K)
     .set_description(""),
 
     Option("filestore_merge_threshold", Option::TYPE_INT, Option::LEVEL_DEV)
@@ -3986,7 +3986,7 @@ std::vector<Option> get_global_options() {
     .set_description(""),
 
     Option("journal_block_size", Option::TYPE_INT, Option::LEVEL_DEV)
-    .set_default(4096)
+    .set_default(4_K)
     .set_description(""),
 
     Option("journal_max_corrupt_search", Option::TYPE_UINT, Option::LEVEL_DEV)
@@ -4111,7 +4111,7 @@ std::vector<Option> get_global_options() {
                           "performance counter data."),
 
     Option("mgr_client_bytes", Option::TYPE_UINT, Option::LEVEL_DEV)
-    .set_default(128*1048576)
+    .set_default(128_M)
     .add_service("mgr"),
 
     Option("mgr_client_messages", Option::TYPE_UINT, Option::LEVEL_DEV)
@@ -4119,7 +4119,7 @@ std::vector<Option> get_global_options() {
     .add_service("mgr"),
 
     Option("mgr_osd_bytes", Option::TYPE_UINT, Option::LEVEL_DEV)
-    .set_default(512*1048576)
+    .set_default(512_M)
     .add_service("mgr"),
 
     Option("mgr_osd_messages", Option::TYPE_UINT, Option::LEVEL_DEV)
@@ -4127,7 +4127,7 @@ std::vector<Option> get_global_options() {
     .add_service("mgr"),
 
     Option("mgr_mds_bytes", Option::TYPE_UINT, Option::LEVEL_DEV)
-    .set_default(128*1048576)
+    .set_default(128_M)
     .add_service("mgr"),
 
     Option("mgr_mds_messages", Option::TYPE_UINT, Option::LEVEL_DEV)
@@ -4135,7 +4135,7 @@ std::vector<Option> get_global_options() {
     .add_service("mgr"),
 
     Option("mgr_mon_bytes", Option::TYPE_UINT, Option::LEVEL_DEV)
-    .set_default(128*1048576)
+    .set_default(128_M)
     .add_service("mgr"),
 
     Option("mgr_mon_messages", Option::TYPE_UINT, Option::LEVEL_DEV)
@@ -5958,7 +5958,7 @@ std::vector<Option> get_mds_client_options() {
     .set_description(""),
 
     Option("client_max_inline_size", Option::TYPE_UINT, Option::LEVEL_ADVANCED)
-    .set_default(4096)
+    .set_default(4_K)
     .set_description(""),
 
     Option("client_inject_release_failure", Option::TYPE_BOOL, Option::LEVEL_DEV)
-- 
2.39.5