Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/1.2/patches/2230191/?format=api
{ "id": 2230191, "url": "http://patchwork.ozlabs.org/api/1.2/patches/2230191/?format=api", "web_url": "http://patchwork.ozlabs.org/project/linux-pci/patch/20260429122617.7324-3-ilpo.jarvinen@linux.intel.com/", "project": { "id": 28, "url": "http://patchwork.ozlabs.org/api/1.2/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, "list_archive_url": "", "list_archive_url_format": "", "commit_url_format": "" }, "msgid": "<20260429122617.7324-3-ilpo.jarvinen@linux.intel.com>", "list_archive_url": null, "date": "2026-04-29T12:26:08", "name": "[02/11] PCI: Rename added to add_list", "commit_ref": null, "pull_url": null, "state": "new", "archived": false, "hash": "410d54b01036c94553500372d993c467e2ca494b", "submitter": { "id": 83553, "url": "http://patchwork.ozlabs.org/api/1.2/people/83553/?format=api", "name": "Ilpo Järvinen", "email": "ilpo.jarvinen@linux.intel.com" }, "delegate": null, "mbox": "http://patchwork.ozlabs.org/project/linux-pci/patch/20260429122617.7324-3-ilpo.jarvinen@linux.intel.com/mbox/", "series": [ { "id": 502050, "url": "http://patchwork.ozlabs.org/api/1.2/series/502050/?format=api", "web_url": "http://patchwork.ozlabs.org/project/linux-pci/list/?series=502050", "date": "2026-04-29T12:26:06", "name": "PCI: pci_resource_alignment() improvement + cleanups", "version": 1, "mbox": "http://patchwork.ozlabs.org/series/502050/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/patches/2230191/comments/", "check": "pending", "checks": "http://patchwork.ozlabs.org/api/patches/2230191/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "\n <linux-pci+bounces-53387-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=T8Dmh9QN;\n\tdkim-atps=neutral", "legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=vger.kernel.org\n (client-ip=172.234.253.10; helo=sea.lore.kernel.org;\n envelope-from=linux-pci+bounces-53387-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=\"T8Dmh9QN\"", "smtp.subspace.kernel.org;\n arc=none smtp.client-ip=192.198.163.18", "smtp.subspace.kernel.org;\n dmarc=pass (p=none dis=none) header.from=linux.intel.com", "smtp.subspace.kernel.org;\n spf=pass smtp.mailfrom=linux.intel.com" ], "Received": [ "from sea.lore.kernel.org (sea.lore.kernel.org [172.234.253.10])\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 4g5GmH6MKJz1yHX\n\tfor <incoming@patchwork.ozlabs.org>; Wed, 29 Apr 2026 22:27:59 +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 8D5673013A69\n\tfor <incoming@patchwork.ozlabs.org>; Wed, 29 Apr 2026 12:26:53 +0000 (UTC)", "from localhost.localdomain (localhost.localdomain [127.0.0.1])\n\tby smtp.subspace.kernel.org (Postfix) with ESMTP id 24EE539D6C6;\n\tWed, 29 Apr 2026 12:26:53 +0000 (UTC)", "from mgamail.intel.com (mgamail.intel.com [192.198.163.18])\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 AA20939C002;\n\tWed, 29 Apr 2026 12:26:51 +0000 (UTC)", "from fmviesa006.fm.intel.com ([10.60.135.146])\n by fmvoesa112.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;\n 29 Apr 2026 05:26:51 -0700", "from ijarvine-mobl1.ger.corp.intel.com (HELO localhost)\n ([10.245.245.212])\n by fmviesa006-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;\n 29 Apr 2026 05:26:47 -0700" ], "ARC-Seal": "i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116;\n\tt=1777465613; cv=none;\n b=XSItwaTuoqKqFKb7G3ZnvA8j2cIxgt82c13qukBoS/b1eS1nB+LoTiVsk1Y2AjOckIANJO736PT/UWT/CYk9aCH3eaeQSwMyvcjUIrc0QHTfFXT0YdSgdjG6oInqqy+ttsrw3KCJ0TDNCaiHCuzDJxwOVZ3QZxmBO4VChi30vqs=", "ARC-Message-Signature": "i=1; a=rsa-sha256; d=subspace.kernel.org;\n\ts=arc-20240116; t=1777465613; c=relaxed/simple;\n\tbh=NlSw3DKYOjHOeHYh/0go2vSw53u8u6nznavTwXNw3m0=;\n\th=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References:\n\t MIME-Version:Content-Type;\n b=FgHpQRXnFQFwROPeSDCCpszSSpxvnvGm1epKYICuZx1SmmM+fn52fpf3ZeP9QNeQN1zV4LrnRYN+9vrCKw4/aqmGgY+IqRN40n0VaZQQ/PAwvA8mijgRN0CkD4MlULt278Rgyb8cTkxTVU8Twkpl3d7CinJye2paJK0JV+mBe/U=", "ARC-Authentication-Results": "i=1; smtp.subspace.kernel.org;\n dmarc=pass (p=none dis=none) header.from=linux.intel.com;\n spf=pass smtp.mailfrom=linux.intel.com;\n dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com\n header.b=T8Dmh9QN; arc=none smtp.client-ip=192.198.163.18", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple;\n d=intel.com; i=@intel.com; q=dns/txt; s=Intel;\n t=1777465611; x=1809001611;\n h=from:to:cc:subject:date:message-id:in-reply-to:\n references:mime-version:content-transfer-encoding;\n bh=NlSw3DKYOjHOeHYh/0go2vSw53u8u6nznavTwXNw3m0=;\n b=T8Dmh9QN7vTMXo+7Abx8weaCfVRCUWL7gMRn9Jvzs7MMSWr98+uB2yB6\n IEIgKCNMs4QiU2KM3JhjxWgSR+F+yIzSU68iePkk9/3IdJKmxrDDOQ4Iz\n /E7hPLG87GHDBf7X3O25ElxfoszCtBzSFOFMt5KK51ZoeRxWYFGqDCpew\n QOX1QPFWvVv4VFBnDdcwv7rJ+Q+9oyeLWbAGsd7YYBu9fDToSYE8+U7Bh\n hvTOC5ByVcKm4JjzYXSrsV+gPCBkZAC+9v24t9qefed0TLMsZNFXog3v1\n USI23fBRVdBnumMLRWZy56ErLdMHnFvn33duudkuK8b2z21RSsaWsfR0J\n g==;", "X-CSE-ConnectionGUID": [ "N22MFhwsQG+DbcgUpkMQiQ==", "rb2ZCEBZRhq5Nh1Tx+QRvA==" ], "X-CSE-MsgGUID": [ "AKDB1Du/RmGXknErETuYew==", "XJYr1JnTROCPcbw42Lz3bw==" ], "X-IronPort-AV": [ "E=McAfee;i=\"6800,10657,11770\"; a=\"77557694\"", "E=Sophos;i=\"6.23,206,1770624000\";\n d=\"scan'208\";a=\"77557694\"", "E=Sophos;i=\"6.23,206,1770624000\";\n d=\"scan'208\";a=\"229669702\"" ], "X-ExtLoop1": "1", "From": "=?utf-8?q?Ilpo_J=C3=A4rvinen?= <ilpo.jarvinen@linux.intel.com>", "To": "linux-pci@vger.kernel.org,\n\tBjorn Helgaas <bhelgaas@google.com>,\n\tShawn Jin <shawn.jin@asteralabs.com>,\n\tlinuxppc-dev@lists.ozlabs.org,\n\tMadhavan Srinivasan <maddy@linux.ibm.com>,\n\tMichael Ellerman <mpe@ellerman.id.au>,\n\tNicholas Piggin <npiggin@gmail.com>,\n\tlinux-kernel@vger.kernel.org", "Cc": "=?utf-8?q?Ilpo_J=C3=A4rvinen?= <ilpo.jarvinen@linux.intel.com>", "Subject": "[PATCH 02/11] PCI: Rename added to add_list", "Date": "Wed, 29 Apr 2026 15:26:08 +0300", "Message-Id": "<20260429122617.7324-3-ilpo.jarvinen@linux.intel.com>", "X-Mailer": "git-send-email 2.39.5", "In-Reply-To": "<20260429122617.7324-1-ilpo.jarvinen@linux.intel.com>", "References": "<20260429122617.7324-1-ilpo.jarvinen@linux.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-Type": "text/plain; charset=UTF-8", "Content-Transfer-Encoding": "8bit" }, "content": "The resource fitting algorithm uses different names from the list\nholding the optional sizes: added, add_head, add_list, and\nrealloc_head. 'add_list' sounds the most natural and some of the\nrelated variables also use 'add' such as 'add_size'.\n\nTo reduce variation, rename 'added' and 'add_head' to 'add_list'.\nAlso rename some 'realloc_head' cases selectively to 'add_list'.\n\nWhile it would be nice to rename every 'realloc_head' to 'add_list' for\nconsistency, it might create a backport headache with all to work going\ninto this algorithm that may need to be eventually backported. Thus,\nit's better to leave 'realloc_head' as is for now.\n\nSigned-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>\n---\n drivers/pci/pci.h | 2 +-\n drivers/pci/setup-bus.c | 26 +++++++++++++-------------\n 2 files changed, 14 insertions(+), 14 deletions(-)", "diff": "diff --git a/drivers/pci/pci.h b/drivers/pci/pci.h\nindex 4a14f88e543a..4fcf5a25ad9e 100644\n--- a/drivers/pci/pci.h\n+++ b/drivers/pci/pci.h\n@@ -515,7 +515,7 @@ int pci_dev_res_add_to_list(struct list_head *head, struct pci_dev *dev,\n void __pci_bus_size_bridges(struct pci_bus *bus,\n \t\t\tstruct list_head *realloc_head);\n void __pci_bus_assign_resources(const struct pci_bus *bus,\n-\t\t\t\tstruct list_head *realloc_head,\n+\t\t\t\tstruct list_head *add_list,\n \t\t\t\tstruct list_head *fail_head);\n bool pci_bus_clip_resource(struct pci_dev *dev, int idx);\n void pci_walk_bus_locked(struct pci_bus *top,\ndiff --git a/drivers/pci/setup-bus.c b/drivers/pci/setup-bus.c\nindex 4cf120ebe5ad..3765693e95f0 100644\n--- a/drivers/pci/setup-bus.c\n+++ b/drivers/pci/setup-bus.c\n@@ -756,13 +756,13 @@ static void __assign_resources_sorted(struct list_head *head,\n }\n \n static void pdev_assign_resources_sorted(struct pci_dev *dev,\n-\t\t\t\t\t struct list_head *add_head,\n+\t\t\t\t\t struct list_head *add_list,\n \t\t\t\t\t struct list_head *fail_head)\n {\n \tLIST_HEAD(head);\n \n \tpdev_sort_resources(dev, &head);\n-\t__assign_resources_sorted(&head, add_head, fail_head);\n+\t__assign_resources_sorted(&head, add_list, fail_head);\n \n }\n \n@@ -1502,13 +1502,13 @@ static void pdev_assign_fixed_resources(struct pci_dev *dev)\n }\n \n void __pci_bus_assign_resources(const struct pci_bus *bus,\n-\t\t\t\tstruct list_head *realloc_head,\n+\t\t\t\tstruct list_head *add_list,\n \t\t\t\tstruct list_head *fail_head)\n {\n \tstruct pci_bus *b;\n \tstruct pci_dev *dev;\n \n-\tpbus_assign_resources_sorted(bus, realloc_head, fail_head);\n+\tpbus_assign_resources_sorted(bus, add_list, fail_head);\n \n \tlist_for_each_entry(dev, &bus->devices, bus_list) {\n \t\tpdev_assign_fixed_resources(dev);\n@@ -1517,7 +1517,7 @@ void __pci_bus_assign_resources(const struct pci_bus *bus,\n \t\tif (!b)\n \t\t\tcontinue;\n \n-\t\t__pci_bus_assign_resources(b, realloc_head, fail_head);\n+\t\t__pci_bus_assign_resources(b, add_list, fail_head);\n \n \t\tswitch (dev->hdr_type) {\n \t\tcase PCI_HEADER_TYPE_BRIDGE:\n@@ -1613,19 +1613,19 @@ void pci_bus_claim_resources(struct pci_bus *b)\n EXPORT_SYMBOL(pci_bus_claim_resources);\n \n static void __pci_bridge_assign_resources(const struct pci_dev *bridge,\n-\t\t\t\t\t struct list_head *add_head,\n+\t\t\t\t\t struct list_head *add_list,\n \t\t\t\t\t struct list_head *fail_head)\n {\n \tstruct pci_bus *b;\n \n \tpdev_assign_resources_sorted((struct pci_dev *)bridge,\n-\t\t\t\t\t add_head, fail_head);\n+\t\t\t\t add_list, fail_head);\n \n \tb = bridge->subordinate;\n \tif (!b)\n \t\treturn;\n \n-\t__pci_bus_assign_resources(b, add_head, fail_head);\n+\t__pci_bus_assign_resources(b, add_list, fail_head);\n \n \tswitch (bridge->class >> 8) {\n \tcase PCI_CLASS_BRIDGE_PCI:\n@@ -2303,7 +2303,7 @@ static int pbus_reassign_bridge_resources(struct pci_bus *bus, struct resource *\n \tunsigned long type = res->flags;\n \tstruct pci_dev_resource *dev_res;\n \tstruct pci_dev *bridge = NULL;\n-\tLIST_HEAD(added);\n+\tLIST_HEAD(add_list);\n \tLIST_HEAD(failed);\n \tunsigned int i;\n \tint ret = 0;\n@@ -2337,10 +2337,10 @@ static int pbus_reassign_bridge_resources(struct pci_bus *bus, struct resource *\n \tif (!bridge)\n \t\treturn -ENOENT;\n \n-\t__pci_bus_size_bridges(bridge->subordinate, &added);\n-\t__pci_bridge_assign_resources(bridge, &added, &failed);\n-\tif (WARN_ON_ONCE(!list_empty(&added)))\n-\t\tpci_dev_res_free_list(&added);\n+\t__pci_bus_size_bridges(bridge->subordinate, &add_list);\n+\t__pci_bridge_assign_resources(bridge, &add_list, &failed);\n+\tif (WARN_ON_ONCE(!list_empty(&add_list)))\n+\t\tpci_dev_res_free_list(&add_list);\n \n \tif (!list_empty(&failed)) {\n \t\tif (pci_required_resource_failed(&failed, type))\n", "prefixes": [ "02/11" ] }