diff options
author | Leo Martins <loemra.dev@gmail.com> | 2024-08-30 13:24:55 -0700 |
---|---|---|
committer | David Sterba <dsterba@suse.com> | 2024-11-11 14:34:22 +0100 |
commit | 7c855e16ab72596d771355050ffe026e6b99f91c (patch) | |
tree | ed2808ca1dfe8f627548f271ab5cfe6d07db20f7 /fs/jbd2/commit.c | |
parent | 69673992b1aea5540199d9b8b658ede72f55a6cf (diff) |
btrfs: remove conditional path allocation in btrfs_read_locked_inode()
Remove conditional path allocation from btrfs_read_locked_inode(). Add
an ASSERT(path) to indicate it should never be called with a NULL path.
Call btrfs_read_locked_inode() directly from btrfs_iget(). This causes
code duplication between btrfs_iget() and btrfs_iget_path(), but I
think this is justifiable as it removes the need for conditionally
allocating the path inside of btrfs_read_locked_inode(). This makes the
code easier to reason about and makes it clear who has the
responsibility of allocating and freeing the path.
Signed-off-by: Leo Martins <loemra.dev@gmail.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/jbd2/commit.c')
0 files changed, 0 insertions, 0 deletions