Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.32 as permitted sender) client-ip=23.128.96.32;
From:   Niklas Cassel <Niklas.Cassel@wdc.com>
To:     David Gow <david@davidgow.net>
CC:     Damien Le Moal <dlemoal@kernel.org>,
        Bagas Sanjaya <bagasdotme@gmail.com>,
        Bjorn Helgaas <bhelgaas@google.com>,
        patenteng <dimitar@daskalov.co.uk>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        Linux Regressions <regressions@lists.linux.dev>,
        Linux IDE and libata <linux-ide@vger.kernel.org>,
        Linux PCI <linux-pci@vger.kernel.org>
Subject: Re: Fwd: Kernel 6.5.2 Causes Marvell Technology Group 88SE9128 PCIe
 SATA to Constantly Reset
Thread-Topic: Fwd: Kernel 6.5.2 Causes Marvell Technology Group 88SE9128 PCIe
 SATA to Constantly Reset
Thread-Index: AQHZ5lSnupdrJGnYN0eHoLtrJK7xDbAbOvsAgAAnCgCAABQ3gIAAIHCAgAA8gQCAAEFTAA==
Date:   Fri, 15 Sep 2023 16:20:47 +0000
Message-ID: <ZQSEXl0GB3iKoqjZ@x1-carbon>
References: <dacb34e4-ce58-bc0e-8206-672d743a3e34@gmail.com>
 <ZQHRQHAPQdG+Nu1o@x1-carbon>
 <59f6ff78-6b45-465a-bd41-28c7a5d10931@davidgow.net>
 <10f65dfe-5e8a-10ab-4d89-efe693c07caa@kernel.org>
 <658b9285-e030-4987-86a7-57cdb6c7f161@davidgow.net>
 <ZQQa0QRhm1BuI5IT@x1-carbon>
 <49d92af6-4968-4066-b33c-0fd06f8fdf28@davidgow.net>
