diff options
author | Liu Hui <onlyflyer@gmail.com> | 2008-11-18 11:30:10 -0500 |
---|---|---|
committer | Chris Mason <chris.mason@oracle.com> | 2008-11-18 11:30:10 -0500 |
commit | b4eec2ca1167bae46295aeb66abd15fd52387845 (patch) | |
tree | bad79258bb437341611116d951f7a8711c4473a0 /fs/btrfs/ctree.c | |
parent | ea9e8b11bd1252dcbc23afefcf1a52ec6aa3c113 (diff) |
Btrfs: Some fixes for batching extent insert.
In insert_extents(), when ret==1 and last is not zero, it should
check if the current inserted item is the last item in this batching
inserts. If so, it should just break from loop. If not, 'cur =
insert_list->next' will make no sense because the list is empty now,
and 'op' will point to an unexpectable place.
There are also some trivial fixs in this patch including one comment
typo error and deleting two redundant lines.
Signed-off-by: Chris Mason <chris.mason@oracle.com>
Diffstat (limited to 'fs/btrfs/ctree.c')
-rw-r--r-- | fs/btrfs/ctree.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/fs/btrfs/ctree.c b/fs/btrfs/ctree.c index dd1c03aea2df..71ef0a2e2daa 100644 --- a/fs/btrfs/ctree.c +++ b/fs/btrfs/ctree.c | |||
@@ -3041,7 +3041,6 @@ int btrfs_insert_some_items(struct btrfs_trans_handle *trans, | |||
3041 | struct btrfs_item *item; | 3041 | struct btrfs_item *item; |
3042 | int ret = 0; | 3042 | int ret = 0; |
3043 | int slot; | 3043 | int slot; |
3044 | int slot_orig; | ||
3045 | int i; | 3044 | int i; |
3046 | u32 nritems; | 3045 | u32 nritems; |
3047 | u32 total_data = 0; | 3046 | u32 total_data = 0; |
@@ -3064,7 +3063,6 @@ int btrfs_insert_some_items(struct btrfs_trans_handle *trans, | |||
3064 | if (ret < 0) | 3063 | if (ret < 0) |
3065 | goto out; | 3064 | goto out; |
3066 | 3065 | ||
3067 | slot_orig = path->slots[0]; | ||
3068 | leaf = path->nodes[0]; | 3066 | leaf = path->nodes[0]; |
3069 | 3067 | ||
3070 | nritems = btrfs_header_nritems(leaf); | 3068 | nritems = btrfs_header_nritems(leaf); |