[PATCH 0/9 V2] Include otime in the snapshot list

2012-08-03 Thread Anand jain
From: Anand Jain anand.j...@oracle.com Thanks to Alexander Block and Liu Bo for the review. Changes from V1 - V2: . renamed crtime to otime . included the dependent btrfs-progs and btrfs kernel patches as below and in email following this respectively . fix bug in the btrfs su

[PATCH] Btrfs: introduce subvol uuids and times

2012-08-03 Thread Anand jain
From: Alexander Block abloc...@googlemail.com This patch introduces uuids for subvolumes. Each subvolume has it's own uuid. In case it was snapshotted, it also contains parent_uuid. In case it was received, it also contains received_uuid. It also introduces subvolume ctime/otime/stime/rtime. The

[PATCH 1/9] Update ctree.h and ioctl.h for the new uuid+times for subvolumes.

2012-08-03 Thread Anand jain
From: Alexander Block abloc...@googlemail.com Signed-off-by: Alexander Block abloc...@googlemail.com --- ctree.h | 40 - ioctl.h | 12 + print-tree.c | 79 - 3 files changed, 112

[PATCH 3/9] Btrfs-progs: refactor resolve_root() function a bit

2012-08-03 Thread Anand jain
From: Ilya Dryomov idryo...@gmail.com Don't pass a pointer to root_id to resolve_root(). It's always the same as ri-root_id, passing a pointer hints that root_id can somehow change which is not true. Signed-off-by: Ilya Dryomov idryo...@gmail.com --- btrfs-list.c | 21 ++---

[PATCH 2/9] Btrfs-progs: search subvolumes with proper objectid

2012-08-03 Thread Anand jain
From: Liu Bo liubo2...@cn.fujitsu.com Btrfs's subvolume/snapshot is limited to [BTRFS_FIRST_FREE_OBJECTID, BTRFS_LAST_FREE_OBJECTID], so just apply the range. Signed-off-by: Liu Bo liubo2...@cn.fujitsu.com --- btrfs-list.c |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff

[PATCH 4/9] Btrfs-progs: nuke redundant zeroing in __list_subvol_search()

2012-08-03 Thread Anand jain
From: Ilya Dryomov idryo...@gmail.com There's no need to zero out things twice. Signed-off-by: Ilya Dryomov idryo...@gmail.com --- btrfs-list.c |4 1 files changed, 0 insertions(+), 4 deletions(-) diff --git a/btrfs-list.c b/btrfs-list.c index 5c21cac..911b238 100644 ---

[PATCH 5/9] Btrfs-progs: bring 'subvol get-default' back in

2012-08-03 Thread Anand jain
From: Ilya Dryomov idryo...@gmail.com Commit bab2c565 accidentally broke 'subvol get-default' command by removing almost all of the underlying code. Bring it back with some fixes and improvements. Signed-off-by: Ilya Dryomov idryo...@gmail.com --- btrfs-list.c | 81

[PATCH 9/9] Btrfs-progs: fix the btrfs subvol list path last char

2012-08-03 Thread Anand jain
From: Anand Jain anand.j...@oracle.com Signed-off-by: Anand Jain anand.j...@oracle.com --- btrfs-list.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/btrfs-list.c b/btrfs-list.c index e0cb782..6e83b31 100644 --- a/btrfs-list.c +++ b/btrfs-list.c @@ -205,7 +205,7 @@

[PATCH 7/9] Btrfs-progs: list snapshots by generation

2012-08-03 Thread Anand jain
From: Liu Bo liubo2...@cn.fujitsu.com The idea is that we usually use snapshot to backup/restore our data, and the common way can be a cron script which makes lots of snapshots, so we can end up with spending some time to find the latest snapshot to restore. This adds a feature for 'btrfs

Re: How can btrfs take 23sec to stat 23K files from an SSD?

2012-08-03 Thread Martin Steigerwald
Am Donnerstag, 2. August 2012 schrieb Marc MERLIN: I'll try plugging this SSD in a totally different PC and see what happens. This may say if it's an AHCI/intel sata driver problem. Seems we will continue until someone starts to complain here. Maybe another list will be more

