Commit 5f0e4ae1 authored by Kent Overstreet's avatar Kent Overstreet Committed by Kent Overstreet

bcachefs: Use __bch2_trans_do() in a few more places

Minor cleanup, it was being open coded.
Signed-off-by: default avatarKent Overstreet <kent.overstreet@gmail.com>
Signed-off-by: default avatarKent Overstreet <kent.overstreet@linux.dev>
parent b3b66e30
...@@ -416,16 +416,12 @@ static int __bch2_link(struct bch_fs *c, ...@@ -416,16 +416,12 @@ static int __bch2_link(struct bch_fs *c,
mutex_lock(&inode->ei_update_lock); mutex_lock(&inode->ei_update_lock);
bch2_trans_init(&trans, c, 4, 1024); bch2_trans_init(&trans, c, 4, 1024);
do { ret = __bch2_trans_do(&trans, NULL, &inode->ei_journal_seq,
bch2_trans_begin(&trans); BTREE_INSERT_NOUNLOCK,
ret = bch2_link_trans(&trans, bch2_link_trans(&trans,
dir->v.i_ino, dir->v.i_ino,
inode->v.i_ino, &dir_u, &inode_u, inode->v.i_ino, &dir_u, &inode_u,
&dentry->d_name) ?: &dentry->d_name));
bch2_trans_commit(&trans, NULL,
&inode->ei_journal_seq,
BTREE_INSERT_NOUNLOCK);
} while (ret == -EINTR);
if (likely(!ret)) { if (likely(!ret)) {
BUG_ON(inode_u.bi_inum != inode->v.i_ino); BUG_ON(inode_u.bi_inum != inode->v.i_ino);
...@@ -472,17 +468,12 @@ static int bch2_unlink(struct inode *vdir, struct dentry *dentry) ...@@ -472,17 +468,12 @@ static int bch2_unlink(struct inode *vdir, struct dentry *dentry)
bch2_lock_inodes(INODE_UPDATE_LOCK, dir, inode); bch2_lock_inodes(INODE_UPDATE_LOCK, dir, inode);
bch2_trans_init(&trans, c, 4, 1024); bch2_trans_init(&trans, c, 4, 1024);
do { ret = __bch2_trans_do(&trans, NULL, &dir->ei_journal_seq,
bch2_trans_begin(&trans); BTREE_INSERT_NOUNLOCK|
BTREE_INSERT_NOFAIL,
ret = bch2_unlink_trans(&trans, bch2_unlink_trans(&trans,
dir->v.i_ino, &dir_u, dir->v.i_ino, &dir_u,
&inode_u, &dentry->d_name) ?: &inode_u, &dentry->d_name));
bch2_trans_commit(&trans, NULL,
&dir->ei_journal_seq,
BTREE_INSERT_NOUNLOCK|
BTREE_INSERT_NOFAIL);
} while (ret == -EINTR);
if (likely(!ret)) { if (likely(!ret)) {
BUG_ON(inode_u.bi_inum != inode->v.i_ino); BUG_ON(inode_u.bi_inum != inode->v.i_ino);
...@@ -599,21 +590,16 @@ static int bch2_rename2(struct mnt_idmap *idmap, ...@@ -599,21 +590,16 @@ static int bch2_rename2(struct mnt_idmap *idmap,
goto err; goto err;
} }
retry: ret = __bch2_trans_do(&trans, NULL, &journal_seq,
bch2_trans_begin(&trans); BTREE_INSERT_NOUNLOCK,
ret = bch2_rename_trans(&trans, bch2_rename_trans(&trans,
src_dir->v.i_ino, &src_dir_u, src_dir->v.i_ino, &src_dir_u,
dst_dir->v.i_ino, &dst_dir_u, dst_dir->v.i_ino, &dst_dir_u,
&src_inode_u, &src_inode_u,
&dst_inode_u, &dst_inode_u,
&src_dentry->d_name, &src_dentry->d_name,
&dst_dentry->d_name, &dst_dentry->d_name,
mode) ?: mode));
bch2_trans_commit(&trans, NULL,
&journal_seq,
BTREE_INSERT_NOUNLOCK);
if (ret == -EINTR)
goto retry;
if (unlikely(ret)) if (unlikely(ret))
goto err; goto err;
......
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