iv/linux
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

bcachefs: Fix bch2_verify_insert_pos()

Browse Source 
We were calling __btree_node_key_to_offset() on a key that wasn't in the
btree node.

Signed-off-by: Kent Overstreet <kent.overstreet@gmail.com>
Signed-off-by: Kent Overstreet <kent.overstreet@linux.dev>
This commit is contained in:
Kent Overstreet 2019-11-26 17:18:23 -05:00 committed by Kent Overstreet
parent 07358a82bb
commit c201e2d976
1 changed files with 6 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
fs/bcachefs/bset.c
View File

@ -241,10 +241,9 @@ void bch2_verify_insert_pos(struct btree *b, struct bkey_packed *where,
bch2_bkey_to_text(&PBUF(buf2), &k2); bch2_bkey_to_text(&PBUF(buf2), &k2);
panic("prev > insert:\n" panic("prev > insert:\n"
"prev key %5u %s\n" "prev key %s\n"
"insert key %5u %s\n", "insert key %s\n",
__btree_node_key_to_offset(b, prev), buf1, buf1, buf2);
__btree_node_key_to_offset(b, insert), buf2);
} }
#endif #endif
#if 0 #if 0
@ -263,10 +262,9 @@ void bch2_verify_insert_pos(struct btree *b, struct bkey_packed *where,
bch2_bkey_to_text(&PBUF(buf2), &k2); bch2_bkey_to_text(&PBUF(buf2), &k2);
panic("insert > next:\n" panic("insert > next:\n"
"insert key %5u %s\n" "insert key %s\n"
"next key %5u %s\n", "next key %s\n",
__btree_node_key_to_offset(b, insert), buf1, buf1, buf2);
__btree_node_key_to_offset(b, next), buf2);
} }
#endif #endif
} }