[Btrfs-devel] a few cleanup codes for ctree.c
Yan Zheng
yanzheng at 21cn.com
Fri Aug 24 05:10:55 PDT 2007
hello,
All changes but one are related to release buffer_head, the change in
balance_node_right is due to test always return false. By the way, I
found a few locking issues in file.c in error condition. I will try to
fix that, if you agree ;)
Regards
YZ
diff -r 6125224d77d0 ctree.c
--- a/ctree.c Fri Aug 10 16:22:09 2007 -0400
+++ b/ctree.c Fri Aug 24 17:13:58 2007 +0800
@@ -234,8 +234,10 @@ int btrfs_realloc_node(struct btrfs_tran
err = __btrfs_cow_block(trans, root, cur_bh, parent, i,
&tmp_bh, search_start,
min(8, end_slot - i));
- if (err)
+ if (err) {
+ brelse(cur_bh);
break;
+ }
search_start = bh_blocknr(tmp_bh);
*last_ret = search_start;
if (parent_level == 1)
@@ -845,7 +847,6 @@ int btrfs_search_slot(struct btrfs_trans
ins_len, int cow)
{
struct buffer_head *b;
- struct buffer_head *cow_buf;
struct btrfs_node *c;
u64 blocknr;
int slot;
@@ -869,12 +870,11 @@ again:
wret = btrfs_cow_block(trans, root, b,
p->nodes[level + 1],
p->slots[level + 1],
- &cow_buf);
+ &b);
if (wret) {
- btrfs_block_release(root, cow_buf);
+ btrfs_block_release(root, b);
return wret;
}
- b = cow_buf;
c = btrfs_buffer_node(b);
}
BUG_ON(!cow && ins_len);
@@ -1039,7 +1039,7 @@ static int balance_node_right(struct btr
max_push = src_nritems / 2 + 1;
/* don't try to empty the node */
- if (max_push > src_nritems)
+ if (max_push >= src_nritems)
return 1;
if (max_push < push_items)
push_items = max_push;
@@ -1429,6 +1429,7 @@ static int push_leaf_left(struct btrfs_t
ret = btrfs_cow_block(trans, root, t, path->nodes[1], slot - 1, &t);
if (ret) {
/* we hit -ENOSPC, but it isn't fatal here */
+ btrfs_block_release(root, t);
return 1;
}
left = btrfs_buffer_leaf(t);
More information about the Btrfs-devel
mailing list