Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/2197771/?format=api
{ "id": 2197771, "url": "http://patchwork.ozlabs.org/api/patches/2197771/?format=api", "web_url": "http://patchwork.ozlabs.org/project/linux-pci/patch/20260218134633.461181-3-Simon.Richter@hogyros.de/", "project": { "id": 28, "url": "http://patchwork.ozlabs.org/api/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": "<20260218134633.461181-3-Simon.Richter@hogyros.de>", "list_archive_url": null, "date": "2026-02-18T13:46:30", "name": "[v2,2/5] vgaarb: pass errors from pci_set_vga_state up", "commit_ref": null, "pull_url": null, "state": "new", "archived": false, "hash": "4656fcf0c659b35068c8524a1abcb844e2aa8e2f", "submitter": { "id": 91044, "url": "http://patchwork.ozlabs.org/api/people/91044/?format=api", "name": "Simon Richter", "email": "Simon.Richter@hogyros.de" }, "delegate": null, "mbox": "http://patchwork.ozlabs.org/project/linux-pci/patch/20260218134633.461181-3-Simon.Richter@hogyros.de/mbox/", "series": [ { "id": 492553, "url": "http://patchwork.ozlabs.org/api/series/492553/?format=api", "web_url": "http://patchwork.ozlabs.org/project/linux-pci/list/?series=492553", "date": "2026-02-18T13:46:28", "name": "Bridges without VGA support", "version": 2, "mbox": "http://patchwork.ozlabs.org/series/492553/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/patches/2197771/comments/", "check": "pending", "checks": "http://patchwork.ozlabs.org/api/patches/2197771/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "\n <linux-pci+bounces-47564-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 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-47564-incoming=patchwork.ozlabs.org@vger.kernel.org;\n receiver=patchwork.ozlabs.org)", "smtp.subspace.kernel.org;\n arc=none smtp.client-ip=185.187.169.70", "smtp.subspace.kernel.org;\n dmarc=none (p=none dis=none) header.from=hogyros.de", "smtp.subspace.kernel.org;\n spf=pass smtp.mailfrom=hogyros.de" ], "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 4fGHqy6h9Qz1xvq\n\tfor <incoming@patchwork.ozlabs.org>; Thu, 19 Feb 2026 00:47:10 +1100 (AEDT)", "from smtp.subspace.kernel.org (conduit.subspace.kernel.org\n [100.90.174.1])\n\tby tor.lore.kernel.org (Postfix) with ESMTP id 0E1D93011A54\n\tfor <incoming@patchwork.ozlabs.org>; Wed, 18 Feb 2026 13:47:06 +0000 (UTC)", "from localhost.localdomain (localhost.localdomain [127.0.0.1])\n\tby smtp.subspace.kernel.org (Postfix) with ESMTP id BBD1D30DEAD;\n\tWed, 18 Feb 2026 13:47:03 +0000 (UTC)", "from psionic.psi5.com (psionic.psi5.com [185.187.169.70])\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 4B780328246\n\tfor <linux-pci@vger.kernel.org>; Wed, 18 Feb 2026 13:47:02 +0000 (UTC)", "from localhost.localdomain (unknown\n [IPv6:2400:2410:b120:f200:2e09:4dff:fe00:2e9])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest\n SHA256)\n\t(Client did not present a certificate)\n\tby psionic.psi5.com (Postfix) with ESMTPSA id 4EE843F206;\n\tWed, 18 Feb 2026 14:46:59 +0100 (CET)" ], "ARC-Seal": "i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116;\n\tt=1771422423; cv=none;\n b=WVh0HHYEHDQBxBcN1L2W1FQQWSgy3+ZHgogs2fXGCDOG0g9FmmYs0V5iWpdJA6Sq4659cqWmHuec0ZHitz4824a+dsgZzlDjy11wddhtFtuO2mnq/dQN29o0FllRkKYsBB24b+vZXwaAxnP8IKchI9bzgmcnXHh/WD8Sub2VJoo=", "ARC-Message-Signature": "i=1; a=rsa-sha256; d=subspace.kernel.org;\n\ts=arc-20240116; t=1771422423; c=relaxed/simple;\n\tbh=tV5Zl7hb5xuXldR8mGLUJCleWHHBrRQDEMUtV7cGqUc=;\n\th=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References:\n\t MIME-Version;\n b=K2vBUKv4CMUIsNKcL514LoNd/NIqxecMLdsH0O0ROSLDj5Id43Wb9elI4XZJ2GWt/9mRWeh0WCwB0nGpJCOxl351Zao6cLNDML9IchG/WQXkrGiUyE9u8yc9MycvBhd0avdBKR3eGV4s6eceOICyU+5b9VEX2wwuv/bOQ6MvVbY=", "ARC-Authentication-Results": "i=1; smtp.subspace.kernel.org;\n dmarc=none (p=none dis=none) header.from=hogyros.de;\n spf=pass smtp.mailfrom=hogyros.de; arc=none smtp.client-ip=185.187.169.70", "From": "Simon Richter <Simon.Richter@hogyros.de>", "To": "linux-pci@vger.kernel.org", "Cc": "intel-xe@lists.freedesktop.org,\n\tdri-devel@lists.freedesktop.org,\n\tSimon Richter <Simon.Richter@hogyros.de>", "Subject": "[PATCH v2 2/5] vgaarb: pass errors from pci_set_vga_state up", "Date": "Wed, 18 Feb 2026 22:46:30 +0900", "Message-ID": "<20260218134633.461181-3-Simon.Richter@hogyros.de>", "X-Mailer": "git-send-email 2.47.3", "In-Reply-To": "<20260218134633.461181-1-Simon.Richter@hogyros.de>", "References": "<20260217170419.236739-1-Simon.Richter@hogyros.de>\n <20260218134633.461181-1-Simon.Richter@hogyros.de>", "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": "pci_set_vga_state returns an error code, which so far has been ignored.\nPass this code through __vga_tryget (via ERR_PTR).\n\nSigned-off-by: Simon Richter <Simon.Richter@hogyros.de>\n---\n drivers/pci/vgaarb.c | 13 +++++++++++--\n 1 file changed, 11 insertions(+), 2 deletions(-)", "diff": "diff --git a/drivers/pci/vgaarb.c b/drivers/pci/vgaarb.c\nindex 188885d30d41..93f695f9d768 100644\n--- a/drivers/pci/vgaarb.c\n+++ b/drivers/pci/vgaarb.c\n@@ -215,6 +215,7 @@ static struct vga_device *__vga_tryget(struct vga_device *vgadev,\n \tstruct vga_device *conflict;\n \tunsigned int pci_bits;\n \tu32 flags = 0;\n+\tint err = 0;\n \n \t/*\n \t * Account for \"normal\" resources to lock. If we decode the legacy,\n@@ -307,7 +308,9 @@ static struct vga_device *__vga_tryget(struct vga_device *vgadev,\n \t\tif (change_bridge)\n \t\t\tflags |= PCI_VGA_STATE_CHANGE_BRIDGE;\n \n-\t\tpci_set_vga_state(conflict->pdev, false, pci_bits, flags);\n+\t\terr = pci_set_vga_state(conflict->pdev, false, pci_bits, flags);\n+\t\tif (unlikely(err))\n+\t\t\treturn ERR_PTR(err);\n \t\tconflict->owns &= ~match;\n \n \t\t/* If we disabled normal decoding, reflect it in owns */\n@@ -337,7 +340,9 @@ static struct vga_device *__vga_tryget(struct vga_device *vgadev,\n \tif (wants & VGA_RSRC_LEGACY_MASK)\n \t\tflags |= PCI_VGA_STATE_CHANGE_BRIDGE;\n \n-\tpci_set_vga_state(vgadev->pdev, true, pci_bits, flags);\n+\terr = pci_set_vga_state(vgadev->pdev, true, pci_bits, flags);\n+\tif (unlikely(err))\n+\t\treturn ERR_PTR(err);\n \n \tvgadev->owns |= wants;\n lock_them:\n@@ -455,6 +460,10 @@ int vga_get(struct pci_dev *pdev, unsigned int rsrc, int interruptible)\n \t\t}\n \t\tconflict = __vga_tryget(vgadev, rsrc);\n \t\tspin_unlock_irqrestore(&vga_lock, flags);\n+\t\tif (IS_ERR(conflict)) {\n+\t\t\trc = PTR_ERR(conflict);\n+\t\t\tbreak;\n+\t\t}\n \t\tif (conflict == NULL)\n \t\t\tbreak;\n \n", "prefixes": [ "v2", "2/5" ] }