From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark Lord Subject: Re: libata fails to recover from HSM violation involving DRQ status Date: Thu, 10 May 2007 23:35:07 -0400 Message-ID: <4643E46B.1000405@rtr.ca> References: <4633AB75.7070107@rtr.ca> <4633B0A6.6090705@garzik.org> <20070428222502.26fc9bbc@the-village.bc.nu> <4633BEE7.8020005@garzik.org> <4633BF6D.40902@rtr.ca> <46340E63.5070209@gmail.com> <4634163D.1040408@gmail.com> <463487F0.4040701@rtr.ca> <46349695.7080706@rtr.ca> <46349A03.9090300@rtr.ca> <4634CAEC.4010700@gmail.com> <4634CC18.4080208@rtr.ca> <463739E4.1030306@rtr.ca> <4643E3F6.4080009@rtr.ca> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from rtr.ca ([64.26.128.89]:3538 "EHLO mail.rtr.ca" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751747AbXEKDfI (ORCPT ); Thu, 10 May 2007 23:35:08 -0400 In-Reply-To: <4643E3F6.4080009@rtr.ca> Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: Tejun Heo Cc: Jeff Garzik , Alan Cox , Alan Cox , IDE/ATA development list Mark Lord wrote: > Mark Lord wrote: > >> I retested this again today on my new pure-SATA notebook with ata_piix. >> In this case, the DRQ drain is not necessary, but also doesn't harm >> anything. >> Tested it both ways. This is with a Hitachi HTS541612J9SA00 SATA drive. >> >> The original fault was on ata_piix SATA, with some kind of external >> bridge (on the motherboard) to a Seagate PATA drive. Sometime in the >> next few days I'll have the exact same drive, but with a SATA interface, >> and we'll try that in the pure-SATA situation. >> >> This will tell us whether it's the bridge, or the drive, that was the >> issue. >> >> The fix remains the same: drain the data fifo when DRQ is left high. > > Okay, I finally got round to testing this with the new pure-SATA > notebook I have here. Same problem: without draining the DRQ fifo, > the system *never* recovers. > > But with the patch to drain DRQ, all is well. That patch is now a keeper > for my own kernels. Tejun, did you want to cook up a better-placed variant > of it for mainline? I'm away for a few days now.. A note for anyone confused by my two postings above: The DRQ drain *is* needed for the Seagate notebook drives (PATA/SATA), but not for the Hitachi notebook SATA drive I also have here. Cheers