Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/1.1/patches/2221672/?format=api
{ "id": 2221672, "url": "http://patchwork.ozlabs.org/api/1.1/patches/2221672/?format=api", "web_url": "http://patchwork.ozlabs.org/project/linux-pci/patch/20260410055040.39233-14-kwilczynski@kernel.org/", "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": "<20260410055040.39233-14-kwilczynski@kernel.org>", "date": "2026-04-10T05:50:33", "name": "[13/20] alpha/PCI: Clean up __pci_mmap_fits()", "commit_ref": null, "pull_url": null, "state": "new", "archived": false, "hash": "0185f98d6c3cbcc14d2424d2e1142b4c30d02fa7", "submitter": { "id": 86709, "url": "http://patchwork.ozlabs.org/api/1.1/people/86709/?format=api", "name": "Krzysztof Wilczyński", "email": "kwilczynski@kernel.org" }, "delegate": null, "mbox": "http://patchwork.ozlabs.org/project/linux-pci/patch/20260410055040.39233-14-kwilczynski@kernel.org/mbox/", "series": [ { "id": 499398, "url": "http://patchwork.ozlabs.org/api/1.1/series/499398/?format=api", "web_url": "http://patchwork.ozlabs.org/project/linux-pci/list/?series=499398", "date": "2026-04-10T05:50:21", "name": "PCI: Convert all dynamic sysfs attributes to static", "version": 1, "mbox": "http://patchwork.ozlabs.org/series/499398/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/patches/2221672/comments/", "check": "pending", "checks": "http://patchwork.ozlabs.org/api/patches/2221672/checks/", "tags": {}, "headers": { "Return-Path": "\n <linux-pci+bounces-52275-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=kernel.org header.i=@kernel.org header.a=rsa-sha256\n header.s=k20201202 header.b=XGDauH65;\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-52275-incoming=patchwork.ozlabs.org@vger.kernel.org;\n receiver=patchwork.ozlabs.org)", "smtp.subspace.kernel.org;\n\tdkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org\n header.b=\"XGDauH65\"", "smtp.subspace.kernel.org;\n arc=none smtp.client-ip=10.30.226.201" ], "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)\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4fsQsj0Zbpz1yGb\n\tfor <incoming@patchwork.ozlabs.org>; Fri, 10 Apr 2026 15:51:37 +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 A8A1630120F9\n\tfor <incoming@patchwork.ozlabs.org>; Fri, 10 Apr 2026 05:51:34 +0000 (UTC)", "from localhost.localdomain (localhost.localdomain [127.0.0.1])\n\tby smtp.subspace.kernel.org (Postfix) with ESMTP id D618E322B9F;\n\tFri, 10 Apr 2026 05:51:31 +0000 (UTC)", "from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org\n [10.30.226.201])\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 B3BF030BF5C;\n\tFri, 10 Apr 2026 05:51:31 +0000 (UTC)", "by smtp.kernel.org (Postfix) with ESMTPSA id 5A7F7C19421;\n\tFri, 10 Apr 2026 05:51:28 +0000 (UTC)" ], "ARC-Seal": "i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116;\n\tt=1775800291; cv=none;\n b=dSHNacMgJ9arRKIoNhwU7eIRpkZXKsoHDYfn8ShMRt5f5bGteref4H5rqQW7M0T/FN0l7IoOpssIXEhvOl/v3/BYZSnYD37STF1qtkkD/2A6KrfiZcZL2ijDLJzeYaurLcRYZMc9ISn7V/0qhGh8rmBoPn6clBIczRlG+gAOpjg=", "ARC-Message-Signature": "i=1; a=rsa-sha256; d=subspace.kernel.org;\n\ts=arc-20240116; t=1775800291; c=relaxed/simple;\n\tbh=Nh7a1ArkRQeNDsZQ4G4s7B27zHMA9oeNqUUdRhEq7Fs=;\n\th=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References:\n\t MIME-Version:Content-Type;\n b=Qix79xH0QKkxAgJUtiotDwhFRVdcH3hLFc/RTmEVib0aiXPuRhyr6PbHZ8IankUw1EyzdM7bvboQeslFd4vMb1pHsWZ3wgRSUXG9QQT2/z36TdpMlpD5kK6bml9c6wSqoRj2vjxEi6RtUP95b78245IhkvXdOKEZrJlG7e4tT2I=", "ARC-Authentication-Results": "i=1; smtp.subspace.kernel.org;\n dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org\n header.b=XGDauH65; arc=none smtp.client-ip=10.30.226.201", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org;\n\ts=k20201202; t=1775800291;\n\tbh=Nh7a1ArkRQeNDsZQ4G4s7B27zHMA9oeNqUUdRhEq7Fs=;\n\th=From:To:Cc:Subject:Date:In-Reply-To:References:From;\n\tb=XGDauH65tg4bJ9ruSw7Z6gY2FgZvSRdoiVLxHgO6FHnL6maslO7dpOby3VlujfHY6\n\t lp71qjGPI9FnJ+YkDygQZLF5ftlEhT8uDDo3MIf7YZBN6Itnjwxnhzm72KF4k+HFD0\n\t GpQmeT9QHiDMxTrn+nKdUqMsIwo+qI9jQVMvVAYuN333KZPyYMJovH9pMhkBeIGf2V\n\t 5iyGh6zq4JQKEamShug3eT7CotSXt3MqJ+1Tg0qxcotMGh+oKGvTXX7Cs6ugCuVz52\n\t aLYqZNvCXjb5KJX1U93HTRBGcVdyqotqEjICJRYp1s9beAn1C1ORBRpyLbEfda9fm/\n\t Li3fkgG20rMiw==", "From": "=?utf-8?q?Krzysztof_Wilczy=C5=84ski?= <kwilczynski@kernel.org>", "To": "Bjorn Helgaas <bhelgaas@google.com>", "Cc": "Bjorn Helgaas <helgaas@kernel.org>,\n Manivannan Sadhasivam <mani@kernel.org>,\n Lorenzo Pieralisi <lpieralisi@kernel.org>,\n Magnus Lindholm <linmag7@gmail.com>, Matt Turner <mattst88@gmail.com>,\n Richard Henderson <richard.henderson@linaro.org>,\n Christophe Leroy <chleroy@kernel.org>,\n Madhavan Srinivasan <maddy@linux.ibm.com>,\n Michael Ellerman <mpe@ellerman.id.au>, Nicholas Piggin <npiggin@gmail.com>,\n Dexuan Cui <decui@microsoft.com>,\n =?utf-8?q?Krzysztof_Ha=C5=82asa?= <khalasa@piap.pl>,\n Lukas Wunner <lukas@wunner.de>, \"Oliver O'Halloran\" <oohall@gmail.com>,\n Saurabh Singh Sengar <ssengar@microsoft.com>,\n Shuan He <heshuan@bytedance.com>, Srivatsa Bhat <srivatsabhat@microsoft.com>,\n\t=?utf-8?q?Ilpo_J=C3=A4rvinen?= <ilpo.jarvinen@linux.intel.com>,\n linux-pci@vger.kernel.org, linux-alpha@vger.kernel.org,\n linuxppc-dev@lists.ozlabs.org", "Subject": "[PATCH 13/20] alpha/PCI: Clean up __pci_mmap_fits()", "Date": "Fri, 10 Apr 2026 05:50:33 +0000", "Message-ID": "<20260410055040.39233-14-kwilczynski@kernel.org>", "X-Mailer": "git-send-email 2.53.0", "In-Reply-To": "<20260410055040.39233-1-kwilczynski@kernel.org>", "References": "<20260410055040.39233-1-kwilczynski@kernel.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": "Currently, __pci_mmap_fits() computes the BAR size using\npci_resource_len() - 1, which wraps to a large value when the\nBAR length is zero, causing the bounds check to incorrectly\nsucceed.\n\nThus, add an early return for empty resources.\n\nAlso, remove the WARN() that fires when userspace attempts to\nmmap beyond the BAR bounds. The check still returns 0 to reject\nthe mapping, but the warning is excessive for normal operation.\n\nA similar warning was removed from the PCI core in the commit\n3b519e4ea618 (\"PCI: fix size checks for mmap() on /proc/bus/pci files\").\n\nSigned-off-by: Krzysztof Wilczyński <kwilczynski@kernel.org>\n---\n arch/alpha/kernel/pci-sysfs.c | 14 ++++++--------\n 1 file changed, 6 insertions(+), 8 deletions(-)", "diff": "diff --git a/arch/alpha/kernel/pci-sysfs.c b/arch/alpha/kernel/pci-sysfs.c\nindex 7aac5e76dcd6..867199b988de 100644\n--- a/arch/alpha/kernel/pci-sysfs.c\n+++ b/arch/alpha/kernel/pci-sysfs.c\n@@ -37,20 +37,18 @@ static int hose_mmap_page_range(struct pci_controller *hose,\n static int __pci_mmap_fits(struct pci_dev *pdev, int num,\n \t\t\t struct vm_area_struct *vma, int sparse)\n {\n+\tresource_size_t len = pci_resource_len(pdev, num);\n \tunsigned long nr, start, size;\n \tint shift = sparse ? 5 : 0;\n \n+\tif (!len)\n+\t\treturn 0;\n+\n \tnr = vma_pages(vma);\n \tstart = vma->vm_pgoff;\n-\tsize = ((pci_resource_len(pdev, num) - 1) >> (PAGE_SHIFT - shift)) + 1;\n+\tsize = ((len - 1) >> (PAGE_SHIFT - shift)) + 1;\n \n-\tif (start < size && size - start >= nr)\n-\t\treturn 1;\n-\tWARN(1, \"process \\\"%s\\\" tried to map%s 0x%08lx-0x%08lx on %s BAR %d \"\n-\t\t\"(size 0x%08lx)\\n\",\n-\t\tcurrent->comm, sparse ? \" sparse\" : \"\", start, start + nr,\n-\t\tpci_name(pdev), num, size);\n-\treturn 0;\n+\treturn start < size && size - start >= nr;\n }\n \n /**\n", "prefixes": [ "13/20" ] }