Skip to content

Commit 62f3502

Browse files
author
Kent Overstreet
committed
bcachefs: Change "accounting overran journal reservation" to a warning
This doesn't need to be a BUG_ON(); the actual serious "things break" condition is if the whole journal write overruns the available space, and that has a fatal error, not a BUG_ON(). This check indicates we screwed something up, but it should be a warning. Signed-off-by: Kent Overstreet <[email protected]>
1 parent be28368 commit 62f3502

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

fs/bcachefs/journal_io.c

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1848,7 +1848,8 @@ static int bch2_journal_write_prep(struct journal *j, struct journal_buf *w)
18481848

18491849
bch2_journal_super_entries_add_common(c, &end, seq);
18501850
u64s = (u64 *) end - (u64 *) start;
1851-
BUG_ON(u64s > j->entry_u64s_reserved);
1851+
1852+
WARN_ON(u64s > j->entry_u64s_reserved);
18521853

18531854
le32_add_cpu(&jset->u64s, u64s);
18541855

0 commit comments

Comments
 (0)