mirror of
https://mirrors.bfsu.edu.cn/git/linux.git
synced 2025-01-10 15:54:39 +08:00
bcachefs: check for failure to downgrade
With the upcoming member seq patch, it's now critical that we don't ever write to a superblock that hasn't been version downgraded - failure to update member seq fields will cause split brain detection to fire erroniously. Signed-off-by: Kent Overstreet <kent.overstreet@linux.dev>
This commit is contained in:
parent
44fd13a4c6
commit
447c1c0105
@ -223,6 +223,7 @@
|
||||
x(BCH_ERR_invalid, invalid_bkey) \
|
||||
x(BCH_ERR_operation_blocked, nocow_lock_blocked) \
|
||||
x(EIO, btree_node_read_err) \
|
||||
x(EIO, sb_not_downgraded) \
|
||||
x(BCH_ERR_btree_node_read_err, btree_node_read_err_fixable) \
|
||||
x(BCH_ERR_btree_node_read_err, btree_node_read_err_want_retry) \
|
||||
x(BCH_ERR_btree_node_read_err, btree_node_read_err_must_retry) \
|
||||
|
@ -987,6 +987,18 @@ int bch2_write_super(struct bch_fs *c)
|
||||
if (!BCH_SB_INITIALIZED(c->disk_sb.sb))
|
||||
goto out;
|
||||
|
||||
if (le16_to_cpu(c->disk_sb.sb->version) > bcachefs_metadata_version_current) {
|
||||
struct printbuf buf = PRINTBUF;
|
||||
prt_printf(&buf, "attempting to write superblock that wasn't version downgraded (");
|
||||
bch2_version_to_text(&buf, le16_to_cpu(c->disk_sb.sb->version));
|
||||
prt_str(&buf, " > ");
|
||||
bch2_version_to_text(&buf, bcachefs_metadata_version_current);
|
||||
prt_str(&buf, ")");
|
||||
bch2_fs_fatal_error(c, "%s", buf.buf);
|
||||
printbuf_exit(&buf);
|
||||
return -BCH_ERR_sb_not_downgraded;
|
||||
}
|
||||
|
||||
for_each_online_member(ca, c, i) {
|
||||
__set_bit(ca->dev_idx, sb_written.d);
|
||||
ca->sb_write_error = 0;
|
||||
|
Loading…
Reference in New Issue
Block a user