summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAnand Avati <avati@redhat.com>2012-06-07 17:09:45 -0700
committerAnand Avati <avati@redhat.com>2012-06-07 17:34:04 -0700
commit7efa697c26b31c52f0f8cc976a505ff34bfbe33d (patch)
tree8fc0e00197c69cf6d7e8da4f65d1134a7e0e85db
parent81e7464a54165b5d8b40847355d3bb6fe7b6e9df (diff)
downloadglusterfs-7efa697c26b31c52f0f8cc976a505ff34bfbe33d.tar.gz
glusterfs-7efa697c26b31c52f0f8cc976a505ff34bfbe33d.tar.xz
glusterfs-7efa697c26b31c52f0f8cc976a505ff34bfbe33d.zip
io-cache,quick-read: bring down log level
log messages were unnecessarily in INFO level. The two functions with the same name were non-static and actually the quick-read's call landed in the io-cache's version: 2012-06-07 17:02:29.848667] I [io-cache.c:1549:check_cache_size_ok] 0-single-master-io-cache: Max cache size is 33791991808 [2012-06-07 17:02:29.848751] I [io-cache.c:1549:check_cache_size_ok] 0-single-master-quick-read: Max cache size is 33791991808 Changed them to static declaration. Change-Id: Id9daf9593b2832e4c261f95eac6181efea8899a5 BUG: 765227 Signed-off-by: Anand Avati <avati@redhat.com> Reviewed-on: http://review.gluster.com/3536 Reviewed-by: Jeff Darcy <jdarcy@redhat.com> Tested-by: Gluster Build System <jenkins@build.gluster.com>
-rw-r--r--xlators/performance/io-cache/src/io-cache.c4
-rw-r--r--xlators/performance/quick-read/src/quick-read.c5
2 files changed, 5 insertions, 4 deletions
diff --git a/xlators/performance/io-cache/src/io-cache.c b/xlators/performance/io-cache/src/io-cache.c
index 7e6e0fb2ed..4dbb1aa01a 100644
--- a/xlators/performance/io-cache/src/io-cache.c
+++ b/xlators/performance/io-cache/src/io-cache.c
@@ -1522,7 +1522,7 @@ mem_acct_init (xlator_t *this)
}
-gf_boolean_t
+static gf_boolean_t
check_cache_size_ok (xlator_t *this, uint64_t cache_size)
{
gf_boolean_t ret = _gf_true;
@@ -1545,7 +1545,7 @@ check_cache_size_ok (xlator_t *this, uint64_t cache_size)
else
max_cache_size = total_mem;
- gf_log (this->name, GF_LOG_INFO, "Max cache size is %"PRIu64,
+ gf_log (this->name, GF_LOG_DEBUG, "Max cache size is %"PRIu64,
max_cache_size);
if (cache_size > max_cache_size) {
diff --git a/xlators/performance/quick-read/src/quick-read.c b/xlators/performance/quick-read/src/quick-read.c
index fed5760157..0513087847 100644
--- a/xlators/performance/quick-read/src/quick-read.c
+++ b/xlators/performance/quick-read/src/quick-read.c
@@ -3600,7 +3600,8 @@ mem_acct_init (xlator_t *this)
return ret;
}
-gf_boolean_t
+
+static gf_boolean_t
check_cache_size_ok (xlator_t *this, int64_t cache_size)
{
int ret = _gf_true;
@@ -3623,7 +3624,7 @@ check_cache_size_ok (xlator_t *this, int64_t cache_size)
else
max_cache_size = total_mem;
- gf_log (this->name, GF_LOG_INFO, "Max cache size is %"PRIu64,
+ gf_log (this->name, GF_LOG_DEBUG, "Max cache size is %"PRIu64,
max_cache_size);
if (cache_size > max_cache_size) {
ret = _gf_false;