summaryrefslogtreecommitdiff
path: root/fs/btrfs
diff options
context:
space:
mode:
authorAnand Jain <anand.jain@oracle.com>2019-04-12 11:02:59 +0300
committerDavid Sterba <dsterba@suse.com>2019-04-29 20:02:44 +0300
commite3de9b159a927fc1fd65e4d090b981b781c58926 (patch)
tree1d9586278f68a8593f681d5c0c36d049001f0fae /fs/btrfs
parent04e6863b19c72279bcbeffa26d85d649ab9c8205 (diff)
downloadlinux-e3de9b159a927fc1fd65e4d090b981b781c58926.tar.xz
btrfs: cleanup btrfs_setxattr_trans and drop transaction parameter
Previous patch made sure that btrfs_setxattr_trans() is called only when transaction NULL. Clean up btrfs_setxattr_trans() and drop the parameter. Signed-off-by: Anand Jain <anand.jain@oracle.com> Reviewed-by: David Sterba <dsterba@suse.com> Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs')
-rw-r--r--fs/btrfs/acl.c2
-rw-r--r--fs/btrfs/props.c13
-rw-r--r--fs/btrfs/xattr.c9
-rw-r--r--fs/btrfs/xattr.h5
4 files changed, 12 insertions, 17 deletions
diff --git a/fs/btrfs/acl.c b/fs/btrfs/acl.c
index 7fe6551bc59b..a0af1b952c4d 100644
--- a/fs/btrfs/acl.c
+++ b/fs/btrfs/acl.c
@@ -96,7 +96,7 @@ static int __btrfs_set_acl(struct btrfs_trans_handle *trans,
if (trans)
ret = btrfs_setxattr(trans, inode, name, value, size, 0);
else
- ret = btrfs_setxattr_trans(NULL, inode, name, value, size, 0);
+ ret = btrfs_setxattr_trans(inode, name, value, size, 0);
out:
kfree(value);
diff --git a/fs/btrfs/props.c b/fs/btrfs/props.c
index a73c1bdc7b05..44b7bf647ab3 100644
--- a/fs/btrfs/props.c
+++ b/fs/btrfs/props.c
@@ -74,9 +74,8 @@ static int btrfs_set_prop(struct btrfs_trans_handle *trans, struct inode *inode,
ret = btrfs_setxattr(trans, inode, handler->xattr_name,
NULL, 0, flags);
else
- ret = btrfs_setxattr_trans(NULL, inode,
- handler->xattr_name, NULL, 0,
- flags);
+ ret = btrfs_setxattr_trans(inode, handler->xattr_name,
+ NULL, 0, flags);
if (ret)
return ret;
@@ -93,8 +92,8 @@ static int btrfs_set_prop(struct btrfs_trans_handle *trans, struct inode *inode,
ret = btrfs_setxattr(trans, inode, handler->xattr_name, value,
value_len, flags);
else
- ret = btrfs_setxattr_trans(NULL, inode, handler->xattr_name,
- value, value_len, flags);
+ ret = btrfs_setxattr_trans(inode, handler->xattr_name, value,
+ value_len, flags);
if (ret)
return ret;
@@ -104,8 +103,8 @@ static int btrfs_set_prop(struct btrfs_trans_handle *trans, struct inode *inode,
btrfs_setxattr(trans, inode, handler->xattr_name, NULL,
0, flags);
else
- btrfs_setxattr_trans(NULL, inode, handler->xattr_name,
- NULL, 0, flags);
+ btrfs_setxattr_trans(inode, handler->xattr_name, NULL,
+ 0, flags);
return ret;
}
diff --git a/fs/btrfs/xattr.c b/fs/btrfs/xattr.c
index 623d508f21a6..10da873d11f5 100644
--- a/fs/btrfs/xattr.c
+++ b/fs/btrfs/xattr.c
@@ -221,16 +221,13 @@ out:
/*
* @value: "" makes the attribute to empty, NULL removes it
*/
-int btrfs_setxattr_trans(struct btrfs_trans_handle *trans,
- struct inode *inode, const char *name,
+int btrfs_setxattr_trans(struct inode *inode, const char *name,
const void *value, size_t size, int flags)
{
struct btrfs_root *root = BTRFS_I(inode)->root;
+ struct btrfs_trans_handle *trans;
int ret;
- if (trans)
- return btrfs_setxattr(trans, inode, name, value, size, flags);
-
trans = btrfs_start_transaction(root, 2);
if (IS_ERR(trans))
return PTR_ERR(trans);
@@ -368,7 +365,7 @@ static int btrfs_xattr_handler_set(const struct xattr_handler *handler,
size_t size, int flags)
{
name = xattr_full_name(handler, name);
- return btrfs_setxattr_trans(NULL, inode, name, buffer, size, flags);
+ return btrfs_setxattr_trans(inode, name, buffer, size, flags);
}
static int btrfs_xattr_handler_set_prop(const struct xattr_handler *handler,
diff --git a/fs/btrfs/xattr.h b/fs/btrfs/xattr.h
index a95834cc3c04..1cd3fc0a8f17 100644
--- a/fs/btrfs/xattr.h
+++ b/fs/btrfs/xattr.h
@@ -14,9 +14,8 @@ int btrfs_getxattr(struct inode *inode, const char *name,
void *buffer, size_t size);
int btrfs_setxattr(struct btrfs_trans_handle *trans, struct inode *inode,
const char *name, const void *value, size_t size, int flags);
-int btrfs_setxattr_trans(struct btrfs_trans_handle *trans,
- struct inode *inode, const char *name,
- const void *value, size_t size, int flags);
+int btrfs_setxattr_trans(struct inode *inode, const char *name,
+ const void *value, size_t size, int flags);
ssize_t btrfs_listxattr(struct dentry *dentry, char *buffer, size_t size);
int btrfs_xattr_security_init(struct btrfs_trans_handle *trans,