From 3defb823843ac6071e38dbd89603344885e7a8aa Mon Sep 17 00:00:00 2001 From: Miao Xie Date: Tue, 18 Sep 2012 17:14:37 +0800 Subject: [PATCH] Btrfs-progs: enhance btrfs subvol list only to show read-only snapshots We want 'btrfs subvolume list' only to list readonly subvolumes, this patch set introduces a new option 'r' to implement it. You can use the command like that: btrfs subvolume list -r Original-Signed-off-by: Zhou Bo Signed-off-by: Miao Xie --- btrfs-list.c | 37 ++++++++++++++++++++++++++----------- btrfs-list.h | 1 + cmds-subvolume.c | 13 +++++++++++-- ctree.h | 2 ++ 4 files changed, 40 insertions(+), 13 deletions(-) diff --git a/btrfs-list.c b/btrfs-list.c index 929f19fe..201f3784 100644 --- a/btrfs-list.c +++ b/btrfs-list.c @@ -59,6 +59,9 @@ struct root_info { /* equal the offset of the root's key */ u64 root_offset; + /* flags of the root */ + u64 flags; + /* the id of the root that references this one */ u64 ref_tree; @@ -394,9 +397,9 @@ static struct root_info *root_tree_search(struct root_lookup *root_tree, } static int update_root(struct root_lookup *root_lookup, - u64 root_id, u64 ref_tree, u64 root_offset, u64 dir_id, - char *name, int name_len, u64 ogen, u64 gen, time_t ot, - void *uuid) + u64 root_id, u64 ref_tree, u64 root_offset, u64 flags, + u64 dir_id, char *name, int name_len, u64 ogen, u64 gen, + time_t ot, void *uuid) { struct root_info *ri; @@ -419,6 +422,8 @@ static int update_root(struct root_lookup *root_lookup, ri->ref_tree = ref_tree; if (root_offset) ri->root_offset = root_offset; + if (flags) + ri->flags = flags; if (dir_id) ri->dir_id = dir_id; if (gen) @@ -450,15 +455,15 @@ static int update_root(struct root_lookup *root_lookup, * uuid: uuid of the root */ static int add_root(struct root_lookup *root_lookup, - u64 root_id, u64 ref_tree, u64 root_offset, u64 dir_id, - char *name, int name_len, u64 ogen, u64 gen, time_t ot, - void *uuid) + u64 root_id, u64 ref_tree, u64 root_offset, u64 flags, + u64 dir_id, char *name, int name_len, u64 ogen, u64 gen, + time_t ot, void *uuid) { struct root_info *ri; int ret; - ret = update_root(root_lookup, root_id, ref_tree, root_offset, dir_id, - name, name_len, ogen, gen, ot, uuid); + ret = update_root(root_lookup, root_id, ref_tree, root_offset, flags, + dir_id, name, name_len, ogen, gen, ot, uuid); if (!ret) return 0; @@ -485,6 +490,8 @@ static int add_root(struct root_lookup *root_lookup, ri->dir_id = dir_id; if (root_offset) ri->root_offset = root_offset; + if (flags) + ri->flags = flags; if (gen) ri->gen = gen; if (ogen) @@ -961,6 +968,7 @@ static int __list_subvol_search(int fd, struct root_lookup *root_lookup) u64 dir_id; u64 gen = 0; u64 ogen; + u64 flags; int i; time_t t; u8 uuid[BTRFS_UUID_SIZE]; @@ -1016,11 +1024,12 @@ static int __list_subvol_search(int fd, struct root_lookup *root_lookup) dir_id = btrfs_stack_root_ref_dirid(ref); add_root(root_lookup, sh->objectid, sh->offset, - 0, dir_id, name, name_len, 0, 0, 0, + 0, 0, dir_id, name, name_len, 0, 0, 0, NULL); } else if (sh->type == BTRFS_ROOT_ITEM_KEY) { ri = (struct btrfs_root_item *)(args.buf + off); gen = btrfs_root_generation(ri); + flags = btrfs_root_flags(ri); if(sh->len > sizeof(struct btrfs_root_item_v0)) { t = ri->otime.sec; @@ -1033,8 +1042,8 @@ static int __list_subvol_search(int fd, struct root_lookup *root_lookup) } add_root(root_lookup, sh->objectid, 0, - sh->offset, 0, NULL, 0, ogen, gen, t, - uuid); + sh->offset, flags, 0, NULL, 0, ogen, + gen, t, uuid); } off += sh->len; @@ -1077,9 +1086,15 @@ static int filter_snapshot(struct root_info *ri, u64 data) return !!ri->root_offset; } +static int filter_flags(struct root_info *ri, u64 flags) +{ + return ri->flags & flags; +} + static btrfs_list_filter_func all_filter_funcs[] = { [BTRFS_LIST_FILTER_ROOTID] = filter_by_rootid, [BTRFS_LIST_FILTER_SNAPSHOT_ONLY] = filter_snapshot, + [BTRFS_LIST_FILTER_FLAGS] = filter_flags, }; struct btrfs_list_filter_set *btrfs_list_alloc_filter_set(void) diff --git a/btrfs-list.h b/btrfs-list.h index 11f12156..21d0fdc7 100644 --- a/btrfs-list.h +++ b/btrfs-list.h @@ -61,6 +61,7 @@ enum btrfs_list_column_enum { enum btrfs_list_filter_enum { BTRFS_LIST_FILTER_ROOTID, BTRFS_LIST_FILTER_SNAPSHOT_ONLY, + BTRFS_LIST_FILTER_FLAGS, BTRFS_LIST_FILTER_MAX, }; diff --git a/cmds-subvolume.c b/cmds-subvolume.c index 59f4dfdd..f385816f 100644 --- a/cmds-subvolume.c +++ b/cmds-subvolume.c @@ -259,13 +259,14 @@ static int cmd_subvol_delete(int argc, char **argv) } static const char * const cmd_subvol_list_usage[] = { - "btrfs subvolume list [-pu] [-s 0|1] ", + "btrfs subvolume list [-pur] [-s 0|1] ", "List subvolumes (and snapshots)", "", "-p print parent ID", "-u print the uuid of subvolumes (and snapshots)", "-s value list snapshots with generation in ascending/descending order", " (1: ascending, 0: descending)", + "-r list readonly subvolumes (including snapshots)", NULL }; @@ -273,6 +274,7 @@ static int cmd_subvol_list(int argc, char **argv) { struct btrfs_list_filter_set *filter_set; struct btrfs_list_comparer_set *comparer_set; + u64 flags = 0; int fd; int ret; int order; @@ -283,7 +285,7 @@ static int cmd_subvol_list(int argc, char **argv) optind = 1; while(1) { - int c = getopt(argc, argv, "ps:u"); + int c = getopt(argc, argv, "ps:ur"); if (c < 0) break; @@ -305,11 +307,18 @@ static int cmd_subvol_list(int argc, char **argv) case 'u': btrfs_list_setup_print_column(BTRFS_LIST_UUID); break; + case 'r': + flags |= BTRFS_ROOT_SUBVOL_RDONLY; + break; default: usage(cmd_subvol_list_usage); } } + if (flags) + btrfs_list_setup_filter(&filter_set, BTRFS_LIST_FILTER_FLAGS, + flags); + if (check_argc_exact(argc - optind, 1)) usage(cmd_subvol_list_usage); diff --git a/ctree.h b/ctree.h index c55d0338..4bb66fff 100644 --- a/ctree.h +++ b/ctree.h @@ -139,6 +139,8 @@ static int btrfs_csum_sizes[] = { 4, 0 }; #define BTRFS_FT_XATTR 8 #define BTRFS_FT_MAX 9 +#define BTRFS_ROOT_SUBVOL_RDONLY (1ULL << 0) + /* * the key defines the order in the tree, and so it also defines (optimal) * block layout. objectid corresonds to the inode number. The flags