btrfs: Remove fs_info argument from btrfs_uuid_tree_add
This function always takes a transaction handle which contains a reference to the fs_info. Use that and remove the extra argument. Signed-off-by: Lu Fengqi <lufq.fnst@cn.fujitsu.com> Reviewed-by: Nikolay Borisov <nborisov@suse.com> Signed-off-by: David Sterba <dsterba@suse.com>
This commit is contained in:
parent
f9ddfd0592
commit
cdb345a877
|
@ -3040,8 +3040,7 @@ void btrfs_update_root_times(struct btrfs_trans_handle *trans,
|
|||
struct btrfs_root *root);
|
||||
|
||||
/* uuid-tree.c */
|
||||
int btrfs_uuid_tree_add(struct btrfs_trans_handle *trans,
|
||||
struct btrfs_fs_info *fs_info, u8 *uuid, u8 type,
|
||||
int btrfs_uuid_tree_add(struct btrfs_trans_handle *trans, u8 *uuid, u8 type,
|
||||
u64 subid);
|
||||
int btrfs_uuid_tree_rem(struct btrfs_trans_handle *trans,
|
||||
struct btrfs_fs_info *fs_info, u8 *uuid, u8 type,
|
||||
|
|
|
@ -718,7 +718,7 @@ static noinline int create_subvol(struct inode *dir,
|
|||
btrfs_ino(BTRFS_I(dir)), index, name, namelen);
|
||||
BUG_ON(ret);
|
||||
|
||||
ret = btrfs_uuid_tree_add(trans, fs_info, root_item->uuid,
|
||||
ret = btrfs_uuid_tree_add(trans, root_item->uuid,
|
||||
BTRFS_UUID_KEY_SUBVOL, objectid);
|
||||
if (ret)
|
||||
btrfs_abort_transaction(trans, ret);
|
||||
|
@ -5043,7 +5043,7 @@ static long _btrfs_ioctl_set_received_subvol(struct file *file,
|
|||
goto out;
|
||||
}
|
||||
if (received_uuid_changed && !btrfs_is_empty_uuid(sa->uuid)) {
|
||||
ret = btrfs_uuid_tree_add(trans, fs_info, sa->uuid,
|
||||
ret = btrfs_uuid_tree_add(trans, sa->uuid,
|
||||
BTRFS_UUID_KEY_RECEIVED_SUBVOL,
|
||||
root->root_key.objectid);
|
||||
if (ret < 0 && ret != -EEXIST) {
|
||||
|
|
|
@ -1634,15 +1634,14 @@ static noinline int create_pending_snapshot(struct btrfs_trans_handle *trans,
|
|||
btrfs_abort_transaction(trans, ret);
|
||||
goto fail;
|
||||
}
|
||||
ret = btrfs_uuid_tree_add(trans, fs_info, new_uuid.b,
|
||||
BTRFS_UUID_KEY_SUBVOL, objectid);
|
||||
ret = btrfs_uuid_tree_add(trans, new_uuid.b, BTRFS_UUID_KEY_SUBVOL,
|
||||
objectid);
|
||||
if (ret) {
|
||||
btrfs_abort_transaction(trans, ret);
|
||||
goto fail;
|
||||
}
|
||||
if (!btrfs_is_empty_uuid(new_root_item->received_uuid)) {
|
||||
ret = btrfs_uuid_tree_add(trans, fs_info,
|
||||
new_root_item->received_uuid,
|
||||
ret = btrfs_uuid_tree_add(trans, new_root_item->received_uuid,
|
||||
BTRFS_UUID_KEY_RECEIVED_SUBVOL,
|
||||
objectid);
|
||||
if (ret && ret != -EEXIST) {
|
||||
|
|
|
@ -79,10 +79,10 @@ out:
|
|||
return ret;
|
||||
}
|
||||
|
||||
int btrfs_uuid_tree_add(struct btrfs_trans_handle *trans,
|
||||
struct btrfs_fs_info *fs_info, u8 *uuid, u8 type,
|
||||
int btrfs_uuid_tree_add(struct btrfs_trans_handle *trans, u8 *uuid, u8 type,
|
||||
u64 subid_cpu)
|
||||
{
|
||||
struct btrfs_fs_info *fs_info = trans->fs_info;
|
||||
struct btrfs_root *uuid_root = fs_info->uuid_root;
|
||||
int ret;
|
||||
struct btrfs_path *path = NULL;
|
||||
|
|
|
@ -4209,8 +4209,7 @@ static int btrfs_uuid_scan_kthread(void *data)
|
|||
}
|
||||
update_tree:
|
||||
if (!btrfs_is_empty_uuid(root_item.uuid)) {
|
||||
ret = btrfs_uuid_tree_add(trans, fs_info,
|
||||
root_item.uuid,
|
||||
ret = btrfs_uuid_tree_add(trans, root_item.uuid,
|
||||
BTRFS_UUID_KEY_SUBVOL,
|
||||
key.objectid);
|
||||
if (ret < 0) {
|
||||
|
@ -4221,7 +4220,7 @@ update_tree:
|
|||
}
|
||||
|
||||
if (!btrfs_is_empty_uuid(root_item.received_uuid)) {
|
||||
ret = btrfs_uuid_tree_add(trans, fs_info,
|
||||
ret = btrfs_uuid_tree_add(trans,
|
||||
root_item.received_uuid,
|
||||
BTRFS_UUID_KEY_RECEIVED_SUBVOL,
|
||||
key.objectid);
|
||||
|
|
Loading…
Reference in New Issue