Re: [PATCH] Snapshot/subvolume listing feature

2009-11-17 Thread TARUISI Hiroaki
Patches for Snapshot/subvolume listing feature are modified. With these patches, ioctl searches for subvolumes under only one tree, and btrfsctl repeatedly creates parameters and calls ioctl until all subvolumes are listed. An intrenal logic was changed but user interfaces (option and contents of

Re: [PATCH] Snapshot/subvolume listing feature

2009-11-16 Thread Andrey Kuzmin
Just for clarity, getdents is exactly the other interface options discussed couple of weeks back (use virtual directories & standard file-system API). Regards, Andrey On Mon, Nov 16, 2009 at 11:58 AM, TARUISI Hiroaki wrote: > Thank you for your advice. > > I'm aware of redundant search, but I

Re: [PATCH] Snapshot/subvolume listing feature

2009-11-16 Thread TARUISI Hiroaki
Thank you for your advice. I'm aware of redundant search, but I didn't think of getdents like interface. I'll remake it without redundant search. Regards, taruisi Yan, Zheng wrote: > 2009/11/16 TARUISI Hiroaki : >> I made Snapshot/subvolume listing feature. >> >> This feature consists of two pa

Re: [PATCH] Snapshot/subvolume listing feature

2009-11-16 Thread Yan, Zheng
2009/11/16 TARUISI Hiroaki : > I made Snapshot/subvolume listing feature. > > This feature consists of two patches, for kernel(ioctl), > and for progs(btrfsctl). I send these two patches as response > of this mail soon. > > New option '-l' is introduced to btrfsctl for listing. > > If this option i

Re: [PATCH] Snapshot/subvolume listing feature

2009-11-15 Thread TARUISI Hiroaki
I'm sorry, I forgot Signed-off line. Please add this line. Signed-off-by: TARUISI Hiroaki -- 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

[PATCH] Snapshot/subvolume listing feature

2009-11-15 Thread TARUISI Hiroaki
I made Snapshot/subvolume listing feature. This feature consists of two patches, for kernel(ioctl), and for progs(btrfsctl). I send these two patches as response of this mail soon. New option '-l' is introduced to btrfsctl for listing. If this option is specified, btrfsctl call new ioctl. New io