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--