marko10_000
/
btrfs-progs
forked from Mirrors/btrfs-progs
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

btrfs-progs: subvol: change set-default to also accept path 

This patch changes "subvol set-default" to also accept the subvolume path
for convenience.

If there are two args, they are assumed as subvol id and path to the fs
(the same as current behavior), and if there is only one arg, it is assumed
as the path to the subvolume.

subvol id is resolved by test_issubvolume() + lookup_path_rootid().
The empty subvol (ino == 2) will get error on test_issubvolume() which
checks whether inode num is 256 or not.

Issue: #35
Signed-off-by: Tomohiro Misono <misono.tomohiro@jp.fujitsu.com>
[ update documentation, use the new multi-line command scheme ]
Signed-off-by: David Sterba <dsterba@suse.com>
master
Misono, Tomohiro 2017-10-06 10:05:08 +09:00 committed by David Sterba
parent c56d61883c
commit 4a5b95abb6
2 changed files with 43 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
Documentation/btrfs-subvolume.asciidoc
View File

@ -142,12 +142,13 @@ you can add \'\+' or \'-' in front of each items, \'+' means ascending,
for --sort you can combine some items together by \',', just like
--sort=+ogen,-gen,path,rootid.
*set-default* <id> <path>::
Set the subvolume of the filesystem <path> which is mounted as
default.
*set-default* [<subvolume>|<id> <path>]::
Set the default subvolume of the (mounted) filesystem.
+
The subvolume is identified by <id>, which is returned by the *subvolume list*
command.
There are two ways how to specify the subvolume, by <id> or by the <subvolume>
path.
The id can be obtained from *btrfs subvolume list*, *btrfs subvolume show* or
*btrfs inspect-internal rootid*.
*show* <path>::
Show information of a given subvolume in the <path>.

45
cmds-subvolume.c
View File

@ -858,8 +858,11 @@ out:
}
static const char * const cmd_subvol_set_default_usage[] = {
"btrfs subvolume set-default <subvolume>\n"
"btrfs subvolume set-default <subvolid> <path>",
"Set the default subvolume of a filesystem",
"Set the default subvolume of the filesystem mounted as default.",
"The subvolume can be specified by its path,",
"or the pair of subvolume id and path to the filesystem.",
NULL
};
@ -873,17 +876,43 @@ static int cmd_subvol_set_default(int argc, char **argv)
clean_args_no_options(argc, argv, cmd_subvol_set_default_usage);
if (check_argc_exact(argc - optind, 2))
if (check_argc_min(argc - optind, 1) ||
check_argc_max(argc - optind, 2))
usage(cmd_subvol_set_default_usage);
subvolid = argv[optind];
path = argv[optind + 1];
if (argc - optind == 1) {
/* path to the subvolume is specified */
path = argv[optind];
objectid = arg_strtou64(subvolid);
ret = test_issubvolume(path);
if (ret < 0) {
error("stat error: %s", strerror(-ret));
return 1;
} else if (!ret) {
error("'%s' is not a subvolume", path);
return 1;
}
fd = btrfs_open_dir(path, &dirstream, 1);
if (fd < 0)
return 1;
fd = btrfs_open_dir(path, &dirstream, 1);
if (fd < 0)
return 1;
ret = lookup_path_rootid(fd, &objectid);
if (ret) {
error("unable to get subvol id: %s", strerror(-ret));
close_file_or_dir(fd, dirstream);
return 1;
}
} else {
/* subvol id and path to the filesystem are specified */
subvolid = argv[optind];
path = argv[optind + 1];
objectid = arg_strtou64(subvolid);
fd = btrfs_open_dir(path, &dirstream, 1);
if (fd < 0)
return 1;
}
ret = ioctl(fd, BTRFS_IOC_DEFAULT_SUBVOL, &objectid);
e = errno;