In-Reply-To: <49d92af6-4968-4066-b33c-0fd06f8fdf28@davidgow.net>
Accept-Language: en-US
Content-Language: en-US
wdcipoutbound: EOP-TRUE
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?/32cIwYGuYxgmNF7Vzt4HKIBQbKc8jMRl2t9ZcgnsL41ZzRq4JUwvTzR7gBy?=
 =?us-ascii?Q?8UQ3zSFuuSoytLlPMv1HYIeJezYKCvLj0xZAAzXE+5jlE4+Fq/9FMFyQtSEE?=
 =?us-ascii?Q?xepd5/WGqAdYQNcs20Ea379ngv9NKEVfss8M/OfI8Tgd2tiQl5XozwmaQeWQ?=
 =?us-ascii?Q?C8shWrR7YZE2XvEgJTYL/+nQhy92w6DJ+cSlptLVhczaC2bUxk6bZxZqy1IH?=
 =?us-ascii?Q?zEmIgFxVmbR1XLS18xqbbP3vzDu5aA8d3CgvXNiezoSZbbPTWkMN3Pv3NuDn?=
 =?us-ascii?Q?GWA905iTTeKF0jITRxZYyl67d7GD6J+ayVJBgOSpCBpGU02odF1pvrckFsD1?=
 =?us-ascii?Q?AA/uqBfnmCRORv3aSgYBFo3DGM1LRdK6jBpJYgqkkigCPD1UX0kNQSqg/SEt?=
 =?us-ascii?Q?XrD9d4LTy6D88Tv9W0HMXeH2iMYwE5aeabRGymx9eto+I2jePxOwvbelZqnB?=
 =?us-ascii?Q?PSVE6qW1DkfDaUeTQXWdM4IIOhB4V/HVYh2i6zQC+QWp//RZ31z3bDxhEkZy?=
 =?us-ascii?Q?e10Q08PfxK3fTnvHMfl4dkTXQt1tOxQ9ehyjDtchJgOSzL7aGDmNj6IOov4n?=
 =?us-ascii?Q?xlBWlts8HtQTLOvb7HTcCi8w/OP6NaeIxVsNXLRbk0UloazZTFRiPj3UhNQL?=
 =?us-ascii?Q?J2P/Qt7DBPALyRBSYh5yF6f+noLYj/mKMJ1OI+VJpDV0OqxT6FWIQB61V0ub?=
 =?us-ascii?Q?06DvU1K4Kxhz85qKlJ6Ryji83kAYQNgi+bj4PE6BgC2R2WCNtVJ9XYcmJ4s3?=
 =?us-ascii?Q?ckWMW4ha3eM4A8AIfdsFaEbVGxJxXBAxVtMk+WkrJSZD53ihPpLWdOBmOxOt?=
 =?us-ascii?Q?KS+B4EQEIulFyX1klAK9ZQd8ZxZOJOicgJjQTdHHmWOx3lfyR35UtfdThWwM?=
 =?us-ascii?Q?7DLyPgjbRIp5WQF5NODRmNgQrTuuztypDlj0+WYHPpiHZA7837N0Z6INWA/x?=
 =?us-ascii?Q?r0f2rE5LlSel6fGXA51GVR+iXjDR1+s8femV78FmPOclAKB+cHUDdH4V5sCJ?=
 =?us-ascii?Q?kbIn6dUKifs2W69OR7byGVKSbT+RJ01/G5MU619oCioh7Pqwg5ZUYXuksNf/?=
 =?us-ascii?Q?HHyxVUlFapEpgl/UuqqGFvEb2r/AehBKj0bhfaPcexh21k52cL8FxeiMHdmV?=
 =?us-ascii?Q?9283FOKRl2ism89fEQhYAIzo+h/wkyJQFkGd+342kKlv4B+cD8+NlVkmVD3I?=
 =?us-ascii?Q?fi8kfK5wVVLqqtRtNpxQQvTjF0WlUIL5qlS4qjnXjRGsEiJP1RHFDrPO3kbu?=
 =?us-ascii?Q?vWpdDHAdpljAZJl0qISiBg1h62YJv+uNZcOATqCEnkmfzEUI5gfLARuOL3JK?=
 =?us-ascii?Q?EMOcYB84ZdBPi8LVe6H9dDSafAGto6EHrevDjjbS5roc8eC+hUWrRAax2VjB?=
 =?us-ascii?Q?VF6WF1Ul8zrhLxilZtlnyDKQaR3RVJOWWdhAytEjrYjqfsguHvuLonIy0R3P?=
 =?us-ascii?Q?A7EJlSNy4vfsgsvYlxktx6zOgnUwHl5CmuKB7IjUTNQNDQgLmLRCd6eB4oGU?=
 =?us-ascii?Q?JUgPpegX5YWFs1zR146FkT320dZJaegpBMupFZahFHScKusSjHSkYwzej3P/?=
 =?us-ascii?Q?vuQ5NNsOzyfMFm5bzSOEXqsLfhl93sTtg2PqBjbsolhnvKdUWuWo4xWx2BE9?=
 =?us-ascii?Q?Aw=3D=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-ID: <BDD9F48D6241B748939595E765B180E9@namprd04.prod.outlook.com>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: =?us-ascii?Q?PPqE4MTgFtYzbHMZSChjZkMGvjGkM5S3YJrAU9jXcVUdnlDQ1LB5ALdAmwSf?=
 =?us-ascii?Q?cOLyvXt/N/eUB8dam1KYoOO3myv/pdrNtmhP+C6xlRXSBAkzKjZ6AHxqnrjn?=
 =?us-ascii?Q?DPe+nStq+Or0S5ybFebl8sMQvPRSqTNaOVqiyjJrpmx7dfBqOnuj83ANErJF?=
 =?us-ascii?Q?Fs2kwHIwl3+qrKQmohWEs6OyB3F5QVau6nMY6uPLPECbjwsiQZpo2fSPxbW3?=
 =?us-ascii?Q?84zfiWkrTFcSrWsSbBBCrYkgrD4le95f/TBwjqOawHZm5NZLESA1xQTzJ3Un?=
 =?us-ascii?Q?fS1yu7z5ovgrSJ/gIXbt7NqDpbPnogpSeGyhklWNaoVw0O4tzxuCGVSB4dSz?=
 =?us-ascii?Q?uSysliO52BmBRP2OM/n/SZ3AQ/apwzRBL+wlGJtYSfRudunUtZ46zgqw8tGg?=
 =?us-ascii?Q?d2AXq980JCdyAv/K+3oBzjYzCBcOuUiD4ZX464SY2O774H5Yx1Za29U8/kWW?=
 =?us-ascii?Q?v4KOEeKn0HKyii1DB/5yLmHeRfybudaT+M7tJ+mZ0Y47dqUgqxF8ogW/FSG9?=
 =?us-ascii?Q?jZye/ofUg4qfyW6e8NwLozm7woxwK2wDx8QbIouh9gM5Gch8zPe6sUIOeqaN?=
 =?us-ascii?Q?7ISula8myGPxx8OX1/RAQDdV11l7duX0AWs19DYnkhYjFCMnB4NJVtYO323+?=
 =?us-ascii?Q?doS9+qPsIrXzcs/hdaKW5WmMjZuB49YOgvN/yvlPw/QQBveV5EoBFgjejo6s?=
 =?us-ascii?Q?/aoScDjl2ktzvRXUjDoqngSHoG9m0ihLqU5I22q/5TmIlStAwKMN6PKzDFLB?=
 =?us-ascii?Q?8HYE2QdiTieRLiGglceLJxVbLJ5BqL5+VOoWE33CBV9ZXWi2+rE0cw+8oS8/?=
 =?us-ascii?Q?BHMr82bwqBQTwvaK4karOL1TC6iSHBtcUSTmE1kJvcdoxbuwB5hLxxlw4eVX?=
 =?us-ascii?Q?LN7oE6b+S+elY1Ipy0+mjnQtdzlEwWRfJD7ottm4Zg+qhlaURd6r9qrhMstC?=
 =?us-ascii?Q?JoK6+oDNhLRGZffpK/LXZZhwGaG2uwieuLYwzgUrcGs=3D?=
