Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/1.1/patches/2222066/?format=api
{ "id": 2222066, "url": "http://patchwork.ozlabs.org/api/1.1/patches/2222066/?format=api", "web_url": "http://patchwork.ozlabs.org/project/linux-pci/patch/20260410191037.782121-1-gloveless@jqluv.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": "<20260410191037.782121-1-gloveless@jqluv.com>", "date": "2026-04-10T19:10:37", "name": "PCI: release empty sibling bridge resources during window resize", "commit_ref": null, "pull_url": null, "state": "new", "archived": false, "hash": "e6eb26b84955e83ec47ce10b42cbcf03720b04e9", "submitter": { "id": 93094, "url": "http://patchwork.ozlabs.org/api/1.1/people/93094/?format=api", "name": "Geramy Loveless", "email": "gloveless@jqluv.com" }, "delegate": null, "mbox": "http://patchwork.ozlabs.org/project/linux-pci/patch/20260410191037.782121-1-gloveless@jqluv.com/mbox/", "series": [ { "id": 499493, "url": "http://patchwork.ozlabs.org/api/1.1/series/499493/?format=api", "web_url": "http://patchwork.ozlabs.org/project/linux-pci/list/?series=499493", "date": "2026-04-10T19:10:37", "name": "PCI: release empty sibling bridge resources during window resize", "version": 1, "mbox": "http://patchwork.ozlabs.org/series/499493/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/patches/2222066/comments/", "check": "pending", "checks": "http://patchwork.ozlabs.org/api/patches/2222066/checks/", "tags": {}, "headers": { "Return-Path": "\n <linux-pci+bounces-52349-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=jqluv-com.20251104.gappssmtp.com\n header.i=@jqluv-com.20251104.gappssmtp.com header.a=rsa-sha256\n header.s=20251104 header.b=kw3+oN6X;\n\tdkim-atps=neutral", "legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=vger.kernel.org\n (client-ip=2600:3c04:e001:36c::12fc:5321; helo=tor.lore.kernel.org;\n envelope-from=linux-pci+bounces-52349-incoming=patchwork.ozlabs.org@vger.kernel.org;\n receiver=patchwork.ozlabs.org)", "smtp.subspace.kernel.org;\n\tdkim=pass (2048-bit key) header.d=jqluv-com.20251104.gappssmtp.com\n header.i=@jqluv-com.20251104.gappssmtp.com header.b=\"kw3+oN6X\"", "smtp.subspace.kernel.org;\n arc=none smtp.client-ip=74.125.82.182", "smtp.subspace.kernel.org;\n dmarc=none (p=none dis=none) header.from=jqluv.com", "smtp.subspace.kernel.org;\n spf=none smtp.mailfrom=jqluv.com" ], "Received": [ "from tor.lore.kernel.org (tor.lore.kernel.org\n [IPv6:2600:3c04:e001:36c::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 4fsmbp0xGRz1yGb\n\tfor <incoming@patchwork.ozlabs.org>; Sat, 11 Apr 2026 05:10:46 +1000 (AEST)", "from smtp.subspace.kernel.org (conduit.subspace.kernel.org\n [100.90.174.1])\n\tby tor.lore.kernel.org (Postfix) with ESMTP id 326753022541\n\tfor <incoming@patchwork.ozlabs.org>; Fri, 10 Apr 2026 19:10:43 +0000 (UTC)", "from localhost.localdomain (localhost.localdomain [127.0.0.1])\n\tby smtp.subspace.kernel.org (Postfix) with ESMTP id 17DB12F87B;\n\tFri, 10 Apr 2026 19:10:42 +0000 (UTC)", "from mail-dy1-f182.google.com (mail-dy1-f182.google.com\n [74.125.82.182])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits))\n\t(No client certificate requested)\n\tby smtp.subspace.kernel.org (Postfix) with ESMTPS id 8DF07330D23\n\tfor <linux-pci@vger.kernel.org>; Fri, 10 Apr 2026 19:10:40 +0000 (UTC)", "by mail-dy1-f182.google.com with SMTP id\n 5a478bee46e88-2b4520f6b32so2605127eec.0\n for <linux-pci@vger.kernel.org>; Fri, 10 Apr 2026 12:10:40 -0700 (PDT)", "from geramyl-MS-S1-MAX.crconsultinggroup.net\n (c-24-7-159-43.hsd1.ca.comcast.net. [24.7.159.43])\n by smtp.gmail.com with ESMTPSA id\n 5a478bee46e88-2d561bde70csm6853067eec.15.2026.04.10.12.10.38\n (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n Fri, 10 Apr 2026 12:10:39 -0700 (PDT)" ], "ARC-Seal": "i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116;\n\tt=1775848242; cv=none;\n b=ZKOUyxEi2bnTVVYkvHSQK7uQM+JWay949W22jpySOm37gh5Zzciq7+wSPuQfaBotYkc2Gco36Qxp83Z6l7TMCcT8BgVD2wO0v3V5YFeuoyTJ4Iv9jhOypSpsXbX6X7jpwwqkU7nmQUc5x+QdQx7zc8UZxkcNNatQi82Kn02iyIk=", "ARC-Message-Signature": "i=1; a=rsa-sha256; d=subspace.kernel.org;\n\ts=arc-20240116; t=1775848242; c=relaxed/simple;\n\tbh=mpqUq4lt5PRhqmRG2nWWVvjeSAyu6CGhlJwAEPGRf7Q=;\n\th=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References:\n\t MIME-Version:Content-Type;\n b=Ah1p+Up5oxBLFNoL9qMfVPGmAs0Q/F7QPPvwmsdWs6UXUyXV78TCMuHBlMo6ef5G8DUY0e3RHGaHHcDSpR78+kY7BdWvzB14eZtaN3TSh/drCLmkSIVDqKxoYV5G8Ar0YfkPXWg4jCVoup9vdOZ/5HLUNJLMlCHERG8I9ZXe6vY=", "ARC-Authentication-Results": "i=1; smtp.subspace.kernel.org;\n dmarc=none (p=none dis=none) header.from=jqluv.com;\n spf=none smtp.mailfrom=jqluv.com;\n dkim=pass (2048-bit key) header.d=jqluv-com.20251104.gappssmtp.com\n header.i=@jqluv-com.20251104.gappssmtp.com header.b=kw3+oN6X;\n arc=none smtp.client-ip=74.125.82.182", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=jqluv-com.20251104.gappssmtp.com; s=20251104; t=1775848239;\n x=1776453039; darn=vger.kernel.org;\n h=content-transfer-encoding:mime-version:references:in-reply-to\n :message-id:date:subject:cc:to:from:from:to:cc:subject:date\n :message-id:reply-to;\n bh=4gIcSBFISy1np7le9AlixVLh62OAhzwxZ2OczFxQ9KE=;\n b=kw3+oN6Xv2Y/Xtar48e04dscV2n8QB1DOjD89sDv7/DNl9xGVKx1Db/kln1emT7AwI\n RzcNusw0wtthr9V35VgFBg1DppQYkSyXEjFW+uutIDFPu+YjMKCYrHGcst0QWRdETCVR\n d6XKviUjFgfKV6eg9lgz8xz5ut6YOPKKFVvI/Z38CO61FEX1M/8yEQ/8U922kacjRtMI\n UKEZ8xbf1Vkuq/zGjuXbyKt8hj8Frp7yTQ/Mn2q7LROX8c6IewjT/2NWql4hfrfe8RmN\n 8pWA2ALcoyKwZ137Gp1wfvBQ7t+bg6q903UjZmoUwLtetYYTQnmtWdywfnem0jWvs3Zo\n Lfag==", "X-Google-DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=1e100.net; s=20251104; t=1775848239; x=1776453039;\n h=content-transfer-encoding:mime-version:references:in-reply-to\n :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from\n :to:cc:subject:date:message-id:reply-to;\n bh=4gIcSBFISy1np7le9AlixVLh62OAhzwxZ2OczFxQ9KE=;\n b=YrY1HoVaHy9bFwLYTHpvdFQYQEZ65xc/VSWmiOLVoFaVb3jVV79Rc0MOuYEKLE+cxP\n JboXGo8t2r4jBxF046AJtWFEpI8irBVHdCckDiM2b24QT9VOfz64sH/+jf/2VTu2rhzX\n pDzPs3ynHF3nWBE5qM2NarwE9fcteKT7Y1kcfInP1Dvq3MbaB3qQIYn2jr112mCqz03/\n sibSbMsilcX2hcdjZgoAc7moMa5JBYiijZ/lkUxRAe/fbBnyXn1yi5iQx6/ylKmzjFAJ\n P6zpeMRvUYQ77yeUgU06xZWYzylJIpW5rK2nnIvVu7ZOI6Ran1el1a5/9Mr0KExtTZhG\n iTrw==", "X-Gm-Message-State": "AOJu0Yyj1M9fQgIQQsQN48ZtGTGPYz+e3B7NuNeRKoFtV3SgO99blzZ/\n\tZWF+sXA+lw9yAj15X4Hn8iQxzoFXcbv8ft/8bmH6AUBerRMyd3Nbpj/BpkZH2IXSrTx6sfjNEFN\n\tTfsKF8wCboA==", "X-Gm-Gg": "AeBDieu+tIQPNFf+tQHRy9+PYi8PM65H/Y3YLND4kOHDe5V+lAo22SfjAmmlEyrHuGe\n\tEs0nzWYkR8UAU6+71BtGrdfbzY2SoKivU1vAYKN+Jqrv7K83S7PgojJbqblulIgbhh5LAjIp+MZ\n\tfRUa/JNleF9Km+6uVjECd/FVHU0Tdb+r1gfD210Arg5ki/bczOg+3mbEgRU5wnyGg8WxgIDtZ1c\n\tSDOpPB1TN3ZLNcfUo88T2Kb3pdpggXoh0X4n4PdyMlS2bRAd/mEwkJcVbYYVUce3AotmJ1VnQw4\n\tx7CA2UQQ/4OAvX6vj0aqmXxwXn9yMxI4AudbImqZA7ZzbcoY4vZgJG8/C7X7FhPCIuvt2QsskjX\n\tfFH2fV9s7uqq5pyQ+wqeeC+ls5e35AS9v6xyQcorEFN0JAWoRiDNyxidyBbojABXjxao1mFXA6o\n\t4mb+Ur58TahXsotRz2bBr/S13MegKINabxcMPrHFIZ+SetmvVz/qmfndXDkIDqk5Nn95a+PBuKB\n\tS3tbJWN7s3St8UfxaQV", "X-Received": "by 2002:a05:693c:310a:b0:2ca:2f21:64da with SMTP id\n 5a478bee46e88-2d5841c5483mr2615126eec.0.1775848239372;\n Fri, 10 Apr 2026 12:10:39 -0700 (PDT)", "From": "Geramy Loveless <gloveless@jqluv.com>", "To": "linux-pci@vger.kernel.org", "Cc": "Cristian Cocos <cristi@ieee.org>,\n =?utf-8?q?Ilpo_J=C3=A4rvinen?= <ilpo.jarvinen@linux.intel.com>, =?utf-8?q?C?=\n\t=?utf-8?q?hristian_K=C3=B6nig?= <christian.koenig@amd.com>,\n Geramy Loveless <gloveless@jqluv.com>", "Subject": "[PATCH] PCI: release empty sibling bridge resources during window\n resize", "Date": "Fri, 10 Apr 2026 12:10:37 -0700", "Message-ID": "<20260410191037.782121-1-gloveless@jqluv.com>", "X-Mailer": "git-send-email 2.51.0", "In-Reply-To": "<ec0d287c884cbdd5131e1b09147b9b3cd56faf1d.camel@ieee.org>", "References": "<ec0d287c884cbdd5131e1b09147b9b3cd56faf1d.camel@ieee.org>", "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-Type": "text/plain; charset=UTF-8", "Content-Transfer-Encoding": "8bit" }, "content": "pbus_reassign_bridge_resources() refuses to release bridge windows that\nhave child resources. On multi-port PCIe switches (e.g. Thunderbolt\ndocks), empty sibling downstream ports hold small reservations that\nprevent the parent window from being freed and re-sized for rebar.\n\nAdd a bottom-up recursive release that saves and frees each empty bridge\nwindow individually, so the parent window can then be released and grown.\n\nSuggested-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>\nSigned-off-by: Geramy Loveless <gloveless@jqluv.com>\n---\n drivers/pci/setup-bus.c | 63 +++++++++++++++++++++++++++++++++++++++--\n 1 file changed, 61 insertions(+), 2 deletions(-)", "diff": "diff --git a/drivers/pci/setup-bus.c b/drivers/pci/setup-bus.c\nindex 4cf120ebe5ad..0c1fb654c9cc 100644\n--- a/drivers/pci/setup-bus.c\n+++ b/drivers/pci/setup-bus.c\n@@ -2292,6 +2292,60 @@ void pci_assign_unassigned_bridge_resources(struct pci_dev *bridge)\n }\n EXPORT_SYMBOL_GPL(pci_assign_unassigned_bridge_resources);\n \n+/*\n+ * pci_bus_release_empty_bridge_resources - Bottom-up release of bridge\n+ * window resources in empty subtrees.\n+ * @bus: PCI bus whose child bridges to process\n+ * @b_win: Ancestor bridge window; only children of this resource are released\n+ * @saved: List to save released resources for rollback\n+ *\n+ * Recurses depth-first into subordinate buses, then releases bridge windows\n+ * on the way back up. Each resource is individually saved before release so\n+ * the entire operation can be rolled back.\n+ */\n+static void pci_bus_release_empty_bridge_resources(struct pci_bus *bus,\n+\t\t\t\t\t\t struct resource *b_win,\n+\t\t\t\t\t\t struct list_head *saved)\n+{\n+\tstruct pci_dev *dev;\n+\n+\tlist_for_each_entry(dev, &bus->devices, bus_list) {\n+\t\tstruct resource *r;\n+\t\tunsigned int i;\n+\n+\t\tif (!dev->subordinate)\n+\t\t\tcontinue;\n+\n+\t\t/* Recurse first — release deepest resources before parents */\n+\t\tpci_bus_release_empty_bridge_resources(dev->subordinate,\n+\t\t\t\t\t\t b_win, saved);\n+\n+\t\tpci_dev_for_each_resource(dev, r, i) {\n+\t\t\tif (!pci_resource_is_bridge_win(i))\n+\t\t\t\tcontinue;\n+\n+\t\t\tif (!resource_assigned(r))\n+\t\t\t\tcontinue;\n+\n+\t\t\tif (r->parent != b_win)\n+\t\t\t\tcontinue;\n+\n+\t\t\tif (r->child) {\n+\t\t\t\tconst char *res_name = pci_resource_name(dev, i);\n+\n+\t\t\t\tpci_info(dev, \"%s %pR: not released, children still present\\n\",\n+\t\t\t\t\t res_name, r);\n+\t\t\t\tcontinue;\n+\t\t\t}\n+\n+\t\t\tif (pci_dev_res_add_to_list(saved, dev, r, 0, 0))\n+\t\t\t\tcontinue;\n+\n+\t\t\tpci_release_resource(dev, i);\n+\t\t}\n+\t}\n+}\n+\n /*\n * Walk to the root bus, find the bridge window relevant for @res and\n * release it when possible. If the bridge window contains assigned\n@@ -2316,7 +2370,12 @@ static int pbus_reassign_bridge_resources(struct pci_bus *bus, struct resource *\n \n \t\ti = pci_resource_num(bridge, res);\n \n-\t\t/* Ignore BARs which are still in use */\n+\t\t/* Release empty sibling bridge windows bottom-up */\n+\t\tif (bridge->subordinate)\n+\t\t\tpci_bus_release_empty_bridge_resources(\n+\t\t\t\tbridge->subordinate, res, saved);\n+\n+\t\t/* Ignore bridge windows which are still in use */\n \t\tif (!res->child) {\n \t\t\tret = pci_dev_res_add_to_list(saved, bridge, res, 0, 0);\n \t\t\tif (ret)\n@@ -2327,7 +2386,7 @@ static int pbus_reassign_bridge_resources(struct pci_bus *bus, struct resource *\n \t\t\tconst char *res_name = pci_resource_name(bridge, i);\n \n \t\t\tpci_warn(bridge,\n-\t\t\t\t \"%s %pR: was not released (still contains assigned resources)\\n\",\n+\t\t\t\t \"%s %pR: not released, active children present\\n\",\n \t\t\t\t res_name, res);\n \t\t}\n \n", "prefixes": [] }