diff options
| author |   Kent Overstreet <kent.overstreet@linux.dev> | 2024-10-14 22:18:12 -0400 |
|---|---|---|
| committer | Kent Overstreet <kent.overstreet@linux.dev> | 2024-10-18 00:49:47 -0400 |
| commit | 74ec2f302402c4cfd172f7254ae0e5b851e31b0a (patch) | |
| tree | 76dbba6442225d749f22b37d1265bea7a6e16964 | |
| parent | bcachefs: Fix sysfs warning in fstests generic/730,731 (diff) | |
| download | wireguard-linux-74ec2f302402c4cfd172f7254ae0e5b851e31b0a.tar.xz wireguard-linux-74ec2f302402c4cfd172f7254ae0e5b851e31b0a.zip | |
bcachefs: fix restart handling in bch2_rename2()
This should be impossible to hit in practice; the first lookup within a
transaction won't return a restart due to lock ordering, but we're
adding fault injection for transaction restarts and shaking out bugs.
Signed-off-by: Kent Overstreet <kent.overstreet@linux.dev>
| -rw-r--r-- | fs/bcachefs/fs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/bcachefs/fs.c b/fs/bcachefs/fs.c index 857175f4bbbe..d5d16d12e03a 100644 --- a/fs/bcachefs/fs.c +++ b/fs/bcachefs/fs.c @@ -871,7 +871,7 @@ static int bch2_rename2(struct mnt_idmap *idmap, ret = bch2_subvol_is_ro_trans(trans, src_dir->ei_inum.subvol) ?: bch2_subvol_is_ro_trans(trans, dst_dir->ei_inum.subvol); if (ret) - goto err; + goto err_tx_restart; if (inode_attr_changing(dst_dir, src_inode, Inode_opt_project)) { ret = bch2_fs_quota_transfer(c, src_inode, |