Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/1.1/patches/2230416/?format=api
{ "id": 2230416, "url": "http://patchwork.ozlabs.org/api/1.1/patches/2230416/?format=api", "web_url": "http://patchwork.ozlabs.org/project/linux-pci/patch/20260429180647.197072-4-thomas.falcon@intel.com/", "project": { "id": 28, "url": "http://patchwork.ozlabs.org/api/1.1/projects/28/?format=api", "name": "Linux PCI development", "link_name": "linux-pci", "list_id": "linux-pci.vger.kernel.org", "list_email": "linux-pci@vger.kernel.org", "web_url": null, "scm_url": null, "webscm_url": null }, "msgid": "<20260429180647.197072-4-thomas.falcon@intel.com>", "date": "2026-04-29T18:06:45", "name": "[RFC,3/4] pcie/aspm: Enable all hardware power-saving states by default", "commit_ref": null, "pull_url": null, "state": "new", "archived": false, "hash": "d8341a3b9b8d284e9c7daa5e1e3a77fabc673691", "submitter": { "id": 93044, "url": "http://patchwork.ozlabs.org/api/1.1/people/93044/?format=api", "name": "Falcon, Thomas", "email": "thomas.falcon@intel.com" }, "delegate": null, "mbox": "http://patchwork.ozlabs.org/project/linux-pci/patch/20260429180647.197072-4-thomas.falcon@intel.com/mbox/", "series": [ { "id": 502124, "url": "http://patchwork.ozlabs.org/api/1.1/series/502124/?format=api", "web_url": "http://patchwork.ozlabs.org/project/linux-pci/list/?series=502124", "date": "2026-04-29T18:06:44", "name": "pcie/aspm: Enable all advertised ASPM states by default", "version": 1, "mbox": "http://patchwork.ozlabs.org/series/502124/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/patches/2230416/comments/", "check": "pending", "checks": "http://patchwork.ozlabs.org/api/patches/2230416/checks/", "tags": {}, "headers": { "Return-Path": "\n <linux-pci+bounces-53418-incoming=patchwork.ozlabs.org@vger.kernel.org>", "X-Original-To": [ "incoming@patchwork.ozlabs.org", "linux-pci@vger.kernel.org" ], "Delivered-To": "patchwork-incoming@legolas.ozlabs.org", "Authentication-Results": [ "legolas.ozlabs.org;\n\tdkim=pass (2048-bit key;\n unprotected) header.d=intel.com header.i=@intel.com header.a=rsa-sha256\n header.s=Intel header.b=RxAfiXc2;\n\tdkim-atps=neutral", "legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=vger.kernel.org\n (client-ip=2600:3c0a:e001:db::12fc:5321; helo=sea.lore.kernel.org;\n envelope-from=linux-pci+bounces-53418-incoming=patchwork.ozlabs.org@vger.kernel.org;\n receiver=patchwork.ozlabs.org)", "smtp.subspace.kernel.org;\n\tdkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com\n header.b=\"RxAfiXc2\"", "smtp.subspace.kernel.org;\n arc=none smtp.client-ip=192.198.163.15", "smtp.subspace.kernel.org;\n dmarc=pass (p=none dis=none) header.from=intel.com", "smtp.subspace.kernel.org;\n spf=pass smtp.mailfrom=intel.com" ], "Received": [ "from sea.lore.kernel.org (sea.lore.kernel.org\n [IPv6:2600:3c0a:e001:db::12fc:5321])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange x25519 server-signature ECDSA (secp384r1) server-digest SHA384)\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4g5QJV29Lyz1xqf\n\tfor <incoming@patchwork.ozlabs.org>; Thu, 30 Apr 2026 04:07:54 +1000 (AEST)", "from smtp.subspace.kernel.org (conduit.subspace.kernel.org\n [100.90.174.1])\n\tby sea.lore.kernel.org (Postfix) with ESMTP id 91CC230440B2\n\tfor <incoming@patchwork.ozlabs.org>; Wed, 29 Apr 2026 18:07:15 +0000 (UTC)", "from localhost.localdomain (localhost.localdomain [127.0.0.1])\n\tby smtp.subspace.kernel.org (Postfix) with ESMTP id 60A0841B366;\n\tWed, 29 Apr 2026 18:07:07 +0000 (UTC)", "from mgamail.intel.com (mgamail.intel.com [192.198.163.15])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby smtp.subspace.kernel.org (Postfix) with ESMTPS id 51047402BAC;\n\tWed, 29 Apr 2026 18:07:03 +0000 (UTC)", "from fmviesa003.fm.intel.com ([10.60.135.143])\n by fmvoesa109.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;\n 29 Apr 2026 11:06:56 -0700", "from iherna2-mobl4.amr.corp.intel.com (HELO tfalcon-desk.intel.com)\n ([10.124.221.251])\n by fmviesa003-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;\n 29 Apr 2026 11:06:55 -0700" ], "ARC-Seal": "i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116;\n\tt=1777486026; cv=none;\n b=UOgDHpA61BaYzbRvzCzuljToPx+y67IQnB6/8cxrOjcbAHNCLr4MXl8lD73nL2M9oVgNy4DgJbbVofBT4Jk+tn0WpSkMBx4w1XI/fu4aO+GOPPbgPi0+gdFSiBWV8yCzzdtCzz3XH2PSs+/bVXM+4CYLMhV4yvfPVMHc+eNGMhg=", "ARC-Message-Signature": "i=1; a=rsa-sha256; d=subspace.kernel.org;\n\ts=arc-20240116; t=1777486026; c=relaxed/simple;\n\tbh=I7KLo2BSP/NmxfPhvBY24w3mWqQHMkCVhmxUuigSc34=;\n\th=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References:\n\t MIME-Version;\n b=FDkWv77cEmH8/1f8MOlWEbh5WJOwZmQanQbHRyR6XLO6/XZ2Y3n/9Zf/KgOezm1NGCMrLGSqmt21eJCFHCbOVS7wtBEMUzKUolK6ZbWedVMqtOCOwJvA2k4feNMaiVbxO9PuRLEVkLopp4R8BE058uAAHwnDzMS/fFP4aNFdk9A=", "ARC-Authentication-Results": "i=1; smtp.subspace.kernel.org;\n dmarc=pass (p=none dis=none) header.from=intel.com;\n spf=pass smtp.mailfrom=intel.com;\n dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com\n header.b=RxAfiXc2; arc=none smtp.client-ip=192.198.163.15", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple;\n d=intel.com; i=@intel.com; q=dns/txt; s=Intel;\n t=1777486024; x=1809022024;\n h=from:to:cc:subject:date:message-id:in-reply-to:\n references:mime-version:content-transfer-encoding;\n bh=I7KLo2BSP/NmxfPhvBY24w3mWqQHMkCVhmxUuigSc34=;\n b=RxAfiXc252OLzkihqnB16CvAfGV8WBYEEtgYjwgeHHg9kzdne+AsP6Ur\n T6GhR9FcrpN2za/bWoan2Gpncc8/Tc+MSLndm2fk7x9phcAsU3s5PF36j\n zBlPZFl3nlLxwxRfGTb2BSO9su6JN0EQuIrBqxji/rO27QyRDrNDjwdCY\n t3t1CP1oRtPssAftoxlTVMAEphQu0aM8R5nZkeb3TMZs8sAJzIBOmCMdd\n xrQDPfwRJj3mThBDoTDbudj7N2CO5B55N+BDDBuVMftwLp6m1L9kowqCI\n lqHO+ZUtvswbtTs/EQl56IV8V1MGt+4O/N01UQF4Ksfg+ZlkmZhDT3O95\n Q==;", "X-CSE-ConnectionGUID": [ "FFJ2sVoZRFqCfE4ByOWCZQ==", "BTx9jMfHRpe7XRmhbK+aXA==" ], "X-CSE-MsgGUID": [ "O1wDWRM0S36dqo3LedTH1w==", "HmB0qf4xQmCz2anyQdoshA==" ], "X-IronPort-AV": [ "E=McAfee;i=\"6800,10657,11771\"; a=\"78532284\"", "E=Sophos;i=\"6.23,206,1770624000\";\n d=\"scan'208\";a=\"78532284\"" ], "X-ExtLoop1": "1", "From": "Thomas Falcon <thomas.falcon@intel.com>", "To": "Bjorn Helgaas <bhelgaas@google.com>,\n\t\"Rafael J . Wysocki\" <rafael@kernel.org>", "Cc": "\"David E . Box\" <david.e.box@linux.intel.com>,\n\tLukas Wunner <lukas@wunner.de>,\n\tManivannan Sadhasivam <manivannan.sadhasivam@oss.qualcomm.com>,\n\tLen Brown <lenb@kernel.org>,\n\tlinux-pci@vger.kernel.org,\n\tlinux-kernel@vger.kernel.org,\n\tThomas Falcon <thomas.falcon@intel.com>", "Subject": "[RFC PATCH 3/4] pcie/aspm: Enable all hardware power-saving states by\n default", "Date": "Wed, 29 Apr 2026 13:06:45 -0500", "Message-ID": "<20260429180647.197072-4-thomas.falcon@intel.com>", "X-Mailer": "git-send-email 2.43.0", "In-Reply-To": "<20260429180647.197072-1-thomas.falcon@intel.com>", "References": "<20260429180647.197072-1-thomas.falcon@intel.com>", "Precedence": "bulk", "X-Mailing-List": "linux-pci@vger.kernel.org", "List-Id": "<linux-pci.vger.kernel.org>", "List-Subscribe": "<mailto:linux-pci+subscribe@vger.kernel.org>", "List-Unsubscribe": "<mailto:linux-pci+unsubscribe@vger.kernel.org>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit" }, "content": "For systems with a BIOS release date starting in 2025, default\nASPM policy to powersupersave if supported in the ACPI FADT.\nProvide a flag, aspm_user_policy, tracking whether a user has\nrequested a specific power state to give those precedence.\nDo not enable all states if user has chosen a specific policy\nor disabled ASPM using the pcie_aspm module parameter.\n\nSuggested-by: David E. Box <david.e.box@linux.intel.com>\nSigned-off-by: Thomas Falcon <thomas.falcon@intel.com>\n---\n drivers/pci/pci-acpi.c | 4 +++-\n drivers/pci/pcie/aspm.c | 17 +++++++++++++++++\n include/linux/pci.h | 1 +\n 3 files changed, 21 insertions(+), 1 deletion(-)", "diff": "diff --git a/drivers/pci/pci-acpi.c b/drivers/pci/pci-acpi.c\nindex 4d0f2cb6c695..d849bc6d0c0c 100644\n--- a/drivers/pci/pci-acpi.c\n+++ b/drivers/pci/pci-acpi.c\n@@ -1523,7 +1523,9 @@ static int __init acpi_pci_init(void)\n \tif (acpi_gbl_FADT.boot_flags & ACPI_FADT_NO_ASPM) {\n \t\tpr_info(\"ACPI FADT declares the system doesn't support PCIe ASPM, so disable it\\n\");\n \t\tpcie_no_aspm();\n-\t}\n+\t} else\n+\t\t/* If ASPM is supported, configure the default policy here. */\n+\t\tpcie_aspm_policy_config_init();\n \n \tif (acpi_pci_disabled)\n \t\treturn 0;\ndiff --git a/drivers/pci/pcie/aspm.c b/drivers/pci/pcie/aspm.c\nindex 84d49aa8a5ba..1c81e2f2e589 100644\n--- a/drivers/pci/pcie/aspm.c\n+++ b/drivers/pci/pcie/aspm.c\n@@ -267,6 +267,8 @@ static int aspm_policy = POLICY_POWER_SUPERSAVE;\n #else\n static int aspm_policy;\n #endif\n+static int aspm_default_policy = POLICY_POWER_SUPERSAVE;\n+static bool aspm_user_policy;\n \n static const char *policy_str[] = {\n \t[POLICY_DEFAULT] = \"default\",\n@@ -1609,6 +1611,7 @@ static int pcie_aspm_set_policy(const char *val,\n \tdown_read(&pci_bus_sem);\n \tmutex_lock(&aspm_lock);\n \taspm_policy = i;\n+\taspm_user_policy = true;\n \tlist_for_each_entry(link, &link_list, sibling) {\n \t\tpcie_config_aspm_link(link, policy_to_aspm_state(link));\n \t\tpcie_set_clkpm(link, policy_to_clkpm_state(link));\n@@ -1810,6 +1813,20 @@ static int __init pcie_aspm_disable(char *str)\n \n __setup(\"pcie_aspm=\", pcie_aspm_disable);\n \n+\n+\n+void __init pcie_aspm_policy_config_init(void)\n+{\n+\t/*\n+\t * Set ASPM policy here, enabling all power-saving states\n+\t * unless ASPM has been disabled or the user has already\n+\t * requested a policy or the systems BIOS release date\n+\t * is before the year 2025. Otherwise use BIOS defaults.\n+\t */\n+\tif (!aspm_disabled && !aspm_user_policy && dmi_get_bios_year() >= 2025)\n+\t\taspm_policy = aspm_default_policy;\n+}\n+\n void pcie_no_aspm(void)\n {\n \t/*\ndiff --git a/include/linux/pci.h b/include/linux/pci.h\nindex 2c4454583c11..36fa5579709c 100644\n--- a/include/linux/pci.h\n+++ b/include/linux/pci.h\n@@ -1915,6 +1915,7 @@ int pci_disable_link_state_locked(struct pci_dev *pdev, int state);\n int pci_enable_link_state(struct pci_dev *pdev, int state);\n int pci_enable_link_state_locked(struct pci_dev *pdev, int state);\n void pcie_no_aspm(void);\n+void pcie_aspm_policy_config_init(void);\n bool pcie_aspm_support_enabled(void);\n bool pcie_aspm_enabled(struct pci_dev *pdev);\n #else\n", "prefixes": [ "RFC", "3/4" ] }