[PATCH] multipath-tools: unreachable controllers maintainers

[PATCH] multipath-tools: unreachable controllers maintainers

[Xose Vazquez Perez]

Replace 3 unreachable controllers maintainers, their emails are bounced,
with the default maintainer(Christophe Varoqui <christophe.varoqui@opensvc.com>)

Compellent Technologies, Inc.: Jim Lester <jim.lester@compellent.com>
NETAPP controller family: Dave Wysochanski <davidw@netapp.com>
IBM NSeries (NETAPP) controller family: Dave Wysochanski <davidw@netapp.com>


Cc: bartb@pivot3.com
Cc: brking@linux.vnet.ibm.com
Cc: egoggin@emc.com
Cc: g.shyamsundar@yahoo.co.in
Cc: matthias.rudolph@hds.com
Cc: sean.stewart@netapp.com
Cc: sramani@pillardata.com
Cc: yacine@alyseo.com
Cc: Hannes Reinecke <hare@suse.de>
Cc: Christophe Varoqui <christophe.varoqui@opensvc.com>
Cc: device-mapper development <dm-devel@redhat.com>
Signed-off-by: Xose Vazquez Perez <xose.vazquez@gmail.com>

---

If anyone in the Cc: list is not currently in charge of a listed controller
at hwtable.c, please drop here an email, thanks.
Ideally, an impersonal address is desired, e.g.: linux-mpt@mycompany.com

---
 libmultipath/hwtable.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/libmultipath/hwtable.c b/libmultipath/hwtable.c
