From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 7848F27F16C; Thu, 31 Jul 2025 23:21:42 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1754004103; cv=none; b=t9QXBAEAlvUi2FE6XdVo1lC0b7U0EKJm5yWClyLvmnlAEuByft9OIYtpLYXs4zRqPCY2oYFmEnV2HG6uQ+Wv9wjCh9JhM/HhgvXEsC9AK1vV/aq9uSrxPBQV4CL4vhLJTts6lDc/QKT5zUL8siEioZKdJFo1tkoOdOEVTsJBI9o= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1754004103; c=relaxed/simple; bh=Co+hWowkbbzPPbt8dWbu3Q+G4ZrGp4O6TjcsY/Swk1E=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=tn7mLAzdmSYBaifKuE/PbsrkSgDnttcxZZ+Pm+r+57oah0GPGRQlzQWbOx3xVGbm3csYbDFxHwlYUZNmfNXGQjA/nrPq7zq+OsJnI2KDLlT3yIn9j/G3aRx+N6gFveRBgHx4NcinAJW3eZmUlu8StUrbvluLNG4TD2maCHj2B90= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=al9jCCJ/; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="al9jCCJ/" Received: by smtp.kernel.org (Postfix) with ESMTPSA id CAC54C4CEEF; Thu, 31 Jul 2025 23:21:38 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1754004102; bh=Co+hWowkbbzPPbt8dWbu3Q+G4ZrGp4O6TjcsY/Swk1E=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=al9jCCJ/2aNrWa0aSZtjnPVel6mWKlP31/9faXs3UmEawdWwFZLkexwJG/0AvtDkK qB4wbZfKLOhirzjnJOUeqCc5XuEuJk+qzAk0hAa/ndZ8KA6TjVXQTfIaAiTPLGnuLY HxQuqW5EvWs1NIbUt7YmKei619SvaovkB2SnV2bPIgU3o7GseRHh+u4sw4PeDhDTqd bPZZK1Hu68wKWzeT2suQb0M9aAzzqMFNjpSSrC9mVPj+1VRtYDYY4zmJEAOq2OnZuh 8qkr4zW30NOkj3ZZ1x37Y48Fmek5lJ2tBL7unOU+2nwEv++GPHlXfgUKPqOIwfiPNy 1VKSGFA/WWBEg== Message-ID: <856aa232-701f-40bf-bae9-1aff8886f473@kernel.org> Date: Fri, 1 Aug 2025 08:21:37 +0900 Precedence: bulk X-Mailing-List: linux-ide@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v7 08/19] scsi: detect support for command duration limits To: Diangang Li Cc: Friedrich Weber , Mira Limbeck , Niklas Cassel , Jens Axboe , "Martin K. Petersen" , "James E.J. Bottomley" , Kashyap Desai , Sumit Saxena , Shivasharan S , Chandrakanth patil , Sathya Prakash Veerichetty , Sreekanth Reddy , megaraidlinux.pdl@broadcom.com, mpi3mr-linuxdrv.pdl@broadcom.com, Bart Van Assche , Christoph Hellwig , Hannes Reinecke , linux-scsi@vger.kernel.org, linux-ide@vger.kernel.org, linux-block@vger.kernel.org, Niklas Cassel References: <3dee186c-285e-4c1c-b879-6445eb2f3edf@proxmox.com> <6fb8499a-b5bc-4d41-bf37-32ebdea43e9a@kernel.org> <2e7d6a7e-4a82-4da5-ab39-267a7400ca49@proxmox.com> <54e0a717-e9fc-4534-bc27-8bc1ee745048@kernel.org> <72bf0fd7-f646-46f7-a2aa-ef815dbfa4e2@proxmox.com> <3b2a6cfe-5bf3-4818-8633-c200d8e6f122@kernel.org> <4cb58e56-d9e2-4868-84ad-8b7253148228@proxmox.com> <75412b1b-3f39-4f6a-93ce-823c15a19bf3@kernel.org> <20250731114832.GA97414@bytedance.com> Content-Language: en-US From: Damien Le Moal Organization: Western Digital Research In-Reply-To: <20250731114832.GA97414@bytedance.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On 7/31/25 20:48, Diangang Li wrote: > On Tue, Jul 22, 2025 at 06:37:50PM +0900, Damien Le Moal wrote: >> On 7/22/25 6:32 PM, Friedrich Weber wrote: >>> On 14/07/2025 04:48, Damien Le Moal wrote: >>>> On 7/10/25 5:41 PM, Friedrich Weber wrote: >>>>> Thanks for looking into this, it is definitely a strange problem. >>>>> >>>>> Considering these drives don't support CDL anyway: Do you think it would >>>>> be possible to provide an "escape hatch" to disable only the CDL checks >>>>> (a module parameter?) so hotplug can work for the user again for their >>>>> device? If I see correctly, disabling just the CDL checks is not >>>>> possible (without recompiling the kernel) -- scsi_mod.dev_flags can be >>>>> used to disable RSOC, but I guess that has other unintended consequences >>>>> too, so a more "targeted" escape hatch would be nice. >>>> >>>> Could you test the attached patch ? That should solve the issue. >>>> >>> >>> Thanks for the patch! The user tested it on top of a 6.15.6 kernel and >>> with the SAS3008 HBA, and indeed: >>> >>> - under 6.15.6, hotplug fails with the log messages mentioned in my >>> first message, >>> - with your patch on top, hotplug works again. >> >> OK. Will post a proper patch then (tomorrow). >> Thanks for testing. >> > > Hi Damien, > > Are you planning to post a formal patch to upstream? I initially thought that the issue was due to some problem with the HBA SAT (SCSI to ATA translation). However, the drives that trigger the issue are SAS drives, so there is no command translation and it is much less likely that the issue is related to the HBA. So I am relunctant to take a big hammer and disable CDL for mpt3sas for SAS drive. HBAs driven by that driver do *not* support CDL for ATA, so disabling CDL would not be an issue for SATA disks. But there is no good reasons to disable CDL for SAS drives. This is being handled as a drive issue now, off-list. -- Damien Le Moal Western Digital Research