diff options
| author | Kent Overstreet <kent.overstreet@gmail.com> | 2018-08-05 15:28:29 -0400 |
|---|---|---|
| committer | Kent Overstreet <kent.overstreet@linux.dev> | 2023-10-22 17:08:09 -0400 |
| commit | df8a42393e038574a84d256720b2ba9745af0ad5 (patch) | |
| tree | 7cd252f476a6dd965fb00d0566bbfdc6a1c9e833 | |
| parent | e4ccb251312003a9feed402e1958d0fef24987f1 (diff) | |
bcachefs: extent_squash() can no longer fail
Signed-off-by: Kent Overstreet <kent.overstreet@linux.dev>
| -rw-r--r-- | fs/bcachefs/extents.c | 15 |
1 files changed, 5 insertions, 10 deletions
diff --git a/fs/bcachefs/extents.c b/fs/bcachefs/extents.c index df04af882c16..6cc6961047cf 100644 --- a/fs/bcachefs/extents.c +++ b/fs/bcachefs/extents.c @@ -1402,7 +1402,7 @@ bch2_extent_can_insert(struct btree_insert *trans, return BTREE_INSERT_OK; } -static enum btree_insert_ret +static void extent_squash(struct extent_insert_state *s, struct bkey_i *insert, struct bset_tree *t, struct bkey_packed *_k, struct bkey_s k, enum bch_extent_overlap overlap) @@ -1496,8 +1496,6 @@ extent_squash(struct extent_insert_state *s, struct bkey_i *insert, break; } } - - return BTREE_INSERT_OK; } static enum btree_insert_ret @@ -1542,7 +1540,7 @@ __bch2_insert_fixup_extent(struct extent_insert_state *s) _k->needs_whiteout = false; } - ret = extent_squash(s, insert, t, _k, k, overlap); + extent_squash(s, insert, t, _k, k, overlap); } else { if (bkey_whiteout(k.k)) goto next; @@ -1575,21 +1573,18 @@ __bch2_insert_fixup_extent(struct extent_insert_state *s) discard.k.needs_whiteout = true; - ret = extent_squash(s, insert, t, _k, k, overlap); - BUG_ON(ret != BTREE_INSERT_OK); + extent_squash(s, insert, t, _k, k, overlap); extent_bset_insert(c, iter, &discard); } else { - ret = extent_squash(s, insert, t, _k, k, overlap); - BUG_ON(ret != BTREE_INSERT_OK); + extent_squash(s, insert, t, _k, k, overlap); } next: bch2_cut_front(s->committed, insert); bch2_btree_iter_set_pos_same_leaf(iter, s->committed); } - if (ret != BTREE_INSERT_OK || - overlap == BCH_EXTENT_OVERLAP_FRONT || + if (overlap == BCH_EXTENT_OVERLAP_FRONT || overlap == BCH_EXTENT_OVERLAP_MIDDLE) break; } |