From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Eric D. Mudama" Subject: Re: [PATCH][2.6.12-rc4][SATA] sil driver - Blacklist Maxtor disk Date: Thu, 26 May 2005 22:01:07 -0600 Message-ID: <311601c9050526210119abb8d8@mail.gmail.com> References: <20050524162710.55401.qmail@web88008.mail.re2.yahoo.com> Reply-To: "Eric D. Mudama" Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7BIT Return-path: Received: from rproxy.gmail.com ([64.233.170.204]:44953 "EHLO rproxy.gmail.com") by vger.kernel.org with ESMTP id S261449AbVE0EBI convert rfc822-to-8bit (ORCPT ); Fri, 27 May 2005 00:01:08 -0400 Received: by rproxy.gmail.com with SMTP id i8so231579rne for ; Thu, 26 May 2005 21:01:07 -0700 (PDT) In-Reply-To: <20050524162710.55401.qmail@web88008.mail.re2.yahoo.com> Content-Disposition: inline Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: Shawn Starr Cc: linux-kernel@vger.kernel.org, linux-ide@vger.kernel.org, jgarzik@pobox.com I don't believe that drive is vulnerable to the traditionally described SI issue, as it's based on Marvell bridge IP. If that drive was vulnerable, so would be the drives of more than one other large vendor. I would bet something else is actually the root cause, and by using smaller transfers or something you're just hiding the race condition, not fixing a root cause. (obviously i'd love to see a bus trace of the hang in action, but not everyone can swap their cars for bus analyzers...) --eric On 5/24/05, Shawn Starr wrote: > Yes, I know we shouldn't do this, but at the current > time, this seems to fix DMA READ/WRITE timeout > problems. This also may fix the sata_nv driver as > well, but this seems indicate that there is a lowlevel > problem with the SATA subsystem. > > Signed-off-by: Shawn Starr > > --- sata_sil.c.old 2005-05-24 12:19:20.312197269 > -0400 > +++ sata_sil.c 2005-05-11 14:05:26.000000000 -0400 > @@ -93,6 +93,7 @@ struct sil_drivelist { > { "ST380011ASL", SIL_QUIRK_MOD15WRITE > }, > { "ST3120022ASL", SIL_QUIRK_MOD15WRITE > }, > { "ST3160021ASL", SIL_QUIRK_MOD15WRITE > }, > + { "Maxtor 6Y080M0", SIL_QUIRK_MOD15WRITE > }, > { "Maxtor 4D060H3", SIL_QUIRK_UDMA5MAX }, > { } > }; > > - > To unsubscribe from this list: send the line "unsubscribe linux-ide" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html >