From mboxrd@z Thu Jan 1 00:00:00 1970 From: Valdis.Kletnieks@vt.edu Subject: Re: [PATCH v2.6.38-rc7] Revert "libata: ahci_start_engine compliant to AHCI spec" Date: Fri, 20 May 2011 14:25:59 -0400 Message-ID: <17610.1305915959@localhost> References: <201105120025.34531.rjw@sisk.pl> <20633.1305565337@localhost> <201105182144.37713.rjw@sisk.pl> <10124.1305906020@localhost> <20110520154341.GL31426@htj.dyndns.org> Mime-Version: 1.0 Content-Type: multipart/signed; boundary="==_Exmh_1305915959_2669P"; micalg=pgp-sha1; protocol="application/pgp-signature" Content-Transfer-Encoding: 7bit Return-path: Received: from lennier.cc.vt.edu ([198.82.162.213]:48200 "EHLO lennier.cc.vt.edu" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S934878Ab1ETS0L (ORCPT ); Fri, 20 May 2011 14:26:11 -0400 In-Reply-To: Your message of "Fri, 20 May 2011 10:02:56 PDT." Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: Jian Peng Cc: Tejun Heo , "Rafael J. Wysocki" , Jeff Garzik , Michael Leun , linux-ide@vger.kernel.org, Linux PM mailing list , LKML --==_Exmh_1305915959_2669P Content-Type: text/plain; charset=us-ascii On Fri, 20 May 2011 10:02:56 PDT, Jian Peng said: > --20cf307f38f6d842a904a3b81730 > You are both right that adding this majic 5ms delay at this place should not > be the final solution. > > If this magic 5ms delay works on other affected systems, I plan to post a > new patch that inside ahci_start_engine(), still perform same check, and > show warning message if failed, but will set a flag, then still set START > bit, and if there is such failure flag, add 5ms delay. > > Valdis, could you apply the following patch and retest it? I should be able to do that this weekend. To clarify - should this be with the problem commit 270dac35c26433d06a89150c51e75ca0181ca7e4 applied, or reverted? --==_Exmh_1305915959_2669P Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (GNU/Linux) Comment: Exmh version 2.5 07/13/2001 iD8DBQFN1rI3cC3lWbTT17ARAholAKDXyBVpYZyAWJ8gqpx+ye9glWfMIwCeLr/V nxlCRWYPv2DT4GrXxJPDTqg= =pEhp -----END PGP SIGNATURE----- --==_Exmh_1305915959_2669P--