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

2016-08-18 Thread Hugo Mills
On Thu, Aug 18, 2016 at 03:26:58PM +0200, David Sterba wrote: > On Thu, Jul 28, 2016 at 11:42:55AM -0400, Sanidhya Solanki 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 > > >

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

2016-08-18 Thread David Sterba
On Thu, Jul 28, 2016 at 11:42:55AM -0400, Sanidhya Solanki 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: 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: Change RAID stripesize to a user-configurable option

2016-07-28 Thread David Sterba
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/metadata/system group, but I think this is totally wrong. The

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
Adds the kernel component of making the RAID stripesize user configurable. Updates the kernel ioctl interface to account for new options. Updates the existing implementations of RAID stripesize in metadata. Make the stripesize an user-configurable option. Convert the existing metadata option of