Subject: Re: kern/33637: ST3500841NS (500G SATA drive) require LBA48 quirk in wd.c
To: None <bouyer@NetBSD.org, gnats-admin@netbsd.org,>
From: Manuel Bouyer <bouyer@antioche.eu.org>
List: netbsd-bugs
Date: 06/05/2006 18:25:02
The following reply was made to PR kern/33637; it has been noted by GNATS.

From: Manuel Bouyer <bouyer@antioche.eu.org>
To: gnats-bugs@NetBSD.org
Cc: kern-bug-people@NetBSD.org, gnats-admin@NetBSD.org,
	netbsd-bugs@NetBSD.org
Subject: Re: kern/33637: ST3500841NS (500G SATA drive) require LBA48 quirk in wd.c
Date: Mon, 5 Jun 2006 20:23:29 +0200

 --7JfCtLOvnd9MIVvH
 Content-Type: text/plain; charset=us-ascii
 Content-Disposition: inline
 
 On Fri, Jun 02, 2006 at 11:50:00PM +0000, Shigeya Suzuki wrote:
 > [...]
 > Index: wd.c
 > ===================================================================
 > RCS file: /cvsroot/src/sys/dev/ata/wd.c,v
 > retrieving revision 1.298.2.7.2.3
 > diff -c -r1.298.2.7.2.3 wd.c
 > *** wd.c	24 May 2006 02:15:21 -0000	1.298.2.7.2.3
 > --- wd.c	2 Jun 2006 22:30:19 -0000
 > ***************
 > *** 244,249 ****
 > --- 244,252 ----
 >   	/* Attemp to catch all seagate drives larger than 200GB */
 >   	{ "ST3[2-9][0-9][0-9][0-9][0-9][0-9]A*",
 >   	  WD_QUIRK_FORCE_LBA48 },
 > + 	/* ST3500841NS and other */
 > + 	{ "ST3[2-9][0-9][0-9][0-9][0-9][0-9]NS",
 > + 	  WD_QUIRK_FORCE_LBA48 },
 >   	{ NULL,
 >   	  0 }
 >   };
 
 Hi,
 I commited the attached patch. Can you check if it works for you ?
 
 -- 
 Manuel Bouyer <bouyer@antioche.eu.org>
      NetBSD: 26 ans d'experience feront toujours la difference
 --
 
 --7JfCtLOvnd9MIVvH
 Content-Type: text/plain; charset=us-ascii
 Content-Disposition: attachment; filename=diff
 
 Index: wd.c
 ===================================================================
 RCS file: /cvsroot/src/sys/dev/ata/wd.c,v
 retrieving revision 1.324
 diff -u -r1.324 wd.c
 --- wd.c	18 May 2006 19:42:09 -0000	1.324
 +++ wd.c	5 Jun 2006 18:21:16 -0000
 @@ -246,7 +246,7 @@
  	{ "ST3160827A*",
  	  WD_QUIRK_FORCE_LBA48 },
  	/* Attempt to catch all seagate drives larger than 200GB */
 -	{ "ST3[2-9][0-9][0-9][0-9][0-9][0-9]A*",
 +	{ "ST3[2-9][0-9][0-9][0-9][0-9][0-9][A-Z]*",
  	  WD_QUIRK_FORCE_LBA48 },
  	{ NULL,
  	  0 }
 
 --7JfCtLOvnd9MIVvH--