On 19/10/12, Miao Xie (mi...@cn.fujitsu.com) wrote:
> On fri, 19 Oct 2012 14:23:53 +0800, Anand jain wrote:
> > From: Anand Jain <anand.j...@oracle.com>
> >
> > This set of patch will make
> > btrfs su list -s <subvol>
> > to list only snapshot(s) of the given subvol.
> >
> > before:
> > btrfs su list -s /btrfs/sv1
> > <nothing>
...
> > with this patch:
> > btrfs su list -s /btrfs/sv1
> > ID 258 gen 6 cgen 6 top level 5 otime 2012-10-18 17:01:56 uuid
> > f648cdda-4efa-6f45-bd6b-041a8ae1538e path ss1
> > ID 261 gen 9 cgen 9 top level 5 otime 2012-10-19 13:37:42 uuid
> > 44560e56-3879-2146-8b24-e9048871892f path ss3
>
> Though the function implemented by your patch is very useful, you change the
> semantic of "-s",
> I think it is not allowed, or the programs that use "btrfs us list -s" may
> fail.
>From my perspective as a user I would be grateful if the following changes in
syntax for listing subvolumes could be considered:
In addition to
btrfs subvolume list [-apurts] [-g [+|-]value] [-c [+|-]value]
[--sort=gen,ogen,rootid,path] <path>
btrfs su list [-apurts] [-g [+|-]value] [-c [+|-]value]
[--sort=gen,ogen,rootid,path] <path>
List subvolumes (and snapshots)
(and Anand's patch to allow <path>/<subpath>)
I believe the following shortcuts may be useful
btrfs subvolumes [-apurts] [-g [+|-]value] [-c [+|-]value]
[--sort=gen,ogen,rootid,path] <path>[</subpath>]
btrfs sl [-apurts] [-g [+|-]value] [-c [+|-]value]
[--sort=gen,ogen,rootid,path] <path>[</subpath>]
Although from a technical perspective 'subvolume list' may be logical, listing
subvolumes (plural) seems like a different sort of operation to those relating
to a subvolume (singular) for create/alter/show/delete/snapshot operations.
--
Rory Campbell-Lange
r...@campbell-lange.net
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
