|
 |
74096c |
From baa566be8832a56fdea7068d84844ec1ec84d8d9 Mon Sep 17 00:00:00 2001
|
|
|
74096c |
From: mohit84 <moagrawa@redhat.com>
|
|
|
74096c |
Date: Thu, 15 Oct 2020 16:28:58 +0530
|
|
|
74096c |
Subject: [PATCH 504/511] io-stats: Configure ios_sample_buf_size based on
|
|
|
74096c |
sample_interval value (#1574)
|
|
|
74096c |
|
|
|
74096c |
io-stats xlator declares a ios_sample_buf_size 64k object(10M) per xlator
|
|
|
74096c |
but in case of sample_interval is 0 this big buffer is not required so
|
|
|
74096c |
declare the default value only while sample_interval is not 0.The new
|
|
|
74096c |
change would be helpful to reduce RSS size for a brick and shd process
|
|
|
74096c |
while the number of volumes are huge.
|
|
|
74096c |
|
|
|
74096c |
> Change-Id: I3e82cca92e40549355edfac32580169f3ce51af8
|
|
|
74096c |
> Fixes: #1542
|
|
|
74096c |
> Signed-off-by: Mohit Agrawal <moagrawa@redhat.com>
|
|
|
74096c |
> (Cherry picked from commit f71660eb879a9cd5761e5adbf10c783e959a990a)
|
|
|
74096c |
> (Reviewed on upstream link https://github.com/gluster/glusterfs/issues/1542)
|
|
|
74096c |
|
|
|
74096c |
Change-Id: I3e82cca92e40549355edfac32580169f3ce51af8
|
|
|
74096c |
BUG: 1898778
|
|
|
74096c |
Signed-off-by: Mohit Agrawal <moagrawa@redhat.com>
|
|
|
74096c |
Reviewed-on: https://code.engineering.redhat.com/gerrit/221183
|
|
|
74096c |
Tested-by: RHGS Build Bot <nigelb@redhat.com>
|
|
|
74096c |
Reviewed-by: Sunil Kumar Heggodu Gopala Acharya <sheggodu@redhat.com>
|
|
|
74096c |
---
|
|
|
74096c |
tests/bugs/glusterd/daemon-log-level-option.t | 8 ++++----
|
|
|
74096c |
xlators/debug/io-stats/src/io-stats.c | 26 ++++++++++++++++++++++----
|
|
|
74096c |
2 files changed, 26 insertions(+), 8 deletions(-)
|
|
|
74096c |
|
|
|
74096c |
diff --git a/tests/bugs/glusterd/daemon-log-level-option.t b/tests/bugs/glusterd/daemon-log-level-option.t
|
|
|
74096c |
index 66e55e3..5352a63 100644
|
|
|
74096c |
--- a/tests/bugs/glusterd/daemon-log-level-option.t
|
|
|
74096c |
+++ b/tests/bugs/glusterd/daemon-log-level-option.t
|
|
|
74096c |
@@ -61,8 +61,8 @@ rm -f /var/log/glusterfs/glustershd.log
|
|
|
74096c |
TEST $CLI volume set all cluster.daemon-log-level WARNING
|
|
|
74096c |
TEST $CLI volume start $V0
|
|
|
74096c |
|
|
|
74096c |
-# log should not have any info messages
|
|
|
74096c |
-EXPECT 0 Info_messages_count "/var/log/glusterfs/glustershd.log"
|
|
|
74096c |
+# log does have 1 info message specific to configure ios_sample_buf_size in io-stats xlator
|
|
|
74096c |
+EXPECT 1 Info_messages_count "/var/log/glusterfs/glustershd.log"
|
|
|
74096c |
|
|
|
74096c |
# log should not have any debug messages
|
|
|
74096c |
EXPECT 0 Debug_messages_count "/var/log/glusterfs/glustershd.log"
|
|
|
74096c |
@@ -78,8 +78,8 @@ rm -f /var/log/glusterfs/glustershd.log
|
|
|
74096c |
TEST $CLI volume set all cluster.daemon-log-level ERROR
|
|
|
74096c |
TEST $CLI volume start $V0
|
|
|
74096c |
|
|
|
74096c |
-# log should not have any info messages
|
|
|
74096c |
-EXPECT 0 Info_messages_count "/var/log/glusterfs/glustershd.log"
|
|
|
74096c |
+# log does have 1 info message specific to configure ios_sample_buf_size in io-stats xlator
|
|
|
74096c |
+EXPECT 1 Info_messages_count "/var/log/glusterfs/glustershd.log"
|
|
|
74096c |
|
|
|
74096c |
# log should not have any warning messages
|
|
|
74096c |
EXPECT 0 Warning_messages_count "/var/log/glusterfs/glustershd.log"
|
|
|
74096c |
diff --git a/xlators/debug/io-stats/src/io-stats.c b/xlators/debug/io-stats/src/io-stats.c
|
|
|
74096c |
index aa91a0a..9b34895 100644
|
|
|
74096c |
--- a/xlators/debug/io-stats/src/io-stats.c
|
|
|
74096c |
+++ b/xlators/debug/io-stats/src/io-stats.c
|
|
|
74096c |
@@ -3724,6 +3724,15 @@ xlator_set_loglevel(xlator_t *this, int log_level)
|
|
|
74096c |
}
|
|
|
74096c |
}
|
|
|
74096c |
|
|
|
74096c |
+void
|
|
|
74096c |
+ios_sample_buf_size_configure(char *name, struct ios_conf *conf)
|
|
|
74096c |
+{
|
|
|
74096c |
+ conf->ios_sample_buf_size = 1024;
|
|
|
74096c |
+ gf_log(name, GF_LOG_INFO,
|
|
|
74096c |
+ "Configure ios_sample_buf "
|
|
|
74096c |
+ " size is 1024 because ios_sample_interval is 0");
|
|
|
74096c |
+}
|
|
|
74096c |
+
|
|
|
74096c |
int
|
|
|
74096c |
reconfigure(xlator_t *this, dict_t *options)
|
|
|
74096c |
{
|
|
|
74096c |
@@ -3779,8 +3788,13 @@ reconfigure(xlator_t *this, dict_t *options)
|
|
|
74096c |
int32, out);
|
|
|
74096c |
GF_OPTION_RECONF("ios-dump-format", dump_format_str, options, str, out);
|
|
|
74096c |
ios_set_log_format_code(conf, dump_format_str);
|
|
|
74096c |
- GF_OPTION_RECONF("ios-sample-buf-size", conf->ios_sample_buf_size, options,
|
|
|
74096c |
- int32, out);
|
|
|
74096c |
+ if (conf->ios_sample_interval) {
|
|
|
74096c |
+ GF_OPTION_RECONF("ios-sample-buf-size", conf->ios_sample_buf_size,
|
|
|
74096c |
+ options, int32, out);
|
|
|
74096c |
+ } else {
|
|
|
74096c |
+ ios_sample_buf_size_configure(this->name, conf);
|
|
|
74096c |
+ }
|
|
|
74096c |
+
|
|
|
74096c |
GF_OPTION_RECONF("sys-log-level", sys_log_str, options, str, out);
|
|
|
74096c |
if (sys_log_str) {
|
|
|
74096c |
sys_log_level = glusterd_check_log_level(sys_log_str);
|
|
|
74096c |
@@ -3947,8 +3961,12 @@ init(xlator_t *this)
|
|
|
74096c |
GF_OPTION_INIT("ios-dump-format", dump_format_str, str, out);
|
|
|
74096c |
ios_set_log_format_code(conf, dump_format_str);
|
|
|
74096c |
|
|
|
74096c |
- GF_OPTION_INIT("ios-sample-buf-size", conf->ios_sample_buf_size, int32,
|
|
|
74096c |
- out);
|
|
|
74096c |
+ if (conf->ios_sample_interval) {
|
|
|
74096c |
+ GF_OPTION_INIT("ios-sample-buf-size", conf->ios_sample_buf_size, int32,
|
|
|
74096c |
+ out);
|
|
|
74096c |
+ } else {
|
|
|
74096c |
+ ios_sample_buf_size_configure(this->name, conf);
|
|
|
74096c |
+ }
|
|
|
74096c |
|
|
|
74096c |
ret = ios_init_sample_buf(conf);
|
|
|
74096c |
if (ret) {
|
|
|
74096c |
--
|
|
|
74096c |
1.8.3.1
|
|
|
74096c |
|