{"id":2197264,"url":"http://patchwork.ozlabs.org/api/1.0/patches/2197264/?format=json","project":{"id":28,"url":"http://patchwork.ozlabs.org/api/1.0/projects/28/?format=json","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":"<20260217160836.2709885-2-kbusch@meta.com>","date":"2026-02-17T16:08:34","name":"[PATCHv5,1/3] PCI: rename __pci_bus_reset and __pci_slot_reset","commit_ref":null,"pull_url":null,"state":"new","archived":false,"hash":"048702cfbb354db98ef901affbaf7b8fe6717493","submitter":{"id":84963,"url":"http://patchwork.ozlabs.org/api/1.0/people/84963/?format=json","name":"Keith Busch","email":"kbusch@meta.com"},"delegate":null,"mbox":"http://patchwork.ozlabs.org/project/linux-pci/patch/20260217160836.2709885-2-kbusch@meta.com/mbox/","series":[{"id":492447,"url":"http://patchwork.ozlabs.org/api/1.0/series/492447/?format=json","date":"2026-02-17T16:08:34","name":"pci slot reset handling fixes","version":1,"mbox":"http://patchwork.ozlabs.org/series/492447/mbox/"}],"check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/2197264/checks/","tags":{},"headers":{"Return-Path":"\n ","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=meta.com header.i=@meta.com header.a=rsa-sha256\n header.s=s2048-2025-q2 header.b=k5XXVM0d;\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-47466-incoming=patchwork.ozlabs.org@vger.kernel.org;\n receiver=patchwork.ozlabs.org)","smtp.subspace.kernel.org;\n\tdkim=pass (2048-bit key) header.d=meta.com header.i=@meta.com\n header.b=\"k5XXVM0d\"","smtp.subspace.kernel.org;\n arc=none smtp.client-ip=67.231.153.30","smtp.subspace.kernel.org;\n dmarc=pass (p=reject dis=none) header.from=meta.com","smtp.subspace.kernel.org;\n spf=pass smtp.mailfrom=meta.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)\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4fFl222cR6z1xwD\n\tfor ; Wed, 18 Feb 2026 03:08:58 +1100 (AEDT)","from smtp.subspace.kernel.org (conduit.subspace.kernel.org\n [100.90.174.1])\n\tby sea.lore.kernel.org (Postfix) with ESMTP id 07AA030125D8\n\tfor ; Tue, 17 Feb 2026 16:08:56 +0000 (UTC)","from localhost.localdomain (localhost.localdomain [127.0.0.1])\n\tby smtp.subspace.kernel.org (Postfix) with ESMTP id 825A4280A29;\n\tTue, 17 Feb 2026 16:08:55 +0000 (UTC)","from mx0b-00082601.pphosted.com (mx0b-00082601.pphosted.com\n [67.231.153.30])\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 C7C6336C0CF\n\tfor ; Tue, 17 Feb 2026 16:08:53 +0000 (UTC)","from pps.filterd (m0148460.ppops.net [127.0.0.1])\n\tby mx0a-00082601.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id\n 61HC1aPK2053945\n\tfor ; Tue, 17 Feb 2026 08:08:52 -0800","from mail.thefacebook.com ([163.114.134.16])\n\tby mx0a-00082601.pphosted.com (PPS) with ESMTPS id 4ccgx2y55h-4\n\t(version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128 verify=NOT)\n\tfor ; Tue, 17 Feb 2026 08:08:52 -0800 (PST)","from twshared25002.15.frc2.facebook.com (2620:10d:c085:208::f) by\n mail.thefacebook.com (2620:10d:c08b:78::2ac9) with Microsoft SMTP Server\n (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id\n 15.2.2562.35; Tue, 17 Feb 2026 16:08:50 +0000","by devbig197.nha3.facebook.com (Postfix, from userid 544533)\n\tid 5C6CB811FAF8; Tue, 17 Feb 2026 08:08:39 -0800 (PST)"],"ARC-Seal":"i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116;\n\tt=1771344535; cv=none;\n b=GiyfJJwjgVGObh5MwwZlAxNv6tIT5Nr6SnjxqCg1/XoIZn1jmASLNJL7uqIgbCr3V3pfrb4joidynBqOVH0iXZafzQI/yHkHLpS7/3wA+gRfe5S8HJXbna958fbw4iNAXorGICVcqXCoowh2VDNOOjRxcXngEKHUPAGgm/0wsz0=","ARC-Message-Signature":"i=1; a=rsa-sha256; d=subspace.kernel.org;\n\ts=arc-20240116; t=1771344535; c=relaxed/simple;\n\tbh=jgDHUr3cr3inVo1HyulVq1ejvs211Lq2ujVKQzh6evM=;\n\th=From:To:CC:Subject:Date:Message-ID:In-Reply-To:References:\n\t MIME-Version:Content-Type;\n b=C/JeWAz4hUexbXMpvRvqTEXpNwl/ymqYlkT35Rzr7CzlxE7WWtBF7doAa2tIROHlBgomRT3UorNw3YyE1Q0NpXoVA7IXKf4z/sPzTk/DcoS+tEbM9qeM3AwQVfm4r3ui3huWU08as3Bz8UOi2xtZgXOJb8aLB0B7MsTl+MdPQ+Y=","ARC-Authentication-Results":"i=1; smtp.subspace.kernel.org;\n dmarc=pass (p=reject dis=none) header.from=meta.com;\n spf=pass smtp.mailfrom=meta.com;\n dkim=pass (2048-bit key) header.d=meta.com header.i=@meta.com\n header.b=k5XXVM0d; arc=none smtp.client-ip=67.231.153.30","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=meta.com; h=cc\n\t:content-transfer-encoding:content-type:date:from:in-reply-to\n\t:message-id:mime-version:references:subject:to; s=s2048-2025-q2;\n\t bh=xgXQ7cFfh0zjc5joR+GiffW1h+2SQFnjJA0jIXipN7E=; b=k5XXVM0dQU2g\n\tXxcH5Rp1O6r+cTRSLz2h6w5GEuqW0fRB9Uh9M/M3Q7+vRdlOboBWUwgKF/I3dEXe\n\tDdmROyEUy89biCB0mzbt2u4frg6qDHS7cDymkh+6BEnBy6imC2nyO/LP6Yo0IV5q\n\tEG4OQNV6rs5+ZqfHuWVuzi2/chV2ueNbYEmgXhgelnY9kXHsPlfhJcGRJHF72dbZ\n\twLPVCnTuwRw+S7DNYM5b2iUctXhbMguQfIC/74ROa3UcWZJGFOZDS19DjMINvetv\n\t2k8x0PtQFk2k5nEMSOmbHxL6MU85do28mV4FdeOMOGcnk6f/cuL0Es4u0+3YfHAF\n\tiUf0QIGebQ==","From":"Keith Busch ","To":", ,\n ","CC":", ,\n Keith Busch\n\t","Subject":"[PATCHv5 1/3] PCI: rename __pci_bus_reset and __pci_slot_reset","Date":"Tue, 17 Feb 2026 08:08:34 -0800","Message-ID":"<20260217160836.2709885-2-kbusch@meta.com>","X-Mailer":"git-send-email 2.47.3","In-Reply-To":"<20260217160836.2709885-1-kbusch@meta.com>","References":"<20260217160836.2709885-1-kbusch@meta.com>","Precedence":"bulk","X-Mailing-List":"linux-pci@vger.kernel.org","List-Id":"","List-Subscribe":"","List-Unsubscribe":"","MIME-Version":"1.0","Content-Transfer-Encoding":"quoted-printable","X-FB-Internal":"Safe","Content-Type":"text/plain","X-Proofpoint-ORIG-GUID":"Q5LOxAxaqSBuhk17iOAVKQXDZ7qH6pnj","X-Proofpoint-GUID":"Q5LOxAxaqSBuhk17iOAVKQXDZ7qH6pnj","X-Authority-Analysis":"v=2.4 cv=QqhTHFyd c=1 sm=1 tr=0 ts=69949294 cx=c_pps\n a=CB4LiSf2rd0gKozIdrpkBw==:117 a=CB4LiSf2rd0gKozIdrpkBw==:17\n a=HzLeVaNsDn8A:10 a=VkNPw1HP01LnGYTKEx00:22 a=Mpw57Om8IfrbqaoTuvik:22\n a=GgsMoib0sEa3-_RKJdDe:22 a=VwQbUJbxAAAA:8 a=QyXUC8HyAAAA:8\n a=piTmfLtOH4KJcwlmKd4A:9","X-Proofpoint-Spam-Details-Enc":"AW1haW4tMjYwMjE3MDEzMSBTYWx0ZWRfX59BppKo9atlL\n pQGOy+ECjuI8wcXP+gobkl4weNU4Ajo4uWU6b4CkLda9Gl7hkD9vj/6BHsF/aPusTvqtmpxiqvm\n d4TjKMZzoune/sJ8Nz3ckJv2J8h3XKjDLCga/gy27cQlFA1s9vdk1mjUFOx5FEBBQY2I1Nrte7G\n MzmGOpirl2L3bFxC7unSZZz5OBdNb63hZq5x0OPbgE9v/H37oW/mGkrsRGRuIq89CGuR2Rxt1m3\n jgJUtLfxzW16hCUEoy4pvYZkTGAGNMPaCCE+fBpqIE/XGSysXvM6ravyO06HVfatov9kjDD0a4F\n rmrvPGoaBetUm3rxadU1AZLBAJZytB6QuxAUUs/fiOpVYaYooeI5CZliZZeLqcrWHR6rA3NpQlA\n OaQJF+91YRS68x72Jm/4VJ3ObOivzToddnwb3vfObFmNum6CXe7nRWYXGbh5SGKO9H9VWByM1rr\n Ziwz/+6OLjhf5Jr5qgQ==","X-Proofpoint-Virus-Version":"vendor=baseguard\n engine=ICAP:2.0.293,Aquarius:18.0.1121,Hydra:6.1.51,FMLib:17.12.100.49\n definitions=2026-02-17_02,2026-02-16_04,2025-10-01_01"},"content":"From: Keith Busch \n\nMake the code a little easier to navigate with more descriptive\nfunction names. The two renamed functions here \"try\" to do to a reset,\nso make that clear in the name to distinguish them from other similarly\nnamed functions.\n\nReviewed-by: Dan Williams \nSigned-off-by: Keith Busch \n---\n drivers/pci/pci-sysfs.c | 2 +-\n drivers/pci/pci.c | 10 +++++-----\n drivers/pci/pci.h | 2 +-\n 3 files changed, 7 insertions(+), 7 deletions(-)","diff":"diff --git a/drivers/pci/pci-sysfs.c b/drivers/pci/pci-sysfs.c\nindex 363187ba4f56c..b44e884fd5372 100644\n--- a/drivers/pci/pci-sysfs.c\n+++ b/drivers/pci/pci-sysfs.c\n@@ -563,7 +563,7 @@ static ssize_t reset_subordinate_store(struct device *dev,\n \t\treturn -EINVAL;\n \n \tif (val) {\n-\t\tint ret = __pci_reset_bus(bus);\n+\t\tint ret = pci_try_reset_bus(bus);\n \n \t\tif (ret)\n \t\t\treturn ret;\ndiff --git a/drivers/pci/pci.c b/drivers/pci/pci.c\nindex 06bb01b3bd6c4..3bd0a830872c4 100644\n--- a/drivers/pci/pci.c\n+++ b/drivers/pci/pci.c\n@@ -5541,7 +5541,7 @@ int pci_probe_reset_slot(struct pci_slot *slot)\n EXPORT_SYMBOL_GPL(pci_probe_reset_slot);\n \n /**\n- * __pci_reset_slot - Try to reset a PCI slot\n+ * pci_try_reset_slot - Try to reset a PCI slot\n * @slot: PCI slot to reset\n *\n * A PCI bus may host multiple slots, each slot may support a reset mechanism\n@@ -5555,7 +5555,7 @@ EXPORT_SYMBOL_GPL(pci_probe_reset_slot);\n *\n * Same as above except return -EAGAIN if the slot cannot be locked\n */\n-static int __pci_reset_slot(struct pci_slot *slot)\n+static int pci_try_reset_slot(struct pci_slot *slot)\n {\n \tint rc;\n \n@@ -5644,12 +5644,12 @@ int pci_probe_reset_bus(struct pci_bus *bus)\n EXPORT_SYMBOL_GPL(pci_probe_reset_bus);\n \n /**\n- * __pci_reset_bus - Try to reset a PCI bus\n+ * pci_try_reset_bus - Try to reset a PCI bus\n * @bus: top level PCI bus to reset\n *\n * Same as above except return -EAGAIN if the bus cannot be locked\n */\n-int __pci_reset_bus(struct pci_bus *bus)\n+int pci_try_reset_bus(struct pci_bus *bus)\n {\n \tint rc;\n \n@@ -5678,7 +5678,7 @@ int __pci_reset_bus(struct pci_bus *bus)\n int pci_reset_bus(struct pci_dev *pdev)\n {\n \treturn (!pci_probe_reset_slot(pdev->slot)) ?\n-\t __pci_reset_slot(pdev->slot) : __pci_reset_bus(pdev->bus);\n+\t pci_try_reset_slot(pdev->slot) : pci_try_reset_bus(pdev->bus);\n }\n EXPORT_SYMBOL_GPL(pci_reset_bus);\n \ndiff --git a/drivers/pci/pci.h b/drivers/pci/pci.h\nindex c8a0522e2e1f5..d1350d54b932d 100644\n--- a/drivers/pci/pci.h\n+++ b/drivers/pci/pci.h\n@@ -231,7 +231,7 @@ bool pci_reset_supported(struct pci_dev *dev);\n void pci_init_reset_methods(struct pci_dev *dev);\n int pci_bridge_secondary_bus_reset(struct pci_dev *dev);\n int pci_bus_error_reset(struct pci_dev *dev);\n-int __pci_reset_bus(struct pci_bus *bus);\n+int pci_try_reset_bus(struct pci_bus *bus);\n \n struct pci_cap_saved_data {\n \tu16\t\tcap_nr;\n","prefixes":["PATCHv5","1/3"]}