Re: BTRFS converted from EXT4 becomes read-only after reboot

2017-05-08 Thread Sanidhya Solanki
On Mon, 8 May 2017 10:16:44 -0400 Alexandru Guzu wrote: > Sean, how would you approach the copy of the data back and forth if > the OS is on it? Would a Send-receive and then back work? You could use a Live-USB and then just dd it to remote or attached storage, if you want

Re: [PATCH v3 2/3] btrfs: Add trace point for qgroup reserved space

2016-11-08 Thread Sanidhya Solanki
On Tue, 8 Nov 2016 10:20:43 +0800 Qu Wenruo wrote: > Introduce the following trace points: > qgroup_update_reserve > qgroup_meta_reserve > > These trace points are handy to trace qgroup reserve space related > problems. > > Signed-off-by: Qu Wenruo

Re: [RFC PATCH v0.8 02/14] btrfs-progs: Allow __btrfs_map_block_v2 to remove unrelated stripes

2016-10-20 Thread Sanidhya Solanki
On Mon, 17 Oct 2016 09:27:31 +0800 Qu Wenruo wrote: > For READ, caller normally hopes to get what they request, other than > full stripe map. > > In this case, we should remove unrelated stripe map, just like the > following case: >32K 96K

Re: [PATCH] btrfs: Change RAID stripesize to a user-configurable option

2016-08-02 Thread Sanidhya Solanki
Any comments? On Thu, 28 Jul 2016 13:32:27 +0200 David Sterba wrote: > I'll comment on the overall approach and skip code-specific comments. > > The changelog does not explain why there's a need for a new blockgroup > type and what's the relation to the existing types. It

Re: [PATCH] btrfs: Change RAID stripesize to a user-configurable option

2016-07-30 Thread Sanidhya Solanki
Any comments? On Thu, 28 Jul 2016 13:32:27 +0200 David Sterba wrote: > I'll comment on the overall approach and skip code-specific comments. > > The changelog does not explain why there's a need for a new blockgroup > type and what's the relation to the existing types. It

Re: [PATCH] btrfs: Change RAID stripesize to a user-configurable option

2016-07-28 Thread Sanidhya Solanki
On Thu, 28 Jul 2016 13:32:27 +0200 David Sterba wrote: > I'll comment on the overall approach and skip code-specific comments. > > The changelog does not explain why there's a need for a new blockgroup > type and what's the relation to the existing types. It seems that it >

Re: [PATCH] btrfs-progs: Make RAID stripesize configurable

2016-07-27 Thread Sanidhya Solanki
On Wed, 27 Jul 2016 18:25:48 +0200 Goffredo Baroncelli wrote: > I am not able to understand this sentence: on the best of my knowledge, > in btrfs the RAID5/RAID6 stripe is composed by several sub-stripes (I am > not sure about the terminology to adopt); the number of

Re: [PATCH] btrfs-progs: Make RAID stripesize configurable

2016-07-27 Thread Sanidhya Solanki
On Tue, 26 Jul 2016 11:14:37 -0600 Chris Murphy <li...@colorremedies.com> wrote: > On Fri, Jul 22, 2016 at 8:58 AM, Austin S. Hemmelgarn > <ahferro...@gmail.com> wrote: > > On 2016-07-22 09:42, Sanidhya Solanki wrote: > > >> +*stripesize=*;; > >&g

Re: [PATCH] btrfs-progs: Make RAID stripesize configurable

2016-07-22 Thread Sanidhya Solanki
On Fri, 22 Jul 2016 10:58:59 -0400 "Austin S. Hemmelgarn" <ahferro...@gmail.com> wrote: > On 2016-07-22 09:42, Sanidhya Solanki wrote: > > +*stripesize=*;; > > +Specifies the new stripe size for a filesystem instance. Multiple BTrFS > > +filesystems mounted

Re: [PATCH] btrfs: Change RAID stripesize to a user-configurable option

2016-07-22 Thread Sanidhya Solanki
Applies to v4.7rc7 release kernel. Sanidhya -- 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] btrfs: Change RAID stripesize to a user-configurable option

2016-07-22 Thread Sanidhya Solanki
of stripesize into the basis for this option. Updates the kernel component of RAID stripesize management. Update the RAID stripe block management. Signed-off-by: Sanidhya Solanki <lkml.p...@gmail.com> --- fs/btrfs/ctree.h| 21 ++-- fs/btrfs/disk-io.c

