diff options
author | Anand Avati <avati@gluster.com> | 2010-02-22 10:59:40 +0000 |
---|---|---|
committer | Anand V. Avati <avati@dev.gluster.com> | 2010-02-22 08:51:44 -0800 |
commit | af2f208ff4f40e44069ede605a5e21148de053de (patch) | |
tree | c4d97c1bbb76e7bb88550a39918bdec8b36260e3 | |
parent | a6f0dca7cd348b2a428b2583ebcfc80425213d5c (diff) |
common-utils: gf_log2 handles 0 input
Signed-off-by: Anand V. Avati <avati@amp.gluster.com>
Signed-off-by: Anand V. Avati <avati@blackhole.gluster.com>
Signed-off-by: Anand V. Avati <avati@dev.gluster.com>
BUG: 583 (filesystem access hangs while deleting large files)
URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=583
-rw-r--r-- | libglusterfs/src/common-utils.c | 4 | ||||
-rw-r--r-- | libglusterfs/src/common-utils.h | 2 | ||||
-rw-r--r-- | xlators/performance/io-cache/src/io-cache.c | 2 |
3 files changed, 4 insertions, 4 deletions
diff --git a/libglusterfs/src/common-utils.c b/libglusterfs/src/common-utils.c index 7b5e02a344c..31727adf813 100644 --- a/libglusterfs/src/common-utils.c +++ b/libglusterfs/src/common-utils.c @@ -58,11 +58,11 @@ struct dnscache6 { int -gf_log2 (unsigned long x) +log_base2 (unsigned long x) { int val = 0; - while (x != 1) { + while (x > 1) { x /= 2; val++; } diff --git a/libglusterfs/src/common-utils.h b/libglusterfs/src/common-utils.h index 05acfd83d29..9c1c93ebffb 100644 --- a/libglusterfs/src/common-utils.h +++ b/libglusterfs/src/common-utils.h @@ -332,7 +332,7 @@ int gf_lockfd (int fd); int gf_unlockfd (int fd); int get_checksum_for_file (int fd, uint32_t *checksum); -int gf_log2 (unsigned long x); +int log_base2 (unsigned long x); #endif /* _COMMON_UTILS_H */ diff --git a/xlators/performance/io-cache/src/io-cache.c b/xlators/performance/io-cache/src/io-cache.c index f01d8e3bb81..0c015caa4cc 100644 --- a/xlators/performance/io-cache/src/io-cache.c +++ b/xlators/performance/io-cache/src/io-cache.c @@ -1409,7 +1409,7 @@ init (xlator_t *this) ret = 0; ctx = this->ctx; - ioc_log2_page_size = gf_log2 (ctx->page_size); + ioc_log2_page_size = log_base2 (ctx->page_size); out: if (ret == -1) { |