alex/bcachefs-tools
1
0
Fork 0
mirror of https://github.com/koverstreet/bcachefs-tools.git synced 2025-12-10 00:00:24 +03:00
Code Issues Actions 6 Packages Projects Releases Wiki Activity

cmd_migrate: Make fsck after migrate-superblock debug-only

Browse Source 
There is no reason to run a fsck there for common use, as the FS is now
modified only by calling consistency-preserving APIs.

Also switch the debug-only fsck to nochanges mode for easier debugging.

Signed-off-by: Nikita Ofitserov <himikof@gmail.com>
Signed-off-by: Kent Overstreet <kent.overstreet@linux.dev>
This commit is contained in:
Nikita Ofitserov 2025-09-07 00:12:41 +03:00 committed by Kent Overstreet
parent 805319c0ef
commit b1021e7b39
1 changed files with 10 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
c_src/cmd_migrate.c
View File

@ -495,20 +495,23 @@ int cmd_migrate_superblock(int argc, char *argv[])
bch2_fs_stop(c);
#if CONFIG_BCACHEFS_DEBUG
/* Verify that filesystem is clean and consistent */
opts = bch2_opts_empty();
opt_set(opts, fsck, true);
opt_set(opts, fix_errors, true);
/*
* Hack: the free space counters are coming out wrong after marking the
* new superblock, but it's just the device counters so it's
* inconsequential:
*/
opt_set(opts, nochanges, true);
c = bch2_fs_open(&devs, &opts);
ret = PTR_ERR_OR_ZERO(c);
if (ret)
die("error opening filesystem: %s", bch2_err_str(ret));
die("error checking filesystem: %s", bch2_err_str(ret));
if (test_bit(BCH_FS_errors, &c->flags) || test_bit(BCH_FS_errors_fixed, &c->flags))
die("Filesystem has errors after migration");
bch2_fs_stop(c);
#endif
return 0;
}