summaryrefslogtreecommitdiffstats
path: root/xlators/mgmt/glusterd/src/glusterd-locks.c
diff options
context:
space:
mode:
authorNandaja Varma <nandaja.varma@gmail.com>2015-07-09 11:37:16 +0530
committerAtin Mukherjee <amukherj@redhat.com>2015-07-29 22:30:35 -0700
commit68059dbe0116150cac06958f95832fd1bda54cfb (patch)
treebe27291fadfe7c86343182add1b88e87a3c385d5 /xlators/mgmt/glusterd/src/glusterd-locks.c
parent62ba572f12d3300cec70a68d959d16a9a74023dd (diff)
glusterd: Porting the left out gf_log_callingfns to new framework.
Change-Id: I1b0ad54238895475ddbacc4fffacac8dc6e887fe BUG: 1244721 Signed-off-by: Nandaja Varma <nandaja.varma@gmail.com> Reviewed-on: http://review.gluster.org/11719 Tested-by: NetBSD Build System <jenkins@build.gluster.org> Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Atin Mukherjee <amukherj@redhat.com>
Diffstat (limited to 'xlators/mgmt/glusterd/src/glusterd-locks.c')
-rw-r--r--xlators/mgmt/glusterd/src/glusterd-locks.c15
1 files changed, 10 insertions, 5 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-locks.c b/xlators/mgmt/glusterd/src/glusterd-locks.c
index a16e499df68..272dd079585 100644
--- a/xlators/mgmt/glusterd/src/glusterd-locks.c
+++ b/xlators/mgmt/glusterd/src/glusterd-locks.c
@@ -541,7 +541,8 @@ glusterd_mgmt_v3_lock (const char *name, uuid_t uuid, uint32_t *op_errno,
is_valid = glusterd_mgmt_v3_is_type_valid (type);
if (is_valid != _gf_true) {
- gf_log_callingfn (this->name, GF_LOG_ERROR,
+ gf_msg_callingfn (this->name, GF_LOG_ERROR,
+ EINVAL, GD_MSG_INVALID_ENTRY,
"Invalid entity. Cannot perform locking "
"operation on %s types", type);
ret = -1;
@@ -570,7 +571,8 @@ glusterd_mgmt_v3_lock (const char *name, uuid_t uuid, uint32_t *op_errno,
/* If the lock has already been held for the given volume
* we fail */
if (!gf_uuid_is_null (owner)) {
- gf_log_callingfn (this->name, GF_LOG_WARNING,
+ gf_msg_callingfn (this->name, GF_LOG_WARNING,
+ 0, GD_MSG_LOCK_ALREADY_HELD,
"Lock for %s held by %s",
name, uuid_utoa (owner));
ret = -1;
@@ -645,7 +647,8 @@ glusterd_mgmt_v3_unlock (const char *name, uuid_t uuid, char *type)
is_valid = glusterd_mgmt_v3_is_type_valid (type);
if (is_valid != _gf_true) {
- gf_log_callingfn (this->name, GF_LOG_ERROR,
+ gf_msg_callingfn (this->name, GF_LOG_ERROR, EINVAL,
+ GD_MSG_INVALID_ENTRY,
"Invalid entity. Cannot perform unlocking "
"operation on %s types", type);
ret = -1;
@@ -673,7 +676,8 @@ glusterd_mgmt_v3_unlock (const char *name, uuid_t uuid, char *type)
}
if (gf_uuid_is_null (owner)) {
- gf_log_callingfn (this->name, GF_LOG_WARNING,
+ gf_msg_callingfn (this->name, GF_LOG_WARNING,
+ 0, GD_MSG_LOCK_NOT_HELD,
"Lock for %s %s not held", type, name);
ret = -1;
goto out;
@@ -681,7 +685,8 @@ glusterd_mgmt_v3_unlock (const char *name, uuid_t uuid, char *type)
ret = gf_uuid_compare (uuid, owner);
if (ret) {
- gf_log_callingfn (this->name, GF_LOG_WARNING,
+ gf_msg_callingfn (this->name, GF_LOG_WARNING,
+ 0, GD_MSG_LOCK_OWNER_MISMATCH,
"Lock owner mismatch. "
"Lock for %s %s held by %s",
type, name, uuid_utoa (owner));