Re: [dm-devel] [PATCH 0/3 v3] add resync speed control for dm-raid1

2013-02-05 Thread Mikulas Patocka
On Wed, 23 Jan 2013, NeilBrown wrote: > On Tue, 22 Jan 2013 20:44:41 -0500 (EST) Mikulas Patocka > wrote: > > > > > > > On Wed, 16 Jan 2013, Guangliang Zhao wrote: > > > > > On Wed, Jan 09, 2013 at 12:43:21AM -0500, Mikulas Patocka wrote: > > > > Hi > > > Hi, > > > > > > I think it is very

Re: [dm-devel] [PATCH 0/3 v3] add resync speed control for dm-raid1

2013-01-22 Thread NeilBrown
On Tue, 22 Jan 2013 20:44:41 -0500 (EST) Mikulas Patocka wrote: > > > On Wed, 16 Jan 2013, Guangliang Zhao wrote: > > > On Wed, Jan 09, 2013 at 12:43:21AM -0500, Mikulas Patocka wrote: > > > Hi > > Hi, > > > > I think it is very good that your patches could be used for other > > targets(snaps

Re: [dm-devel] [PATCH 0/3 v3] add resync speed control for dm-raid1

2013-01-22 Thread Mikulas Patocka
On Wed, 16 Jan 2013, Guangliang Zhao wrote: > On Wed, Jan 09, 2013 at 12:43:21AM -0500, Mikulas Patocka wrote: > > Hi > Hi, > > I think it is very good that your patches could be used for other > targets(snapshot, thin) after reviewing yours, but I find some issues > (maybe not, please correct

Re: [dm-devel] [PATCH 0/3 v3] add resync speed control for dm-raid1

2013-01-16 Thread Guangliang Zhao
On Wed, Jan 09, 2013 at 12:43:21AM -0500, Mikulas Patocka wrote: > Hi Hi, I think it is very good that your patches could be used for other targets(snapshot, thin) after reviewing yours, but I find some issues (maybe not, please correct me if I am wrong). > > I did this already some times ago. >

Re: [dm-devel] [PATCH 0/3 v3] add resync speed control for dm-raid1

2013-01-08 Thread Mikulas Patocka
Hi I did this already some times ago. I'm sending my patches in the next mail. Basically, my and Guangliang's patches have the following differences: my patch: uses per-module throttle settings Guangliang's patch: uses per-device settings (my patch could be changed to use per-device throttle too