Re: [PATCH v3 06/11] Btrfs: delete dead code in btrfs_orphan_commit_root()

2018-05-11 Thread Josef Bacik
On Fri, May 11, 2018 at 12:56:11AM -0700, Omar Sandoval wrote:
> From: Omar Sandoval 
> 
> btrfs_orphan_commit_root() tries to delete an orphan item for a
> subvolume in the tree root, but we don't actually insert that item in
> the first place. See commit 0a0d4415e338 ("Btrfs: delete dead code in
> btrfs_orphan_add()"). We can get rid of it.
> 
> Signed-off-by: Omar Sandoval 

Reviewed-by: Josef Bacik 

Thanks,

Josef
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH v3 06/11] Btrfs: delete dead code in btrfs_orphan_commit_root()

2018-05-11 Thread Omar Sandoval
From: Omar Sandoval 

btrfs_orphan_commit_root() tries to delete an orphan item for a
subvolume in the tree root, but we don't actually insert that item in
the first place. See commit 0a0d4415e338 ("Btrfs: delete dead code in
btrfs_orphan_add()"). We can get rid of it.

Signed-off-by: Omar Sandoval 
---
 fs/btrfs/inode.c | 12 
 1 file changed, 12 deletions(-)

diff --git a/fs/btrfs/inode.c b/fs/btrfs/inode.c
index e22f8c9f6459..6110387f0218 100644
--- a/fs/btrfs/inode.c
+++ b/fs/btrfs/inode.c
@@ -3302,7 +3302,6 @@ void btrfs_orphan_commit_root(struct btrfs_trans_handle 
*trans,
 {
struct btrfs_fs_info *fs_info = root->fs_info;
struct btrfs_block_rsv *block_rsv;
-   int ret;
 
if (atomic_read(&root->orphan_inodes) ||
root->orphan_cleanup_state != ORPHAN_CLEANUP_DONE)
@@ -3323,17 +3322,6 @@ void btrfs_orphan_commit_root(struct btrfs_trans_handle 
*trans,
root->orphan_block_rsv = NULL;
spin_unlock(&root->orphan_lock);
 
-   if (test_bit(BTRFS_ROOT_ORPHAN_ITEM_INSERTED, &root->state) &&
-   btrfs_root_refs(&root->root_item) > 0) {
-   ret = btrfs_del_orphan_item(trans, fs_info->tree_root,
-   root->root_key.objectid);
-   if (ret)
-   btrfs_abort_transaction(trans, ret);
-   else
-   clear_bit(BTRFS_ROOT_ORPHAN_ITEM_INSERTED,
- &root->state);
-   }
-
if (block_rsv) {
WARN_ON(block_rsv->size > 0);
btrfs_free_block_rsv(fs_info, block_rsv);
-- 
2.17.0

--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html