linux-btrfs.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Goffredo Baroncelli <kreijack@tiscalinet.it>
To: Anand Jain <Anand.Jain@oracle.com>
Cc: kreijack@inwind.it, Goffredo Baroncelli <kreijack@gmail.com>,
	linux-btrfs <linux-btrfs@vger.kernel.org>
Subject: Re: [PATCH 00/11 V3] add show command to the subvol sub command
Date: Tue, 15 Jan 2013 19:37:55 +0100	[thread overview]
Message-ID: <50F5A203.1020205@tiscalinet.it> (raw)
In-Reply-To: <50F50DC1.8080201@oracle.com>

Hi Anand
On 01/15/2013 09:05 AM, Anand Jain wrote:
> 
> Goffredo,
> 
>  Thanks for the review.
> 
>> I expected also from test1 and test2 something.
> 
>  actually it is working as intended. which is
>  to show more info of any item under btrfs su list output,
>  and root itself won't be in the btrfs su list (unless
>  -a option is used), otherwise any suggestion what
>  is good to have for btrfs su show / ?

If I understood correctly, btrfs su show is capable to lists the
subvolume snapshots. It could be useful to list the snapshot of the root
subvolume.



> 
> Anand
> 
> On 01/15/2013 02:25 AM, Goffredo Baroncelli wrote:
>> On 01/14/2013 05:04 AM, Anand Jain wrote:
>>>
>>>
>>> Any comments on this new sub-command, please. ?
>>>
>>> Thanks, Anand
>>>
>> I am trying to use this new command. Very nice. However I tried to use
>> it against the root of filesystem, without success:
>>
>> The root of filesystem is under /var/btrfs; I used a subvolume as root:
>>
>> $ cat /proc/self/mountinfo  | grep sdc3
>> 19 1 0:15 /__active / rw,noatime,nodiratime - btrfs /dev/sdc3
>> rw,space_cache
>> 25 19 0:15 / /var/btrfs rw,noatime,nodiratime - btrfs /dev/sdc3
>> rw,space_cache
>>
>>
>> If I do:
>>
>> $ #test 1
>> $ sudo ./btrfs su show /
>>
>> I got nothing
>>
>> If I do
>>
>> $ #test 2
>> $ sudo ./btrfs su show /var/btrfs/
>>
>> still, I got nothing
>>
>> $ #test 3
>> $  sudo ./btrfs su show /var/btrfs/__active
>>
>> I, finally, got:
>>
>> /var/btrfs/__active
>>     uuid:             835c96b8-c066-554b-9230-1c531e831ff6
>>     Parent uuid:         -
>>     Creation time:         -
>>     Object ID:         256
>>     Generation (Gen):     75774
>>     Gen at creation:     0
>>     Parent:         5
>>     Top Level:         5
>>     Snapshot(s):
>>
>>
>> I expected also from test1 and test2 something.
>>
>> BR
>> G.Baroncelli
>>
>>
>>
>>
>>
> 


-- 
gpg @keyserver.linux.it: Goffredo Baroncelli (kreijackATinwind.it>
Key fingerprint BBF5 1610 0B64 DAC6 5F7D  17B2 0EDA 9B37 8B82 E0B5

  reply	other threads:[~2013-01-15 18:37 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-12-28  3:12 [PATCH 0/8 v2] add show command to the subvol sub command Anand jain
2012-12-28  3:12 ` [PATCH 1/8] Btrfs-progs: move open_file_or_dir() to utils.c Anand jain
2012-12-28  3:12 ` [PATCH 2/8] Btrfs-progs: Move printing outside of btrfs_list_subvols Anand jain
2012-12-28  3:12 ` [PATCH 3/8] Btrfs-progs: add parent uuid for snapshots Anand jain
2012-12-28  3:12 ` [PATCH 4/8] Btrfs-progs: move struct root_info to btrfs-list.h Anand jain
2012-12-28  3:12 ` [PATCH 5/8] Btrfs-progs: function to get root_info of a subvol Anand jain
2012-12-28  3:13 ` [PATCH 6/8] Btrfs-progs: provide method to check if filter is set Anand jain
2012-12-28  3:13 ` [PATCH 7/8] Btrfs-progs: add method to filter snapshots by parent uuid Anand jain
2012-12-28  3:13 ` [PATCH 8/8] Btrfs-progs: add show command to display detailed information about the subvol Anand jain
2013-01-10 11:41 ` [PATCH 00/11 V3] add show command to the subvol sub command Anand Jain
2013-01-10 11:41   ` [PATCH 01/11] Btrfs-progs: move open_file_or_dir() to utils.c Anand Jain
2013-01-10 11:41   ` [PATCH 02/11] Btrfs-progs: Move printing outside of btrfs_list_subvols Anand Jain
2013-01-10 11:41   ` [PATCH 03/11] Btrfs-progs: add parent uuid for snapshots Anand Jain
2013-01-10 11:41   ` [PATCH 04/11] Btrfs-progs: move struct root_info to btrfs-list.h Anand Jain
2013-01-10 11:41   ` [PATCH 05/11] Btrfs-progs: function to get root_info of a subvol Anand Jain
2013-01-10 11:41   ` [PATCH 06/11] Btrfs-progs: provide method to check if filter is set Anand Jain
2013-01-10 11:41   ` [PATCH 07/11] Btrfs-progs: add method to filter snapshots by parent uuid Anand Jain
2013-01-10 11:41   ` [PATCH 08/11] Btrfs-progs: put find_mount_root() in commands.h Anand Jain
2013-01-10 11:41   ` [PATCH 09/11] Btrfs-progs: maintain similar case in heading prefix Anand Jain
2013-01-10 11:41   ` [PATCH 10/11] Btrfs-progs: accommodate different layout for printing subvol list Anand Jain
2013-01-10 11:41   ` [PATCH 11/11] Btrfs-progs: add show to display all known parameters of the given subvol Anand Jain
2013-01-14  4:04   ` [PATCH 00/11 V3] add show command to the subvol sub command Anand Jain
2013-01-14 18:25     ` Goffredo Baroncelli
2013-01-15  8:05       ` Anand Jain
2013-01-15 18:37         ` Goffredo Baroncelli [this message]
2013-01-21  0:19   ` Gene Czarcinski
2013-01-21 10:35     ` Anand Jain
2013-01-23  8:30   ` Anand Jain

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=50F5A203.1020205@tiscalinet.it \
    --to=kreijack@tiscalinet.it \
    --cc=Anand.Jain@oracle.com \
    --cc=kreijack@gmail.com \
    --cc=kreijack@inwind.it \
    --cc=linux-btrfs@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).