Re: [dm-devel] [PATCH] multipath-tools: add VIOLIN arrays to hardware table

2016-10-17 Thread Xose Vazquez Perez 
On 10/16/2016 06:56 PM, Hannes Reinecke wrote:

>> diff --git a/libmultipath/hwtable.c b/libmultipath/hwtable.c
>> index aca46ab..e34032d 100644
>> --- a/libmultipath/hwtable.c
>> +++ b/libmultipath/hwtable.c
>> @@ -944,6 +944,32 @@ static struct hwentry default_hw[] = {
>>  .pgpolicy  = MULTIBUS,
>>  .no_path_retry = 12,
>>  },
>> +/*
>> + * Violin Memory
>> + */
>> +{
>> +/* V3000 / V6000 Series */
>> +.vendor= "VIOLIN",
>> +.product   = "SAN ARRAY$",
>> +.pgpolicy  = GROUP_BY_SERIAL,
>> +.no_path_retry = 30,
>> +},
>> +{
>> +.vendor= "VIOLIN",
>> +.product   = "SAN ARRAY ALUA",
>> +.hwhandler = "1 alua",
>> +.pgpolicy  = GROUP_BY_PRIO,
>> +.pgfailback= -FAILBACK_IMMEDIATE,
>> +.prio_name = PRIO_ALUA,
>> +.no_path_retry = 30,
>> +},
>> +{
>> +/* FSP 7000 family */
>> +.vendor= "VIOLIN",
>> +.product   = "CONCERTO ARRAY",
>> +.pgpolicy  = MULTIBUS,
>> +.no_path_retry = 30,
>> +},
>>  #if 0
>>  /*
>>   * Copy this TEMPLATE to add new hardware.
>>
> Where's the point of the first and the third entry?
> They'll be identical to the default settings, no?

for VIOLIN/"SAN ARRAY"  .pgpolicy  = GROUP_BY_SERIAL,
for VIOLIN/"CONCERTO ARRAY" .pgpolicy  = MULTIBUS,

and DEFAULT_PGPOLICY is FAILOVER

--
dm-devel mailing list
dm-devel@redhat.com
https://www.redhat.com/mailman/listinfo/dm-devel

Re: [dm-devel] [PATCH] multipath-tools: add VIOLIN arrays to hardware table

2016-10-16 Thread Hannes Reinecke 

On 10/15/2016 01:50 PM, Xose Vazquez Perez wrote:

Based on documentation provided by the manufacturer:
https://drive.google.com/open?id=0B_B6YmEmO7cDQlMzc1BsaUxZRVU

Cc: Christophe Varoqui 
Cc: device-mapper development 
Signed-off-by: Xose Vazquez Perez 
---
 libmultipath/hwtable.c | 26 ++
 1 file changed, 26 insertions(+)

diff --git a/libmultipath/hwtable.c b/libmultipath/hwtable.c
index aca46ab..e34032d 100644
--- a/libmultipath/hwtable.c
+++ b/libmultipath/hwtable.c
@@ -944,6 +944,32 @@ static struct hwentry default_hw[] = {
.pgpolicy  = MULTIBUS,
.no_path_retry = 12,
},
+   /*
+* Violin Memory
+*/
+   {
+   /* V3000 / V6000 Series */
+   .vendor= "VIOLIN",
+   .product   = "SAN ARRAY$",
+   .pgpolicy  = GROUP_BY_SERIAL,
+   .no_path_retry = 30,
+   },
+   {
+   .vendor= "VIOLIN",
+   .product   = "SAN ARRAY ALUA",
+   .hwhandler = "1 alua",
+   .pgpolicy  = GROUP_BY_PRIO,
+   .pgfailback= -FAILBACK_IMMEDIATE,
+   .prio_name = PRIO_ALUA,
+   .no_path_retry = 30,
+   },
+   {
+   /* FSP 7000 family */
+   .vendor= "VIOLIN",
+   .product   = "CONCERTO ARRAY",
+   .pgpolicy  = MULTIBUS,
+   .no_path_retry = 30,
+   },
 #if 0
/*
 * Copy this TEMPLATE to add new hardware.


Where's the point of the first and the third entry?
They'll be identical to the default settings, no?

Cheers,

Hannes
--
Dr. Hannes Reinecke   zSeries & Storage
h...@suse.de  +49 911 74053 688
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: J. Hawn, J. Guild, F. Imendörffer, HRB 16746 (AG Nürnberg)

--
dm-devel mailing list
dm-devel@redhat.com
https://www.redhat.com/mailman/listinfo/dm-devel

[dm-devel] [PATCH] multipath-tools: add VIOLIN arrays to hardware table

2016-10-15 Thread Xose Vazquez Perez 
Based on documentation provided by the manufacturer:
https://drive.google.com/open?id=0B_B6YmEmO7cDQlMzc1BsaUxZRVU

Cc: Christophe Varoqui 
Cc: device-mapper development 
Signed-off-by: Xose Vazquez Perez 
---
 libmultipath/hwtable.c | 26 ++
 1 file changed, 26 insertions(+)

diff --git a/libmultipath/hwtable.c b/libmultipath/hwtable.c
index aca46ab..e34032d 100644
--- a/libmultipath/hwtable.c
+++ b/libmultipath/hwtable.c
@@ -944,6 +944,32 @@ static struct hwentry default_hw[] = {
.pgpolicy  = MULTIBUS,
.no_path_retry = 12,
},
+   /*
+* Violin Memory
+*/
+   {
+   /* V3000 / V6000 Series */
+   .vendor= "VIOLIN",
+   .product   = "SAN ARRAY$",
+   .pgpolicy  = GROUP_BY_SERIAL,
+   .no_path_retry = 30,
+   },
+   {
+   .vendor= "VIOLIN",
+   .product   = "SAN ARRAY ALUA",
+   .hwhandler = "1 alua",
+   .pgpolicy  = GROUP_BY_PRIO,
+   .pgfailback= -FAILBACK_IMMEDIATE,
+   .prio_name = PRIO_ALUA,
+   .no_path_retry = 30,
+   },
+   {
+   /* FSP 7000 family */
+   .vendor= "VIOLIN",
+   .product   = "CONCERTO ARRAY",
+   .pgpolicy  = MULTIBUS,
+   .no_path_retry = 30,
+   },
 #if 0
/*
 * Copy this TEMPLATE to add new hardware.
-- 
2.10.1

--
dm-devel mailing list
dm-devel@redhat.com
https://www.redhat.com/mailman/listinfo/dm-devel