diff options
| author | arao <arao@redhat.com> | 2015-06-22 11:10:05 +0530 |
|---|---|---|
| committer | Raghavendra G <rgowdapp@redhat.com> | 2015-06-27 02:29:27 -0700 |
| commit | 3741804bec65a33d400af38dcc80700c8a668b81 (patch) | |
| tree | a10008485e263087a52764d2d1ae413bf58c9240 /xlators/performance/open-behind/src/open-behind.c | |
| parent | 911e9228f31e89fe5df6e2282ce449b2a94c42b1 (diff) | |
| download | glusterfs-3741804bec65a33d400af38dcc80700c8a668b81.tar.gz glusterfs-3741804bec65a33d400af38dcc80700c8a668b81.tar.xz glusterfs-3741804bec65a33d400af38dcc80700c8a668b81.zip | |
Logging: Porting the performance translator
logs to new logging framework.
Change-Id: Ie6aaf8d30bd4457bb73c48e23e6b1dea27598644
BUG: 1194640
Signed-off-by: arao <arao@redhat.com>
Reviewed-on: http://review.gluster.org/9822
Tested-by: NetBSD Build System <jenkins@build.gluster.org>
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Raghavendra G <rgowdapp@redhat.com>
Tested-by: Raghavendra G <rgowdapp@redhat.com>
Diffstat (limited to 'xlators/performance/open-behind/src/open-behind.c')
| -rw-r--r-- | xlators/performance/open-behind/src/open-behind.c | 15 |
1 files changed, 10 insertions, 5 deletions
diff --git a/xlators/performance/open-behind/src/open-behind.c b/xlators/performance/open-behind/src/open-behind.c index 742e4df3fd..97c12fab41 100644 --- a/xlators/performance/open-behind/src/open-behind.c +++ b/xlators/performance/open-behind/src/open-behind.c @@ -13,6 +13,7 @@ #include "statedump.h" #include "call-stub.h" #include "defaults.h" +#include "open-behind-messages.h" typedef struct ob_conf { gf_boolean_t use_anonymous_fd; /* use anonymous FDs wherever safe @@ -340,8 +341,8 @@ ob_open (call_frame_t *frame, xlator_t *this, loc_t *loc, int flags, return 0; err: - gf_log (this->name, GF_LOG_ERROR, "%s: %s", loc->path, - strerror (op_errno)); + gf_msg (this->name, GF_LOG_ERROR, op_errno, OPEN_BEHIND_MSG_NO_MEMORY, + "%s", loc->path); STACK_UNWIND_STRICT (open, frame, -1, op_errno, 0, 0); @@ -887,7 +888,9 @@ mem_acct_init (xlator_t *this) ret = xlator_mem_acct_init (this, gf_ob_mt_end + 1); if (ret) - gf_log (this->name, GF_LOG_ERROR, "Memory accounting failed"); + gf_msg (this->name, GF_LOG_ERROR, ENOMEM, + OPEN_BEHIND_MSG_NO_MEMORY, + "Memory accounting failed"); return ret; } @@ -920,14 +923,16 @@ init (xlator_t *this) ob_conf_t *conf = NULL; if (!this->children || this->children->next) { - gf_log (this->name, GF_LOG_ERROR, + gf_msg (this->name, GF_LOG_ERROR, 0, + OPEN_BEHIND_MSG_XLATOR_CHILD_MISCONFIGURED, "FATAL: volume (%s) not configured with exactly one " "child", this->name); return -1; } if (!this->parents) - gf_log (this->name, GF_LOG_WARNING, + gf_msg (this->name, GF_LOG_WARNING, 0, + OPEN_BEHIND_MSG_VOL_MISCONFIGURED, "dangling volume. check volfile "); conf = GF_CALLOC (1, sizeof (*conf), gf_ob_mt_conf_t); |