github-mirror/linux
1
0
Fork 0
mirror of https://github.com/torvalds/linux.git synced 2026-08-01 12:11:59 +02:00
Code Issues Actions Packages Projects Releases Wiki Activity Graph

btrfs: push cleanup into btrfs_read_locked_inode()

Browse Source 
Move btrfs_add_inode_to_root() so it can be called from
btrfs_read_locked_inode(), no changes were made to the function.

Move cleanup code from btrfs_iget_path() to btrfs_read_locked_inode.
This improves readability and improves a leaky abstraction. Previously
btrfs_iget_path() had to handle a positive error case as a result of a
call to btrfs_search_slot(), but it makes more sense to handle this
closer to the source of the call.

Signed-off-by: Leo Martins <loemra.dev@gmail.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
This commit is contained in:
Leo Martins 2024-08-30 13:24:54 -07:00 committed by David Sterba
parent df3b8ca604
commit 69673992b1
1 changed files with 53 additions and 48 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

101
fs/btrfs/inode.c
View File

@ -3759,8 +3759,41 @@ static int btrfs_init_file_extent_tree(struct btrfs_inode *inode)
return 0; return 0;
} }
static int btrfs_add_inode_to_root(struct btrfs_inode *inode, bool prealloc)
{
struct btrfs_root *root = inode->root;
struct btrfs_inode *existing;
const u64 ino = btrfs_ino(inode);
int ret;
if (inode_unhashed(&inode->vfs_inode))
return 0;
if (prealloc) {
ret = xa_reserve(&root->inodes, ino, GFP_NOFS);
if (ret)
return ret;
}
existing = xa_store(&root->inodes, ino, inode, GFP_ATOMIC);
if (xa_is_err(existing)) {
ret = xa_err(existing);
ASSERT(ret != -EINVAL);
ASSERT(ret != -ENOMEM);
return ret;
} else if (existing) {
WARN_ON(!(existing->vfs_inode.i_state & (I_WILL_FREE | I_FREEING)));
}
return 0;
}
/* /*
* read an inode from the btree into the in-memory inode * Read a locked inode from the btree into the in-memory inode and add it to
* its root list/tree.
*
* On failure clean up the inode.
*/ */
static int btrfs_read_locked_inode(struct inode *inode, static int btrfs_read_locked_inode(struct inode *inode,
struct btrfs_path *in_path) struct btrfs_path *in_path)
@ -3780,7 +3813,7 @@ static int btrfs_read_locked_inode(struct inode *inode,
ret = btrfs_init_file_extent_tree(BTRFS_I(inode)); ret = btrfs_init_file_extent_tree(BTRFS_I(inode));
if (ret) if (ret)
return ret; goto out;
ret = btrfs_fill_inode(inode, &rdev); ret = btrfs_fill_inode(inode, &rdev);
if (!ret) if (!ret)
@ -3789,7 +3822,7 @@ static int btrfs_read_locked_inode(struct inode *inode,
if (!path) { if (!path) {
path = btrfs_alloc_path(); path = btrfs_alloc_path();
if (!path) if (!path)
return -ENOMEM; goto out;
} }
btrfs_get_inode_key(BTRFS_I(inode), &location); btrfs_get_inode_key(BTRFS_I(inode), &location);
@ -3798,7 +3831,13 @@ static int btrfs_read_locked_inode(struct inode *inode,
if (ret) { if (ret) {
if (path != in_path) if (path != in_path)
btrfs_free_path(path); btrfs_free_path(path);
return ret; /*
* ret > 0 can come from btrfs_search_slot called by
* btrfs_lookup_inode(), this means the inode was not found.
*/
if (ret > 0)
ret = -ENOENT;
goto out;
} }
leaf = path->nodes[0]; leaf = path->nodes[0];
@ -3961,7 +4000,15 @@ static int btrfs_read_locked_inode(struct inode *inode,
} }
btrfs_sync_inode_flags_to_i_flags(inode); btrfs_sync_inode_flags_to_i_flags(inode);
ret = btrfs_add_inode_to_root(BTRFS_I(inode), true);
if (ret)
goto out;
return 0; return 0;
out:
iget_failed(inode);
return ret;
} }
/* /*
@ -5470,35 +5517,7 @@ static int fixup_tree_root_location(struct btrfs_fs_info *fs_info,
return err; return err;
} }
static int btrfs_add_inode_to_root(struct btrfs_inode *inode, bool prealloc)
{
struct btrfs_root *root = inode->root;
struct btrfs_inode *existing;
const u64 ino = btrfs_ino(inode);
int ret;
if (inode_unhashed(&inode->vfs_inode))
return 0;
if (prealloc) {
ret = xa_reserve(&root->inodes, ino, GFP_NOFS);
if (ret)
return ret;
}
existing = xa_store(&root->inodes, ino, inode, GFP_ATOMIC);
if (xa_is_err(existing)) {
ret = xa_err(existing);
ASSERT(ret != -EINVAL);
ASSERT(ret != -ENOMEM);
return ret;
} else if (existing) {
WARN_ON(!(existing->vfs_inode.i_state & (I_WILL_FREE | I_FREEING)));
}
return 0;
}
static void btrfs_del_inode_from_root(struct btrfs_inode *inode) static void btrfs_del_inode_from_root(struct btrfs_inode *inode)
{ {
@ -5579,25 +5598,11 @@ struct inode *btrfs_iget_path(u64 ino, struct btrfs_root *root,
return inode; return inode;
ret = btrfs_read_locked_inode(inode, path); ret = btrfs_read_locked_inode(inode, path);
/* if (ret)
* ret > 0 can come from btrfs_search_slot called by return ERR_PTR(ret);
* btrfs_read_locked_inode(), this means the inode item was not found.
*/
if (ret > 0)
ret = -ENOENT;
if (ret < 0)
goto error;
ret = btrfs_add_inode_to_root(BTRFS_I(inode), true);
if (ret < 0)
goto error;
unlock_new_inode(inode); unlock_new_inode(inode);
return inode; return inode;
error:
iget_failed(inode);
return ERR_PTR(ret);
} }
struct inode *btrfs_iget(u64 ino, struct btrfs_root *root) struct inode *btrfs_iget(u64 ino, struct btrfs_root *root)