Re: [dm-devel] [PATCH v2] dm pref-path: provides preferred path load balance policy

2016-01-30 Thread Benjamin Marzinski
On Sat, Jan 30, 2016 at 02:25:25PM -0600, Benjamin Marzinski wrote: > Before this commit, it always used the pref bit. Again, like I said > before, I'm saying that this was the wrong thing to do. The Spec is Oops. I meant: "I'm NOT saying that this was the wrong thing to do". I am also fine with

Re: [PATCH v2] dm pref-path: provides preferred path load balance policy

2016-01-30 Thread Benjamin Marzinski
On Sat, Jan 30, 2016 at 09:32:53AM +0100, Hannes Reinecke wrote: > On 01/29/2016 06:50 PM, Benjamin Marzinski wrote: > >On Fri, Jan 29, 2016 at 02:10:52PM +, Nalla, Ravikanth wrote: > >>Hi Mike, Hannes, Ben > >>>This seems like a problem that has already been solved with path groups. > >>>If th

Re: [PATCH v2] dm pref-path: provides preferred path load balance policy

2016-01-30 Thread Hannes Reinecke
On 01/29/2016 06:50 PM, Benjamin Marzinski wrote: On Fri, Jan 29, 2016 at 02:10:52PM +, Nalla, Ravikanth wrote: Hi Mike, Hannes, Ben This seems like a problem that has already been solved with path groups. If the path(s) in your preferred path group are there, multipath will >>> use them.

Re: [PATCH v2] dm pref-path: provides preferred path load balance policy

2016-01-29 Thread Benjamin Marzinski
On Fri, Jan 29, 2016 at 02:10:52PM +, Nalla, Ravikanth wrote: > Hi Mike, Hannes, Ben > > This seems like a problem that has already been solved with path groups. > > If the path(s) in your preferred path group are there, multipath will use > > them. If not, then it will use your less preferre

RE: [PATCH v2] dm pref-path: provides preferred path load balance policy

2016-01-29 Thread Nalla, Ravikanth
Hi Mike, Hannes, Ben On 1/22/2016 10:29 PM, Mike Snitzer wrote: [Hannes please fix your mail client, seems you dropped all the original CCs] On Fri, Jan 22 2016 at 8:42am -0500, Hannes Reinecke wrote: > On 01/22/2016 02:31 PM, Ravikanth Nalla wrote: > > > > Functionality provided in this mo

Re: [dm-devel] [PATCH v2] dm pref-path: provides preferred path load balance policy

2016-01-22 Thread Benjamin Marzinski
On Fri, Jan 22, 2016 at 06:31:42AM -0700, Ravikanth Nalla wrote: > v2: > - changes merged with latest mainline and functionality re-verified. > - performed additional tests to illustrate performance benefits of > using this feature in certain configuration. > > In a dm multipath environmen

Re: [PATCH v2] dm pref-path: provides preferred path load balance policy

2016-01-22 Thread Mike Snitzer
[Hannes please fix your mail client, seems you dropped all the original CCs] On Fri, Jan 22 2016 at 8:42am -0500, Hannes Reinecke wrote: > On 01/22/2016 02:31 PM, Ravikanth Nalla wrote: > > v2: > > - changes merged with latest mainline and functionality re-verified. > > - performed addition

[PATCH v2] dm pref-path: provides preferred path load balance policy

2016-01-22 Thread Ravikanth Nalla
v2: - changes merged with latest mainline and functionality re-verified. - performed additional tests to illustrate performance benefits of using this feature in certain configuration. In a dm multipath environment, providing end user with an option of selecting preferred path for an I/O i