index 66b5123..5c49e32 100644
--- a/libmultipath/hwtable.c
+++ b/libmultipath/hwtable.c
@@ -20,8 +20,8 @@ static struct hwentry default_hw[] = {
 	/*
 	 * Compellent Technologies, Inc.
 	 *
-	 * Maintainer : Jim Lester, Compellent
-	 * Mail : jim.lester@compellent.com
+	 * Maintainer : Christophe Varoqui
+	 * Mail : christophe.varoqui@opensvc.com
 	 */
 	{
 		.vendor        = "COMPELNT",
@@ -790,8 +790,8 @@ static struct hwentry default_hw[] = {
 	/*
 	 * NETAPP controller family
 	 *
-	 * Maintainer : Dave Wysochanski
-	 * Mail : davidw@netapp.com
+	 * Maintainer : Christophe Varoqui
+	 * Mail : christophe.varoqui@opensvc.com
 	 */
 	{
 		.vendor        = "NETAPP",
@@ -835,8 +835,8 @@ static struct hwentry default_hw[] = {
 	/*
 	 * IBM NSeries (NETAPP) controller family
 	 *
-	 * Maintainer : Dave Wysochanski
-	 * Mail : davidw@netapp.com
+	 * Maintainer : Christophe Varoqui
+	 * Mail : christophe.varoqui@opensvc.com
 	 */
 	{
 		.vendor        = "IBM",
-- 
2.5.5

Re: [PATCH] multipath-tools: unreachable controllers maintainers

[Xose Vazquez Perez]

On 06/05/2016 09:54 PM, Xose Vazquez Perez wrote:

> Replace 3 unreachable controllers maintainers, their emails are bounced,
> with the default maintainer(Christophe Varoqui <christophe.varoqui@opensvc.com>)
> 
> Compellent Technologies, Inc.: Jim Lester <jim.lester@compellent.com>
> NETAPP controller family: Dave Wysochanski <davidw@netapp.com>
> IBM NSeries (NETAPP) controller family: Dave Wysochanski <davidw@netapp.com>

> Cc: sramani@pillardata.com

Hi Christophe,

pillardata.com was bought by Oracle time ago, and it has no MX record. So
Pillar Data controller family: Srinivasan Ramani <sramani@pillardata.com>
is also orphan.

Re: [PATCH] multipath-tools: unreachable controllers maintainers

[Martin K. Petersen]

>>>>> "Xose" == Xose Vazquez Perez <xose.vazquez@gmail.com> writes:

Xose> pillardata.com was bought by Oracle time ago, and it has no MX
Xose> record. So Pillar Data controller family: Srinivasan Ramani
Xose> <sramani@pillardata.com> is also orphan.

Srinivasan Ramani <srinivas.ramani@oracle.com> is the correct address.

-- 
Martin K. Petersen	Oracle Linux Engineering

Re: [PATCH] multipath-tools: unreachable controllers maintainers

[Xose Vazquez Perez]

On 06/05/2016 09:54 PM, Xose Vazquez Perez wrote:

> Replace 3 unreachable controllers maintainers, their emails are bounced,
> with the default maintainer(Christophe Varoqui <christophe.varoqui@opensvc.com>)
> 
> Compellent Technologies, Inc.: Jim Lester <jim.lester@compellent.com>

FYI, Compellent was bought by DELL some time ago.

> NETAPP controller family: Dave Wysochanski <davidw@netapp.com>
> IBM NSeries (NETAPP) controller family: Dave Wysochanski <davidw@netapp.com>

Re: [PATCH] multipath-tools: unreachable controllers maintainers

[Xose Vazquez Perez]

On 06/05/2016 09:54 PM, Xose Vazquez Perez wrote:
> Replace 3 unreachable controllers maintainers, their emails are bounced,
> with the default maintainer(Christophe Varoqui <christophe.varoqui@opensvc.com>)

Please DROP this patch.

> Compellent Technologies, Inc.: Jim Lester <jim.lester@compellent.com>
> NETAPP controller family: Dave Wysochanski <davidw@netapp.com>
> IBM NSeries (NETAPP) controller family: Dave Wysochanski <davidw@netapp.com>
> 
> 
> Cc: bartb@pivot3.com
> Cc: brking@linux.vnet.ibm.com
> Cc: egoggin@emc.com
> Cc: g.shyamsundar@yahoo.co.in
> Cc: matthias.rudolph@hds.com
> Cc: sean.stewart@netapp.com
> Cc: sramani@pillardata.com
> Cc: yacine@alyseo.com
> Cc: Hannes Reinecke <hare@suse.de>
> Cc: Christophe Varoqui <christophe.varoqui@opensvc.com>
> Cc: device-mapper development <dm-devel@redhat.com>
> Signed-off-by: Xose Vazquez Perez <xose.vazquez@gmail.com>
> 
> ---
> 
> If anyone in the Cc: list is not currently in charge of a listed controller
> at hwtable.c, please drop here an email, thanks.
> Ideally, an impersonal address is desired, e.g.: linux-mpt@mycompany.com
> 
> ---
>  libmultipath/hwtable.c | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/libmultipath/hwtable.c b/libmultipath/hwtable.c
> index 66b5123..5c49e32 100644
> --- a/libmultipath/hwtable.c
> +++ b/libmultipath/hwtable.c
> @@ -20,8 +20,8 @@ static struct hwentry default_hw[] = {
>  	/*
>  	 * Compellent Technologies, Inc.
>  	 *
> -	 * Maintainer : Jim Lester, Compellent
> -	 * Mail : jim.lester@compellent.com
> +	 * Maintainer : Christophe Varoqui
> +	 * Mail : christophe.varoqui@opensvc.com
>  	 */
>  	{
>  		.vendor        = "COMPELNT",
> @@ -790,8 +790,8 @@ static struct hwentry default_hw[] = {
>  	/*
>  	 * NETAPP controller family
>  	 *
> -	 * Maintainer : Dave Wysochanski
> -	 * Mail : davidw@netapp.com
> +	 * Maintainer : Christophe Varoqui
> +	 * Mail : christophe.varoqui@opensvc.com
>  	 */
>  	{
>  		.vendor        = "NETAPP",
> @@ -835,8 +835,8 @@ static struct hwentry default_hw[] = {
>  	/*
>  	 * IBM NSeries (NETAPP) controller family
>  	 *
> -	 * Maintainer : Dave Wysochanski
> -	 * Mail : davidw@netapp.com
> +	 * Maintainer : Christophe Varoqui
> +	 * Mail : christophe.varoqui@opensvc.com
>  	 */
>  	{
>  		.vendor        = "IBM",
> 

Re: [PATCH] multipath-tools: unreachable controllers maintainers

[Martin George]

On 6/6/2016 1:24 AM, Xose Vazquez Perez wrote:
> Replace 3 unreachable controllers maintainers, their emails are bounced,
> with the default maintainer(Christophe Varoqui <christophe.varoqui@opensvc.com>)
>
> @@ -790,8 +790,8 @@ static struct hwentry default_hw[] = {
>   	/*
>   	 * NETAPP controller family
>   	 *
> -	 * Maintainer : Dave Wysochanski
> -	 * Mail : davidw@netapp.com
> +	 * Maintainer : Christophe Varoqui
> +	 * Mail : christophe.varoqui@opensvc.com
>   	 */
>   	{
>   		.vendor        = "NETAPP",
> @@ -835,8 +835,8 @@ static struct hwentry default_hw[] = {
>   	/*
>   	 * IBM NSeries (NETAPP) controller family
>   	 *
> -	 * Maintainer : Dave Wysochanski
> -	 * Mail : davidw@netapp.com
> +	 * Maintainer : Christophe Varoqui
> +	 * Mail : christophe.varoqui@opensvc.com
>   	 */
>   	{
>   		.vendor        = "IBM",
>

I can take up the maintainer's role here for the NetApp controller 
family. Please let me know if this mail suffices, or whether you need me 
to send a separate patch for this.

Thanks,
-Martin George (marting@netapp.com)

NetApp Ontap

[Xose Vazquez Perez]

On 06/22/2016 04:38 AM, Martin George wrote:

> I can take up the maintainer's role here for the NetApp controller family.
> Please let me know if this mail suffices, or whether you need me to send a
> separate patch for this.

Hi Martin,

I already sent a patch for this. You can reply adding to the body of the
message:
Acked-by: or Signed-off-by: Martin George <marting@netapp.com>


And one question, should NETAPP/LUN.* and IBM/Nseries.* share the same options?


Thank you.

Re: NetApp Ontap

[Martin George]

On 6/24/2016 1:12 AM, Xose Vazquez Perez wrote:
> On 06/22/2016 04:38 AM, Martin George wrote:
>
>> I can take up the maintainer's role here for the NetApp controller family.
>> Please let me know if this mail suffices, or whether you need me to send a
>> separate patch for this.
>
> Hi Martin,
>
> I already sent a patch for this. You can reply adding to the body of the
> message:
> Acked-by: or Signed-off-by: Martin George <marting@netapp.com>
>

I just did.

>
> And one question, should NETAPP/LUN.* and IBM/Nseries.* share the same options?
>

Yes, let them share the same options for now. I have some further 
updates on this which I will share later. Thanks.

-Martin George

Re: NetApp Ontap

[Xose Vazquez Perez]

On 06/24/2016 07:52 PM, Martin George wrote:

> On 6/24/2016 1:12 AM, Xose Vazquez Perez wrote:
>>
>> And one question, should NETAPP/LUN.* and IBM/Nseries.* share the same options?
>>
> Yes, let them share the same options for now. I have some further updates on
> this which I will share later. Thanks.

Right now they are different:

--- netapp-ontap        2016-06-24 20:05:35.182539928 +0200
+++ ibm-ontap   2016-06-24 20:05:26.750552635 +0200
@@ -1,18 +1,14 @@
        {
-               .vendor        = "NETAPP",
-               .product       = "LUN.*",
-               .features      = "3 queue_if_no_path pg_init_retries 50",
+               .vendor        = "IBM",
+               .product       = "Nseries.*",
+               .features      = "1 queue_if_no_path",
                .hwhandler     = DEFAULT_HWHANDLER,
                .pgpolicy      = GROUP_BY_PRIO,
                .pgfailback    = -FAILBACK_IMMEDIATE,
-               .flush_on_last_del = FLUSH_ENABLED,
                .rr_weight     = RR_WEIGHT_NONE,
                .no_path_retry = NO_PATH_RETRY_UNDEF,
                .minio         = 128,
-               .dev_loss      = MAX_DEV_LOSS_TMO,
-               .checker_name  = TUR,
+               .checker_name  = DIRECTIO,
                .prio_name     = PRIO_ONTAP,
                .prio_args     = NULL,
-               .retain_hwhandler = RETAIN_HWHANDLER_ON,
-               .detect_prio   = DETECT_PRIO_ON,

Re: NetApp Ontap

[Martin George]

On 6/24/2016 11:41 PM, Xose Vazquez Perez wrote:
> On 06/24/2016 07:52 PM, Martin George wrote:
>
>> On 6/24/2016 1:12 AM, Xose Vazquez Perez wrote:
>>>
>>> And one question, should NETAPP/LUN.* and IBM/Nseries.* share the same options?
>>>
>> Yes, let them share the same options for now. I have some further updates on
>> this which I will share later. Thanks.
>
> Right now they are different:
>
> --- netapp-ontap        2016-06-24 20:05:35.182539928 +0200
> +++ ibm-ontap   2016-06-24 20:05:26.750552635 +0200
> @@ -1,18 +1,14 @@
>          {
> -               .vendor        = "NETAPP",
> -               .product       = "LUN.*",
> -               .features      = "3 queue_if_no_path pg_init_retries 50",
> +               .vendor        = "IBM",
> +               .product       = "Nseries.*",
> +               .features      = "1 queue_if_no_path",
>                  .hwhandler     = DEFAULT_HWHANDLER,
>                  .pgpolicy      = GROUP_BY_PRIO,
>                  .pgfailback    = -FAILBACK_IMMEDIATE,
> -               .flush_on_last_del = FLUSH_ENABLED,
>                  .rr_weight     = RR_WEIGHT_NONE,
>                  .no_path_retry = NO_PATH_RETRY_UNDEF,
>                  .minio         = 128,
> -               .dev_loss      = MAX_DEV_LOSS_TMO,
> -               .checker_name  = TUR,
> +               .checker_name  = DIRECTIO,
>                  .prio_name     = PRIO_ONTAP,
>                  .prio_args     = NULL,
> -               .retain_hwhandler = RETAIN_HWHANDLER_ON,
> -               .detect_prio   = DETECT_PRIO_ON,
>

Yes, I know. I actually meant keeping these settings as it is for now. I 
will send a patch for NSeries later. Thanks.

-Martin George