diff options
author | Anand Avati <avati@redhat.com> | 2012-06-07 17:09:45 -0700 |
---|---|---|
committer | Anand Avati <avati@redhat.com> | 2012-06-07 17:34:04 -0700 |
commit | 7efa697c26b31c52f0f8cc976a505ff34bfbe33d (patch) | |
tree | 8fc0e00197c69cf6d7e8da4f65d1134a7e0e85db | |
parent | 81e7464a54165b5d8b40847355d3bb6fe7b6e9df (diff) | |
download | glusterfs-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.c | 4 | ||||
-rw-r--r-- | xlators/performance/quick-read/src/quick-read.c | 5 |
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; |