btrfs: merge _btrfs_set_prop helpers
btrfs_set_prop() is a redirect to __btrfs_set_prop() with the transaction handle equal to NULL. __btrfs_set_prop() in turn passes this to do_setxattr() which then transaction is actually created. Instead merge __btrfs_set_prop() to btrfs_set_prop(), and update the caller with NULL argument. Signed-off-by: Anand Jain <anand.jain@oracle.com> Reviewed-by: David Sterba <dsterba@suse.com> Signed-off-by: David Sterba <dsterba@suse.com>
This commit is contained in:
@@ -85,12 +85,9 @@ find_prop_handler(const char *name,
|
||||
return NULL;
|
||||
}
|
||||
|
||||
static int __btrfs_set_prop(struct btrfs_trans_handle *trans,
|
||||
struct inode *inode,
|
||||
const char *name,
|
||||
const char *value,
|
||||
size_t value_len,
|
||||
int flags)
|
||||
int btrfs_set_prop(struct btrfs_trans_handle *trans, struct inode *inode,
|
||||
const char *name, const char *value, size_t value_len,
|
||||
int flags)
|
||||
{
|
||||
const struct prop_handler *handler;
|
||||
int ret;
|
||||
@@ -133,15 +130,6 @@ static int __btrfs_set_prop(struct btrfs_trans_handle *trans,
|
||||
return 0;
|
||||
}
|
||||
|
||||
int btrfs_set_prop(struct inode *inode,
|
||||
const char *name,
|
||||
const char *value,
|
||||
size_t value_len,
|
||||
int flags)
|
||||
{
|
||||
return __btrfs_set_prop(NULL, inode, name, value, value_len, flags);
|
||||
}
|
||||
|
||||
static int iterate_object_props(struct btrfs_root *root,
|
||||
struct btrfs_path *path,
|
||||
u64 objectid,
|
||||
@@ -313,8 +301,8 @@ static int inherit_props(struct btrfs_trans_handle *trans,
|
||||
num_bytes, BTRFS_RESERVE_NO_FLUSH);
|
||||
if (ret)
|
||||
goto out;
|
||||
ret = __btrfs_set_prop(trans, inode, h->xattr_name,
|
||||
value, strlen(value), 0);
|
||||
ret = btrfs_set_prop(trans, inode, h->xattr_name, value,
|
||||
strlen(value), 0);
|
||||
btrfs_block_rsv_release(fs_info, trans->block_rsv, num_bytes);
|
||||
if (ret)
|
||||
goto out;
|
||||
|
Reference in New Issue
Block a user