[PATCH] btrfs-progs: Make RAID stripesize configurable

2016-07-22 Thread Sanidhya Solanki
for new options. Updates the user-space component of RAID stripesize management. Updates the TODO list for future tasks. Patch applies to the v4.6.1 release branch. Signed-off-by: Sanidhya Solanki <lkml.p...@gmail.com> --- Documentation/btrfs-balance.asciidoc | 14 + btrfs-con

Re: [PATCH] btrfs: Fix BUG_ON condition in scrub_setup_recheck_block()

2016-05-04 Thread Sanidhya Solanki
On Fri, 29 Apr 2016 18:33:59 -0700 Ashish Samant wrote: > pagev array in scrub_block{} is of size SCRUB_MAX_PAGES_PER_BLOCK. > page_index should be checked with the same to trigger BUG_ON(). > > Signed-off-by: Ashish Samant > --- >

[no subject]

2016-03-30 Thread Sanidhya Solanki
subscribe linux-btrfs -- 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

Re: [PATCH] BTRFS: Runs the xor function if a Block has failed

2016-01-05 Thread Sanidhya Solanki
On Tue, 5 Jan 2016 10:22:36 +0100 David Sterba wrote: > If the data a rerecovered, why is -EIO still returned? In the other places in the file where the code appears, the submitted patch is all that is required to do the xor. I think we also need to include the following line:

Re: [PATCH] BTRFS: Adds the files and options needed for Hybrid Storage

2016-01-02 Thread Sanidhya Solanki
On Sat, 02 Jan 2016 12:40:46 +0100 Martin Steigerwald wrote: > Or is this something different? Yes, Martin this patch starts the implementation that I hope will lead to the implementation of a Hybrid Cache to the BTRFS. However, there is no need to limit ourselves to just

Re: [PATCH] BTRFS: Adds an option to select RAID Stripe size

2016-01-02 Thread Sanidhya Solanki
On Tue, 29 Dec 2015 19:06:44 +0100 David Sterba wrote: > In theory this is possible with current on-disk data structures. The > stripe length is property of btrfs_chunk and changing it should be > possible the same way we do other raid transformations. The > implementation might

[PATCH] BTRFS: Adds the files and options needed for Hybrid Storage

2016-01-01 Thread Sanidhya Solanki
This patch adds the file required for Hybrid Storage. It contains the memory, time and size limits for the cache and the statistics that will be provided while the cache is operating. It also adds the Makefile changes needed to add the Hybrid Storage. Signed-off-by: Sanidhya Solanki <jpag

Re: [PATCH] BTRFS: Adds an option to select RAID Stripe size

2015-12-30 Thread Sanidhya Solanki
On Wed, 30 Dec 2015 16:58:05 +0100 David Sterba <dste...@suse.cz> wrote: > On Wed, Dec 30, 2015 at 06:15:23AM -0500, Sanidhya Solanki wrote: > > - Implement a way to do an in-place Stripe Length change. > > How are you going to implement that? I've suggested the b

Re: [PATCH] BTRFS: Adds an option to select RAID Stripe size

2015-12-30 Thread Sanidhya Solanki
On Wed, 30 Dec 2015 17:17:22 +0100 David Sterba wrote: > Let me note that a good reputation is also built from patch reviews > (hint hint). Unfortunately, not too many patches coming in for BTRFS presently. Mailing list activity is down to 25-35 mails per day. Mostly feature

Re: [PATCH] BTRFS: Runs the xor function if a Block has failed

2015-12-30 Thread Sanidhya Solanki
On Wed, 30 Dec 2015 18:18:26 +0100 David Sterba wrote: > That's just the comment copied, the changelog does not explain why > it's ok to do just the run_xor there. It does not seem trivial to me. > Please describe that the end result after the code change is expected. In the

[PATCH] BTRFS: Runs the xor function if a Block has failed

2015-12-30 Thread Sanidhya Solanki
The patch adds the xor function after the P stripe has failed, without bad data or the Q stripe. Signed-off-by: Sanidhya Solanki <jpage.l...@gmail.com> --- fs/btrfs/raid56.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/btrfs/raid56.c b/fs/btrfs/raid56.c index 1

Re: [PATCH] BTRFS: Adds an option to select RAID Stripe size

