protocol/client: log enhancements

* print paths wherever it is possible to log, to help debugging.
* bring uniformity in log level.

Change-Id: I2fa85b629de5dd0f0057ed96cba08ecb0ff1a798
BUG: 3473
Reviewed-on: http://review.gluster.com/328
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Amar Tumballi <amar@gluster.com>

1 files changed, 109 insertions, 90 deletions

diff --git a/xlators/protocol/client/src/client3_1-fops.c b/xlators/protocol/client/src/client3_1-fops.c
index 700e5da6beb..8e2c153d4bb 100644
--- a/xlators/protocol/client/src/client3_1-fops.c
+++ b/xlators/protocol/client/src/client3_1-fops.c
@@ -171,15 +171,16 @@ client3_1_symlink_cbk (struct rpc_req *req, struct iovec *iov, int count,
 out:
         frame->local = NULL;
         if (rsp.op_ret == -1) {
-                gf_log (this->name, GF_LOG_INFO, "remote operation failed: %s",
-                        strerror (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 : "--");
         }
         STACK_UNWIND_STRICT (symlink, frame, rsp.op_ret,
                              gf_error_to_errno (rsp.op_errno), inode, &stbuf,
                              &preparent, &postparent);
 
-        if (local)
-                client_local_wipe (local);
+         client_local_wipe (local);
 
         return 0;
 }
@@ -230,15 +231,16 @@ client3_1_mknod_cbk (struct rpc_req *req, struct iovec *iov, int count,
 out:
         frame->local = NULL;
         if (rsp.op_ret == -1) {
-                gf_log (this->name, GF_LOG_ERROR, "remote operation failed: %s",
-                        strerror (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 : "--");
         }
         STACK_UNWIND_STRICT (mknod, frame, rsp.op_ret,
                              gf_error_to_errno (rsp.op_errno), inode,
                              &stbuf, &preparent, &postparent);
 
-        if (local)
-                client_local_wipe (local);
+        client_local_wipe (local);
 
         return 0;
 }
@@ -262,7 +264,6 @@ client3_1_mkdir_cbk (struct rpc_req *req, struct iovec *iov, int count,
         frame = myframe;
 
         local = frame->local;
-        frame->local = NULL;
         inode = local->loc.inode;
 
         if (-1 == req->rpc_status) {
@@ -286,16 +287,18 @@ client3_1_mkdir_cbk (struct rpc_req *req, struct iovec *iov, int count,
         }
 
 out:
+        frame->local = NULL;
         if (rsp.op_ret == -1) {
-                gf_log (this->name, GF_LOG_INFO, "remote operation failed: %s",
-                        strerror (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 : "--");
         }
         STACK_UNWIND_STRICT (mkdir, frame, rsp.op_ret,
                              gf_error_to_errno (rsp.op_errno), inode,
                              &stbuf, &preparent, &postparent);
 
-        if (local)
-                client_local_wipe (local);
+        client_local_wipe (local);
 
         return 0;
 }
@@ -364,8 +367,10 @@ client3_1_open_cbk (struct rpc_req *req, struct iovec *iov, int count,
 out:
         frame->local = NULL;
         if (rsp.op_ret == -1) {
-                gf_log (this->name, GF_LOG_INFO, "remote operation failed: %s",
-                        strerror (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 : "--");
         }
         STACK_UNWIND_STRICT (open, frame, rsp.op_ret,
                              gf_error_to_errno (rsp.op_errno), fd);
@@ -409,7 +414,7 @@ client3_1_stat_cbk (struct rpc_req *req, struct iovec *iov, int count,
 
 out:
         if (rsp.op_ret == -1) {
-                gf_log (this->name, GF_LOG_INFO, "remote operation failed: %s",
+                gf_log (this->name, GF_LOG_WARNING, "remote operation failed: %s",
                         strerror (gf_error_to_errno (rsp.op_errno)));
         }
         STACK_UNWIND_STRICT (stat, frame, rsp.op_ret,
@@ -451,7 +456,7 @@ client3_1_readlink_cbk (struct rpc_req *req, struct iovec *iov, int count,
 
 out:
         if (rsp.op_ret == -1) {
-                gf_log (this->name, GF_LOG_INFO, "remote operation failed: %s",
+                gf_log (this->name, GF_LOG_WARNING, "remote operation failed: %s",
                         strerror (gf_error_to_errno (rsp.op_errno)));
         }
         STACK_UNWIND_STRICT (readlink, frame, rsp.op_ret,
@@ -500,7 +505,7 @@ client3_1_unlink_cbk (struct rpc_req *req, struct iovec *iov, int count,
 
 out:
         if (rsp.op_ret == -1) {
-                gf_log (this->name, GF_LOG_INFO, "remote operation failed: %s",
+                gf_log (this->name, GF_LOG_WARNING, "remote operation failed: %s",
                         strerror (gf_error_to_errno (rsp.op_errno)));
         }
         STACK_UNWIND_STRICT (unlink, frame, rsp.op_ret,
@@ -545,7 +550,7 @@ client3_1_rmdir_cbk (struct rpc_req *req, struct iovec *iov, int count,
 
 out:
         if (rsp.op_ret == -1) {
-                gf_log (this->name, GF_LOG_INFO, "remote operation failed: %s",
+                gf_log (this->name, GF_LOG_WARNING, "remote operation failed: %s",
                         strerror (gf_error_to_errno (rsp.op_errno)));
         }
         STACK_UNWIND_STRICT (rmdir, frame, rsp.op_ret,
@@ -591,7 +596,7 @@ client3_1_truncate_cbk (struct rpc_req *req, struct iovec *iov, int count,
 
 out:
         if (rsp.op_ret == -1) {
-                gf_log (this->name, GF_LOG_INFO, "remote operation failed: %s",
+                gf_log (this->name, GF_LOG_WARNING, "remote operation failed: %s",
                         strerror (gf_error_to_errno (rsp.op_errno)));
         }
         STACK_UNWIND_STRICT (truncate, frame, rsp.op_ret,
@@ -635,7 +640,7 @@ client3_1_statfs_cbk (struct rpc_req *req, struct iovec *iov, int count,
 
 out:
         if (rsp.op_ret == -1) {
-                gf_log (this->name, GF_LOG_INFO, "remote operation failed: %s",
+                gf_log (this->name, GF_LOG_WARNING, "remote operation failed: %s",
                         strerror (gf_error_to_errno (rsp.op_errno)));
         }
         STACK_UNWIND_STRICT (statfs, frame, rsp.op_ret,
@@ -681,7 +686,7 @@ client3_1_writev_cbk (struct rpc_req *req, struct iovec *iov, int count,
 
 out:
         if (rsp.op_ret == -1) {
-                gf_log (this->name, GF_LOG_INFO, "remote operation failed: %s",
+                gf_log (this->name, GF_LOG_WARNING, "remote operation failed: %s",
                         strerror (gf_error_to_errno (rsp.op_errno)));
         }
         STACK_UNWIND_STRICT (writev, frame, rsp.op_ret,
@@ -726,18 +731,19 @@ client3_1_flush_cbk (struct rpc_req *req, struct iovec *iov, int count,
                         (long long unsigned) local->owner, ret);
         }
 
-        frame->local = NULL;
-        if (local)
-                client_local_wipe (local);
-
 out:
+        frame->local = NULL;
         if (rsp.op_ret == -1) {
-                gf_log (this->name, GF_LOG_INFO, "remote operation failed: %s",
-                        strerror (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 : "--");
         }
         STACK_UNWIND_STRICT (flush, frame, rsp.op_ret,
                              gf_error_to_errno (rsp.op_errno));
 
+        client_local_wipe (local);
+
         return 0;
 }
 
@@ -777,7 +783,7 @@ client3_1_fsync_cbk (struct rpc_req *req, struct iovec *iov, int count,
 
 out:
         if (rsp.op_ret == -1) {
-                gf_log (this->name, GF_LOG_INFO, "remote operation failed: %s",
+                gf_log (this->name, GF_LOG_WARNING, "remote operation failed: %s",
                         strerror (gf_error_to_errno (rsp.op_errno)));
         }
         STACK_UNWIND_STRICT (fsync, frame, rsp.op_ret,
@@ -816,7 +822,7 @@ client3_1_setxattr_cbk (struct rpc_req *req, struct iovec *iov, int count,
 
 out:
         if (rsp.op_ret == -1) {
-                gf_log (this->name, GF_LOG_INFO, "remote operation failed: %s",
+                gf_log (this->name, GF_LOG_WARNING, "remote operation failed: %s",
                         strerror (gf_error_to_errno (rsp.op_errno)));
         }
         STACK_UNWIND_STRICT (setxattr, frame, rsp.op_ret,
@@ -888,8 +894,10 @@ client3_1_getxattr_cbk (struct rpc_req *req, struct iovec *iov, int count,
 
 out:
         if (rsp.op_ret == -1) {
-                gf_log (this->name, GF_LOG_INFO, "remote operation failed: %s",
-                        strerror (op_errno));
+                gf_log (this->name, GF_LOG_WARNING,
+                        "remote operation failed: %s. Path: %s",
+                        strerror (op_errno),
+                        (local) ? local->loc.path : "--");
         }
         STACK_UNWIND_STRICT (getxattr, frame, op_ret, op_errno, dict);
 
@@ -971,8 +979,10 @@ 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_INFO, "remote operation failed: %s",
-                        strerror (op_errno));
+                gf_log (this->name, GF_LOG_WARNING,
+                        "remote operation failed: %s. Path: %s",
+                        strerror (op_errno),
+                        (local) ? local->loc.path : "--");
         }
         STACK_UNWIND_STRICT (fgetxattr, frame, op_ret, op_errno, dict);
         if (rsp.dict.dict_val) {
@@ -988,8 +998,7 @@ out:
         if (dict)
                 dict_unref (dict);
 
-        if (local)
-                client_local_wipe (local);
+        client_local_wipe (local);
 
         return 0;
 }
@@ -1023,7 +1032,7 @@ client3_1_removexattr_cbk (struct rpc_req *req, struct iovec *iov, int count,
 
 out:
         if (rsp.op_ret == -1) {
-                gf_log (this->name, GF_LOG_INFO, "remote operation failed: %s",
+                gf_log (this->name, GF_LOG_WARNING, "remote operation failed: %s",
                         strerror (gf_error_to_errno (rsp.op_errno)));
         }
         STACK_UNWIND_STRICT (removexattr, frame, rsp.op_ret,
@@ -1060,7 +1069,7 @@ client3_1_fsyncdir_cbk (struct rpc_req *req, struct iovec *iov, int count,
 
 out:
         if (rsp.op_ret == -1) {
-                gf_log (this->name, GF_LOG_INFO, "remote operation failed: %s",
+                gf_log (this->name, GF_LOG_WARNING, "remote operation failed: %s",
                         strerror (gf_error_to_errno (rsp.op_errno)));
         }
         STACK_UNWIND_STRICT (fsyncdir, frame, rsp.op_ret,
@@ -1097,7 +1106,7 @@ client3_1_access_cbk (struct rpc_req *req, struct iovec *iov, int count,
 
 out:
         if (rsp.op_ret == -1) {
-                gf_log (this->name, GF_LOG_INFO, "remote operation failed: %s",
+                gf_log (this->name, GF_LOG_WARNING, "remote operation failed: %s",
                         strerror (gf_error_to_errno (rsp.op_errno)));
         }
         STACK_UNWIND_STRICT (access, frame, rsp.op_ret,
@@ -1142,7 +1151,7 @@ client3_1_ftruncate_cbk (struct rpc_req *req, struct iovec *iov, int count,
 
 out:
         if (rsp.op_ret == -1) {
-                gf_log (this->name, GF_LOG_INFO, "remote operation failed: %s",
+                gf_log (this->name, GF_LOG_WARNING, "remote operation failed: %s",
                         strerror (gf_error_to_errno (rsp.op_errno)));
         }
         STACK_UNWIND_STRICT (ftruncate, frame, rsp.op_ret,
@@ -1185,7 +1194,7 @@ client3_1_fstat_cbk (struct rpc_req *req, struct iovec *iov, int count,
 
 out:
         if (rsp.op_ret == -1) {
-                gf_log (this->name, GF_LOG_INFO, "remote operation failed: %s",
+                gf_log (this->name, GF_LOG_WARNING, "remote operation failed: %s",
                         strerror (gf_error_to_errno (rsp.op_errno)));
         }
         STACK_UNWIND_STRICT (fstat, frame, rsp.op_ret,
@@ -1224,7 +1233,7 @@ client3_1_inodelk_cbk (struct rpc_req *req, struct iovec *iov, int count,
 out:
         if ((rsp.op_ret == -1) &&
             (EAGAIN != gf_error_to_errno (rsp.op_errno))) {
-                gf_log (this->name, GF_LOG_INFO, "remote operation failed: %s",
+                gf_log (this->name, GF_LOG_WARNING, "remote operation failed: %s",
                         strerror (gf_error_to_errno (rsp.op_errno)));
         }
         STACK_UNWIND_STRICT (inodelk, frame, rsp.op_ret,
@@ -1262,7 +1271,7 @@ client3_1_finodelk_cbk (struct rpc_req *req, struct iovec *iov, int count,
 out:
         if ((rsp.op_ret == -1) &&
             (EAGAIN != gf_error_to_errno (rsp.op_errno))) {
-                gf_log (this->name, GF_LOG_INFO, "remote operation failed: %s",
+                gf_log (this->name, GF_LOG_WARNING, "remote operation failed: %s",
                         strerror (gf_error_to_errno (rsp.op_errno)));
         }
         STACK_UNWIND_STRICT (finodelk, frame, rsp.op_ret,
@@ -1301,7 +1310,7 @@ out:
 
         if ((rsp.op_ret == -1) &&
             (EAGAIN != gf_error_to_errno (rsp.op_errno))) {
-                gf_log (this->name, GF_LOG_INFO, "remote operation failed: %s",
+                gf_log (this->name, GF_LOG_WARNING, "remote operation failed: %s",
                         strerror (gf_error_to_errno (rsp.op_errno)));
         }
         STACK_UNWIND_STRICT (entrylk, frame, rsp.op_ret,
@@ -1339,7 +1348,7 @@ client3_1_fentrylk_cbk (struct rpc_req *req, struct iovec *iov, int count,
 out:
         if ((rsp.op_ret == -1) &&
             (EAGAIN != gf_error_to_errno (rsp.op_errno))) {
-                gf_log (this->name, GF_LOG_INFO, "remote operation failed: %s",
+                gf_log (this->name, GF_LOG_WARNING, "remote operation failed: %s",
                         strerror (gf_error_to_errno (rsp.op_errno)));
         }
         STACK_UNWIND_STRICT (fentrylk, frame, rsp.op_ret,
@@ -1410,8 +1419,10 @@ client3_1_xattrop_cbk (struct rpc_req *req, struct iovec *iov, int count,
 out:
 
         if (rsp.op_ret == -1) {
-                gf_log (this->name, GF_LOG_INFO, "remote operation failed: %s",
-                        strerror (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 : "--");
         }
         STACK_UNWIND_STRICT (xattrop, frame, op_ret,
                              gf_error_to_errno (op_errno), dict);
@@ -1496,8 +1507,10 @@ client3_1_fxattrop_cbk (struct rpc_req *req, struct iovec *iov, int count,
 out:
 
         if (rsp.op_ret == -1) {
-                gf_log (this->name, GF_LOG_INFO, "remote operation failed: %s",
-                        strerror (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 : "--");
         }
         STACK_UNWIND_STRICT (fxattrop, frame, op_ret,
                              gf_error_to_errno (op_errno), dict);
@@ -1547,7 +1560,7 @@ client3_1_fsetxattr_cbk (struct rpc_req *req, struct iovec *iov, int count,
 
 out:
         if (rsp.op_ret == -1) {
-                gf_log (this->name, GF_LOG_INFO, "remote operation failed: %s",
+                gf_log (this->name, GF_LOG_WARNING, "remote operation failed: %s",
                         strerror (gf_error_to_errno (rsp.op_errno)));
         }
         STACK_UNWIND_STRICT (fsetxattr, frame, rsp.op_ret,
@@ -1591,7 +1604,7 @@ client3_1_fsetattr_cbk (struct rpc_req *req, struct iovec *iov, int count,
 
 out:
         if (rsp.op_ret == -1) {
-                gf_log (this->name, GF_LOG_INFO, "remote operation failed: %s",
+                gf_log (this->name, GF_LOG_WARNING, "remote operation failed: %s",
                         strerror (gf_error_to_errno (rsp.op_errno)));
         }
         STACK_UNWIND_STRICT (fsetattr, frame, rsp.op_ret,
@@ -1638,7 +1651,7 @@ client3_1_setattr_cbk (struct rpc_req *req, struct iovec *iov, int count,
 
 out:
         if (rsp.op_ret == -1) {
-                gf_log (this->name, GF_LOG_INFO, "remote operation failed: %s",
+                gf_log (this->name, GF_LOG_WARNING, "remote operation failed: %s",
                         strerror (gf_error_to_errno (rsp.op_errno)));
         }
         STACK_UNWIND_STRICT (setattr, frame, rsp.op_ret,
@@ -1720,8 +1733,10 @@ client3_1_create_cbk (struct rpc_req *req, struct iovec *iov, int count,
 out:
         frame->local = NULL;
         if (rsp.op_ret == -1) {
-                gf_log (this->name, GF_LOG_INFO, "remote operation failed: %s",
-                        strerror (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 : "--");
         }
         STACK_UNWIND_STRICT (create, frame, rsp.op_ret,
                              gf_error_to_errno (rsp.op_errno), fd, inode,
@@ -1761,7 +1776,7 @@ client3_1_rchecksum_cbk (struct rpc_req *req, struct iovec *iov, int count,
 
 out:
         if (rsp.op_ret == -1) {
-                gf_log (this->name, GF_LOG_INFO, "remote operation failed: %s",
+                gf_log (this->name, GF_LOG_WARNING, "remote operation failed: %s",
                         strerror (gf_error_to_errno (rsp.op_errno)));
         }
         STACK_UNWIND_STRICT (rchecksum, frame, rsp.op_ret,
@@ -1826,18 +1841,19 @@ client3_1_lk_cbk (struct rpc_req *req, struct iovec *iov, int count,
         }
         */
 
-        frame->local = NULL;
-        client_local_wipe (local);
-
 out:
+        frame->local = NULL;
         if ((rsp.op_ret == -1) &&
             (EAGAIN != gf_error_to_errno (rsp.op_errno))) {
-                gf_log (this->name, GF_LOG_INFO, "remote operation failed: %s",
-                        strerror (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 : "--");
         }
         STACK_UNWIND_STRICT (lk, frame, rsp.op_ret,
                              gf_error_to_errno (rsp.op_errno), &lock);
 
+        client_local_wipe (local);
         return 0;
 }
 
@@ -1879,8 +1895,10 @@ 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_INFO, "remote operation failed: %s",
-                        strerror (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 : "--");
         }
         STACK_UNWIND_STRICT (readdir, frame, rsp.op_ret,
                              gf_error_to_errno (rsp.op_errno), &entries);
@@ -1935,8 +1953,10 @@ 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_INFO, "remote operation failed: %s",
-                        strerror (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 : "--");
         }
         STACK_UNWIND_STRICT (readdirp, frame, rsp.op_ret,
                              gf_error_to_errno (rsp.op_errno), &entries);
@@ -1997,7 +2017,7 @@ client3_1_rename_cbk (struct rpc_req *req, struct iovec *iov, int count,
 
 out:
         if (rsp.op_ret == -1) {
-                gf_log (this->name, GF_LOG_INFO, "remote operation failed: %s",
+                gf_log (this->name, GF_LOG_WARNING, "remote operation failed: %s",
                         strerror (gf_error_to_errno (rsp.op_errno)));
         }
         STACK_UNWIND_STRICT (rename, frame, rsp.op_ret,
@@ -2054,8 +2074,10 @@ client3_1_link_cbk (struct rpc_req *req, struct iovec *iov, int count,
 out:
         frame->local = NULL;
         if (rsp.op_ret == -1) {
-                gf_log (this->name, GF_LOG_INFO, "remote operation failed: %s",
-                        strerror (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 : "--");
         }
         STACK_UNWIND_STRICT (link, frame, rsp.op_ret,
                              gf_error_to_errno (rsp.op_errno), inode,
@@ -2130,8 +2152,10 @@ client3_1_opendir_cbk (struct rpc_req *req, struct iovec *iov, int count,
 out:
         frame->local = NULL;
         if (rsp.op_ret == -1) {
-                gf_log (this->name, GF_LOG_INFO, "remote operation failed: %s",
-                        strerror (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 : "--");
         }
         STACK_UNWIND_STRICT (opendir, frame, rsp.op_ret,
                              gf_error_to_errno (rsp.op_errno), fd);
@@ -2226,8 +2250,10 @@ out:
         if (rsp.op_ret == -1) {
                 /* any error other than ENOENT */
                 if (rsp.op_errno != ENOENT)
-                        gf_log (this->name, GF_LOG_INFO, "remote operation failed: %s",
-                                strerror (rsp.op_errno));
+                        gf_log (this->name, GF_LOG_WARNING,
+                                "remote operation failed: %s. Path: %s",
+                                strerror (rsp.op_errno),
+                                (local) ? local->loc.path : "--");
                 else
                         gf_log (this->name, GF_LOG_TRACE, "not found on remote node");
 
@@ -2300,16 +2326,16 @@ 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_INFO, "remote operation failed: %s",
-                        strerror (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 : "--");
         }
         STACK_UNWIND_STRICT (readv, frame, rsp.op_ret,
                              gf_error_to_errno (rsp.op_errno), vector, rspcount,
                              &stat, iobref);
 
-        if (local) {
-                client_local_wipe (local);
-        }
+        client_local_wipe (local);
 
         return 0;
 }
@@ -2615,8 +2641,7 @@ unwind:
         STACK_UNWIND_STRICT (lookup, frame, -1, op_errno, NULL, NULL, NULL,
                              NULL);
 
-        if (local)
-                client_local_wipe (local);
+        client_local_wipe (local);
 
         if (req.dict.dict_val)
                 GF_FREE (req.dict.dict_val);
@@ -3015,8 +3040,7 @@ unwind:
 
         STACK_UNWIND_STRICT (symlink, frame, -1, op_errno, NULL, NULL, NULL, NULL);
 
-        if (local)
-                client_local_wipe (local);
+        client_local_wipe (local);
         if (req.dict.dict_val) {
                 GF_FREE (req.dict.dict_val);
         }
@@ -3210,8 +3234,7 @@ unwind:
 
         STACK_UNWIND_STRICT (mknod, frame, -1, op_errno, NULL, NULL, NULL, NULL);
 
-        if (local)
-                client_local_wipe (local);
+        client_local_wipe (local);
         if (req.dict.dict_val) {
                 GF_FREE (req.dict.dict_val);
         }
@@ -3291,8 +3314,7 @@ unwind:
 
         STACK_UNWIND_STRICT (mkdir, frame, -1, op_errno, NULL, NULL, NULL, NULL);
 
-        if (local)
-                client_local_wipe (local);
+        client_local_wipe (local);
         if (req.dict.dict_val) {
                 GF_FREE (req.dict.dict_val);
         }
@@ -3374,8 +3396,7 @@ unwind:
 
         STACK_UNWIND_STRICT (create, frame, -1, op_errno, NULL, NULL, NULL,
                              NULL, NULL);
-        if (local)
-                client_local_wipe (local);
+        client_local_wipe (local);
         if (req.dict.dict_val) {
                 GF_FREE (req.dict.dict_val);
         }
@@ -3441,8 +3462,7 @@ unwind:
 
         STACK_UNWIND_STRICT (open, frame, -1, op_errno, NULL);
 
-        if (local)
-                client_local_wipe (local);
+        client_local_wipe (local);
         return 0;
 }
 
@@ -3758,8 +3778,7 @@ unwind:
         if (frame)
                 frame->local = NULL;
         STACK_UNWIND_STRICT (opendir, frame, -1, op_errno, NULL);
-        if (local)
-                client_local_wipe (local);
+        client_local_wipe (local);
         return 0;
 }