Re: ST310211A and HPA ( was Re: Bug#401035: linux-image-2.6-686 - another misbehaving Seagate )

[Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>]

On Wednesday 06 February 2008, Mikko Rapeli wrote:
> (cc'ing linux-ide)
> 
> On Mon, Feb 04, 2008 at 02:32:59AM +0100, Bart Champagne wrote:
> > Just found another type of Seagate drives that shows this behaviour : 
> > ST310211A
> > Can this one be blacklisted as well ?
> 
> Yes, do you have a patch? And a dmesg dump of the drive in use with and 
> without a patch?
> 
> If you do, please send them to upstream linux-ide@vger.kernel.org and hope
> that the fix gets applied to Debian kernels too. Also, please test the
> sata/pata driver with your drive and add a blacklist there too.
> 
> If you don't have a patch, here's one for 2.6.24 to go.
> 
> From 4a8b40c53d2e539bad3f93996f599d69a16af94b Mon Sep 17 00:00:00 2001
> From: Mikko Rapeli <mikko.rapeli@teleca.com>
> Date: Wed, 6 Feb 2008 09:41:14 +0200
> Subject: [PATCH] ST310211A has buggy HPA too.
> 
> Signed-off-by: Mikko Rapeli <mikko.rapeli@teleca.com>

applied (this was confirmed by Bart Champagne to fix the issue)

Thanks.

> ---
>  drivers/ata/libata-core.c |    1 +
>  drivers/ide/ide-disk.c    |    1 +
>  2 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/ata/libata-core.c b/drivers/ata/libata-core.c
> index 6380726..d08f869 100644
> --- a/drivers/ata/libata-core.c
> +++ b/drivers/ata/libata-core.c
> @@ -4169,6 +4169,7 @@ static const struct ata_blacklist_entry ata_device_blacklist [] = {
>  	/* Devices which report 1 sector over size HPA */
>  	{ "ST340823A",		NULL,		ATA_HORKAGE_HPA_SIZE, },
>  	{ "ST320413A",		NULL,		ATA_HORKAGE_HPA_SIZE, },
> +	{ "ST310211A",		NULL,		ATA_HORKAGE_HPA_SIZE, },
>  
>  	/* Devices which get the IVB wrong */
>  	{ "QUANTUM FIREBALLlct10 05", "A03.0900", ATA_HORKAGE_IVB, },
> diff --git a/drivers/ide/ide-disk.c b/drivers/ide/ide-disk.c
> index b178190..a0f9940 100644
> --- a/drivers/ide/ide-disk.c
> +++ b/drivers/ide/ide-disk.c
> @@ -462,6 +462,7 @@ static inline int idedisk_supports_lba48(const struct hd_driveid *id)
>  static const struct drive_list_entry hpa_list[] = {
>  	{ "ST340823A",	NULL },
>  	{ "ST320413A",	NULL },
> +	{ "ST310211A",	NULL },
>  	{ NULL,		NULL }
>  };