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 to be absolutely sure

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 > --- > Although I hope these trace points w

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 >|<-reques

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 seems that it > extend

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 seems that it > extend

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 > extends the data/meta

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 sub-stripe is equal > to the

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

2016-07-26 Thread Sanidhya Solanki
On Tue, 26 Jul 2016 11:14:37 -0600 Chris Murphy wrote: > On Fri, Jul 22, 2016 at 8:58 AM, Austin S. Hemmelgarn > wrote: > > On 2016-07-22 09:42, Sanidhya Solanki wrote: > > >> +*stripesize=*;; > >> +Specifies the new stripe size > > It'd be ni

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" wrote: > On 2016-07-22 09:42, Sanidhya Solanki wrote: > > +*stripesize=*;; > > +Specifies the new stripe size for a filesystem instance. Multiple BTrFS > > +filesystems mounted in parallel with varying stri

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
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 --- fs/btrfs/ctree.h| 21 ++-- fs/btrfs/disk-io.c | 12 ++- fs/btrfs

[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 --- Documentation/btrfs-balance.asciidoc | 14 + btrfs-convert.c | 59

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 > --- > fs/btrfs/scrub.c | 2 +- > 1 file changed, 1 insertion(+), 1 d

[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: memcpy(pointers[n

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 be tricky at some

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 Flash devices. Depending

[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 --- fs

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 RAID 6 case after a

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 and bug requests. I

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 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 balance filter > style of c

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 problem. I do not run BTRFS on m

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 and configuring it as they

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 a runtime option t

[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 --- 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 1a33d3e..d33734a 100644 --- a/fs

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 usable and you jus

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? I will try and re

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 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? Crashes? > What do you

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 > time... > The result

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 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 implemented as well... That sounds like an a

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 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 in the BTRFS File todo lis

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 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 > > in the BTRFS Config setup &g

[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 --- fs/btrfs/Kconfig | 42 ++ fs/btrfs/Makefile