From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 2050DC433F5 for ; Wed, 23 Mar 2022 08:24:40 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S242590AbiCWI0G (ORCPT ); Wed, 23 Mar 2022 04:26:06 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:58514 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S238221AbiCWI0F (ORCPT ); Wed, 23 Mar 2022 04:26:05 -0400 Received: from esa1.hgst.iphmx.com (esa1.hgst.iphmx.com [68.232.141.245]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 3EE4F6E7B6 for ; Wed, 23 Mar 2022 01:24:36 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=wdc.com; i=@wdc.com; q=dns/txt; s=dkim.wdc.com; t=1648023876; x=1679559876; h=message-id:date:mime-version:subject:to:cc:references: from:in-reply-to:content-transfer-encoding; bh=vHgfnIqvx4gqj9+Qz5CGEkAMw0dBTsI78EXcq1RRvVg=; b=nkatZ/G+9IFrUXVqdJkIM0jF0EwOT/CFL2yb0IlezSoPvtZIBWPTWc6s Mke/XylVEQ0HZQMWF+EDZo7c3Q7Y5PutalPo2HYTKqyya3uTrHBJVIpRO eyZVkmn8L+iT4N2SPRso16yWe1N1s1qsx9Bx0LWjYUqphty0haDFf94a5 RbHgifVHZqZUaLLNP7TJc1ELyb+0LxGEK4MLrwztyQSR1gf/dOLGhPR7p 7pGnghNoTVtd4KzB5/nGusUhaBcAI6V61Fx3z17SKVfPxkDwYX/R3Z4yf 0jncn/almzDTZ4NOoFY0WKuHmfvNSUn81BOQbWp59smP0SYb1hov4WMfq Q==; X-IronPort-AV: E=Sophos;i="5.90,203,1643644800"; d="scan'208";a="307996881" Received: from h199-255-45-14.hgst.com (HELO uls-op-cesaep01.wdc.com) ([199.255.45.14]) by ob1.hgst.iphmx.com with ESMTP; 23 Mar 2022 16:24:35 +0800 IronPort-SDR: YD0B1eBLdFsexjoHT+wdoyPbATQGhAZMHPMITtnErv99atK06sAH5MDFeVLJNtjBt0TX2Sl8NI AeSet7YvUURzZitzblMcu0esryw9GhZw3Tls3Q945g731MtNnR/0ZSq7JzGNw6wnU8JrVRa4To ARPICaXKi4arCMxikol0wyMsg17DOI3CseUeQTSfULjrshBsHu48tJi17MrWCmpn3Nrq/47qKb SVAXKxXb6iJNJVnJuWcPXFBOfTtQITfviw6gAtCeOR/xfffoA4dRp7MIBeJGN8AEBFg8OW11hL y8hn4sDoWkzQKahfJ2h+6vTV Received: from uls-op-cesaip02.wdc.com ([10.248.3.37]) by uls-op-cesaep01.wdc.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 23 Mar 2022 00:56:29 -0700 IronPort-SDR: Wl0o8mLbEBWOBgrQE1Z/EYFPlOE2+02idtI0CxwGCCYU0LKsiHLjcMpt3n3ZHYF98MrZEiTXrZ bxeDkVK81ll6tZAgFt6cNHsdozRc4wNpBgcJl+htaOtCxD4AcC525+OCA9ujX0rgTjKjiAs6UZ B9zhAgnLXjqQ908kL1ypGVtzkvCOyIJNTEYsdtU2VX8yj1FmKE96futBRSYmLBY26RJ+Yr2SCs NyepmwIXWBq3cxZvzccRP1iL3mqTYNlpO1WxlhLRu5ESKHjzssFwAshqZd2W5fcZzzXqkKW0bg MFg= WDCIronportException: Internal Received: from usg-ed-osssrv.wdc.com ([10.3.10.180]) by uls-op-cesaip02.wdc.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 23 Mar 2022 01:24:36 -0700 Received: from usg-ed-osssrv.wdc.com (usg-ed-osssrv.wdc.com [127.0.0.1]) by usg-ed-osssrv.wdc.com (Postfix) with ESMTP id 4KNhGq3msfz1SHwl for ; Wed, 23 Mar 2022 01:24:35 -0700 (PDT) Authentication-Results: usg-ed-osssrv.wdc.com (amavisd-new); dkim=pass reason="pass (just generated, assumed good)" header.d=opensource.wdc.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d= opensource.wdc.com; h=content-transfer-encoding:content-type :in-reply-to:organization:from:references:to:content-language :subject:user-agent:mime-version:date:message-id; s=dkim; t= 1648023875; x=1650615876; bh=vHgfnIqvx4gqj9+Qz5CGEkAMw0dBTsI78EX cq1RRvVg=; b=QeOURQO6QMEdw+C1fIb7ToRGR8cxhk33N7trqAHH9Sr+0EL9rJn 944TIHiRTDEE83Ro4gjVFiI7WHug/np+CLTES31GfU+BRt4ye60EdGdz+zc9QsiL xHAf8Y5u5w+k05+IT8VM0h/U8Z6iVbTHP4j8PZwyb2jz3nCRNpLkwNdDe5L54PP7 1GtClfpc1gYGWYgkKu6aBuns3gF8LHK85kRSrSSxdn3lNAwotDEYFYuSfO5TA2oP 8Gjy8Mbtqj49WlDroOLnWiYlppda35TsachpBipMjnvn8J7gppPLRNGHyyK9qlrN hWdsYSdoen5nKmkmaVq3UVa/cfCFJtGRYhQ== X-Virus-Scanned: amavisd-new at usg-ed-osssrv.wdc.com Received: from usg-ed-osssrv.wdc.com ([127.0.0.1]) by usg-ed-osssrv.wdc.com (usg-ed-osssrv.wdc.com [127.0.0.1]) (amavisd-new, port 10026) with ESMTP id jn_Wv8ejVXHa for ; Wed, 23 Mar 2022 01:24:35 -0700 (PDT) Received: from [10.225.163.114] (unknown [10.225.163.114]) by usg-ed-osssrv.wdc.com (Postfix) with ESMTPSA id 4KNhGp0rMzz1Rvlx; Wed, 23 Mar 2022 01:24:33 -0700 (PDT) Message-ID: Date: Wed, 23 Mar 2022 17:24:33 +0900 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.6.2 Subject: Re: [PATCH v2 3/3] ata: ahci: Skip 200 ms debounce delay for AMD 300 Series Chipset SATA Controller Content-Language: en-US To: Paul Menzel , Mario Limonciello Cc: Hans de Goede , linux-ide@vger.kernel.org, LKML , Nehal-bakulchandra Shah References: <20220321212431.13717-1-pmenzel@molgen.mpg.de> <20220321212431.13717-3-pmenzel@molgen.mpg.de> <5fb6af7b-d84f-cbae-7eb1-543f3a7e53e4@molgen.mpg.de> From: Damien Le Moal Organization: Western Digital Research In-Reply-To: <5fb6af7b-d84f-cbae-7eb1-543f3a7e53e4@molgen.mpg.de> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-ide@vger.kernel.org On 3/23/22 15:55, Paul Menzel wrote: > Dear Damien, >=20 >=20 > Am 23.03.22 um 06:01 schrieb Damien Le Moal: >> On 3/22/22 06:51, Limonciello, Mario wrote: >>> [Public] >>> >>>> -----Original Message----- >>>> From: Paul Menzel >>>> Sent: Monday, March 21, 2022 16:25 >=20 > [=E2=80=A6] >=20 >>> I seem to recall that we were talking about trying to drop the deboun= ce delay for >>> everything, weren't we? >>> >>> So perhaps it would be right to add a 4th patch in the series to do j= ust that. Then >>> If this turns out to be problematic for anything other than the contr= ollers in the >>> series that you identified as not problematic then that 4th patch can= potentially >>> be reverted alone? >> >> Not quite everything :) But you are right, let's try to switch the def= ault >> to no delay. I will be posting patches today for that. >> >> Paul, >> >> With these patches, your patches are not necessary anymore as the AMD >> chipset falls under the default no-delay. >=20 > I am all for improving the situation for all devices, but I am unable t= o=20 > judge the regression potential of changing this, as it affects a lot of= =20 > devices. I guess it=E2=80=99d would go through the next tree, and hopef= ully the=20 > company QA teams can give it a good spin. I hoped that my patches, as I= =20 > have tested them, and AMD will hopefully too, could go into the current= =20 > merge window. Yes, correct, the plan is to get the generic series queued as soon as rc1 so that it can spend plenty of time in linux-next for people to test. Tha= t will hopefully reduce the risk of breaking things in the field. Same for the default LPM change. With the default removal of the debounce delay, your patches addressing only the AMD adapter are not needed anymore: this adapter will not have a debounce delay unless the ATA_LFLAG_DEBOUNCE_DELAY flag is set. >=20 >> It would be nice if you can test though. >=20 > Of course, I am going to that either way. Series posted with you on CC. Please test ! >=20 >=20 > Kind regards, >=20 > Paul --=20 Damien Le Moal Western Digital Research