diff --git a/fs/orangefs/inode.c b/fs/orangefs/inode.c index d2923dc91388..4e923ece1e09 100644 --- 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 8fc55c6f58db..b3ae3749a932 100644 --- 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 36eedd6a8335..2c47f159d1d8 100644 --- 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 */