Re: [PATCH v3] fstests: btrfs: add test for qgroup handle extent de-reference

2016-06-13 Thread Lu Fengqi
At 06/13/2016 11:04 AM, Eryu Guan wrote: On Mon, Jun 13, 2016 at 10:10:50AM +0800, Lu Fengqi wrote: Test if qgroup can handle extent de-reference during reallocation. "extent de-reference" means that reducing an extent's reference count or freeing an extent. Although current qgroup can handle

Re: [PATCH v3] fstests: btrfs: add test for qgroup handle extent de-reference

2016-06-13 Thread Lu Fengqi
At 06/13/2016 03:29 PM, Lu Fengqi wrote: At 06/13/2016 11:04 AM, Eryu Guan wrote: On Mon, Jun 13, 2016 at 10:10:50AM +0800, Lu Fengqi wrote: Test if qgroup can handle extent de-reference during reallocation. "extent de-reference" means that reducing an extent's reference count or freeing an

Re: [PATCH v3] fstests: btrfs: add test for qgroup handle extent de-reference

2016-06-13 Thread Filipe Manana
On Mon, Jun 13, 2016 at 9:06 AM, Lu Fengqi wrote: > At 06/13/2016 03:29 PM, Lu Fengqi wrote: >> >> At 06/13/2016 11:04 AM, Eryu Guan wrote: >>> >>> On Mon, Jun 13, 2016 at 10:10:50AM +0800, Lu Fengqi wrote: Test if qgroup can handle extent de-reference during

Re: [PATCH 1/1] btrfs: Use correct format specifier

2016-06-13 Thread David Sterba
On Sat, Jun 11, 2016 at 06:11:10PM +0200, Heinrich Schuchardt wrote: > Component mirror_num of struct btrfsic_block is defined > as unsigned int. Use %u as format specifier. > > Signed-off-by: Heinrich Schuchardt Reviewed-by: David Sterba -- To unsubscribe

Re: [PATCH v2] Btrfs: fix eb memory leak due to readpage failure

2016-06-13 Thread David Sterba
On Fri, Jun 03, 2016 at 12:08:38PM -0700, Liu Bo wrote: > eb->io_pages is set in read_extent_buffer_pages(). > > In case of readpage failure, for pages that have been added to bio, > it calls bio_endio and later readpage_io_failed_hook() does the work. > > When this eb's page (couldn't be the

Re: Cannot balance FS (No space left on device)

2016-06-13 Thread Austin S. Hemmelgarn
On 2016-06-10 18:39, Hans van Kranenburg wrote: On 06/11/2016 12:10 AM, ojab // wrote: On Fri, Jun 10, 2016 at 9:56 PM, Hans van Kranenburg wrote: You can work around it by either adding two disks (like Henk said), or by temporarily converting some chunks to

[PATCH] btrfs-progs: doc: correct the destination of btrfs-receive

2016-06-13 Thread Satoru Takeuchi
We can set not only btrfs mount point but also any path belong to btrfs mount point as btrfs-receive's destination. Signed-off-by: Satoru Takeuchi --- Documentation/btrfs-receive.asciidoc | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git

Re: Replacing drives with larger ones in a 4 drive raid1

2016-06-13 Thread Austin S. Hemmelgarn
On 2016-06-12 06:35, boli wrote: It has now been doing "btrfs device delete missing /mnt" for about 90 hours. These 90 hours seem like a rather long time, given that a rebalance/convert from 4-disk-raid5 to 4-disk-raid1 took about 20 hours months ago, and a scrub takes about 7 hours

Re: [PATCH 2/2] btrfs: prefix fsid to all trace events

2016-06-13 Thread David Sterba
On Thu, Jun 09, 2016 at 07:48:01PM -0400, je...@suse.com wrote: > +#define TP_printk_btrfs(fmt, args...) \ > + TP_printk("%pU: " fmt, __entry->fsid, args) So it's identified by the UUID. As there's no previous format of the filesystem identifier (unlike the syslog messages), I'm ok with using

Re: [PATCH 2/2] btrfs: prefix fsid to all trace events

2016-06-13 Thread Jeff Mahoney
On 6/13/16 11:48 AM, David Sterba wrote: > On Thu, Jun 09, 2016 at 07:48:01PM -0400, je...@suse.com wrote: >> +#define TP_printk_btrfs(fmt, args...) \ >> +TP_printk("%pU: " fmt, __entry->fsid, args) > > So it's identified by the UUID. As there's no previous format of the > filesystem

Re: [PATCH v3] fstests: btrfs: add test for qgroup handle extent de-reference

2016-06-13 Thread Qu Wenruo
At 06/13/2016 05:49 PM, Filipe Manana wrote: On Mon, Jun 13, 2016 at 9:06 AM, Lu Fengqi wrote: At 06/13/2016 03:29 PM, Lu Fengqi wrote: At 06/13/2016 11:04 AM, Eryu Guan wrote: On Mon, Jun 13, 2016 at 10:10:50AM +0800, Lu Fengqi wrote: Test if qgroup can

[PATCH v4] fstests: btrfs: add test for qgroup handle extent de-reference

2016-06-13 Thread Lu Fengqi
Test if qgroup can handle extent de-reference during reallocation. "extent de-reference" means that reducing an extent's reference count or freeing an extent. Although current qgroup can handle it, we still need to prevent any regression which may break current qgroup. Signed-off-by: Lu Fengqi

Re: Allocator behaviour during device delete

2016-06-13 Thread Austin S. Hemmelgarn
On 2016-06-10 15:26, Henk Slager wrote: On Thu, Jun 9, 2016 at 3:54 PM, Brendan Hide wrote: On 06/09/2016 03:07 PM, Austin S. Hemmelgarn wrote: On 2016-06-09 08:34, Brendan Hide wrote: Hey, all I noticed this odd behaviour while migrating from a 1TB spindle to

Re: [PATCH v3] fstests: btrfs: add test for qgroup handle extent de-reference

2016-06-13 Thread Lu Fengqi
At 06/13/2016 05:49 PM, Filipe Manana wrote: On Mon, Jun 13, 2016 at 9:06 AM, Lu Fengqi wrote: At 06/13/2016 03:29 PM, Lu Fengqi wrote: At 06/13/2016 11:04 AM, Eryu Guan wrote: On Mon, Jun 13, 2016 at 10:10:50AM +0800, Lu Fengqi wrote: Test if qgroup can handle