From patchwork Tue Dec 11 14:25:00 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Lukas Czerner X-Patchwork-Id: 1862641 Return-Path: X-Original-To: patchwork-linux-btrfs@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork1.kernel.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by patchwork1.kernel.org (Postfix) with ESMTP id 4F49F3FC71 for ; Tue, 11 Dec 2012 14:25:29 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753802Ab2LKOZW (ORCPT ); Tue, 11 Dec 2012 09:25:22 -0500 Received: from mx1.redhat.com ([209.132.183.28]:23811 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753783Ab2LKOZV (ORCPT ); Tue, 11 Dec 2012 09:25:21 -0500 Received: from int-mx02.intmail.prod.int.phx2.redhat.com (int-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.12]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id qBBEPJ2i026326 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK); Tue, 11 Dec 2012 09:25:19 -0500 Received: from dhcp-1-104.brq.redhat.com (dhcp-1-104.brq.redhat.com [10.34.1.104]) by int-mx02.intmail.prod.int.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id qBBEP2V6015324; Tue, 11 Dec 2012 09:25:17 -0500 From: Lukas Czerner To: linux-btrfs@vger.kernel.org Cc: chris.mason@fusionio.com, cwillu@cwillu.com, Lukas Czerner Subject: [PATCH 3/3] Btrfs-progs: List all subvolumes by default Date: Tue, 11 Dec 2012 15:25:00 +0100 Message-Id: <1355235900-10252-3-git-send-email-lczerner@redhat.com> In-Reply-To: <1355235900-10252-1-git-send-email-lczerner@redhat.com> References: <1355235900-10252-1-git-send-email-lczerner@redhat.com> X-Scanned-By: MIMEDefang 2.67 on 10.5.11.12 Sender: linux-btrfs-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-btrfs@vger.kernel.org Commit a1e89891eb6af5381539d9875b85c196150171b6 changed subvolume list command so that we list only subvolumes under the specified directory. However this is confusing and unnecessary obstacle, because one usually want to see all subvolumes in the file system. It was introduced with the notion the full_path may be invalid which is not exactly true as the full_path is always relative to the root subvolume which makes perfect sense. Simply making option '-a' default is not enough since it introduces the relative/absolute path distinction effectively obfuscating the subvolume nesting. This commit returns the subvolume list command behaviour before commit a1e89891eb6af5381539d9875b85c196150171b6 where we list all subvolumes in the filesystem with path naming from root subovolume. IMO this is the best default as it is well understood and gives all the important information about file system subvolumes including subvolume nesting without the need to parse additional information. Signed-off-by: Lukas Czerner --- btrfs-list.c | 8 ++++---- btrfs-list.h | 2 +- cmds-subvolume.c | 7 ++++--- 3 files changed, 9 insertions(+), 8 deletions(-) diff --git a/btrfs-list.c b/btrfs-list.c index 77d99f8..ac6e1f0 100644 --- a/btrfs-list.c +++ b/btrfs-list.c @@ -1283,12 +1283,11 @@ static void __filter_and_sort_subvol(struct root_lookup *all_subvols, struct root_lookup *sort_tree, struct btrfs_list_filter_set *filter_set, struct btrfs_list_comparer_set *comp_set, - int fd) + u64 top_id) { struct rb_node *n; struct root_info *entry; int ret; - u64 top_id = btrfs_list_get_path_rootid(fd); root_lookup_init(sort_tree); @@ -1457,11 +1456,12 @@ static void print_all_volume_info(struct root_lookup *sorted_tree, int btrfs_list_subvols(int fd, struct btrfs_list_filter_set *filter_set, struct btrfs_list_comparer_set *comp_set, - int is_tab_result) + int is_tab_result, int full_path) { struct root_lookup root_lookup; struct root_lookup root_sort; int ret; + u64 top_id = (full_path ? 0 : btrfs_list_get_path_rootid(fd)); ret = __list_subvol_search(fd, &root_lookup); if (ret) { @@ -1479,7 +1479,7 @@ int btrfs_list_subvols(int fd, struct btrfs_list_filter_set *filter_set, return ret; __filter_and_sort_subvol(&root_lookup, &root_sort, filter_set, - comp_set, fd); + comp_set, top_id); print_all_volume_info(&root_sort, is_tab_result); __free_all_subvolumn(&root_lookup); diff --git a/btrfs-list.h b/btrfs-list.h index f7fbea6..658e1ad 100644 --- a/btrfs-list.h +++ b/btrfs-list.h @@ -101,7 +101,7 @@ int btrfs_list_setup_comparer(struct btrfs_list_comparer_set **comp_set, int btrfs_list_subvols(int fd, struct btrfs_list_filter_set *filter_set, struct btrfs_list_comparer_set *comp_set, - int is_tab_result); + int is_tab_result, int full_path); int btrfs_list_find_updated_files(int fd, u64 root_id, u64 oldest_gen); int btrfs_list_get_default_subvolume(int fd, u64 *default_id); char *btrfs_list_path_for_root(int fd, u64 root); diff --git a/cmds-subvolume.c b/cmds-subvolume.c index 0aa7467..88158f0 100644 --- a/cmds-subvolume.c +++ b/cmds-subvolume.c @@ -310,7 +310,7 @@ static int cmd_subvol_list(int argc, char **argv) char *subvol; int is_tab_result = 0; int is_list_all = 0; - int is_only_in_path = 1; + int is_only_in_path = 0; struct option long_options[] = { {"sort", 1, NULL, 'S'}, {0, 0, 0, 0} @@ -418,7 +418,8 @@ static int cmd_subvol_list(int argc, char **argv) top_id); ret = btrfs_list_subvols(fd, filter_set, comparer_set, - is_tab_result); + is_tab_result, + !is_list_all && !is_only_in_path); if (ret) return 19; return 0; @@ -624,7 +625,7 @@ static int cmd_subvol_get_default(int argc, char **argv) btrfs_list_setup_filter(&filter_set, BTRFS_LIST_FILTER_ROOTID, default_id); - ret = btrfs_list_subvols(fd, filter_set, NULL, 0); + ret = btrfs_list_subvols(fd, filter_set, NULL, 0, 1); if (ret) return 19; return 0;