diff options
| author | Kent Overstreet <kent.overstreet@linux.dev> | 2022-12-09 14:09:14 -0500 |
|---|---|---|
| committer | Kent Overstreet <kent.overstreet@linux.dev> | 2023-10-22 17:09:47 -0400 |
| commit | 4f948723eda1ca44e470c31fcab4a453ed53aa13 (patch) | |
| tree | 02e7115b493981b4c197a552d0e6f4f447e6b4ad /fs/bcachefs/recovery.c | |
| parent | ac9fa4bdc79bb54e639f5dc262f9f5976a5ccd21 (diff) | |
| download | lwn-4f948723eda1ca44e470c31fcab4a453ed53aa13.tar.gz lwn-4f948723eda1ca44e470c31fcab4a453ed53aa13.zip | |
bcachefs: Fix bch2_journal_keys_peek_upto()
bch2_journal_keys_peek_upto() was comparing against btree_id & level
incorrectly - fix this by using __journal_key_cmp().
Signed-off-by: Kent Overstreet <kent.overstreet@linux.dev>
Diffstat (limited to 'fs/bcachefs/recovery.c')
| -rw-r--r-- | fs/bcachefs/recovery.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/fs/bcachefs/recovery.c b/fs/bcachefs/recovery.c index aff813e3e360..15a676196e2f 100644 --- a/fs/bcachefs/recovery.c +++ b/fs/bcachefs/recovery.c @@ -128,12 +128,12 @@ search: if (!*idx) *idx = __bch2_journal_key_search(keys, btree_id, level, pos); - while (*idx < keys->nr && - (k = idx_to_key(keys, *idx), - k->btree_id == btree_id && - k->level == level && - bpos_le(k->k->k.p, end_pos))) { - if (bpos_ge(k->k->k.p, pos) && !k->overwritten) + while ((k = *idx < keys->nr ? idx_to_key(keys, *idx) : NULL)) { + if (__journal_key_cmp(btree_id, level, end_pos, k) < 0) + return NULL; + + if (__journal_key_cmp(btree_id, level, pos, k) <= 0 && + !k->overwritten) return k->k; (*idx)++; |