orangefs: formatting cleanups
Signed-off-by: Mike Marshall <hubcap@omnibond.com>
This commit is contained in:
parent
f6a4b4c9d0
commit
95f5f88f89
|
@ -281,14 +281,17 @@ restart:
|
||||||
ret = copy_to_user(buf, &proto_ver, sizeof(__s32));
|
ret = copy_to_user(buf, &proto_ver, sizeof(__s32));
|
||||||
if (ret != 0)
|
if (ret != 0)
|
||||||
goto error;
|
goto error;
|
||||||
ret = copy_to_user(buf+sizeof(__s32), &magic, sizeof(__s32));
|
ret = copy_to_user(buf + sizeof(__s32), &magic, sizeof(__s32));
|
||||||
if (ret != 0)
|
if (ret != 0)
|
||||||
goto error;
|
goto error;
|
||||||
ret = copy_to_user(buf+2 * sizeof(__s32), &cur_op->tag, sizeof(__u64));
|
ret = copy_to_user(buf + 2 * sizeof(__s32),
|
||||||
|
&cur_op->tag,
|
||||||
|
sizeof(__u64));
|
||||||
if (ret != 0)
|
if (ret != 0)
|
||||||
goto error;
|
goto error;
|
||||||
ret = copy_to_user(buf+2*sizeof(__s32)+sizeof(__u64), &cur_op->upcall,
|
ret = copy_to_user(buf + 2 * sizeof(__s32) + sizeof(__u64),
|
||||||
sizeof(struct orangefs_upcall_s));
|
&cur_op->upcall,
|
||||||
|
sizeof(struct orangefs_upcall_s));
|
||||||
if (ret != 0)
|
if (ret != 0)
|
||||||
goto error;
|
goto error;
|
||||||
|
|
||||||
|
@ -381,7 +384,7 @@ static ssize_t orangefs_devreq_write_iter(struct kiocb *iocb,
|
||||||
(unsigned int) MAX_DEV_REQ_DOWNSIZE);
|
(unsigned int) MAX_DEV_REQ_DOWNSIZE);
|
||||||
return -EFAULT;
|
return -EFAULT;
|
||||||
}
|
}
|
||||||
|
|
||||||
if (!copy_from_iter_full(&head, head_size, iter)) {
|
if (!copy_from_iter_full(&head, head_size, iter)) {
|
||||||
gossip_err("%s: failed to copy head.\n", __func__);
|
gossip_err("%s: failed to copy head.\n", __func__);
|
||||||
return -EFAULT;
|
return -EFAULT;
|
||||||
|
@ -426,7 +429,7 @@ static ssize_t orangefs_devreq_write_iter(struct kiocb *iocb,
|
||||||
goto wakeup;
|
goto wakeup;
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* We've successfully peeled off the head and the downcall.
|
* We've successfully peeled off the head and the downcall.
|
||||||
* Something has gone awry if total doesn't equal the
|
* Something has gone awry if total doesn't equal the
|
||||||
* sum of head_size, downcall_size and trailer_size.
|
* sum of head_size, downcall_size and trailer_size.
|
||||||
*/
|
*/
|
||||||
|
@ -477,7 +480,7 @@ static ssize_t orangefs_devreq_write_iter(struct kiocb *iocb,
|
||||||
wakeup:
|
wakeup:
|
||||||
/*
|
/*
|
||||||
* Return to vfs waitqueue, and back to service_operation
|
* Return to vfs waitqueue, and back to service_operation
|
||||||
* through wait_for_matching_downcall.
|
* through wait_for_matching_downcall.
|
||||||
*/
|
*/
|
||||||
spin_lock(&op->lock);
|
spin_lock(&op->lock);
|
||||||
if (unlikely(op_is_cancel(op))) {
|
if (unlikely(op_is_cancel(op))) {
|
||||||
|
|
|
@ -162,7 +162,7 @@ populate_shared_memory:
|
||||||
else
|
else
|
||||||
ret = 0;
|
ret = 0;
|
||||||
break;
|
break;
|
||||||
/*
|
/*
|
||||||
* If the op was in progress when the interrupt
|
* If the op was in progress when the interrupt
|
||||||
* occurred, then the client-core was able to
|
* occurred, then the client-core was able to
|
||||||
* trigger the write.
|
* trigger the write.
|
||||||
|
|
|
@ -181,16 +181,15 @@ static int orangefs_setattr_size(struct inode *inode, struct iattr *iattr)
|
||||||
new_op->upcall.req.truncate.refn = orangefs_inode->refn;
|
new_op->upcall.req.truncate.refn = orangefs_inode->refn;
|
||||||
new_op->upcall.req.truncate.size = (__s64) iattr->ia_size;
|
new_op->upcall.req.truncate.size = (__s64) iattr->ia_size;
|
||||||
|
|
||||||
ret = service_operation(new_op, __func__,
|
ret = service_operation(new_op,
|
||||||
get_interruptible_flag(inode));
|
__func__,
|
||||||
|
get_interruptible_flag(inode));
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* the truncate has no downcall members to retrieve, but
|
* the truncate has no downcall members to retrieve, but
|
||||||
* the status value tells us if it went through ok or not
|
* the status value tells us if it went through ok or not
|
||||||
*/
|
*/
|
||||||
gossip_debug(GOSSIP_INODE_DEBUG,
|
gossip_debug(GOSSIP_INODE_DEBUG, "%s: ret:%d:\n", __func__, ret);
|
||||||
"orangefs: orangefs_truncate got return value of %d\n",
|
|
||||||
ret);
|
|
||||||
|
|
||||||
op_release(new_op);
|
op_release(new_op);
|
||||||
|
|
||||||
|
@ -212,8 +211,9 @@ int orangefs_setattr(struct dentry *dentry, struct iattr *iattr)
|
||||||
struct inode *inode = dentry->d_inode;
|
struct inode *inode = dentry->d_inode;
|
||||||
|
|
||||||
gossip_debug(GOSSIP_INODE_DEBUG,
|
gossip_debug(GOSSIP_INODE_DEBUG,
|
||||||
"orangefs_setattr: called on %pd\n",
|
"%s: called on %pd\n",
|
||||||
dentry);
|
__func__,
|
||||||
|
dentry);
|
||||||
|
|
||||||
ret = setattr_prepare(dentry, iattr);
|
ret = setattr_prepare(dentry, iattr);
|
||||||
if (ret)
|
if (ret)
|
||||||
|
@ -230,15 +230,16 @@ int orangefs_setattr(struct dentry *dentry, struct iattr *iattr)
|
||||||
|
|
||||||
ret = orangefs_inode_setattr(inode, iattr);
|
ret = orangefs_inode_setattr(inode, iattr);
|
||||||
gossip_debug(GOSSIP_INODE_DEBUG,
|
gossip_debug(GOSSIP_INODE_DEBUG,
|
||||||
"orangefs_setattr: inode_setattr returned %d\n",
|
"%s: orangefs_inode_setattr returned %d\n",
|
||||||
ret);
|
__func__,
|
||||||
|
ret);
|
||||||
|
|
||||||
if (!ret && (iattr->ia_valid & ATTR_MODE))
|
if (!ret && (iattr->ia_valid & ATTR_MODE))
|
||||||
/* change mod on a file that has ACLs */
|
/* change mod on a file that has ACLs */
|
||||||
ret = posix_acl_chmod(inode, inode->i_mode);
|
ret = posix_acl_chmod(inode, inode->i_mode);
|
||||||
|
|
||||||
out:
|
out:
|
||||||
gossip_debug(GOSSIP_INODE_DEBUG, "orangefs_setattr: returning %d\n", ret);
|
gossip_debug(GOSSIP_INODE_DEBUG, "%s: ret:%d:\n", __func__, ret);
|
||||||
return ret;
|
return ret;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -312,7 +313,7 @@ int orangefs_update_time(struct inode *inode, struct timespec *time, int flags)
|
||||||
return orangefs_inode_setattr(inode, &iattr);
|
return orangefs_inode_setattr(inode, &iattr);
|
||||||
}
|
}
|
||||||
|
|
||||||
/* ORANGEDS2 implementation of VFS inode operations for files */
|
/* ORANGEFS2 implementation of VFS inode operations for files */
|
||||||
static const struct inode_operations orangefs_file_inode_operations = {
|
static const struct inode_operations orangefs_file_inode_operations = {
|
||||||
.get_acl = orangefs_get_acl,
|
.get_acl = orangefs_get_acl,
|
||||||
.set_acl = orangefs_set_acl,
|
.set_acl = orangefs_set_acl,
|
||||||
|
@ -350,8 +351,8 @@ static int orangefs_init_iops(struct inode *inode)
|
||||||
}
|
}
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* Given a ORANGEFS object identifier (fsid, handle), convert it into a ino_t type
|
* Given an ORANGEFS object identifier (fsid, handle), convert it into
|
||||||
* that will be used as a hash-index from where the handle will
|
* a ino_t type that will be used as a hash-index from where the handle will
|
||||||
* be searched for in the VFS hash table of inodes.
|
* be searched for in the VFS hash table of inodes.
|
||||||
*/
|
*/
|
||||||
static inline ino_t orangefs_handle_hash(struct orangefs_object_kref *ref)
|
static inline ino_t orangefs_handle_hash(struct orangefs_object_kref *ref)
|
||||||
|
@ -381,8 +382,10 @@ static int orangefs_test_inode(struct inode *inode, void *data)
|
||||||
struct orangefs_inode_s *orangefs_inode = NULL;
|
struct orangefs_inode_s *orangefs_inode = NULL;
|
||||||
|
|
||||||
orangefs_inode = ORANGEFS_I(inode);
|
orangefs_inode = ORANGEFS_I(inode);
|
||||||
return (!ORANGEFS_khandle_cmp(&(orangefs_inode->refn.khandle), &(ref->khandle))
|
/* test handles and fs_ids... */
|
||||||
&& orangefs_inode->refn.fs_id == ref->fs_id);
|
return (!ORANGEFS_khandle_cmp(&(orangefs_inode->refn.khandle),
|
||||||
|
&(ref->khandle)) &&
|
||||||
|
orangefs_inode->refn.fs_id == ref->fs_id);
|
||||||
}
|
}
|
||||||
|
|
||||||
/*
|
/*
|
||||||
|
@ -390,16 +393,21 @@ static int orangefs_test_inode(struct inode *inode, void *data)
|
||||||
* file handle.
|
* file handle.
|
||||||
*
|
*
|
||||||
* @sb: the file system super block instance.
|
* @sb: the file system super block instance.
|
||||||
* @ref: The ORANGEFS object for which we are trying to locate an inode structure.
|
* @ref: The ORANGEFS object for which we are trying to locate an inode.
|
||||||
*/
|
*/
|
||||||
struct inode *orangefs_iget(struct super_block *sb, struct orangefs_object_kref *ref)
|
struct inode *orangefs_iget(struct super_block *sb,
|
||||||
|
struct orangefs_object_kref *ref)
|
||||||
{
|
{
|
||||||
struct inode *inode = NULL;
|
struct inode *inode = NULL;
|
||||||
unsigned long hash;
|
unsigned long hash;
|
||||||
int error;
|
int error;
|
||||||
|
|
||||||
hash = orangefs_handle_hash(ref);
|
hash = orangefs_handle_hash(ref);
|
||||||
inode = iget5_locked(sb, hash, orangefs_test_inode, orangefs_set_inode, ref);
|
inode = iget5_locked(sb,
|
||||||
|
hash,
|
||||||
|
orangefs_test_inode,
|
||||||
|
orangefs_set_inode,
|
||||||
|
ref);
|
||||||
if (!inode || !(inode->i_state & I_NEW))
|
if (!inode || !(inode->i_state & I_NEW))
|
||||||
return inode;
|
return inode;
|
||||||
|
|
||||||
|
|
|
@ -184,7 +184,7 @@ orangefs_bufmap_free(struct orangefs_bufmap *bufmap)
|
||||||
}
|
}
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* XXX: Can the size and shift change while the caller gives up the
|
* XXX: Can the size and shift change while the caller gives up the
|
||||||
* XXX: lock between calling this and doing something useful?
|
* XXX: lock between calling this and doing something useful?
|
||||||
*/
|
*/
|
||||||
|
|
||||||
|
@ -496,7 +496,7 @@ void orangefs_readdir_index_put(int buffer_index)
|
||||||
}
|
}
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* we've been handed an iovec, we need to copy it to
|
* we've been handed an iovec, we need to copy it to
|
||||||
* the shared memory descriptor at "buffer_index".
|
* the shared memory descriptor at "buffer_index".
|
||||||
*/
|
*/
|
||||||
int orangefs_bufmap_copy_from_iovec(struct iov_iter *iter,
|
int orangefs_bufmap_copy_from_iovec(struct iov_iter *iter,
|
||||||
|
|
|
@ -183,9 +183,9 @@ static inline int copy_attributes_from_inode(struct inode *inode,
|
||||||
attrs->mask |= ORANGEFS_ATTR_SYS_CTIME;
|
attrs->mask |= ORANGEFS_ATTR_SYS_CTIME;
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* ORANGEFS cannot set size with a setattr operation. Probably not likely
|
* ORANGEFS cannot set size with a setattr operation. Probably not
|
||||||
* to be requested through the VFS, but just in case, don't worry about
|
* likely to be requested through the VFS, but just in case, don't
|
||||||
* ATTR_SIZE
|
* worry about ATTR_SIZE
|
||||||
*/
|
*/
|
||||||
|
|
||||||
if (iattr->ia_valid & ATTR_MODE) {
|
if (iattr->ia_valid & ATTR_MODE) {
|
||||||
|
@ -200,14 +200,16 @@ static inline int copy_attributes_from_inode(struct inode *inode,
|
||||||
tmp_mode -= S_ISVTX;
|
tmp_mode -= S_ISVTX;
|
||||||
} else {
|
} else {
|
||||||
gossip_debug(GOSSIP_UTILS_DEBUG,
|
gossip_debug(GOSSIP_UTILS_DEBUG,
|
||||||
"User attempted to set sticky bit on non-root directory; returning EINVAL.\n");
|
"%s: setting sticky bit not supported.\n",
|
||||||
|
__func__);
|
||||||
return -EINVAL;
|
return -EINVAL;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
if (tmp_mode & (S_ISUID)) {
|
if (tmp_mode & (S_ISUID)) {
|
||||||
gossip_debug(GOSSIP_UTILS_DEBUG,
|
gossip_debug(GOSSIP_UTILS_DEBUG,
|
||||||
"Attempting to set setuid bit (not supported); returning EINVAL.\n");
|
"%s: setting setuid bit not supported.\n",
|
||||||
|
__func__);
|
||||||
return -EINVAL;
|
return -EINVAL;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -521,7 +523,9 @@ int orangefs_normalize_to_errno(__s32 error_code)
|
||||||
error_code = -ETIMEDOUT;
|
error_code = -ETIMEDOUT;
|
||||||
} else {
|
} else {
|
||||||
/* assume a default error code */
|
/* assume a default error code */
|
||||||
gossip_err("orangefs: warning: got error code without errno equivalent: %d.\n", error_code);
|
gossip_err("%s: bad error code :%d:.\n",
|
||||||
|
__func__,
|
||||||
|
error_code);
|
||||||
error_code = -EINVAL;
|
error_code = -EINVAL;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -538,7 +542,7 @@ int orangefs_normalize_to_errno(__s32 error_code)
|
||||||
* there is a bug somewhere.
|
* there is a bug somewhere.
|
||||||
*/
|
*/
|
||||||
} else {
|
} else {
|
||||||
gossip_err("orangefs: orangefs_normalize_to_errno: got error code which is not from ORANGEFS.\n");
|
gossip_err("%s: unknown error code.\n", __func__);
|
||||||
error_code = -EINVAL;
|
error_code = -EINVAL;
|
||||||
}
|
}
|
||||||
return error_code;
|
return error_code;
|
||||||
|
|
|
@ -156,9 +156,10 @@ static int orangefs_statfs(struct dentry *dentry, struct kstatfs *buf)
|
||||||
sb = dentry->d_sb;
|
sb = dentry->d_sb;
|
||||||
|
|
||||||
gossip_debug(GOSSIP_SUPER_DEBUG,
|
gossip_debug(GOSSIP_SUPER_DEBUG,
|
||||||
"orangefs_statfs: called on sb %p (fs_id is %d)\n",
|
"%s: called on sb %p (fs_id is %d)\n",
|
||||||
sb,
|
__func__,
|
||||||
(int)(ORANGEFS_SB(sb)->fs_id));
|
sb,
|
||||||
|
(int)(ORANGEFS_SB(sb)->fs_id));
|
||||||
|
|
||||||
new_op = op_alloc(ORANGEFS_VFS_OP_STATFS);
|
new_op = op_alloc(ORANGEFS_VFS_OP_STATFS);
|
||||||
if (!new_op)
|
if (!new_op)
|
||||||
|
@ -198,7 +199,7 @@ static int orangefs_statfs(struct dentry *dentry, struct kstatfs *buf)
|
||||||
|
|
||||||
out_op_release:
|
out_op_release:
|
||||||
op_release(new_op);
|
op_release(new_op);
|
||||||
gossip_debug(GOSSIP_SUPER_DEBUG, "orangefs_statfs: returning %d\n", ret);
|
gossip_debug(GOSSIP_SUPER_DEBUG, "%s: returning %d\n", __func__, ret);
|
||||||
return ret;
|
return ret;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue