diff options
author | Kent Overstreet <kent.overstreet@gmail.com> | 2019-12-18 13:18:33 -0500 |
---|---|---|
committer | Kent Overstreet <kent.overstreet@linux.dev> | 2023-10-22 17:08:33 -0400 |
commit | c45d473df77f41a104ecf78275be49b67f6d6295 (patch) | |
tree | 81c52da9d69186aa3e45b5cde9b8531f47f3614f /fs/bcachefs/fs-io.c | |
parent | b5a5c4c1033af72b94c1ba7a71f61e4231f27832 (diff) |
bcachefs: Fix for an assertion on filesystem error
Normally the in memory i_size is always greater than or equal to i_size
on disk; this doesn't hold on filesystem error.
Signed-off-by: Kent Overstreet <kent.overstreet@gmail.com>
Signed-off-by: Kent Overstreet <kent.overstreet@linux.dev>
Diffstat (limited to 'fs/bcachefs/fs-io.c')
-rw-r--r-- | fs/bcachefs/fs-io.c | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/fs/bcachefs/fs-io.c b/fs/bcachefs/fs-io.c index 5beb47805a68..c30c028c869d 100644 --- a/fs/bcachefs/fs-io.c +++ b/fs/bcachefs/fs-io.c @@ -2258,6 +2258,15 @@ int bch2_truncate(struct bch_inode_info *inode, struct iattr *iattr) if (ret) goto err; + /* + * check this before next assertion; on filesystem error our normal + * invariants are a bit broken (truncate has to truncate the page cache + * before the inode). + */ + ret = bch2_journal_error(&c->journal); + if (ret) + goto err; + BUG_ON(inode->v.i_size < inode_u.bi_size); if (iattr->ia_size > inode->v.i_size) { |