From patchwork Wed Sep 20 13:54:16 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Damien Le Moal X-Patchwork-Id: 1837274 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@legolas.ozlabs.org Authentication-Results: legolas.ozlabs.org; dkim=pass (2048-bit key; unprotected) header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256 header.s=k20201202 header.b=cDtOo7ZF; dkim-atps=neutral Authentication-Results: legolas.ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=vger.kernel.org (client-ip=2620:137:e000::1:20; helo=out1.vger.email; envelope-from=linux-ide-owner@vger.kernel.org; receiver=patchwork.ozlabs.org) Received: from out1.vger.email (out1.vger.email [IPv6:2620:137:e000::1:20]) by legolas.ozlabs.org (Postfix) with ESMTP id 4RrKlq4Sdhz1ynx for ; Wed, 20 Sep 2023 23:54:47 +1000 (AEST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236427AbjITNyt (ORCPT ); Wed, 20 Sep 2023 09:54:49 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:45930 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S236024AbjITNys (ORCPT ); Wed, 20 Sep 2023 09:54:48 -0400 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 910BDAC; Wed, 20 Sep 2023 06:54:42 -0700 (PDT) Received: by smtp.kernel.org (Postfix) with ESMTPSA id B3EB6C433C8; Wed, 20 Sep 2023 13:54:40 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1695218082; bh=qiVW4csX0ux9p+HEmO2rNC5T7bsjgj7AO5Nj4pgbFWE=; h=From:To:Cc:Subject:Date:From; b=cDtOo7ZFAqHHnqecqwjInk9QeJoMnWf3aRb0MW/tYCt8E3gYPkP5Bwh6kUMgdR47T jIuAuppseVK41gQAN/Toc3S0Zz46Qyqlfc0gtyJYBbugOlweBzCRpErXCefq6c5+c/ 4ymUeAV5n03ngoQoFO3V4KfooiBOsPqLfjzDcCFR8k6BL8+yiRF2p7LzUO9/oPoyt0 WVrT6nhJ5Qa4ebUUIFhCe1FPpUThD8vBglv4jR+KrAgfEzlDwbNIG3qJTsZwJ97XYL QnngJXVOGVL7fYsqG9MAQknmhmV0Y2sDKVJkAFK0N9eZCCpUd1yafqYK4XR81OdOg5 vIvCaFsdPb4bA== From: Damien Le Moal To: linux-ide@vger.kernel.org Cc: linux-scsi@vger.kernel.org, "Martin K . Petersen" , John Garry , Rodrigo Vivi , Paul Ausbeck , Kai-Heng Feng , Joe Breuer , Geert Uytterhoeven , Chia-Lin Kao Subject: [PATCH v4 00/23] Fix libata suspend/resume handling and code cleanup Date: Wed, 20 Sep 2023 22:54:16 +0900 Message-ID: <20230920135439.929695-1-dlemoal@kernel.org> X-Mailer: git-send-email 2.41.0 MIME-Version: 1.0 X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF, RCVD_IN_DNSWL_BLOCKED,SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-ide@vger.kernel.org The first 9 patches of this series fix several issues with suspend/resume power management operations in scsi and libata. The most significant changes introduced are in patch 4 and 5, where the manage_start_stop flag of scsi devices is split into the manage_system_start_stop and manage_runtime_start_stop flags to allow keeping scsi runtime power operations for spining up/down ATA devices but have libata do its own system suspend/resume device power state management using EH. The remaining patches are code cleanup that do not introduce any significant functional change. This series was tested on qemu and on various PCs and servers. I am CC-ing people who recently reported issues with suspend/resume. Additional testing would be much appreciated. Changes from v3: * Corrected pathc 1 (typo in commit message and WARN_ON() removal) * Changed path 3 as suggested by Niklas (moved definition of ->slave_alloc) * Rebased on rc2 * Added review tags Changes from v2: * Added patch 4 as simply disabling manage_start_stop from libata was breaking individual disk runtime suspend/autosuspend. Patch 5 was reworked accordingly to the changes in patch 4. * Fixed patch 3: applying the link creation was missing and the link creation itself was also incorrect, preventing sd probe to execute correctly. Thanks to Geert for testing and reporting this issue. * Split the "Fix delayed scsi_rescan_device() execution" patch into patch 6 (scsi part) and patch 7 (ata part). * Modified patch 9 to not call sd_shutdown() from sd_remove() for devices that are not running. * Added Chia-Lin Tested tag to unchanged patches Changes from v1: * Added patch 8 and 9 to fix compilation warnings with W=1 * Addressed John comment in patch 19 * Fixed patch 20 commit message (Sergei) * Added Hannes Review tag Damien Le Moal (23): ata: libata-core: Fix ata_port_request_pm() locking ata: libata-core: Fix port and device removal ata: libata-scsi: link ata port and scsi device scsi: sd: Differentiate system and runtime start/stop management ata: libata-scsi: Disable scsi device manage_system_start_stop scsi: Do not attempt to rescan suspended devices ata: libata-scsi: Fix delayed scsi_rescan_device() execution ata: libata-core: Do not register PM operations for SAS ports scsi: sd: Do not issue commands to suspended disks on shutdown ata: libata-core: Fix compilation warning in ata_dev_config_ncq() ata: libata-eh: Fix compilation warning in ata_eh_link_report() scsi: Remove scsi device no_start_on_resume flag ata: libata-scsi: Cleanup ata_scsi_start_stop_xlat() ata: libata-core: Synchronize ata_port_detach() with hotplug ata: libata-core: Detach a port devices on shutdown ata: libata-core: Remove ata_port_suspend_async() ata: libata-core: Remove ata_port_resume_async() ata: libata-core: Do not poweroff runtime suspended ports ata: libata-core: Do not resume runtime suspended ports ata: libata-sata: Improve ata_sas_slave_configure() ata: libata-eh: Improve reset error messages ata: libata-eh: Reduce "disable device" message verbosity ata: libata: Cleanup inline DMA helper functions drivers/ata/libata-core.c | 242 +++++++++++++++++++++++++-------- drivers/ata/libata-eh.c | 76 +++++++++-- drivers/ata/libata-sata.c | 5 +- drivers/ata/libata-scsi.c | 146 ++++++++++---------- drivers/ata/libata-transport.c | 9 +- drivers/ata/libata.h | 7 + drivers/firewire/sbp2.c | 9 +- drivers/scsi/scsi_scan.c | 18 ++- drivers/scsi/sd.c | 88 ++++++++---- include/linux/libata.h | 26 ++-- include/scsi/scsi_device.h | 4 +- include/scsi/scsi_host.h | 2 +- 12 files changed, 444 insertions(+), 188 deletions(-) Tested-by: Geert Uytterhoeven