X-OriginatorOrg: wdc.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: MN2PR04MB6272.namprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 54389ab7-a9ab-4491-03ec-08dbb607b845
X-MS-Exchange-CrossTenant-originalarrivaltime: 15 Sep 2023 16:20:47.5546
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: b61c8803-16f3-4c35-9b17-6f65f441df86
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: fw9KG8foLl35mUjodWPbwfE7I6huNTWNkIZWmzmpBVF5Cti1uTAjSJXISrUaZt2BO67nrvSF23SiyV0ScBtKLw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY1PR04MB8653
X-Spam-Status: No, score=-0.8 required=5.0 tests=DKIM_SIGNED,DKIM_VALID,
	HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,
	SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on agentk.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (agentk.vger.email [0.0.0.0]); Fri, 15 Sep 2023 09:21:52 -0700 (PDT)

On Fri, Sep 15, 2023 at 08:26:58PM +0800, David Gow wrote:
> In any case, the bisect is done:
>=20
> 624885209f31eb9985bf51abe204ecbffe2fdeea is the first bad commit
> commit 624885209f31eb9985bf51abe204ecbffe2fdeea
> Author: Damien Le Moal <dlemoal@kernel.org>
> Date:   Thu May 11 03:13:41 2023 +0200
>=20
>     scsi: core: Detect support for command duration limits
>=20
>     Introduce the function scsi_cdl_check() to detect if a device support=
s
>     command duration limits (CDL). Support for the READ 16, WRITE 16, REA=
D
> 32
>     and WRITE 32 commands are checked using the function
> scsi_report_opcode()
>     to probe the rwcdlp and cdlp bits as they indicate the mode page
> defining
>     the command duration limits descriptors that apply to the command bei=
ng
>     tested.
>=20
>     If any of these commands support CDL, the field cdl_supported of stru=
ct
>     scsi_device is set to 1 to indicate that the device supports CDL.
>=20
>     Support for CDL for a device is advertizes through sysfs using the ne=
w
>     cdl_supported device attribute. This attribute value is 1 for a devic=
e
>     supporting CDL and 0 otherwise.
>=20
>     Signed-off-by: Damien Le Moal <dlemoal@kernel.org>
>     Reviewed-by: Hannes Reinecke <hare@suse.de>
>     Co-developed-by: Niklas Cassel <niklas.cassel@wdc.com>
>     Signed-off-by: Niklas Cassel <niklas.cassel@wdc.com>
>     Link: https://lore.kernel.org/r/20230511011356.227789-9-nks@flawful.o=
rg
>     Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
>=20
>  Documentation/ABI/testing/sysfs-block-device |  9 ++++
>  drivers/scsi/scsi.c                          | 81
> ++++++++++++++++++++++++++++
>  drivers/scsi/scsi_scan.c                     |  3 ++
>  drivers/scsi/scsi_sysfs.c                    |  2 +
>  include/scsi/scsi_device.h                   |  3 ++
>  5 files changed, 98 insertions(+)
>=20
>=20
> This seems to match what was found on the Arch Linux forums, too:
> https://bbs.archlinux.org/viewtopic.php?id=3D288723&p=3D3
>=20
> I haven't tried it yet, but according to that forum thread, removing the
> calls to scsi_cdl_check() seems to resolve the issue. This is all well
> beyond my SCSI knowledge, but maybe a quirk to disable these CDL checks f=
or
> these older marvell controllers is required? Though it seems odd that the
> device would be rescanned and/or scsi_add_lun called multiple times a sec=
ond
> -- is that normal?
>=20
> In any case, this seems to be the cause.

Hello David,

Thank you very much for your effort of bisecting this.

Could you please try this patch and see if it improves things for you:
https://lore.kernel.org/linux-scsi/20230915022034.678121-1-dlemoal@kernel.o=
rg/


Kind regards,
Niklas=