bcachefs: Fix for bch2_copygc() spuriously returning -EEXIST

Signed-off-by: Kent Overstreet <kent.overstreet@linux.dev>

diff --git a/fs/bcachefs/movinggc.c b/fs/bcachefs/movinggc.c
index 5242f20bb680fc2c0d9fc8ebd54188db87dfa8ab..256431a6dc0caf502b11808877b625848048ad75 100644 (file)
--- a/fs/bcachefs/movinggc.c
+++ b/fs/bcachefs/movinggc.c
@@ -220,8 +220,10 @@ static int bch2_copygc(struct btree_trans *trans,
 
                f = move_bucket_in_flight_add(buckets_in_flight, *i);
                ret = PTR_ERR_OR_ZERO(f);
-               if (ret == -EEXIST) /* rare race: copygc_get_buckets returned same bucket more than once */
+               if (ret == -EEXIST) { /* rare race: copygc_get_buckets returned same bucket more than once */
+                       ret = 0;
                        continue;
+               }
                if (ret == -ENOMEM) { /* flush IO, continue later */
                        ret = 0;
                        break;