Re: [PATCH 00/15] btrfs: Hot spare and Auto replace

2015-11-17 Thread Austin S Hemmelgarn
On 2015-11-16 17:07, Anand Jain wrote: On 11/16/2015 09:41 PM, Austin S Hemmelgarn wrote: On 2015-11-09 05:56, Anand Jain wrote: These set of patches provides btrfs hot spare and auto replace support for you review and comments. First, here below are the simple example steps to configure

Re: [PATCH 00/15] btrfs: Hot spare and Auto replace

2015-11-16 Thread Austin S Hemmelgarn
On 2015-11-13 19:54, Qu Wenruo wrote: 在 2015年11月13日 18:20, Anand Jain 写道: Thanks for commenting. I'm sorry but I didn't quite see the benefit of a spare device. Aside from what Duncan said (and I happen to agree with him), there is also the fact that hot-spares are (at least

Re: [PATCH 00/15] btrfs: Hot spare and Auto replace

2015-11-16 Thread Austin S Hemmelgarn
On 2015-11-09 05:56, Anand Jain wrote: These set of patches provides btrfs hot spare and auto replace support for you review and comments. First, here below are the simple example steps to configure the same: Add a spare device: btrfs spare add /dev/sde -f OR if there is a spare device

Re: [PATCH 00/15] btrfs: Hot spare and Auto replace

2015-11-16 Thread Anand Jain
On 11/16/2015 09:41 PM, Austin S Hemmelgarn wrote: On 2015-11-09 05:56, Anand Jain wrote: These set of patches provides btrfs hot spare and auto replace support for you review and comments. First, here below are the simple example steps to configure the same: Add a spare device: btrfs

Re: [PATCH 00/15] btrfs: Hot spare and Auto replace

2015-11-15 Thread Christoph Anton Mitterer
Hey. You guys may want to update: https://btrfs.wiki.kernel.org/index.php/Project_ideas#Hot_spare_support Cheers, Chris. smime.p7s Description: S/MIME cryptographic signature

Re: [PATCH 00/15] btrfs: Hot spare and Auto replace

2015-11-14 Thread Goffredo Baroncelli
On 2015-11-13 11:20, Anand Jain wrote: > > Thanks for comments. > > On 11/13/2015 03:21 AM, Goffredo Baroncelli wrote: >> On 2015-11-09 11:56, Anand Jain wrote: >>> These set of patches provides btrfs hot spare and auto replace support >>> for you review and comments. >> >> Hi Anand, >> >> is

Re: [PATCH 00/15] btrfs: Hot spare and Auto replace

2015-11-13 Thread Anand Jain
Thanks for the comments. Sorry for the delay. Trying to find out if there is any pending concerns... Hopefully, per-filesystem hot-spares will be a high priority too, as that type of usage is pretty much required for many enterprise type uses, although that doesn't need to be the same code

Re: [PATCH 00/15] btrfs: Hot spare and Auto replace

2015-11-13 Thread Anand Jain
Thanks for commenting. I'm sorry but I didn't quite see the benefit of a spare device. Aside from what Duncan said (and I happen to agree with him), there is also the fact that hot-spares are (at least traditionally in most RAID systems) usually used with RAID5 or RAID6 (or some other parity

Re: [PATCH 00/15] btrfs: Hot spare and Auto replace

2015-11-13 Thread Qu Wenruo
在 2015年11月13日 18:20, Anand Jain 写道: Thanks for commenting. I'm sorry but I didn't quite see the benefit of a spare device. Aside from what Duncan said (and I happen to agree with him), there is also the fact that hot-spares are (at least traditionally in most RAID systems) usually used

Re: [PATCH 00/15] btrfs: Hot spare and Auto replace

2015-11-13 Thread Anand Jain
Thanks for the comments. Let's take the following example: 1) 2 RAID1 + 1 spare (A + B) + C 2) 3 RAID1 (A + B + C) At least in normal operation case, case 1) makes device C useless, and Yes. For case 2), we can just relocate and recover the bad chunks in B. It it should only

Re: [PATCH 00/15] btrfs: Hot spare and Auto replace

2015-11-13 Thread Anand Jain
Thanks for comments. On 11/13/2015 03:21 AM, Goffredo Baroncelli wrote: On 2015-11-09 11:56, Anand Jain wrote: These set of patches provides btrfs hot spare and auto replace support for you review and comments. Hi Anand, is there any reason to put this kind of logic in the kernel space ? I

Re: [PATCH 00/15] btrfs: Hot spare and Auto replace

2015-11-13 Thread Austin S Hemmelgarn
On 2015-11-13 05:17, Anand Jain wrote: Thanks for the comments. Sorry for the delay. Trying to find out if there is any pending concerns... FWIW, I'm planning on setting up a VM to test this over the weekend (I would have already, but I've been kind of busy at work this week), so I'll

Re: [PATCH 00/15] btrfs: Hot spare and Auto replace

2015-11-12 Thread Qu Wenruo
Austin S Hemmelgarn wrote on 2015/11/12 08:04 -0500: On 2015-11-11 21:15, Qu Wenruo wrote: Hi Anand, Nice work. But I have some small questions about it. Anand Jain wrote on 2015/11/09 18:56 +0800: These set of patches provides btrfs hot spare and auto replace support for you review and

Re: [PATCH 00/15] btrfs: Hot spare and Auto replace

2015-11-12 Thread Austin S Hemmelgarn
On 2015-11-11 21:15, Qu Wenruo wrote: Hi Anand, Nice work. But I have some small questions about it. Anand Jain wrote on 2015/11/09 18:56 +0800: These set of patches provides btrfs hot spare and auto replace support for you review and comments. First, here below are the simple example steps

Re: [PATCH 00/15] btrfs: Hot spare and Auto replace

2015-11-12 Thread Goffredo Baroncelli
On 2015-11-09 11:56, Anand Jain wrote: > These set of patches provides btrfs hot spare and auto replace support > for you review and comments. Hi Anand, is there any reason to put this kind of logic in the kernel space ? I think that it could be more simply to create a daemon which checks the

Re: [PATCH 00/15] btrfs: Hot spare and Auto replace

2015-11-12 Thread Goffredo Baroncelli
On 2015-11-12 03:15, Qu Wenruo wrote: > Hi Anand, > > Nice work. > But I have some small questions about it. > > Anand Jain wrote on 2015/11/09 18:56 +0800: >> These set of patches provides btrfs hot spare and auto replace support >> for you review and comments. >> >> First, here below are the

Re: [PATCH 00/15] btrfs: Hot spare and Auto replace

2015-11-11 Thread Qu Wenruo
Hi Anand, Nice work. But I have some small questions about it. Anand Jain wrote on 2015/11/09 18:56 +0800: These set of patches provides btrfs hot spare and auto replace support for you review and comments. First, here below are the simple example steps to configure the same: Add a spare

Re: [PATCH 00/15] btrfs: Hot spare and Auto replace

2015-11-11 Thread Duncan
Qu Wenruo posted on Thu, 12 Nov 2015 10:15:09 +0800 as excerpted: > Anand Jain wrote on 2015/11/09 18:56 +0800: >> These set of patches provides btrfs hot spare and auto replace support >> for you review and comments. >> >> First, here below are the simple example steps to configure the same: >>

Re: [PATCH 00/15] btrfs: Hot spare and Auto replace

2015-11-10 Thread Austin S Hemmelgarn
On 2015-11-09 16:29, Duncan wrote: Austin S Hemmelgarn posted on Mon, 09 Nov 2015 09:09:07 -0500 as excerpted: btrfs fi show Label: none uuid: 52f170c1-725c-457d-8cfd-d57090460091 Total devices 2 FS bytes used 112.00KiB devid1 size 2.00GiB used 417.50MiB path /dev/sdc

Re: [PATCH 00/15] btrfs: Hot spare and Auto replace

2015-11-09 Thread Austin S Hemmelgarn
On 2015-11-09 05:56, Anand Jain wrote: These set of patches provides btrfs hot spare and auto replace support for you review and comments. It's absolutely awesome to see that someone picked up this project, it's something that's very useful and helps BTRFS to compete with many established

Re: [PATCH 00/15] btrfs: Hot spare and Auto replace

2015-11-09 Thread Duncan
Austin S Hemmelgarn posted on Mon, 09 Nov 2015 09:09:07 -0500 as excerpted: >> btrfs fi show >> Label: none uuid: 52f170c1-725c-457d-8cfd-d57090460091 >>Total devices 2 FS bytes used 112.00KiB >>devid1 size 2.00GiB used 417.50MiB path /dev/sdc >>devid2 size 2.00GiB used

[PATCH 00/15] btrfs: Hot spare and Auto replace

2015-11-09 Thread Anand Jain
These set of patches provides btrfs hot spare and auto replace support for you review and comments. First, here below are the simple example steps to configure the same: Add a spare device: btrfs spare add /dev/sde -f OR if there is a spare device which is already added before the, just run