[PATCH] Additional BLIST_LARGELUN devices

[PATCH] Additional BLIST_LARGELUN devices

[Kurt Garloff]

[-- Attachment #1.1: Type: text/plain, Size: 554 bytes --]

Hi Marcello, Dave,

here are some additional devices that need the BLIST_LARGELUN flag.
When I sent the patch to implement the flag, I only knew about the EMC
Symmetrix. Some more have turned up ...
The ones here have been collected from EMC, RH, SuSE sources.

Please consider inclusion into the next 2.4.19 RC.

Regards,
-- 
Kurt Garloff  <garloff@suse.de>                          Eindhoven, NL
GPG key: See mail header, key servers         Linux kernel development
SuSE Linux AG, Nuernberg, DE                            SCSI, Security

[-- Attachment #1.2: 50_scsi-largelun-2.4.19-pre7aa1 --]
[-- Type: text/plain, Size: 2127 bytes --]

diff -urN linux-2.4.18.tmp/drivers/scsi/scsi_scan.c linux-2.4.18.SuSE/drivers/scsi/scsi_scan.c
--- linux-2.4.18.tmp/drivers/scsi/scsi_scan.c	Wed Apr 24 11:26:44 2002
+++ linux-2.4.18.SuSE/drivers/scsi/scsi_scan.c	Wed Apr 24 11:35:48 2002
@@ -149,17 +149,17 @@
  	{"TOSHIBA","CDROM","*", BLIST_ISROM},
  	{"TOSHIBA","CD-ROM","*", BLIST_ISROM},
 	{"MegaRAID", "LD", "*", BLIST_FORCELUN},
-	{"DGC",  "RAID",      "*", BLIST_SPARSELUN}, // Dell PV 650F (tgt @ LUN 0)
-	{"DGC",  "DISK",      "*", BLIST_SPARSELUN}, // Dell PV 650F (no tgt @ LUN 0) 
-	{"DELL", "PV660F",   "*", BLIST_SPARSELUN},
-	{"DELL", "PV660F   PSEUDO",   "*", BLIST_SPARSELUN},
-	{"DELL", "PSEUDO DEVICE .",   "*", BLIST_SPARSELUN}, // Dell PV 530F
-	{"DELL", "PV530F",    "*", BLIST_SPARSELUN}, // Dell PV 530F
+	{"DGC",  "RAID",      "*", BLIST_SPARSELUN | BLIST_LARGELUN}, // Dell PV 650F (tgt @ LUN 0)
+	{"DGC",  "DISK",      "*", BLIST_SPARSELUN | BLIST_LARGELUN}, // Dell PV 650F (no tgt @ LUN 0) 
+	{"DELL", "PV660F",   "*", BLIST_SPARSELUN | BLIST_LARGELUN},
+	{"DELL", "PV660F   PSEUDO",   "*", BLIST_SPARSELUN | BLIST_LARGELUN},
+	{"DELL", "PSEUDO DEVICE .",   "*", BLIST_SPARSELUN | BLIST_LARGELUN}, // Dell PV 530F
+	{"DELL", "PV530F",    "*", BLIST_SPARSELUN | BLIST_LARGELUN}, // Dell PV 530F
 	{"EMC", "SYMMETRIX", "*", BLIST_SPARSELUN | BLIST_LARGELUN | BLIST_FORCELUN},
 	{"HP", "A6189A", "*", BLIST_SPARSELUN |  BLIST_LARGELUN}, // HP VA7400, by Alar Aun
-	{"CMD", "CRA-7280", "*", BLIST_SPARSELUN},   // CMD RAID Controller
-	{"CNSI", "G7324", "*", BLIST_SPARSELUN},     // Chaparral G7324 RAID
-	{"CNSi", "G8324", "*", BLIST_SPARSELUN},     // Chaparral G8324 RAID
+	{"CMD", "CRA-7280", "*", BLIST_SPARSELUN | BLIST_LARGELUN},   // CMD RAID Controller
+	{"CNSI", "G7324", "*", BLIST_SPARSELUN | BLIST_LARGELUN},     // Chaparral G7324 RAID
+	{"CNSi", "G8324", "*", BLIST_SPARSELUN | BLIST_LARGELUN},     // Chaparral G8324 RAID
 	{"Zzyzx", "RocketStor 500S", "*", BLIST_SPARSELUN},
 	{"Zzyzx", "RocketStor 2000", "*", BLIST_SPARSELUN},
 	{"SONY", "TSL",       "*", BLIST_FORCELUN},  // DDS3 & DDS4 autoloaders

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [PATCH] Additional BLIST_LARGELUN devices

[Kurt Garloff]

[-- Attachment #1: Type: text/plain, Size: 780 bytes --]

Hi,

sorry to follow up myself:

On Wed, Jun 26, 2002 at 06:47:44PM +0200, Kurt Garloff wrote:
> Hi Marcello, Dave,
> 
> here are some additional devices that need the BLIST_LARGELUN flag.
> When I sent the patch to implement the flag, I only knew about the EMC
> Symmetrix. Some more have turned up ...

A related question: It might be useful to such a flag at the kernel command
line. Like a "largelun=c2b0t9" parameter. Would people consider this useful?
Would it be accepted into the std. kernel? If yes², I can code a patch ...

Regards.
-- 
Kurt Garloff  <garloff@suse.de>                          Eindhoven, NL
GPG key: See mail header, key servers         Linux kernel development
SuSE Linux AG, Nuernberg, DE                            SCSI, Security

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [PATCH] Additional BLIST_LARGELUN devices

[Michael Heinz]

Gee, I thought I suggested this the other week and got sneered at.

On 6/26/02 6:07 PM, "    Kurt Garloff" <garloff@suse.de> wrote:

> Hi,
> 
> sorry to follow up myself:
> 
> On Wed, Jun 26, 2002 at 06:47:44PM +0200, Kurt Garloff wrote:
>> Hi Marcello, Dave,
>> 
>> here are some additional devices that need the BLIST_LARGELUN flag.
>> When I sent the patch to implement the flag, I only knew about the EMC
>> Symmetrix. Some more have turned up ...
> 
> A related question: It might be useful to such a flag at the kernel command
> line. Like a "largelun=c2b0t9" parameter. Would people consider this useful?
> Would it be accepted into the std. kernel? If yes², I can code a patch ...
> 
> Regards.

-- 
Michael Heinz <mheinz@infiniconsys.com>
Principal Software Engineer
InfiniCon Systems, Inc.

-
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] Additional BLIST_LARGELUN devices

[Michael Clark]

On 06/27/02 00:47, Kurt Garloff wrote:

>Hi Marcello, Dave,
>
>here are some additional devices that need the BLIST_LARGELUN flag.
>

[snip]

>-	{"CNSI", "G7324", "*", BLIST_SPARSELUN},     // Chaparral G7324 RAID
>-	{"CNSi", "G8324", "*", BLIST_SPARSELUN},     // Chaparral G8324 RAID
>+	{"CMD", "CRA-7280", "*", BLIST_SPARSELUN | BLIST_LARGELUN},   // CMD RAID Controller
>+	{"CNSI", "G7324", "*", BLIST_SPARSELUN | BLIST_LARGELUN},     // Chaparral G7324 RAID
>+	{"CNSi", "G8324", "*", BLIST_SPARSELUN | BLIST_LARGELUN},     // Chaparral G8324 RAID
>

The G8324 correctly identifies itself as SCSI-3 so doesn't
need the BLIST_LARGELUN flag (proc output below
from 2.4.18 with automated large lun scanning).

This is as long as my fix for scanning large luns automatically
on SCSI-3 devices that went into 2.4.17-pre5 hasn't been
reverted in 2.4.19-pre

# cat /proc/scsi/scsi
Attached devices:
Host: scsi0 Channel: 00 Id: 00 Lun: 00
  Vendor: SEAGATE  Model: ST318405LC       Rev: 0105
  Type:   Direct-Access                    ANSI SCSI revision: 03
Host: scsi0 Channel: 00 Id: 06 Lun: 00
  Vendor: ESG-SHV  Model: SCA HSBP M10     Rev: 0.06
  Type:   Processor                        ANSI SCSI revision: 02
Host: scsi2 Channel: 00 Id: 00 Lun: 00
  Vendor: CNSi     Model: G8324            Rev: L411
  Type:   Direct-Access                    ANSI SCSI revision: 03
Host: scsi2 Channel: 00 Id: 00 Lun: 50
  Vendor: CNSi     Model: G8324            Rev: L411
  Type:   Processor                        ANSI SCSI revision: 03
Host: scsi2 Channel: 00 Id: 00 Lun: 51
  Vendor: CNSi     Model: G8324            Rev: L411
  Type:   Processor                        ANSI SCSI revision: 03
Host: scsi2 Channel: 00 Id: 00 Lun: 52
  Vendor: CNSi     Model: G8324            Rev: L411
  Type:   Processor                        ANSI SCSI revision: 03
Host: scsi2 Channel: 00 Id: 00 Lun: 53
  Vendor: CNSi     Model: G8324            Rev: L411
  Type:   Processor                        ANSI SCSI revision: 03
Host: scsi2 Channel: 00 Id: 00 Lun: 63
  Vendor: CNSi     Model: G8324            Rev: L411
  Type:   Processor                        ANSI SCSI revision: 03

Re: [PATCH] Additional BLIST_LARGELUN devices

[Kurt Garloff]

[-- Attachment #1.1: Type: text/plain, Size: 883 bytes --]

Hi Michael,

On Thu, Jun 27, 2002 at 10:09:11PM +0800, Michael Clark wrote:
> On 06/27/02 00:47, Kurt Garloff wrote:
> >here are some additional devices that need the BLIST_LARGELUN flag.
> >
> >+	{"CNSi", "G8324", "*", BLIST_SPARSELUN | BLIST_LARGELUN},     // 
> >Chaparral G8324 RAID
> 
> The G8324 correctly identifies itself as SCSI-3 so doesn't
> need the BLIST_LARGELUN flag (proc output below
> from 2.4.18 with automated large lun scanning).

OK, it got in that list by accident then. Won't hurt, of course.
I removed the flag. New patch attached.

Regards,
-- 
Kurt Garloff                   <kurt@garloff.de>         [Eindhoven, NL]
Physics: Plasma simulations    <K.Garloff@TUE.NL>     [TU Eindhoven, NL]
Linux: SCSI, Security          <garloff@suse.de>    [SuSE Nuernberg, DE]
 (See mail header or public key servers for PGP2 and GPG public keys.)

[-- Attachment #1.2: 50_scsi-largelun-2.4.19-pre7aa1.new --]
[-- Type: text/plain, Size: 2110 bytes --]

diff -urN linux-2.4.18.tmp/drivers/scsi/scsi_scan.c linux-2.4.18.SuSE/drivers/scsi/scsi_scan.c
--- linux-2.4.18.tmp/drivers/scsi/scsi_scan.c	Wed Apr 24 11:26:44 2002
+++ linux-2.4.18.SuSE/drivers/scsi/scsi_scan.c	Wed Apr 24 11:35:48 2002
@@ -149,17 +149,17 @@
  	{"TOSHIBA","CDROM","*", BLIST_ISROM},
  	{"TOSHIBA","CD-ROM","*", BLIST_ISROM},
 	{"MegaRAID", "LD", "*", BLIST_FORCELUN},
-	{"DGC",  "RAID",      "*", BLIST_SPARSELUN}, // Dell PV 650F (tgt @ LUN 0)
-	{"DGC",  "DISK",      "*", BLIST_SPARSELUN}, // Dell PV 650F (no tgt @ LUN 0) 
-	{"DELL", "PV660F",   "*", BLIST_SPARSELUN},
-	{"DELL", "PV660F   PSEUDO",   "*", BLIST_SPARSELUN},
-	{"DELL", "PSEUDO DEVICE .",   "*", BLIST_SPARSELUN}, // Dell PV 530F
-	{"DELL", "PV530F",    "*", BLIST_SPARSELUN}, // Dell PV 530F
+	{"DGC",  "RAID",      "*", BLIST_SPARSELUN | BLIST_LARGELUN}, // Dell PV 650F (tgt @ LUN 0)
+	{"DGC",  "DISK",      "*", BLIST_SPARSELUN | BLIST_LARGELUN}, // Dell PV 650F (no tgt @ LUN 0) 
+	{"DELL", "PV660F",   "*", BLIST_SPARSELUN | BLIST_LARGELUN},
+	{"DELL", "PV660F   PSEUDO",   "*", BLIST_SPARSELUN | BLIST_LARGELUN},
+	{"DELL", "PSEUDO DEVICE .",   "*", BLIST_SPARSELUN | BLIST_LARGELUN}, // Dell PV 530F
+	{"DELL", "PV530F",    "*", BLIST_SPARSELUN | BLIST_LARGELUN}, // Dell PV 530F
 	{"EMC", "SYMMETRIX", "*", BLIST_SPARSELUN | BLIST_LARGELUN | BLIST_FORCELUN},
 	{"HP", "A6189A", "*", BLIST_SPARSELUN |  BLIST_LARGELUN}, // HP VA7400, by Alar Aun
-	{"CMD", "CRA-7280", "*", BLIST_SPARSELUN},   // CMD RAID Controller
-	{"CNSI", "G7324", "*", BLIST_SPARSELUN},     // Chaparral G7324 RAID
-	{"CNSi", "G8324", "*", BLIST_SPARSELUN},     // Chaparral G8324 RAID
+	{"CMD", "CRA-7280", "*", BLIST_SPARSELUN | BLIST_LARGELUN},   // CMD RAID Controller
+	{"CNSI", "G7324", "*", BLIST_SPARSELUN | BLIST_LARGELUN},     // Chaparral G7324 RAID
+	{"CNSi", "G8324", "*", BLIST_SPARSELUN},     // Chaparral G8324 RAID
 	{"Zzyzx", "RocketStor 500S", "*", BLIST_SPARSELUN},
 	{"Zzyzx", "RocketStor 2000", "*", BLIST_SPARSELUN},
 	{"SONY", "TSL",       "*", BLIST_FORCELUN},  // DDS3 & DDS4 autoloaders

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [PATCH] Additional BLIST_LARGELUN devices

[Marcelo Tosatti]

On Thu, 27 Jun 2002, Kurt Garloff wrote:

> Hi Michael,
>
> On Thu, Jun 27, 2002 at 10:09:11PM +0800, Michael Clark wrote:
> > On 06/27/02 00:47, Kurt Garloff wrote:
> > >here are some additional devices that need the BLIST_LARGELUN flag.
> > >
> > >+	{"CNSi", "G8324", "*", BLIST_SPARSELUN | BLIST_LARGELUN},     //
> > >Chaparral G8324 RAID
> >
> > The G8324 correctly identifies itself as SCSI-3 so doesn't
> > need the BLIST_LARGELUN flag (proc output below
> > from 2.4.18 with automated large lun scanning).
>
> OK, it got in that list by accident then. Won't hurt, of course.
> I removed the flag. New patch attached.

Could you please send me a patch to remove that one list entry because I
already applied the previous one.

Re: [PATCH] Additional BLIST_LARGELUN devices

[Kurt Garloff]

[-- Attachment #1.1: Type: text/plain, Size: 641 bytes --]

Hi Marcelo,

On Thu, Jun 27, 2002 at 05:22:14PM -0300, Marcelo Tosatti wrote:
> On Thu, 27 Jun 2002, Kurt Garloff wrote:
> > OK, it got in that list by accident then. Won't hurt, of course.
> > I removed the flag. New patch attached.
> 
> Could you please send me a patch to remove that one list entry because I
> already applied the previous one.

Attached.
Thanks for applying the other patch!

Regards,
-- 
Kurt Garloff  <garloff@suse.de>                          Eindhoven, NL
GPG key: See mail header, key servers         Linux kernel development
SuSE Linux AG, Nuernberg, DE                            SCSI, Security

[-- Attachment #1.2: 2419rc1-larglun-devs-fix1.diff --]
[-- Type: text/plain, Size: 906 bytes --]

diff -uNr linux-2.4.18.S18.fixed/drivers/scsi/scsi_scan.c linux-2.4.18.S18.fixed.fixlargelun/drivers/scsi/scsi_scan.c
--- linux-2.4.18.S18.fixed/drivers/scsi/scsi_scan.c	Wed Jun 12 11:37:13 2002
+++ linux-2.4.18.S18.fixed.fixlargelun/drivers/scsi/scsi_scan.c	Thu Jun 27 23:25:13 2002
@@ -161,7 +161,7 @@
 	{"HP", "A6189A", "*", BLIST_SPARSELUN |  BLIST_LARGELUN}, // HP VA7400, by Alar Aun
 	{"CMD", "CRA-7280", "*", BLIST_SPARSELUN | BLIST_LARGELUN},   // CMD RAID Controller
 	{"CNSI", "G7324", "*", BLIST_SPARSELUN | BLIST_LARGELUN},     // Chaparral G7324 RAID
-	{"CNSi", "G8324", "*", BLIST_SPARSELUN | BLIST_LARGELUN},     // Chaparral G8324 RAID
+	{"CNSi", "G8324", "*", BLIST_SPARSELUN},     // Chaparral G8324 RAID
 	{"Zzyzx", "RocketStor 500S", "*", BLIST_SPARSELUN},
 	{"Zzyzx", "RocketStor 2000", "*", BLIST_SPARSELUN},
 	{"SONY", "TSL",       "*", BLIST_FORCELUN},  // DDS3 & DDS4 autoloaders

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]