2015-12-30 Thread Sanidhya Solanki
On Tue, 29 Dec 2015 18:06:11 +0100 David Sterba wrote: > So you want to make the stripe size configurable?... As I see it there are 3 ways to do it: -Make it a compile time option that only configures it for a single system with any devices that are added to the RAID. -Make it

Re: [PATCH] BTRFS: Adds an option to select RAID Stripe size

2015-12-30 Thread Sanidhya Solanki
On Wed, 30 Dec 2015 19:59:16 +0800 Qu Wenruo wrote: > Not really sure about the difference between 2 and 3. I should have made it clear before, I was asking the exact use case in mind when listing the choices. Option 2 would be for SysAdmins running production software

Re: [PATCH] BTRFS: Adds an option to select RAID Stripe size

2015-12-30 Thread Sanidhya Solanki
On Wed, 30 Dec 2015 22:10:44 +0800 Qu Wenruo wrote: > Understood now. Good. > I totally understand that implement ... to polish your > skill. That has got to be the most hilarious way I believe I have seen someone delegate a task. But it was effective. Only one

Re: [PATCH] BTRFS: Adds an option to select RAID Stripe size

2015-12-29 Thread Sanidhya Solanki
On Tue, 29 Dec 2015 14:39:07 +0100 David Sterba wrote: > The stripe size depends on how the filesystem was made, at the moment > the stripesize parameter is missing from mkfs. The kernel module > should support all sizes at runtime, so it's not a compile-time > option. No good?

Re: [PATCH] BTRFS: Adds an option to select RAID Stripe size

2015-12-29 Thread Sanidhya Solanki
On Tue, 29 Dec 2015 18:06:11 +0100 David Sterba wrote: > I don't know about all implications from changing the define to > sb->stripesize, also we want to define the allowed range etc. It would > be better to add more description to the patch. So, is the patch atleast somewhat

Re: [PATCH] BTRFS: Adds an option to select RAID Stripe size

2015-12-28 Thread Sanidhya Solanki
On Tue, 29 Dec 2015 02:21:09 +0100 Christoph Anton Mitterer <cales...@scientia.net> wrote: > On Mon, 2015-12-28 at 15:38 -0500, Sanidhya Solanki wrote: > > > Shouldn't that rather eventually become configurable per > > > filesystem? > > Don't know. It was i

[PATCH] BTRFS: Adds an option to select RAID Stripe size

2015-12-28 Thread Sanidhya Solanki
An option to select the RAID Stripe size is made available in the BTRFS Filesystem, via an option in the BTRFS Config setup, with minimal change to the existing code base. Signed-off-by: Sanidhya Solanki <jpage.l...@gmail.com> --- fs/btrfs/Kconfig

Re: [PATCH] BTRFS: Adds an option to select RAID Stripe size

2015-12-28 Thread Sanidhya Solanki
On Tue, 29 Dec 2015 04:42:08 +0100 Christoph Anton Mitterer <cales...@scientia.net> wrote: > On Mon, 2015-12-28 at 16:43 -0500, Sanidhya Solanki wrote: > May get even much more complicated, if reshaping (i.e. conversion from > one chunk size to another) should get impl

Re: [PATCH] BTRFS: Adds an option to select RAID Stripe size

2015-12-28 Thread Sanidhya Solanki
On Tue, 29 Dec 2015 05:26:28 +0100 Christoph Anton Mitterer wrote: > I spoke largely from the user/admin side,... running a quite big > storage Tier-2, we did many IO benchmarks over time (with different > hardware RAID controllers) and also as our IO patterns changed over

Re: [PATCH] BTRFS: Adds an option to select RAID Stripe size

2015-12-28 Thread Sanidhya Solanki
On Mon, 28 Dec 2015 23:19:55 +0100 Christoph Anton Mitterer <cales...@scientia.net> wrote: > On Mon, 2015-12-28 at 07:24 -0500, Sanidhya Solanki wrote: > > An option to select the RAID Stripe size is made > > available in the BTRFS Filesystem, via an option > >

Re: [PATCH] BTRFS: Adds an option to select RAID Stripe size

2015-12-28 Thread Sanidhya Solanki
On Tue, 29 Dec 2015 07:03:11 +0100 Christoph Anton Mitterer <cales...@scientia.net> wrote: > On Mon, 2015-12-28 at 20:31 -0500, Sanidhya Solanki wrote: > > What is your experience like about running a production system on > > what > > is essentially a beta product?