korg/btrfs-progs
0
0
Fork 0
mirror of https://git.kernel.org/pub/scm/linux/kernel/git/kdave/btrfs-progs.git synced 2024-11-16 08:44:31 +08:00
Code Issues Packages Projects Releases Wiki Activity

btrfs-progs: add ask_user confirmation for btrfstune clear seeding flag

Browse Source 
If we do the following:
	# mkfs.btrfs -f <dev>
	# mount <dev> <mnt>
	# dd if=/dev/urandom of=<mnt>/data bs=1M count=100
	# umount <dev>
	# btrfstune -S 1 <dev> 		<--- make seeding device
	# mount <dev> <mnt>
	# btrfs dev add -f <dev2> <mnt>
	# umount <dev>
	# btrfstune -S 0 <dev>		<--- clear seeding flag
	# mount <dev2> <mnt>		<=== new device not mountable

When mounting the new device, btrfs will check whether
the seeding flag is set when try to open seeding device.
If the user clears the seeding flag of the seeding device,
the new device will not be mountable. Even set the seeding
flag back will not recovery this problem, because the generation
has been changed. So clear the seeding flag has the chance to
damage the derived new fs.

So I add user confirmation check when clearing seeding flag.

Signed-off-by: Gui Hecheng <guihc.fnst@cn.fujitsu.com>
Signed-off-by: David Sterba <dsterba@suse.cz>
This commit is contained in:
Gui Hecheng 2014-07-07 09:54:52 +08:00 committed by David Sterba
parent cfee4bc139
commit 3db4c0a3d3
1 changed files with 16 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
btrfstune.c
View File

@ -56,6 +56,7 @@ static int update_seeding_flag(struct btrfs_root *root, int set_flag)
return 1;
}
super_flags &= ~BTRFS_SUPER_FLAG_SEEDING;
fprintf(stderr, "Warning: Seeding flag cleared.\n");
}
trans = btrfs_start_transaction(root, 1);
@ -103,6 +104,7 @@ static void print_usage(void)
fprintf(stderr, "\t-S value\tpositive value will enable seeding, zero to disable, negative is not allowed\n");
fprintf(stderr, "\t-r \t\tenable extended inode refs\n");
fprintf(stderr, "\t-x \t\tenable skinny metadata extent refs\n");
fprintf(stderr, "\t-f \t\tforce to clear flags, make sure that you are aware of the dangers\n");
}
int main(int argc, char *argv[])
@ -113,11 +115,12 @@ int main(int argc, char *argv[])
int seeding_flag = 0;
u64 seeding_value = 0;
int skinny_flag = 0;
int force = 0;
int ret;
optind = 1;
while(1) {
int c = getopt(argc, argv, "S:rx");
int c = getopt(argc, argv, "S:rxf");
if (c < 0)
break;
switch(c) {
@ -131,6 +134,9 @@ int main(int argc, char *argv[])
case 'x':
skinny_flag = 1;
break;
case 'f':
force = 1;
break;
default:
print_usage();
return 1;
@ -170,6 +176,15 @@ int main(int argc, char *argv[])
}
if (seeding_flag) {
if (!seeding_value && !force) {
fprintf(stderr, "Warning: This is dangerous, clearing the seeding flag may cause the derived device not to be mountable!\n");
ret = ask_user("We are going to clear the seeding flag, are you sure?");
if (!ret) {
fprintf(stderr, "Clear seeding flag canceled\n");
return 1;
}
}
ret = update_seeding_flag(root, seeding_value);
if (!ret)
success++;