Orangefs: make some gossip statements more helpful.

Signed-off-by: Mike Marshall <hubcap@omnibond.com>

diff --git a/fs/orangefs/inode.c b/fs/orangefs/inode.c
index d2923dc..4e923ec 100644 (file)
--- a/fs/orangefs/inode.c
+++ b/fs/orangefs/inode.c
@@ -443,8 +443,8 @@ struct inode *orangefs_new_inode(struct super_block *sb, struct inode *dir,
        int error;
 
        gossip_debug(GOSSIP_INODE_DEBUG,
-                    "orangefs_get_custom_inode_common: called\n"
-                    "(sb is %p | MAJOR(dev)=%u | MINOR(dev)=%u mode=%o)\n",
+                    "%s:(sb is %p | MAJOR(dev)=%u | MINOR(dev)=%u mode=%o)\n",
+                    __func__,
                     sb,
                     MAJOR(dev),
                     MINOR(dev),

diff --git a/fs/orangefs/namei.c b/fs/orangefs/namei.c
index 8fc55c6..b3ae374 100644 (file)
--- a/fs/orangefs/namei.c
+++ b/fs/orangefs/namei.c
@@ -24,7 +24,9 @@ static int orangefs_create(struct inode *dir,
        struct inode *inode;
        int ret;
 
-       gossip_debug(GOSSIP_NAME_DEBUG, "%s: called\n", __func__);
+       gossip_debug(GOSSIP_NAME_DEBUG, "%s: %s\n",
+                    __func__,
+                    dentry->d_name.name);
 
        new_op = op_alloc(ORANGEFS_VFS_OP_CREATE);
        if (!new_op)
@@ -41,35 +43,39 @@ static int orangefs_create(struct inode *dir,
        ret = service_operation(new_op, __func__, get_interruptible_flag(dir));
 
        gossip_debug(GOSSIP_NAME_DEBUG,
-                    "Create Got ORANGEFS handle %pU on fsid %d (ret=%d)\n",
+                    "%s: %s: handle:%pU: fsid:%d: new_op:%p: ret:%d:\n",
+                    __func__,
+                    dentry->d_name.name,
                     &new_op->downcall.resp.create.refn.khandle,
-                    new_op->downcall.resp.create.refn.fs_id, ret);
+                    new_op->downcall.resp.create.refn.fs_id,
+                    new_op,
+                    ret);
 
-       if (ret < 0) {
-               gossip_debug(GOSSIP_NAME_DEBUG,
-                            "%s: failed with error code %d\n",
-                            __func__, ret);
+       if (ret < 0)
                goto out;
-       }
 
        inode = orangefs_new_inode(dir->i_sb, dir, S_IFREG | mode, 0,
                                &new_op->downcall.resp.create.refn);
        if (IS_ERR(inode)) {
-               gossip_err("*** Failed to allocate orangefs file inode\n");
+               gossip_err("%s: Failed to allocate inode for file :%s:\n",
+                          __func__,
+                          dentry->d_name.name);
                ret = PTR_ERR(inode);
                goto out;
        }
 
        gossip_debug(GOSSIP_NAME_DEBUG,
-                    "Assigned file inode new number of %pU\n",
-                    get_khandle_from_ino(inode));
+                    "%s: Assigned inode :%pU: for file :%s:\n",
+                    __func__,
+                    get_khandle_from_ino(inode),
+                    dentry->d_name.name);
 
        d_instantiate(dentry, inode);
        unlock_new_inode(inode);
 
        gossip_debug(GOSSIP_NAME_DEBUG,
-                    "Inode (Regular File) %pU -> %s\n",
-                    get_khandle_from_ino(inode),
+                    "%s: dentry instantiated for %s\n",
+                    __func__,
                     dentry->d_name.name);
 
        SetMtimeFlag(parent);
@@ -78,7 +84,11 @@ static int orangefs_create(struct inode *dir,
        ret = 0;
 out:
        op_release(new_op);
-       gossip_debug(GOSSIP_NAME_DEBUG, "%s: returning %d\n", __func__, ret);
+       gossip_debug(GOSSIP_NAME_DEBUG,
+                    "%s: %s: returning %d\n",
+                    __func__,
+                    dentry->d_name.name,
+                    ret);
        return ret;
 }
 
@@ -229,7 +239,11 @@ static int orangefs_unlink(struct inode *dir, struct dentry *dentry)
        ret = service_operation(new_op, "orangefs_unlink",
                                get_interruptible_flag(inode));
 
-       /* when request is serviced properly, free req op struct */
+       gossip_debug(GOSSIP_NAME_DEBUG,
+                    "%s: service_operation returned:%d:\n",
+                    __func__,
+                    ret);
+
        op_release(new_op);
 
        if (!ret) {

diff --git a/fs/orangefs/waitqueue.c b/fs/orangefs/waitqueue.c
index 36eedd6..2c47f15 100644 (file)
--- a/fs/orangefs/waitqueue.c
+++ b/fs/orangefs/waitqueue.c
@@ -67,11 +67,10 @@ int service_operation(struct orangefs_kernel_op_s *op,
 retry_servicing:
        op->downcall.status = 0;
        gossip_debug(GOSSIP_WAIT_DEBUG,
-                    "orangefs: service_operation: %s %p\n",
+                    "%s: %s op:%p: process:%s: pid:%d:\n",
+                    __func__,
                     op_name,
-                    op);
-       gossip_debug(GOSSIP_WAIT_DEBUG,
-                    "orangefs: operation posted by process: %s, pid: %i\n",
+                    op,
                     current->comm,
                     current->pid);
 
@@ -122,6 +121,13 @@ retry_servicing:
 
        ret = wait_for_matching_downcall(op, timeout,
                                         flags & ORANGEFS_OP_INTERRUPTIBLE);
+
+       gossip_debug(GOSSIP_WAIT_DEBUG,
+                    "%s: wait_for_matching_downcall returned %d for %p\n",
+                    __func__,
+                    ret,
+                    op);
+
        if (!ret) {
                spin_unlock(&op->lock);
                /* got matching downcall; make sure status is in errno format */