From patchwork Mon Apr 25 09:26:15 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Abhishek Sahu X-Patchwork-Id: 566412 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 5F0F2C433F5 for ; Mon, 25 Apr 2022 09:28:20 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236700AbiDYJbM (ORCPT ); Mon, 25 Apr 2022 05:31:12 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:50504 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S238442AbiDYJav (ORCPT ); Mon, 25 Apr 2022 05:30:51 -0400 Received: from NAM10-BN7-obe.outbound.protection.outlook.com (mail-bn7nam10on2053.outbound.protection.outlook.com [40.107.92.53]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 3064B24F00; Mon, 25 Apr 2022 02:27:17 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=V/tj9IooEr7UL6OlBccnrEXRYwl0NSA1Fmy3v5AFLPAlrkCmu7s+JLOT8LCrXCNeFCbNPcK37p+XC4pKjRZ2knOGWw7LyHCDcoaGdebrf2pUQPEh/QVuMkKYhuAyyUyEZcyODyoJZEE06xiMwnEaI4k8l/KvgDWyKNP1nuxMDBVR9SmDUWhn2k3xK8XbwXdPSzJf0vgf5KhIz9AJkeWS4V+4AG/IRWL/GVCB+jfVHHgBARcQfsy6PsWEFNSTDPSocoXjzik+xQIK16ySUDbRKHGaqWKA+GvZYZoBAaGNkWvYEgkvqa4J1dqavbpgPL/tzzxsU7++XnmWCYgZbxxNew== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=xJelIOk3JzOwPDtODVJXcBMpjbWJ1tDEICmaqobWOBo=; b=VnmP2o4qA8IjY8jgLM/BJk4Fq7CzQSuUp3YSjuyP2XQk5Vj86/pD3NE5YJ8fOqJs+qnvJcXqioTW5rBZBUAfsvkRnmzjbeqCCDdECsDMuCnFZO53eM05gaKwchq0jbGOCWXrjksFqSkNrHtgb/Od9LgWOMGoDwjAxtzmLOgJfhYOSsmFrd0kZh9uHjbjlBXgWd06NgKH57SeLqnFfrGqDfqDyObL5JcpUZ4EasoYaHwKROM20/u7Sc8fTzGxP/3Z389JDQSp6eytoTOdgUk94brKyQVFMd/5JN87L3ldaEYZPJmcmvDT2jATOf4HlgOkHOlChS+h+AUE4IBROilFXA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 12.22.5.238) smtp.rcpttodomain=vger.kernel.org smtp.mailfrom=nvidia.com; dmarc=pass (p=reject sp=reject pct=100) action=none header.from=nvidia.com; dkim=none (message not signed); arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Nvidia.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=xJelIOk3JzOwPDtODVJXcBMpjbWJ1tDEICmaqobWOBo=; b=XpBp1SUMl0dWeIp016nFyjzdP8QNlEyL3UrZAZuL906mx0deFJmOPvlpX0Zz7oi4K12NiJwfM3zIpGeb/YSLMVgwBVjcOgMT615fWk3OYJiZpfTdvbwIh+lu5Jbc/EYXMBZBBsqrBPKJiGbztGt915jpADARc1uiVndImIGVf1/opZPi8RTqT99M0tnQrdjXLVqHJZC6VtgX6gIejwQuYID5Mh/bkmcbnujb10tdmoIDhUoA44CSGXKz5mzGFbno7uwshXr8bghdaKdObEM7TsEiMHw2W42ou014GYN2ciOBB3lLUDotM9Nh2KVPorbiOQatvnEUY8N0ptsD+5VBng== Received: from MW4PR03CA0148.namprd03.prod.outlook.com (2603:10b6:303:8c::33) by CH2PR12MB3974.namprd12.prod.outlook.com (2603:10b6:610:2e::27) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5186.15; Mon, 25 Apr 2022 09:27:14 +0000 Received: from CO1NAM11FT006.eop-nam11.prod.protection.outlook.com (2603:10b6:303:8c:cafe::2f) by MW4PR03CA0148.outlook.office365.com (2603:10b6:303:8c::33) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5186.15 via Frontend Transport; Mon, 25 Apr 2022 09:27:14 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 12.22.5.238) smtp.mailfrom=nvidia.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=nvidia.com; Received-SPF: Pass (protection.outlook.com: domain of nvidia.com designates 12.22.5.238 as permitted sender) receiver=protection.outlook.com; client-ip=12.22.5.238; helo=mail.nvidia.com; Received: from mail.nvidia.com (12.22.5.238) by CO1NAM11FT006.mail.protection.outlook.com (10.13.174.246) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384) id 15.20.5186.14 via Frontend Transport; Mon, 25 Apr 2022 09:27:13 +0000 Received: from rnnvmail205.nvidia.com (10.129.68.10) by DRHQMAIL105.nvidia.com (10.27.9.14) with Microsoft SMTP Server (TLS) id 15.0.1497.32; Mon, 25 Apr 2022 09:27:13 +0000 Received: from rnnvmail204.nvidia.com (10.129.68.6) by rnnvmail205.nvidia.com (10.129.68.10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.986.22; Mon, 25 Apr 2022 02:27:12 -0700 Received: from nvidia-abhsahu-1.nvidia.com (10.127.8.10) by mail.nvidia.com (10.129.68.6) with Microsoft SMTP Server id 15.2.986.22 via Frontend Transport; Mon, 25 Apr 2022 02:27:07 -0700 From: Abhishek Sahu To: Alex Williamson , Cornelia Huck , Yishai Hadas , Jason Gunthorpe , Shameer Kolothum , Kevin Tian , "Rafael J . Wysocki" CC: Max Gurtovoy , Bjorn Helgaas , , , , , Abhishek Sahu Subject: [PATCH v3 8/8] vfio/pci: Add the support for PCI D3cold state Date: Mon, 25 Apr 2022 14:56:15 +0530 Message-ID: <20220425092615.10133-9-abhsahu@nvidia.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20220425092615.10133-1-abhsahu@nvidia.com> References: <20220425092615.10133-1-abhsahu@nvidia.com> X-NVConfidentiality: public MIME-Version: 1.0 X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: a48dd775-db94-490d-7fdd-08da269dc85c X-MS-TrafficTypeDiagnostic: CH2PR12MB3974:EE_ X-Microsoft-Antispam-PRVS: X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: JnoSoLNn6OY6+zdw+0BAHq2h/HlYYzezqM2ubkZ1QXBEvKfG9e5i7TcBa61IqJr4zmPVtwZ2YvmqnBJemIdWCMXXmTaqTUTzNNV0kGJdxeEqPUbSPAOCzAzjR0buLgtI+7zRqYfTHDTb8rd6plNFoneJbfJjI8Hz0sfGFV6JrkSrG301rKBDfxwfAwb/lI3lLRbbKBKQUhbg6C8L9MlLBwT9esT7EEiKcrKz6WGCjQoBHiHgpB3KiQPG9e6SMD/c7UK3aNW7RfnjWi/Mu/D76Ia7oPYPyd82X//ABL9lpdZ8uzYOvbzGBpmcrVsbttnHDcxkmbmC99cBRDmPFQTYGHJLSYTbNvWJ+IPVMJQPcHdWl4XZBpVjDG9CFE2RjLUxB+vCVfmIAk6GF61azwnw0skH6T6kE4//FADIlXamirQKHVROj3qiyeaxSL6ZatpNDsc7jj1PsbXgPlJb6020o0oxaToRHwSFnRamZzoPUgzjN6q5MF46eJEqhApDKVMUIsY3ihFkKh4dTGf9n27uCQhibc8iReM43qy6NGjGCmur2R6wS79wrpB0TduGYpQVHmeIl4i8U7Fa2zs/nkbRB6hheXDTOmMrvjY+LIUBRIKnuXhacZSq127sHy4UZeaksfiV235Q2DZT+3J5ooGCENNMPgMjeM4/ikOWGSioJw10UbCtJoea7gxSlaLenqOOf5sTBEU0BubE5km6UmVdOHjw3ldufy58TNyML7uAIjU= X-Forefront-Antispam-Report: CIP:12.22.5.238; CTRY:US; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:mail.nvidia.com; PTR:InfoNoRecords; CAT:NONE; SFS:(13230001)(4636009)(40470700004)(46966006)(36840700001)(426003)(336012)(6666004)(8936002)(36756003)(110136005)(47076005)(4326008)(5660300002)(26005)(508600001)(82310400005)(36860700001)(8676002)(83380400001)(40460700003)(70206006)(7696005)(54906003)(70586007)(356005)(86362001)(7416002)(2616005)(186003)(316002)(107886003)(1076003)(2906002)(81166007)(30864003)(32563001)(36900700001); DIR:OUT; SFP:1101; X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 25 Apr 2022 09:27:13.8166 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: a48dd775-db94-490d-7fdd-08da269dc85c X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=43083d15-7273-40c1-b7db-39efd9ccc17a; Ip=[12.22.5.238]; Helo=[mail.nvidia.com] X-MS-Exchange-CrossTenant-AuthSource: CO1NAM11FT006.eop-nam11.prod.protection.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH2PR12MB3974 Precedence: bulk List-ID: X-Mailing-List: linux-pm@vger.kernel.org Currently, if the runtime power management is enabled for vfio pci based device in the guest OS, then guest OS will do the register write for PCI_PM_CTRL register. This write request will be handled in vfio_pm_config_write() where it will do the actual register write of PCI_PM_CTRL register. With this, the maximum D3hot state can be achieved for low power. If we can use the runtime PM framework, then we can achieve the D3cold state which will help in saving maximum power. 1. Since D3cold state can't be achieved by writing PCI standard PM config registers, so this patch adds a new feature in the existing VFIO_DEVICE_FEATURE IOCTL. This IOCTL can be used to change the PCI device from D3hot to D3cold state and then D3cold to D0 state. The device feature uses low power term instead of D3cold so that if other vfio driver wants to implement low power support, then the same IOCTL can be used. 2. The hypervisors can implement virtual ACPI methods. For example, in guest linux OS if PCI device ACPI node has _PR3 and _PR0 power resources with _ON/_OFF method, then guest linux OS makes the _OFF call during D3cold transition and then _ON during D0 transition. The hypervisor can tap these virtual ACPI calls and then do the D3cold related IOCTL in the vfio driver. 3. The vfio driver uses runtime PM framework to achieve the D3cold state. For the D3cold transition, decrement the usage count and for the D0 transition, increment the usage count. 4. For D3cold, the device current power state should be D3hot. Then during runtime suspend, the pci_platform_power_transition() is required for D3cold state. If the D3cold state is not supported, then the device will still be in D3hot state. But with the runtime PM, the root port can now also go into suspended state. 5. For most of the systems, the D3cold is supported at the root port level. So, when root port will transition to D3cold state, then the vfio PCI device will go from D3hot to D3cold state during its runtime suspend. If root port does not support D3cold, then the root will go into D3hot state. 6. The runtime suspend callback can now happen for 2 cases: there are no users of vfio device and the case where user has initiated D3cold. The 'platform_pm_engaged' flag can help to distinguish between these 2 cases. 7. In D3cold, all kind of BAR related access needs to be disabled like D3hot. Additionally, the config space will also be disabled in D3cold state. To prevent access of config space in D3cold state, do increment the runtime PM usage count before doing any config space access. 8. If user has engaged low power entry through IOCTL, then user should do low power exit first. The user can issue config access or IOCTL after low power entry. We can add an explicit error check but since we are already waking-up device, so IOCTL and config access can be fulfilled. But 'power_state_d3' won't be cleared without issuing low power exit so all BAR related access will still return error till user do low power exit. 9. Since multiple layers are involved, so following is the high level code flow for D3cold entry and exit. D3cold entry: a. User put the PCI device into D3hot by writing into standard config register (vfio_pm_config_write() -> vfio_lock_and_set_power_state() -> vfio_pci_set_power_state()). The device power state will be D3hot and power_state_d3 will be true. b. Set vfio_device_feature_power_management::low_power_state = VFIO_DEVICE_LOW_POWER_STATE_ENTER and call VFIO_DEVICE_FEATURE IOCTL. c. Inside vfio_device_fops_unl_ioctl(), pm_runtime_resume_and_get() will be called first which will make the usage count as 2 and then vfio_pci_core_ioctl_feature() will be invoked. d. vfio_pci_core_feature_pm() will be called and it will go inside VFIO_DEVICE_LOW_POWER_STATE_ENTER switch case. platform_pm_engaged will be true and pm_runtime_put_noidle() will decrement the usage count to 1. e. Inside vfio_device_fops_unl_ioctl() while returning the pm_runtime_put() will make the usage count to 0 and the runtime PM framework will engage the runtime suspend entry. f. pci_pm_runtime_suspend() will be called and invokes driver runtime suspend callback. g. vfio_pci_core_runtime_suspend() will change the power state to D0 and do the INTx mask related handling. h. pci_pm_runtime_suspend() will take care of saving the PCI state and all power management handling for D3cold. D3cold exit: a. Set vfio_device_feature_power_management::low_power_state = VFIO_DEVICE_LOW_POWER_STATE_EXIT and call VFIO_DEVICE_FEATURE IOCTL. b. Inside vfio_device_fops_unl_ioctl(), pm_runtime_resume_and_get() will be called first which will make the usage count as 1. c. pci_pm_runtime_resume() will take care of moving the device into D0 state again and then vfio_pci_core_runtime_resume() will be called. d. vfio_pci_core_runtime_resume() will do the INTx unmask related handling. e. vfio_pci_core_ioctl_feature() will be invoked. f. vfio_pci_core_feature_pm() will be called and it will go inside VFIO_DEVICE_LOW_POWER_STATE_EXIT switch case. platform_pm_engaged and power_state_d3 will be cleared and pm_runtime_get_noresume() will make the usage count as 2. g. Inside vfio_device_fops_unl_ioctl() while returning the pm_runtime_put() will make the usage count to 1 and the device will be in D0 state only. Signed-off-by: Abhishek Sahu --- drivers/vfio/pci/vfio_pci_config.c | 11 ++- drivers/vfio/pci/vfio_pci_core.c | 131 ++++++++++++++++++++++++++++- include/linux/vfio_pci_core.h | 1 + include/uapi/linux/vfio.h | 18 ++++ 4 files changed, 159 insertions(+), 2 deletions(-) diff --git a/drivers/vfio/pci/vfio_pci_config.c b/drivers/vfio/pci/vfio_pci_config.c index af0ae80ef324..65b1bc9586ab 100644 --- a/drivers/vfio/pci/vfio_pci_config.c +++ b/drivers/vfio/pci/vfio_pci_config.c @@ -25,6 +25,7 @@ #include #include #include +#include #include @@ -1936,16 +1937,23 @@ static ssize_t vfio_config_do_rw(struct vfio_pci_core_device *vdev, char __user ssize_t vfio_pci_config_rw(struct vfio_pci_core_device *vdev, char __user *buf, size_t count, loff_t *ppos, bool iswrite) { + struct device *dev = &vdev->pdev->dev; size_t done = 0; int ret = 0; loff_t pos = *ppos; pos &= VFIO_PCI_OFFSET_MASK; + ret = pm_runtime_resume_and_get(dev); + if (ret < 0) + return ret; + while (count) { ret = vfio_config_do_rw(vdev, buf, count, &pos, iswrite); - if (ret < 0) + if (ret < 0) { + pm_runtime_put(dev); return ret; + } count -= ret; done += ret; @@ -1953,6 +1961,7 @@ ssize_t vfio_pci_config_rw(struct vfio_pci_core_device *vdev, char __user *buf, pos += ret; } + pm_runtime_put(dev); *ppos += done; return done; diff --git a/drivers/vfio/pci/vfio_pci_core.c b/drivers/vfio/pci/vfio_pci_core.c index 05a68ca9d9e7..beac6e05f97f 100644 --- a/drivers/vfio/pci/vfio_pci_core.c +++ b/drivers/vfio/pci/vfio_pci_core.c @@ -234,7 +234,14 @@ int vfio_pci_set_power_state(struct vfio_pci_core_device *vdev, pci_power_t stat ret = pci_set_power_state(pdev, state); if (!ret) { - vdev->power_state_d3 = (pdev->current_state >= PCI_D3hot); + /* + * If 'platform_pm_engaged' is true then 'power_state_d3' can + * be cleared only when user makes the explicit request to + * move out of low power state by using power management ioctl. + */ + if (!vdev->platform_pm_engaged) + vdev->power_state_d3 = + (pdev->current_state >= PCI_D3hot); /* D3 might be unsupported via quirk, skip unless in D3 */ if (needs_save && pdev->current_state >= PCI_D3hot) { @@ -266,6 +273,25 @@ static int vfio_pci_core_runtime_suspend(struct device *dev) { struct vfio_pci_core_device *vdev = dev_get_drvdata(dev); + down_read(&vdev->memory_lock); + + /* 'platform_pm_engaged' will be false if there are no users. */ + if (!vdev->platform_pm_engaged) { + up_read(&vdev->memory_lock); + return 0; + } + + /* + * The user will move the device into D3hot state first before invoking + * power management ioctl. Move the device into D0 state here and then + * the pci-driver core runtime PM suspend will move the device into + * low power state. Also, for the devices which have NoSoftRst-, + * it will help in restoring the original state (saved locally in + * 'vdev->pm_save'). + */ + vfio_pci_set_power_state(vdev, PCI_D0); + up_read(&vdev->memory_lock); + /* * If INTx is enabled, then mask INTx before going into runtime * suspended state and unmask the same in the runtime resume. @@ -395,6 +421,19 @@ void vfio_pci_core_disable(struct vfio_pci_core_device *vdev) /* * This function can be invoked while the power state is non-D0. + * This non-D0 power state can be with or without runtime PM. + * Increment the usage count corresponding to pm_runtime_put() + * called during setting of 'platform_pm_engaged'. The device will + * wake up if it has already went into suspended state. Otherwise, + * the next vfio_pci_set_power_state() will change the + * device power state to D0. + */ + if (vdev->platform_pm_engaged) { + pm_runtime_resume_and_get(&pdev->dev); + vdev->platform_pm_engaged = false; + } + + /* * This function calls __pci_reset_function_locked() which internally * can use pci_pm_reset() for the function reset. pci_pm_reset() will * fail if the power state is non-D0. Also, for the devices which @@ -1192,6 +1231,80 @@ long vfio_pci_core_ioctl(struct vfio_device *core_vdev, unsigned int cmd, } EXPORT_SYMBOL_GPL(vfio_pci_core_ioctl); +#ifdef CONFIG_PM +static int vfio_pci_core_feature_pm(struct vfio_device *device, u32 flags, + void __user *arg, size_t argsz) +{ + struct vfio_pci_core_device *vdev = + container_of(device, struct vfio_pci_core_device, vdev); + struct pci_dev *pdev = vdev->pdev; + struct vfio_device_feature_power_management vfio_pm = { 0 }; + int ret = 0; + + ret = vfio_check_feature(flags, argsz, + VFIO_DEVICE_FEATURE_SET | + VFIO_DEVICE_FEATURE_GET, + sizeof(vfio_pm)); + if (ret != 1) + return ret; + + if (flags & VFIO_DEVICE_FEATURE_GET) { + down_read(&vdev->memory_lock); + vfio_pm.low_power_state = vdev->platform_pm_engaged ? + VFIO_DEVICE_LOW_POWER_STATE_ENTER : + VFIO_DEVICE_LOW_POWER_STATE_EXIT; + up_read(&vdev->memory_lock); + if (copy_to_user(arg, &vfio_pm, sizeof(vfio_pm))) + return -EFAULT; + return 0; + } + + if (copy_from_user(&vfio_pm, arg, sizeof(vfio_pm))) + return -EFAULT; + + /* + * The vdev power related fields are protected with memory_lock + * semaphore. + */ + down_write(&vdev->memory_lock); + switch (vfio_pm.low_power_state) { + case VFIO_DEVICE_LOW_POWER_STATE_ENTER: + if (!vdev->power_state_d3 || vdev->platform_pm_engaged) { + ret = EINVAL; + break; + } + + vdev->platform_pm_engaged = true; + + /* + * The pm_runtime_put() will be called again while returning + * from ioctl after which the device can go into runtime + * suspended. + */ + pm_runtime_put_noidle(&pdev->dev); + break; + + case VFIO_DEVICE_LOW_POWER_STATE_EXIT: + if (!vdev->platform_pm_engaged) { + ret = EINVAL; + break; + } + + vdev->platform_pm_engaged = false; + vdev->power_state_d3 = false; + pm_runtime_get_noresume(&pdev->dev); + break; + + default: + ret = EINVAL; + break; + } + + up_write(&vdev->memory_lock); + return ret; +} +#endif + static int vfio_pci_core_feature_token(struct vfio_device *device, u32 flags, void __user *arg, size_t argsz) { @@ -1226,6 +1339,10 @@ int vfio_pci_core_ioctl_feature(struct vfio_device *device, u32 flags, switch (flags & VFIO_DEVICE_FEATURE_MASK) { case VFIO_DEVICE_FEATURE_PCI_VF_TOKEN: return vfio_pci_core_feature_token(device, flags, arg, argsz); +#ifdef CONFIG_PM + case VFIO_DEVICE_FEATURE_POWER_MANAGEMENT: + return vfio_pci_core_feature_pm(device, flags, arg, argsz); +#endif default: return -ENOTTY; } @@ -2189,6 +2306,15 @@ static int vfio_pci_dev_set_hot_reset(struct vfio_device_set *dev_set, goto err_unlock; } + /* + * Some of the devices in the dev_set can be in the runtime suspended + * state. Increment the usage count for all the devices in the dev_set + * before reset and decrement the same after reset. + */ + ret = vfio_pci_dev_set_pm_runtime_get(dev_set); + if (ret) + goto err_unlock; + list_for_each_entry(cur_vma, &dev_set->device_list, vdev.dev_set_list) { /* * Test whether all the affected devices are contained by the @@ -2244,6 +2370,9 @@ static int vfio_pci_dev_set_hot_reset(struct vfio_device_set *dev_set, else mutex_unlock(&cur->vma_lock); } + + list_for_each_entry(cur, &dev_set->device_list, vdev.dev_set_list) + pm_runtime_put(&cur->pdev->dev); err_unlock: mutex_unlock(&dev_set->lock); return ret; diff --git a/include/linux/vfio_pci_core.h b/include/linux/vfio_pci_core.h index e84f31e44238..337983a877d6 100644 --- a/include/linux/vfio_pci_core.h +++ b/include/linux/vfio_pci_core.h @@ -126,6 +126,7 @@ struct vfio_pci_core_device { bool needs_pm_restore; bool power_state_d3; bool pm_intx_masked; + bool platform_pm_engaged; struct pci_saved_state *pci_saved_state; struct pci_saved_state *pm_save; int ioeventfds_nr; diff --git a/include/uapi/linux/vfio.h b/include/uapi/linux/vfio.h index fea86061b44e..53ff890dbd27 100644 --- a/include/uapi/linux/vfio.h +++ b/include/uapi/linux/vfio.h @@ -986,6 +986,24 @@ enum vfio_device_mig_state { VFIO_DEVICE_STATE_RUNNING_P2P = 5, }; +/* + * Use platform-based power management for moving the device into low power + * state. This low power state is device specific. + * + * For PCI, this low power state is D3cold. The native PCI power management + * does not support the D3cold power state. For moving the device into D3cold + * state, change the PCI state to D3hot with standard configuration registers + * and then call this IOCTL to setting the D3cold state. Similarly, if the + * device in D3cold state, then call this IOCTL to exit from D3cold state. + */ +struct vfio_device_feature_power_management { +#define VFIO_DEVICE_LOW_POWER_STATE_EXIT 0x0 +#define VFIO_DEVICE_LOW_POWER_STATE_ENTER 0x1 + __u64 low_power_state; +}; + +#define VFIO_DEVICE_FEATURE_POWER_MANAGEMENT 3 + /* -------- API for Type1 VFIO IOMMU -------- */ /**