Re: filesystem finder / fixer

2012-08-03 Thread serialhex
Alex Elsayed wrote: Just realized I messed up sending this to the list. Roman Mamedov wrote: On Mon, 30 Jul 2012 23:26:42 -0400 (EDT) serial...@lavabit.com wrote: 1) is there a tool to help me recover data from my fs? I don't have a backup of my partition table and so I have about 500GB

Re: [PATCH V3 2/2] Btrfs: fix the snapshot that should not exist

2012-08-03 Thread Josef Bacik
On Thu, Aug 02, 2012 at 07:53:36PM -0600, Miao Xie wrote: On Thu, 2 Aug 2012 13:46:31 +0200, David Sterba wrote: Hi, appologies for late reply, On Thu, Aug 02, 2012 at 12:40:46PM +0800, Miao Xie wrote: Changelog v1 - v2: - add comment to explain why we need deal with the delayed

[PATCH] Btrfs: use a slab for btrfs_dio_private

2012-08-03 Thread Josef Bacik
This is in the IO path, let's try to avoid latencies by having our own slab. Thanks, Signed-off-by: Josef Bacik jba...@fusionio.com --- fs/btrfs/inode.c | 15 --- 1 files changed, 12 insertions(+), 3 deletions(-) diff --git a/fs/btrfs/inode.c b/fs/btrfs/inode.c index

Re: [PATCH] Btrfs: remove mnt_want_write call in btrfs_mksubvol

2012-08-03 Thread Josef Bacik
On Thu, Aug 02, 2012 at 05:16:20PM -0600, Alexander Block wrote: We got a recursive lock in mksubvol because the caller already held a lock. I think we got into this due to a merge error. Commit a874a63 removed the mnt_want_write call from btrfs_mksubvol and added a replacement call to

Re: kernel BUG at fs/btrfs/extent-tree.c:5038 (linux 3.4.7)

2012-08-03 Thread Olivier Bonvalet
On 02/08/2012 18:18, Olivier Bonvalet wrote: On 02/08/2012 15:53, David Sterba wrote: On Thu, Aug 02, 2012 at 03:41:03PM +0200, Olivier Bonvalet wrote: Yes... it's a copy from my /var/log/kern.log. Is it really disabled ? I was mistaken, it really is enabled unconditionally in ctree.h:55.

Re: kernel BUG at fs/btrfs/extent-tree.c:5038 (linux 3.4.7)

2012-08-03 Thread Olivier Bonvalet
On 04/08/2012 00:41, Olivier Bonvalet wrote: On 02/08/2012 18:18, Olivier Bonvalet wrote: On 02/08/2012 15:53, David Sterba wrote: On Thu, Aug 02, 2012 at 03:41:03PM +0200, Olivier Bonvalet wrote: Yes... it's a copy from my /var/log/kern.log. Is it really disabled ? I was mistaken, it really

Re: [PATCH] Btrfs: remove mnt_want_write call in btrfs_mksubvol

2012-08-03 Thread Alexander Block
On Fri, Aug 3, 2012 at 11:13 PM, Josef Bacik jba...@fusionio.com wrote: On Thu, Aug 02, 2012 at 05:16:20PM -0600, Alexander Block wrote: We got a recursive lock in mksubvol because the caller already held a lock. I think we got into this due to a merge error. Commit a874a63 removed the

Re: [PATCH V3 2/2] Btrfs: fix the snapshot that should not exist

2012-08-03 Thread Miao Xie
On fri, 3 Aug 2012 17:03:01 -0400, Josef Bacik wrote: On Thu, Aug 02, 2012 at 07:53:36PM -0600, Miao Xie wrote: On Thu, 2 Aug 2012 13:46:31 +0200, David Sterba wrote: Hi, appologies for late reply, On Thu, Aug 02, 2012 at 12:40:46PM +0800, Miao Xie wrote: Changelog v1 - v2: - add comment