Commit 3301958b authored by Jan Schmidt's avatar Jan Schmidt

Btrfs: add inodes before dropping the extent lock in find_all_leafs

We must build up the inode list with the extent lock held after following
indirect refs.

This also requires an extension to ulists, which allows to modify the stored
aux value in case a key already exists in the list.
Signed-off-by: default avatarJan Schmidt <list.btrfs@jan-o-sch.net>
parent 95a06077
...@@ -106,6 +106,7 @@ struct __prelim_ref { ...@@ -106,6 +106,7 @@ struct __prelim_ref {
struct btrfs_key key_for_search; struct btrfs_key key_for_search;
int level; int level;
int count; int count;
struct extent_inode_elem *inode_list;
u64 parent; u64 parent;
u64 wanted_disk_byte; u64 wanted_disk_byte;
}; };
...@@ -166,6 +167,7 @@ static int __add_prelim_ref(struct list_head *head, u64 root_id, ...@@ -166,6 +167,7 @@ static int __add_prelim_ref(struct list_head *head, u64 root_id,
else else
memset(&ref->key_for_search, 0, sizeof(ref->key_for_search)); memset(&ref->key_for_search, 0, sizeof(ref->key_for_search));
ref->inode_list = NULL;
ref->level = level; ref->level = level;
ref->count = count; ref->count = count;
ref->parent = parent; ref->parent = parent;
...@@ -181,14 +183,21 @@ static int add_all_parents(struct btrfs_root *root, struct btrfs_path *path, ...@@ -181,14 +183,21 @@ static int add_all_parents(struct btrfs_root *root, struct btrfs_path *path,
const u64 *extent_item_pos) const u64 *extent_item_pos)
{ {
int ret; int ret;
int slot; int slot = path->slots[level];
struct extent_buffer *eb = path->nodes[level]; struct extent_buffer *eb = path->nodes[level];
struct btrfs_file_extent_item *fi; struct btrfs_file_extent_item *fi;
struct extent_inode_elem *eie = NULL;
u64 disk_byte; u64 disk_byte;
u64 wanted_objectid = key->objectid; u64 wanted_objectid = key->objectid;
add_parent: add_parent:
ret = ulist_add(parents, eb->start, 0, GFP_NOFS); if (level == 0 && extent_item_pos) {
fi = btrfs_item_ptr(eb, slot, struct btrfs_file_extent_item);
ret = check_extent_in_eb(key, eb, fi, *extent_item_pos, &eie);
if (ret < 0)
return ret;
}
ret = ulist_add(parents, eb->start, (unsigned long)eie, GFP_NOFS);
if (ret < 0) if (ret < 0)
return ret; return ret;
...@@ -202,6 +211,7 @@ static int add_all_parents(struct btrfs_root *root, struct btrfs_path *path, ...@@ -202,6 +211,7 @@ static int add_all_parents(struct btrfs_root *root, struct btrfs_path *path,
* repeat this until we don't find any additional EXTENT_DATA items. * repeat this until we don't find any additional EXTENT_DATA items.
*/ */
while (1) { while (1) {
eie = NULL;
ret = btrfs_next_leaf(root, path); ret = btrfs_next_leaf(root, path);
if (ret < 0) if (ret < 0)
return ret; return ret;
...@@ -346,6 +356,8 @@ static int __resolve_indirect_refs(struct btrfs_fs_info *fs_info, ...@@ -346,6 +356,8 @@ static int __resolve_indirect_refs(struct btrfs_fs_info *fs_info,
ULIST_ITER_INIT(&uiter); ULIST_ITER_INIT(&uiter);
node = ulist_next(parents, &uiter); node = ulist_next(parents, &uiter);
ref->parent = node ? node->val : 0; ref->parent = node ? node->val : 0;
ref->inode_list =
node ? (struct extent_inode_elem *)node->aux : 0;
/* additional parents require new refs being added here */ /* additional parents require new refs being added here */
while ((node = ulist_next(parents, &uiter))) { while ((node = ulist_next(parents, &uiter))) {
...@@ -356,6 +368,8 @@ static int __resolve_indirect_refs(struct btrfs_fs_info *fs_info, ...@@ -356,6 +368,8 @@ static int __resolve_indirect_refs(struct btrfs_fs_info *fs_info,
} }
memcpy(new_ref, ref, sizeof(*ref)); memcpy(new_ref, ref, sizeof(*ref));
new_ref->parent = node->val; new_ref->parent = node->val;
new_ref->inode_list =
(struct extent_inode_elem *)node->aux;
list_add(&new_ref->list, &ref->list); list_add(&new_ref->list, &ref->list);
} }
ulist_reinit(parents); ulist_reinit(parents);
...@@ -879,7 +893,7 @@ static int find_parent_nodes(struct btrfs_trans_handle *trans, ...@@ -879,7 +893,7 @@ static int find_parent_nodes(struct btrfs_trans_handle *trans,
} }
if (ref->count && ref->parent) { if (ref->count && ref->parent) {
struct extent_inode_elem *eie = NULL; struct extent_inode_elem *eie = NULL;
if (extent_item_pos) { if (extent_item_pos && !ref->inode_list) {
u32 bsz; u32 bsz;
struct extent_buffer *eb; struct extent_buffer *eb;
bsz = btrfs_level_size(fs_info->extent_root, bsz = btrfs_level_size(fs_info->extent_root,
...@@ -889,10 +903,22 @@ static int find_parent_nodes(struct btrfs_trans_handle *trans, ...@@ -889,10 +903,22 @@ static int find_parent_nodes(struct btrfs_trans_handle *trans,
BUG_ON(!eb); BUG_ON(!eb);
ret = find_extent_in_eb(eb, bytenr, ret = find_extent_in_eb(eb, bytenr,
*extent_item_pos, &eie); *extent_item_pos, &eie);
ref->inode_list = eie;
free_extent_buffer(eb); free_extent_buffer(eb);
} }
ret = ulist_add(refs, ref->parent, ret = ulist_add_merge(refs, ref->parent,
(unsigned long)eie, GFP_NOFS); (unsigned long)ref->inode_list,
(unsigned long *)&eie, GFP_NOFS);
if (!ret && extent_item_pos) {
/*
* we've recorded that parent, so we must extend
* its inode list here
*/
BUG_ON(!eie);
while (eie->next)
eie = eie->next;
eie->next = ref->inode_list;
}
BUG_ON(ret < 0); BUG_ON(ret < 0);
} }
kfree(ref); kfree(ref);
......
...@@ -145,13 +145,22 @@ EXPORT_SYMBOL(ulist_free); ...@@ -145,13 +145,22 @@ EXPORT_SYMBOL(ulist_free);
*/ */
int ulist_add(struct ulist *ulist, u64 val, unsigned long aux, int ulist_add(struct ulist *ulist, u64 val, unsigned long aux,
unsigned long gfp_mask) unsigned long gfp_mask)
{
return ulist_add_merge(ulist, val, aux, NULL, gfp_mask);
}
int ulist_add_merge(struct ulist *ulist, u64 val, unsigned long aux,
unsigned long *old_aux, unsigned long gfp_mask)
{ {
int i; int i;
for (i = 0; i < ulist->nnodes; ++i) { for (i = 0; i < ulist->nnodes; ++i) {
if (ulist->nodes[i].val == val) if (ulist->nodes[i].val == val) {
if (old_aux)
*old_aux = ulist->nodes[i].aux;
return 0; return 0;
} }
}
if (ulist->nnodes >= ulist->nodes_alloced) { if (ulist->nnodes >= ulist->nodes_alloced) {
u64 new_alloced = ulist->nodes_alloced + 128; u64 new_alloced = ulist->nodes_alloced + 128;
......
...@@ -67,6 +67,8 @@ struct ulist *ulist_alloc(unsigned long gfp_mask); ...@@ -67,6 +67,8 @@ struct ulist *ulist_alloc(unsigned long gfp_mask);
void ulist_free(struct ulist *ulist); void ulist_free(struct ulist *ulist);
int ulist_add(struct ulist *ulist, u64 val, unsigned long aux, int ulist_add(struct ulist *ulist, u64 val, unsigned long aux,
unsigned long gfp_mask); unsigned long gfp_mask);
int ulist_add_merge(struct ulist *ulist, u64 val, unsigned long aux,
unsigned long *old_aux, unsigned long gfp_mask);
struct ulist_node *ulist_next(struct ulist *ulist, struct ulist_node *ulist_next(struct ulist *ulist,
struct ulist_iterator *uiter); struct ulist_iterator *uiter);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment