Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 594a24eb authored by Chris Mason's avatar Chris Mason
Browse files

Fix btrfs_del_ordered_inode to allow forcing the drop during unlinks



This allows us to delete an unlinked inode with dirty pages from the list
instead of forcing commit to write these out before deleting the inode.

Signed-off-by: default avatarChris Mason <chris.mason@oracle.com>
parent 051e1b9f
Loading
Loading
Loading
Loading
+13 −5
Original line number Diff line number Diff line
@@ -1245,7 +1245,10 @@ int btrfs_search_slot(struct btrfs_trans_handle *trans, struct btrfs_root
	int level;
	int should_reada = p->reada;
	int lowest_unlock = 1;
	int blocksize;
	u8 lowest_level = 0;
	u64 blocknr;
	u64 gen;

	lowest_level = p->lowest_level;
	WARN_ON(lowest_level && ins_len);
@@ -1320,11 +1323,12 @@ int btrfs_search_slot(struct btrfs_trans_handle *trans, struct btrfs_root
				reada_for_search(root, p, level, slot,
						 key->objectid);

			tmp = btrfs_find_tree_block(root,
					  btrfs_node_blockptr(b, slot),
					  btrfs_level_size(root, level - 1));
			if (tmp && btrfs_buffer_uptodate(tmp,
				   btrfs_node_ptr_generation(b, slot))) {
			blocknr = btrfs_node_blockptr(b, slot);
			gen = btrfs_node_ptr_generation(b, slot);
			blocksize = btrfs_level_size(root, level - 1);

			tmp = btrfs_find_tree_block(root, blocknr, blocksize);
			if (tmp && btrfs_buffer_uptodate(tmp, gen)) {
				b = tmp;
			} else {
				/*
@@ -1334,6 +1338,10 @@ int btrfs_search_slot(struct btrfs_trans_handle *trans, struct btrfs_root
				 */
				if (level > 1) {
					btrfs_release_path(NULL, p);
					if (tmp)
						free_extent_buffer(tmp);
					tmp = read_tree_block(root, blocknr,
							 blocksize, gen);
					if (tmp)
						free_extent_buffer(tmp);
					goto again;
+1 −1
Original line number Diff line number Diff line
@@ -976,7 +976,7 @@ static ssize_t btrfs_file_write(struct file *file, const char __user *buf,

int btrfs_release_file(struct inode * inode, struct file * filp)
{
	btrfs_del_ordered_inode(inode);
	btrfs_del_ordered_inode(inode, 0);
	if (filp->private_data)
		btrfs_ioctl_trans_end(filp);
	return 0;
+1 −3
Original line number Diff line number Diff line
@@ -852,7 +852,7 @@ static int btrfs_unlink(struct inode *dir, struct dentry *dentry)
		 * we don't need to worry about
		 * data=ordered
		 */
		btrfs_del_ordered_inode(inode);
		btrfs_del_ordered_inode(inode, 1);
	}

	btrfs_end_transaction(trans, root);
@@ -1276,14 +1276,12 @@ void btrfs_delete_inode(struct inode *inode)

	btrfs_end_transaction(trans, root);
	btrfs_btree_balance_dirty(root, nr);
	btrfs_throttle(root);
	return;

no_delete_lock:
	nr = trans->blocks_used;
	btrfs_end_transaction(trans, root);
	btrfs_btree_balance_dirty(root, nr);
	btrfs_throttle(root);
no_delete:
	clear_inode(inode);
}
+3 −3
Original line number Diff line number Diff line
@@ -254,7 +254,7 @@ static void __btrfs_del_ordered_inode(struct btrfs_ordered_inode_tree *tree,
	return;
}

void btrfs_del_ordered_inode(struct inode *inode)
void btrfs_del_ordered_inode(struct inode *inode, int force)
{
	struct btrfs_root *root = BTRFS_I(inode)->root;
	u64 root_objectid = root->root_key.objectid;
@@ -263,8 +263,8 @@ void btrfs_del_ordered_inode(struct inode *inode)
		return;
	}

	if (mapping_tagged(inode->i_mapping, PAGECACHE_TAG_DIRTY) ||
	    mapping_tagged(inode->i_mapping, PAGECACHE_TAG_WRITEBACK))
	if (!force && (mapping_tagged(inode->i_mapping, PAGECACHE_TAG_DIRTY) ||
	    mapping_tagged(inode->i_mapping, PAGECACHE_TAG_WRITEBACK)))
		return;

	spin_lock(&root->fs_info->new_trans_lock);
+1 −1
Original line number Diff line number Diff line
@@ -38,6 +38,6 @@ int btrfs_find_del_first_ordered_inode(struct btrfs_ordered_inode_tree *tree,
int btrfs_find_first_ordered_inode(struct btrfs_ordered_inode_tree *tree,
				       u64 *root_objectid, u64 *objectid,
				       struct inode **inode);
void btrfs_del_ordered_inode(struct inode *inode);
void btrfs_del_ordered_inode(struct inode *inode, int force);
int btrfs_ordered_throttle(struct btrfs_root *root, struct inode *inode);
#endif