>Number:         26834
>Category:       kern
>Synopsis:       amr(8) driver does not recognize LSI Logic Controller
>Confidential:   no
>Severity:       serious
>Priority:       high
>Responsible:    kern-bug-people
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Thu Sep 02 15:48:00 UTC 2004
>Closed-Date:
>Last-Modified:
>Originator:     Hauke Fath <hf@spg.tu-darmstadt.de>
>Release:        NetBSD 2.0_BETA
>Organization:
-- 
/~\  The ASCII Ribbon Campaign                    Hauke Fath
\ /    No HTML/RTF in email	        Institut für Nachrichtentechnik
 X     No Word docs in email	                  TU Darmstadt
/ \  Respect for open standards              Ruf +49-6151-16-3281
>Environment:
	
	
System: NetBSD heiligenberg 2.0_BETA NetBSD 2.0_BETA (HEILIGENBERG) #3: Wed Jul 28 16:42:21 CEST 2004 hf@heiligenberg:/var/obj/netbsd-builds/2_0/i386/sys/arch/i386/compile/HEILIGENBERG i386
Architecture: i386
Machine: i386
>Description:

	We have just aquired two server machines with LSI Logic
	MegaRAID 320-2 RAID controllers. Although the amr(8) manpage
	claims the model as supported, all the INSTALL kernel has to
	say is

Vendor 0x1000 product 0x1960 (RAID mass storage, revision 0x01) at pci3 dev6 function 0 not configured

	After a helpful hint from Eric Haszlakiewicz it turned out
	that dev/pci/amr.c has a list of second source HW IDs, but not
	LSI Logic / Symbios.

	I make this PR priority 'high' because I think the patch
	should ship with NetBSD 2.0.


>How-To-Repeat:
	
	Buy a machine with a LSI Logic MegaRAID 320-2 RAID
	controller. Attempt to install NetBSD, find you cannot because
	there is "no disk".


>Fix:

	Applying the one-line

[hf@heiligenberg] /<4>dev/pci > cvs diff -u amr.c
Index: amr.c
===================================================================
RCS file: /cvsroot/src/sys/dev/pci/amr.c,v
retrieving revision 1.19
diff -u -u -r1.19 amr.c
--- amr.c       5 Nov 2003 16:03:27 -0000       1.19
+++ amr.c       2 Sep 2004 15:31:39 -0000
@@ -132,6 +132,7 @@
        { PCI_VENDOR_AMI,   PCI_PRODUCT_AMI_MEGARAID,  0 },
        { PCI_VENDOR_AMI,   PCI_PRODUCT_AMI_MEGARAID2, 0 },
        { PCI_VENDOR_AMI,   PCI_PRODUCT_AMI_MEGARAID3, AT_QUARTZ },
+       { PCI_VENDOR_SYMBIOS,   PCI_PRODUCT_AMI_MEGARAID3, AT_QUARTZ },
        { PCI_VENDOR_INTEL, PCI_PRODUCT_AMI_MEGARAID3, AT_QUARTZ | AT_SIG },
        { PCI_VENDOR_DELL,  PCI_PRODUCT_DELL_PERC_4DI, AT_QUARTZ },
        { PCI_VENDOR_DELL,  PCI_PRODUCT_DELL_PERC_4DI_2, AT_QUARTZ },

	makes amr(8) recognize our adapters just fine.

>Release-Note:
>Audit-Trail:
>Unformatted: