diff options
| author | Rajesh Amaravathi <rajesh.amaravathi@gmail.com> | 2011-09-19 13:22:54 +0530 |
|---|---|---|
| committer | Vijay Bellur <vijay@gluster.com> | 2011-09-19 01:53:28 -0700 |
| commit | 909f80181c9a6985c49964de4854f5d5460dbf09 (patch) | |
| tree | 4ac05782c115ec818c809ea974673e97e189d39f | |
| parent | d70f640ea121643fadef2f19a33d18d81c9f5bb5 (diff) | |
protocol/client: minor log enhancements
minor changes to the log enhancements of bug 3473.
Change-Id: Id38d29db5a744e0ab7342d10ead6d16866228062
BUG: 3473
Reviewed-on: http://review.gluster.com/452
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Amar Tumballi <amar@gluster.com>
| -rw-r--r-- | xlators/protocol/client/src/client3_1-fops.c | 35 |
1 files changed, 14 insertions, 21 deletions
diff --git a/xlators/protocol/client/src/client3_1-fops.c b/xlators/protocol/client/src/client3_1-fops.c index 8e2c153d4..e9c5d626b 100644 --- a/xlators/protocol/client/src/client3_1-fops.c +++ b/xlators/protocol/client/src/client3_1-fops.c @@ -735,9 +735,8 @@ out: frame->local = NULL; if (rsp.op_ret == -1) { gf_log (this->name, GF_LOG_WARNING, - "remote operation failed: %s. Path: %s", - strerror (gf_error_to_errno (rsp.op_errno)), - (local) ? local->loc.path : "--"); + "remote operation failed: %s", + strerror (gf_error_to_errno (rsp.op_errno))); } STACK_UNWIND_STRICT (flush, frame, rsp.op_ret, gf_error_to_errno (rsp.op_errno)); @@ -980,9 +979,8 @@ client3_1_fgetxattr_cbk (struct rpc_req *req, struct iovec *iov, int count, out: if (rsp.op_ret == -1) { gf_log (this->name, GF_LOG_WARNING, - "remote operation failed: %s. Path: %s", - strerror (op_errno), - (local) ? local->loc.path : "--"); + "remote operation failed: %s", + strerror (op_errno)); } STACK_UNWIND_STRICT (fgetxattr, frame, op_ret, op_errno, dict); if (rsp.dict.dict_val) { @@ -1508,9 +1506,8 @@ out: if (rsp.op_ret == -1) { gf_log (this->name, GF_LOG_WARNING, - "remote operation failed: %s. Path: %s", - strerror (gf_error_to_errno (rsp.op_errno)), - (local) ? local->loc.path : "--"); + "remote operation failed: %s", + strerror (gf_error_to_errno (rsp.op_errno))); } STACK_UNWIND_STRICT (fxattrop, frame, op_ret, gf_error_to_errno (op_errno), dict); @@ -1846,9 +1843,8 @@ out: if ((rsp.op_ret == -1) && (EAGAIN != gf_error_to_errno (rsp.op_errno))) { gf_log (this->name, GF_LOG_WARNING, - "remote operation failed: %s. Path: %s", - strerror (gf_error_to_errno (rsp.op_errno)), - (local) ? local->loc.path : "--"); + "remote operation failed: %s", + strerror (gf_error_to_errno (rsp.op_errno))); } STACK_UNWIND_STRICT (lk, frame, rsp.op_ret, gf_error_to_errno (rsp.op_errno), &lock); @@ -1896,9 +1892,8 @@ client3_1_readdir_cbk (struct rpc_req *req, struct iovec *iov, int count, out: if (rsp.op_ret == -1) { gf_log (this->name, GF_LOG_WARNING, - "remote operation failed: %s. Path: %s", - strerror (gf_error_to_errno (rsp.op_errno)), - (local) ? local->loc.path : "--"); + "remote operation failed: %s", + strerror (gf_error_to_errno (rsp.op_errno))); } STACK_UNWIND_STRICT (readdir, frame, rsp.op_ret, gf_error_to_errno (rsp.op_errno), &entries); @@ -1954,9 +1949,8 @@ client3_1_readdirp_cbk (struct rpc_req *req, struct iovec *iov, int count, out: if (rsp.op_ret == -1) { gf_log (this->name, GF_LOG_WARNING, - "remote operation failed: %s. Path: %s", - strerror (gf_error_to_errno (rsp.op_errno)), - (local) ? local->loc.path : "--"); + "remote operation failed: %s", + strerror (gf_error_to_errno (rsp.op_errno))); } STACK_UNWIND_STRICT (readdirp, frame, rsp.op_ret, gf_error_to_errno (rsp.op_errno), &entries); @@ -2327,9 +2321,8 @@ client3_1_readv_cbk (struct rpc_req *req, struct iovec *iov, int count, out: if (rsp.op_ret == -1) { gf_log (this->name, GF_LOG_WARNING, - "remote operation failed: %s. Path: %s", - strerror (gf_error_to_errno (rsp.op_errno)), - (local) ? local->loc.path : "--"); + "remote operation failed: %s", + strerror (gf_error_to_errno (rsp.op_errno))); } STACK_UNWIND_STRICT (readv, frame, rsp.op_ret, gf_error_to_errno (rsp.op_errno), vector, rspcount, |