From 788992bbf57373913fc416939f72d58568f6648b Mon Sep 17 00:00:00 2001 From: Samuel Just Date: Mon, 10 Dec 2012 13:38:24 -0800 Subject: [PATCH] config_opts.h: adjust recovery defaults osd max backfills: 5 was too low for a default, 10 seems to work better in testing. The message priority system should minimize disruption of push and pull operations anyway. osd recovery max chunk: 1MB was too small for a default. 8MB is reasonable for a single push and will allow us to recover an rbd block in one push rather then 4 reducing client io latency during log-based recovery. osd recovery op priority: 10 rather than 30 will further reduce the client io latency impact of push and pull operations. Signed-off-by: Samuel Just --- src/common/config_opts.h | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/common/config_opts.h b/src/common/config_opts.h index 24875d7114ba2..d312ef28e0c85 100644 --- a/src/common/config_opts.h +++ b/src/common/config_opts.h @@ -280,7 +280,7 @@ OPTION(osd_auto_upgrade_tmap, OPT_BOOL, true) OPTION(osd_tmapput_sets_uses_tmap, OPT_BOOL, false) // Maximum number of backfills to or from a single osd -OPTION(osd_max_backfills, OPT_U64, 5) +OPTION(osd_max_backfills, OPT_U64, 10) // Refuse backfills when OSD full ratio is above this value OPTION(osd_backfill_full_ratio, OPT_FLOAT, 0.85) @@ -335,8 +335,8 @@ OPTION(osd_default_data_pool_replay_window, OPT_INT, 45) OPTION(osd_preserve_trimmed_log, OPT_BOOL, false) OPTION(osd_auto_mark_unfound_lost, OPT_BOOL, false) OPTION(osd_recovery_delay_start, OPT_FLOAT, 15) -OPTION(osd_recovery_max_active, OPT_INT, 5) -OPTION(osd_recovery_max_chunk, OPT_U64, 1<<20) // max size of push chunk +OPTION(osd_recovery_max_active, OPT_INT, 20) +OPTION(osd_recovery_max_chunk, OPT_U64, 8<<20) // max size of push chunk OPTION(osd_recovery_forget_lost_objects, OPT_BOOL, false) // off for now OPTION(osd_max_scrubs, OPT_INT, 1) OPTION(osd_scrub_load_threshold, OPT_FLOAT, 0.5) @@ -374,7 +374,7 @@ OPTION(osd_target_transaction_size, OPT_INT, 300) // to adjust various trans * 1..63. */ OPTION(osd_client_op_priority, OPT_INT, 63) -OPTION(osd_recovery_op_priority, OPT_INT, 30) +OPTION(osd_recovery_op_priority, OPT_INT, 10) OPTION(filestore, OPT_BOOL, false) OPTION(filestore_debug_omap_check, OPT_BOOL, 0) // Expensive debugging check on sync -- 2.39.5