From nobody Sun Apr 12 22:39:08 2026 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) client-ip=192.237.175.120; envelope-from=xen-devel-bounces@lists.xenproject.org; helo=lists.xenproject.org; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1 dmarc=pass fromdomain=epam.com); dmarc=pass(p=quarantine dis=none) header.from=epam.com ARC-Seal: i=2; a=rsa-sha256; t=1775742666; cv=pass; d=zohomail.com; s=zohoarc; b=K5hJcKEe+sweLXN+AX+FgV5KkxxxTIW64tHZ8KK6wct/BySt3D7k+aZvWTqDFLujdBO7godoQzHACTgirYvEp2PHflgh0v6cTfs4cmYSO2P9hmES8P+8qtFm5yRiZ0HfgUjI+qx6sa3rEiYiK4bHdtLBKaVGFXEpzRT9ij/e8Q4= ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1775742666; h=Content-Type:Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=ZJQmSlEhor8w37ZfEXBDxIsCyB/leW+DBAzgDszdUnQ=; b=QcHOLrYqHa3CJHSYJ1KRE335OwA5uBsgbcCnYZ2RJ5+FRjw9OoZzK/Y/8ItWiu0YygXTfx/kRcM3kqchysGy3Dkb6P4dg/Z2Xzn+XpzNO3DpHXXN455YZaXuhAxcd/Mp21WorroIVZ0NhecSJwyamA26uos0+d/qcvCd7TOAHlg= ARC-Authentication-Results: i=2; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1 dmarc=pass fromdomain=epam.com); dmarc=pass header.from= (p=quarantine dis=none) Return-Path: Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) by mx.zohomail.com with SMTPS id 1775742666269366.7656799376282; Thu, 9 Apr 2026 06:51:06 -0700 (PDT) Received: from list by lists.xenproject.org with outflank-mailman.1277413.1562614 (Exim 4.92) (envelope-from ) id 1wApmB-0000BK-JP; Thu, 09 Apr 2026 13:50:31 +0000 Received: by outflank-mailman (output) from mailman id 1277413.1562614; Thu, 09 Apr 2026 13:50:31 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1wApmB-0000Az-Dv; Thu, 09 Apr 2026 13:50:31 +0000 Received: by outflank-mailman (input) for mailman id 1277413; Thu, 09 Apr 2026 13:50:29 +0000 Received: from mx.expurgate.net ([195.190.135.10]) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1wApm9-00007I-DN for xen-devel@lists.xenproject.org; Thu, 09 Apr 2026 13:50:29 +0000 Received: from mx.expurgate.net (helo=localhost) by mx.expurgate.net with esmtp id 1wApm8-005hHg-PX for xen-devel@lists.xenproject.org; Thu, 09 Apr 2026 15:50:28 +0200 Received: from [10.42.69.8] (helo=localhost) by localhost with ESMTP (eXpurgate MTA 0.9.1) (envelope-from ) id 69d7ae9a-bab6-0a2a0a5309dd-0a2a4508a406-24 for ; Thu, 09 Apr 2026 15:50:28 +0200 Received: from [52.101.65.135] (helo=DU2PR03CU002.outbound.protection.outlook.com) by tlsNG-c1860d.mxtls.expurgate.net with ESMTPS (eXpurgate 4.56.0) (envelope-from ) id 69d7aea3-fab6-0a2a45080019-34654187b92d-4 for ; Thu, 09 Apr 2026 15:50:28 +0200 Received: from DU5PR03MB10263.eurprd03.prod.outlook.com (2603:10a6:10:519::5) by AM0PR03MB6132.eurprd03.prod.outlook.com (2603:10a6:20b:145::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9769.17; Thu, 9 Apr 2026 13:50:26 +0000 Received: from DU5PR03MB10263.eurprd03.prod.outlook.com ([fe80::8c9e:b301:61c0:3908]) by DU5PR03MB10263.eurprd03.prod.outlook.com ([fe80::8c9e:b301:61c0:3908%5]) with mapi id 15.20.9769.020; Thu, 9 Apr 2026 13:50:26 +0000 X-Outflank-Mailman: Message body and most headers restored to incoming version X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" Authentication-Results: eu.smtp.expurgate.cloud; dkim=pass header.s=selector1 header.d=epam.com header.i="@epam.com" header.h="From:Date:Subject:Message-ID:Content-Type:MIME-Version:x-ms-exchange-senderadcheck" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=rXT2lbc3X/fm+9bKFloG8300ettErwjeBOwUEqXy1RMYBe+uvL03Bsz2jTV9pRIjmag2o3w+MDhY87y+B2z3Lg+BQiO/HuJINWypXiiLB8Pki5mDzY8BNeYP5dTAxQHZbMzxXGkOq9LOa8nbK8ijaJHSziadxeonFcjk8xiisS1tDM6hazArWKgqSHku+2dxtxmPCbKe2bYDVYFWE6z69eOWsHe1CYjGMO41s0qYeCY1piVYr7sZeGd0DrvKLDiZ2xDt6pA2GjccGnCax/6rYE70CoxLd4yZoEK0kqZYmnp0rGOA8uqyN4XMMRSybB/VX1zfTGTYF11O0nJlq0JWlQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=ZJQmSlEhor8w37ZfEXBDxIsCyB/leW+DBAzgDszdUnQ=; b=f3PDGMfKHqH5J/FkNBeQm/8BL7lpENbIhD5VyvjtrGz/G3a+R1HR+Xudjr3mQTlNUPuI73p1ej6iZ7v6SaJxQnTDj1G/qrGDKjATSVHB3MJ56XbYmnUC/+zoFfXLMdR4K9xfWgsXjNaC3jxz4qzNHu4Ys7b47keQqcRvv98ecWYnuVgbAL+cSUkuJdKpwLh2bh4H4BCzwyEIB4SK53qPJ0USxXSFxfFP1ao3i7elRIHB5yiCTUyUtMsP/ZtaJrd53Vmu/uwysKxXmlthHTzjzpbL3o+XdVGSmGFRz/jf+TmIClHOjtAh+v8+KHZIFyK7V5fL6Y7n8wQoJEEeBAeA3w== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=epam.com; dmarc=pass action=none header.from=epam.com; dkim=pass header.d=epam.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=epam.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=ZJQmSlEhor8w37ZfEXBDxIsCyB/leW+DBAzgDszdUnQ=; b=NesIK6UO8ap3wXRt/fl96o7qN6X6nCOZxFN5IXxEoVnCGYX91cTKMccGrBiH/pekBPQP7htV83Tdgm45/LQ4Os3lMmRP9DntwGTIRYQRpuemMKChf+6cSX86ip/NcODqjf5cu7MqcASvtdve4QAVTb53rZsdOPavNHdR+xmZMneabNNJhRYAwVJLG1cliXJ0Hk1NdRkWD+ykeQ8KtSlWFMGWi/zvWQtWiQerlGGe2TOKhKFDH/+Ys6X939twX6Qd37vErdn7kYrRiae9JPcSI+j5ZfAezj3TuTWnghmvqobkuHssgkX7cggQyStaNulpSM1HY22c8OqrB4MP1KL5UA== From: Oleksii Moisieiev To: "xen-devel@lists.xenproject.org" CC: Bertrand Marquis , Julien Grall , Michal Orzel , Stefano Stabellini , Volodymyr Babchuk , Oleksii Moisieiev Subject: [PATCH v2 1/4] xen/drivers/char: fix SCIF IRQ registration failure propagation Thread-Topic: [PATCH v2 1/4] xen/drivers/char: fix SCIF IRQ registration failure propagation Thread-Index: AQHcyCfREdHUeOlrFkG05EP4j66oig== Date: Thu, 9 Apr 2026 13:50:26 +0000 Message-ID: <276f353ec36258dc545bef581389e60a162988b6.1775742562.git.oleksii_moisieiev@epam.com> References: In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=epam.com; x-ms-publictraffictype: Email x-ms-traffictypediagnostic: DU5PR03MB10263:EE_|AM0PR03MB6132:EE_ x-ms-office365-filtering-correlation-id: 16f0f59e-698a-48f6-cabc-08de963ef43c x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0;ARA:13230040|366016|376014|1800799024|38070700021|22082099003|56012099003|18002099003; x-microsoft-antispam-message-info: Lx3kwz7BWIHYmWzK2VomXitW+3ufFHDNQ2KsmGr+V8no/zy5snDIHzSP/vbhBtSDY2TyZCrhARkM2cHhCx/fOmcluL6Up5IA7AySrGaMlN1q8CrsGglOEcR9/7g3RjJQcDrk5JKE6Adeamxk9J3J2tzCgASvi8uhnU5vPJBiji1jSrFjKvBXfWuFSm8h9bhHYsKmZoi+8CyWNvl9aeYVQsb9UmOPUAklzoxbhmD/T+HeXqJgUUZhzeydgYqFzg9RkBr2XqDqIo0TMdzbXNjqg0c8OHrWkUDQzX+oZHDNeHD0Gwp0ujVFPlovbWfsON4KVnG/kMfRHOo7Jyuh0qBFnZ72yYV0mLmzpsg9opsPYm0uKeG8TOXgWrFvpozGkWppERzHkgr5/0PLfHLTe9IDZqgJlD8SXfbQmXRltT7XUHAOYSL0U6N0l5OruaxZFpTFv8giLAl4ng0zj+cyJXsqwyoNFKzm7GKqsgobhDBgrBQZpcvK5KaUsQXonRD7ilpnqes4GBnNPuumwVr5erBGY0PxSWVf0NV3hvHSenpdHGalx9+8O3zZBkjNQzpBCK9HIcL0uhZumK+NdtMMPiVp7zDsqmAvTPhsAMdVcWPw65QQfgz4lCkMXDzLdSO1ygElEDkyhjo87Tw1ftXnCo4zmI8n0mMqnM8Ftm3NwfrnG38VT/cU3eK9XbsLQyw2+ZWW/VK2Q762ic2xfqtSHadnYwMSFYt8AY989Vl3d4t5ObTVneWeJO18rJDE8nXUfYgWzbfWhp+OXP+m8B3BPAskFoKmTNwUjZRd/YYxpZe4u1c= x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DU5PR03MB10263.eurprd03.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(366016)(376014)(1800799024)(38070700021)(22082099003)(56012099003)(18002099003);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?iso-8859-1?Q?WuZypSncnQ5s+MFB583ydNyjY+Xqx9+oM+nnpqwWXl5U4KfTCsQ/+Etjor?= =?iso-8859-1?Q?Fgt2hI7UwdVDrhUipNQKRTHTAm9feX9j1jutycaKa9VXmcV6raundNp5Pm?= =?iso-8859-1?Q?naTzTb25DhUDUN/rhd4G/Mo7HiqwBEvA0LWO3yuvXS48y83aDsqDArkm0g?= =?iso-8859-1?Q?eJosNz/U+GLdAJG+Ecz8+SIFOmT6fgTJqfAmOGtvfs0Bt3Ymio4bzZxkHJ?= =?iso-8859-1?Q?iRWZmEwRBurIFWoWq9fgutbGXC2TO3XwdGvDxNLGh57oLZdH0lJSplIgF5?= =?iso-8859-1?Q?IgsNBCgPnwGBednGrVx2LIB9o6tq0rCRhGxYsJMamxrgnv0vgZKOx5isyF?= =?iso-8859-1?Q?1/GGAqxGoEv75fKGvF5gdTMFtAT+EL3ngyDdtm5peba95qctui5S2+35kJ?= =?iso-8859-1?Q?Hkf0nwcclVO03jUoJhQ6AguS+7uPKBJ0nEEIrWATB2MbJWLuYZi2eiIeI6?= =?iso-8859-1?Q?VBBXmBSOd0b+c5ATh2acWLUM26eNemaHR/dL/5h9eaP6cEftRXKzp1lR8D?= =?iso-8859-1?Q?eIw3wooifT8O4OUgb8pvdKWt6iju+2e9W2mOBAL/I2qagiWdjkRrkKL67H?= =?iso-8859-1?Q?6jVSKo8ph74r7MnLZFtL1YY23Tpq0n/XrFkXPqM6N4LsYRVd/tfAJXFtDe?= =?iso-8859-1?Q?UrNLgbnybzbZpNOLUEXL1nxAkUNO6Ddxgrv4VoA3n8GFUhQsrucHtAvqkH?= =?iso-8859-1?Q?sGEKEB4aA8AszFPDRKaCSRZ10dOvRyYNVvQAFAefy1N2lyRIDb3cpMHpNE?= =?iso-8859-1?Q?GiPEKLypTOXTQkX7oSgPRRt+yQrUvEo3HSiUXNqr+GtTijdSYlJPHj1kL7?= =?iso-8859-1?Q?QKvWN2e6MDsqATV1eLfQxzY2eDMCEgogIdJScsnsL69t9HQ9aAWWzRlZRk?= =?iso-8859-1?Q?6GnmsFkXIOYrrG2dIZWgO5AVYuCBQ6sEh3OL+YWcZunVE2icGgnUZwN1ZZ?= =?iso-8859-1?Q?f4iydjA+hOZ6mGD8Vl6c1+9VtJwYo8Ya7m64SXEdksSLSsG7nl89OZ9jiP?= =?iso-8859-1?Q?Vb+RKEGrlf8fPDDXO5agn8rzhU4JQvtUlI7az04wQgkyIGOA5aMzlx91Dg?= =?iso-8859-1?Q?49K0kTDAUoyEqiRqf6OpCR0XzLwmi3rla1+DAem84iIoMC0hx15K2o+n3n?= =?iso-8859-1?Q?JIPLYR7GZORLvI1e1s+n4glUTl9jMcMewQth3ZMponmTrrjeGqvGWYwte7?= =?iso-8859-1?Q?Cw3wbe9B/C+ITDf/qo9XbGOG+jCxiMIM+BxWIgRTkZlY1Yuq/ZwuIoFcNs?= =?iso-8859-1?Q?FUYL8lTEhPptYNnFFD5d9XBdqpWrAXM/CiZUXXs2WUtbMTjkt0Sn9l6e28?= =?iso-8859-1?Q?awF1Q3Yb9JQG93J0Eg8KxOCoU8ksQ/8R4OPvbr0/unyaV5djVw+RJMdxQz?= =?iso-8859-1?Q?JelcV0q9Y+G5w+zyT78SV8/6p4AAjpRHEWOb0fnbHkc2vvdCk+eSySqfx+?= =?iso-8859-1?Q?CEjBBZsRQy4GdNVd+BseQ5P40vTTf5NtpELFf67WU0m/I7U+FiHv4DmPDp?= =?iso-8859-1?Q?qPKGSwBoBAIXC3/3Lb752MlBq//Ar7xvcZLykuKTm5FvuY44nygYs3Otsr?= =?iso-8859-1?Q?VFPPM5pYF3UkP/mCEHkNITuaybSaVJLgxyIAO6rvciSmLuRIhd6cMvoL2j?= =?iso-8859-1?Q?waRBffZwXqc4R1sht1mZ/aQqPymupXuf5gTfrk/dddVPnA6NfJdye61D7S?= =?iso-8859-1?Q?Iw+JrVlvUqiO5IJhLtRwM2w+XjDgCv0brfcnMNbJsYivjlcCQkzm4pVHOC?= =?iso-8859-1?Q?AMGlHGR4YBq1rHj95gzP5MBK9o4KRcAVyLTP9pJV5YqiIogIcL7GgA0lFa?= =?iso-8859-1?Q?zHH8NYWeLiCnr40SBGQiV6iy/pDaN9k=3D?= Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-OriginatorOrg: epam.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: DU5PR03MB10263.eurprd03.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 16f0f59e-698a-48f6-cabc-08de963ef43c X-MS-Exchange-CrossTenant-originalarrivaltime: 09 Apr 2026 13:50:26.1973 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: b41b72d0-4e9f-4c26-8a69-f949f367c91d X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: BpFa/9/axK9PnGrP+eIBjYjHjpp0Nd41/OfIGhufHZxZHCn+BEpCUY1W2XulPRRuXua+AZ09irkjz9LMC2/E7pKi3J6bV0BMVLFxrfBeOG8= X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM0PR03MB6132 X-purgate-ID: tlsNG-c1860d/1775742628-7455F497-4396C1B6/0/0 X-purgate-type: clean X-purgate-size: 3017 X-ZohoMail-DKIM: pass (identity @epam.com) X-ZM-MESSAGEID: 1775742668188158500 Content-Type: text/plain; charset="utf-8" In scif_uart_init_postirq(), when setup_irq() returns an error the failure was only logged via dprintk() and execution continued, unconditionally writing TIE|RIE|REIE into the Serial Control Register (SCSCR). This armed all three hardware interrupt lines (TX FIFO empty, RX data ready, receive error) with no handler registered to service them. On platforms where the GIC receives these asserted lines, the result is either repeated spurious-interrupt warnings or an unhandled interrupt fault. The fix adds an early return inside the error branch. The interrupt-enable write to SCSCR is skipped entirely when no handler is registered. SCIF TX continues to operate correctly after this change. The Xen serial framework never calls serial_async_transmit() for SCIF, so port->txbuf is always NULL. This causes __serial_putc() to take the synchronous finite-capacity path, which polls the SCFSR_TDFE hardware flag directly and does not depend on the interrupt mechanism. RX wouldn't work if irq wasn't registered. As a secondary clean-up, the hardware error-flag clearing sequence is moved to before the setup_irq() call so that error bits accumulated since init_preirq() are cleared unconditionally, regardless of whether IRQ registration succeeds. Signed-off-by: Oleksii Moisieiev Reviewed-by: Michal Orzel --- Changes in v2: - Extend fix to pl011, cadence-uart and exynos4210 - fix typo in patch 1 description xen/drivers/char/scif-uart.c | 16 ++++++++++++---- 1 file changed, 12 insertions(+), 4 deletions(-) diff --git a/xen/drivers/char/scif-uart.c b/xen/drivers/char/scif-uart.c index 888821a3b8..673a2d3800 100644 --- a/xen/drivers/char/scif-uart.c +++ b/xen/drivers/char/scif-uart.c @@ -187,16 +187,24 @@ static void __init scif_uart_init_postirq(struct seri= al_port *port) uart->irqaction.name =3D "scif_uart"; uart->irqaction.dev_id =3D port; =20 - if ( (rc =3D setup_irq(uart->irq, 0, &uart->irqaction)) !=3D 0 ) - dprintk(XENLOG_ERR, "Failed to allocated scif_uart IRQ %d\n", - uart->irq); - /* Clear all errors */ if ( scif_readw(uart, params->status_reg) & params->error_mask ) scif_writew(uart, params->status_reg, ~params->error_mask); if ( scif_readw(uart, params->overrun_reg) & params->overrun_mask ) scif_writew(uart, params->overrun_reg, ~params->overrun_mask); =20 + if ( (rc =3D setup_irq(uart->irq, 0, &uart->irqaction)) !=3D 0 ) + { + dprintk(XENLOG_ERR, "Failed to allocated scif_uart IRQ %d\n", + uart->irq); + /* + * If the IRQ handler could not be installed (setup_irq failed), + * do not enable TX/RX or error interrupts. Serial transmit will + * fall back to polling mode. + */ + return; + } + /* Enable TX/RX and Error Interrupts */ scif_writew(uart, SCIF_SCSCR, scif_readw(uart, SCIF_SCSCR) | params->irq_flags); --=20 2.43.0 From nobody Sun Apr 12 22:39:08 2026 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) client-ip=192.237.175.120; envelope-from=xen-devel-bounces@lists.xenproject.org; helo=lists.xenproject.org; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1 dmarc=pass fromdomain=epam.com); dmarc=pass(p=quarantine dis=none) header.from=epam.com ARC-Seal: i=2; a=rsa-sha256; t=1775742658; cv=pass; d=zohomail.com; s=zohoarc; b=LSI0y3qFs6JuHvxjpg+E34SKXelrcpBFSaEkf3IJEBqtGTZX8Fwuw1ZZaRtXlxnDAeoWvecnFyYjk+GdA6MKBzIBNHjgbMh5nNVukwXt499H+YlcE2KvKo4YmX11GHjgP4y2WYcDIM0EzaVdeHTSP1cbX/6z2Flx7jY8B6M/Iuo= ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1775742658; h=Content-Type:Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=oAfXz+4yU574RkHhJeRj5xLLKToN+/aJ1HqmB1NjL18=; b=YZF7rS36HANoMimhlNnHVErz21L6h+64HMcoy07W5JANi/GgTGMzhCzxCIILOMRvWovdS6JQcXWbYtqDiolHjtf3+iCqFWCHjUKIfvJuxyD2ua/AuWJSs+6dcT7naIXEMQHQPHebm8xVAFxSBzAPd65BgW2D3xwzKdhJYiG/MH4= ARC-Authentication-Results: i=2; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1 dmarc=pass fromdomain=epam.com); dmarc=pass header.from= (p=quarantine dis=none) Return-Path: Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) by mx.zohomail.com with SMTPS id 1775742658067589.4077837538243; Thu, 9 Apr 2026 06:50:58 -0700 (PDT) Received: from list by lists.xenproject.org with outflank-mailman.1277414.1562620 (Exim 4.92) (envelope-from ) id 1wApmB-0000Ha-TL; Thu, 09 Apr 2026 13:50:31 +0000 Received: by outflank-mailman (output) from mailman id 1277414.1562620; Thu, 09 Apr 2026 13:50:31 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1wApmB-0000Fr-NP; Thu, 09 Apr 2026 13:50:31 +0000 Received: by outflank-mailman (input) for mailman id 1277414; Thu, 09 Apr 2026 13:50:29 +0000 Received: from mx.expurgate.net ([195.190.135.10]) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1wApm9-00007Q-NA for xen-devel@lists.xenproject.org; Thu, 09 Apr 2026 13:50:29 +0000 Received: from mx.expurgate.net (helo=localhost) by mx.expurgate.net with esmtp id 1wApm9-005hHg-2v for xen-devel@lists.xenproject.org; Thu, 09 Apr 2026 15:50:29 +0200 Received: from [10.42.69.8] (helo=localhost) by localhost with ESMTP (eXpurgate MTA 0.9.1) (envelope-from ) id 69d7ae9a-bab6-0a2a0a5309dd-0a2a4508a406-28 for ; Thu, 09 Apr 2026 15:50:29 +0200 Received: from [52.101.65.135] (helo=DU2PR03CU002.outbound.protection.outlook.com) by tlsNG-c1860d.mxtls.expurgate.net with ESMTPS (eXpurgate 4.56.0) (envelope-from ) id 69d7aea3-fab6-0a2a45080019-34654187b92d-5 for ; Thu, 09 Apr 2026 15:50:28 +0200 Received: from DU5PR03MB10263.eurprd03.prod.outlook.com (2603:10a6:10:519::5) by AM0PR03MB6132.eurprd03.prod.outlook.com (2603:10a6:20b:145::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9769.17; Thu, 9 Apr 2026 13:50:26 +0000 Received: from DU5PR03MB10263.eurprd03.prod.outlook.com ([fe80::8c9e:b301:61c0:3908]) by DU5PR03MB10263.eurprd03.prod.outlook.com ([fe80::8c9e:b301:61c0:3908%5]) with mapi id 15.20.9769.020; Thu, 9 Apr 2026 13:50:26 +0000 X-Outflank-Mailman: Message body and most headers restored to incoming version X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" Authentication-Results: eu.smtp.expurgate.cloud; dkim=pass header.s=selector1 header.d=epam.com header.i="@epam.com" header.h="From:Date:Subject:Message-ID:Content-Type:MIME-Version:x-ms-exchange-senderadcheck" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=ZEfJF4ynxgz78H1SxcSfW8YAO81vVYou7L3/yqCJ9By+AdlBgJDI85NWr1y//DKwfEa1aDxPyx2OkwTBVjw1cQjlQs7CvGFJxzTUD6bJI835auTSTeK7sE2ubxXc5PSxq4ubGJnZwcqZjQlPBgUsiWQn6dgikC87MK9T/qqmtZihB27M1IOboMRhZmc0EJGf/hX/VbbA/0DXFEUx1TXo/ObPQX2GTg5JdfcFSguNw4nxTy9KANDFYhVWBpHMVBpOC0PtpUhDfqTexb2sRG9bHk7j99EigABQMRVHH1hWuR1A4+/Kzdp7VTgMTXSVV8cRB7B7R5vkliqhm4wZ1Hqivg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=oAfXz+4yU574RkHhJeRj5xLLKToN+/aJ1HqmB1NjL18=; b=lBFWPfHaIA9xCfDpBnKdi7P+xkmArhsVhfSbBUACKq+vsv7MaCpNgpaZOclRTdo48D5taV9/5riGPl5z4PhcWUNlfJlVkOTlVlk59FxdsFH9sTt8d8zVRe6kjNctdIU7NodHQxS6OSXjltgxqgSrtbX0yKiNyn99BWEJ5cE600xGvkgNj43DcXV88KOjhcsh2ZkstL12T9yzMD8v8Or6jdDvluCQ5CsKEk3nS4gLmgGOTKWLunAvv3Ikf9Kf+J/XUoIYXFLkk5b4Hy0IijU5YpeM6uzkaaIMjiSA1OZh/c0gjS5PbVcMxKMULL2tIXkmqoiMsLv1SvxoqgKFcQ2IsA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=epam.com; dmarc=pass action=none header.from=epam.com; dkim=pass header.d=epam.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=epam.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=oAfXz+4yU574RkHhJeRj5xLLKToN+/aJ1HqmB1NjL18=; b=IrwzPCUHzxmG0xos1tr2bSlrobF1JZ/zyStLP42/C0YyZIa55mRizxR8rILdmgCU1QDpwrfbcbhe403KSVYE6/ee5GhpFnFGrkdlMjEwINHcrEbYJK0I9s5sXiHagvHJHZSC+MuijkdrrvMHa3k5GpAjYMR2A0lf8fZyNdHCfTPUdKbF8e9mo1b3o9IfmBuPCKA1Lym09AcuqM5i55YGrWofXkKBWTfHFsYB9dJR9SsYJqhzcpAlq9o4VqdPuGTEfZ1RwvrBfJy1DBDKQRufiq2eEXS3calQ9jaayCKZgY4/Qmqgr6bTwhcgcK0FizHYovzwyXnslIGfh2US7s/GaQ== From: Oleksii Moisieiev To: "xen-devel@lists.xenproject.org" CC: Bertrand Marquis , Julien Grall , Michal Orzel , Stefano Stabellini , Volodymyr Babchuk , Oleksii Moisieiev Subject: [PATCH v2 2/4] xen/drivers/char/pl011: fix IRQ registration failure propagation Thread-Topic: [PATCH v2 2/4] xen/drivers/char/pl011: fix IRQ registration failure propagation Thread-Index: AQHcyCfRI6UUL+j1Mk2LAfnsFV7fPA== Date: Thu, 9 Apr 2026 13:50:26 +0000 Message-ID: <9148b1721adfacf20c5621eb583f6fe87dead767.1775742562.git.oleksii_moisieiev@epam.com> References: In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=epam.com; x-ms-publictraffictype: Email x-ms-traffictypediagnostic: DU5PR03MB10263:EE_|AM0PR03MB6132:EE_ x-ms-office365-filtering-correlation-id: 588abf02-8138-40f9-2dc5-08de963ef465 x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0;ARA:13230040|366016|376014|1800799024|38070700021|22082099003|56012099003|18002099003; x-microsoft-antispam-message-info: bq/IPq+gM8Q4WGk61Vyhx6odZd5dQ/kue6bSmt2osxz19XcHf72joTWDhZY+Ekpg/l0+ndj1XOP+FdMPZjqkkBETFFtA8jQlhZZrsFkKiy07oLGZcDk/l0vPS9povQN+R5p8IzJjTHDuzJ6J2iTGTHEdqK+HS0hFlBOI3m2PDU5W3uU3rcDkzbacZL/CCFmVB70/NMmgi1uLec2S6j5T1JtuqwC2SuMuNz0C/u3sCLTHeW6Q1foqQ2T+4EPY0HMV7aDvLmmSK76PkybHOaSzMIyc7ge4lwm9u3OgNYp4Az/7+91Pafit43w09F5RmOGnh/c2KOrsNXRTUUhIFuRGTG3r3xKMnyqtLSBccy1apEqCFjDvNRqRQBFo9Ha8dGfT1uFw8ADnto6hmtp1yvqOwwSBlGS2Wc0tCBfr0b9UJ224a188vvHyrpsWPQ04YHvJaWkUFALDoV/Ygq4o5Rux4OCujx3waFWMOPr0n8bKVv+91twkxeCya+dxCwLECXRuSrMEwcCZJJ7TkBcqua4BDeNUH8vmUxQAzhDdFEqS3Q9ZXJJluTjZ9KFjDhrGnU602zT3f31AHXzntu/k6kJPyfRc7YH+bpT47LFDvodLI2wEPJZ2pw18wd9WzugKC4+EHSTZyHdhydcAzutNxjwh2HxkBFYodwjBs9qQH3Lj6xpcP/RvKkY1RJmeeD6zy53GkpVWZsj18uZSoitcuRUJk9GM3w54wZ2+cjYlCVea+RkT0oKQrnPuakVfYqn685HL+EHdm8y09DUGdHxA+YyCRzGyzmcSTqwSSwSp2FKnwUw= x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DU5PR03MB10263.eurprd03.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(366016)(376014)(1800799024)(38070700021)(22082099003)(56012099003)(18002099003);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?iso-8859-1?Q?evqDq4NKjZ11jID0xRJLv1M4athjaJ0MQzHAMTpvNCN1CxVm0tr2Rn3bVw?= =?iso-8859-1?Q?IiuY/LwdHIIfvj1XIIaaSgG89C5wgHZIx1G3M1CRBwWxXUollQT38yQ9t1?= =?iso-8859-1?Q?BJxK53QP8VHcl0Mom1EoLD2SwvsYGvKxHXiWUS8w+rH75B+yDDdCTG1L/p?= =?iso-8859-1?Q?8cGEtMfZHzCoorh6TlZfvlxsZv+81MgM+2POSon+Tn7GHSBO+YOJwYE38c?= =?iso-8859-1?Q?kL4ljvCRgsgrLNl8MpRnk/6/uicWB4CpglmZEmm/u7Ht2J42S8EaZ865b3?= =?iso-8859-1?Q?n2dSh1fuDGnH6f/FY0ltHJhf/m5gRpwxMFKjA+bU8YzvL6RwE08xQNgnN0?= =?iso-8859-1?Q?DDdjSjOOog5hq+7sC9sjMvxOGA5qdUHPMHIgdiublpzmpFYlleYEMGz54R?= =?iso-8859-1?Q?STREeu5M5HffIVMG1wiVGYB4vzeTk8B3NVIVBXKlYnBdZjEyJicFoVJ12G?= =?iso-8859-1?Q?ltWhWkKgWKXbi9yzmh2zdVpkmzKU+doFh1rbClxbGiDNVvDb6FXmrgtpOp?= =?iso-8859-1?Q?B2bJ3fiHc55MaLnv1+VlxGsH9Vav0bQSC2ntWI2d158T3TcNKU1G317iDg?= =?iso-8859-1?Q?dwJOMNUxByzkkevNcocEGXVTfbr6edhoMdkMmbbxNnYETQG70Wj1K6542t?= =?iso-8859-1?Q?uSHQ6zpftVDnbb4ge0wBg5eHAycNIGtEZ8Ifm+sYA8kiTMdHCQh6RIK4zH?= =?iso-8859-1?Q?w4CRm6vX5+y/jWXqiiUnlPKv9bHqjkOXLCZGzHhFLzjK9DDiHMkf5Bez3v?= =?iso-8859-1?Q?fVGQMPDNdCNiCd6Jov93WVZ6w9j8jbdnLLJ7YkBQ+1xMQ+wB5zpwAyjFLf?= =?iso-8859-1?Q?z5zWYfCK2crF0u/x9DmW9zOACRlm4C3ifmw9V62HAR7FJAj1Q3PPh+85rH?= =?iso-8859-1?Q?ynnqiAMunGDH/s8CE9YweHLza410/Y6MIfA73RZrSXjVeZ1RgLEpw0eGJI?= =?iso-8859-1?Q?MtpFOo0t0JD9OEEzoMLL/RDlRMZyMOfRqWuk7bwrt2sMPfaz2ElKs30mL0?= =?iso-8859-1?Q?f4RC7n0UywlA5JZ6jK/cAbXHF3YD5/R+B9sJ1sdon2c5GHSqQJwLAh06HA?= =?iso-8859-1?Q?slvhSfw5/U9Fe+ZaiVwcp7JUm6GkzM+SIvOjoeddBcrJ7CaniuUF/l68oM?= =?iso-8859-1?Q?CNNEotlu/4mgn+vAQZJ1kP5zoWMe2LHID1A1O70bhdfSt2sRBP6N2D4ww2?= =?iso-8859-1?Q?T4eDI/qEoLb26SZdWCctCwXMrUDLkt6yUoD/JM7L8NsLdlTNSdmc9FG9iK?= =?iso-8859-1?Q?qKoqDKSY0a4z2gCLivSpcufsL0fv7S59NFz2yFaR4MFmPSDfERmI1R5ewT?= =?iso-8859-1?Q?0Am+sWhQ6inyydrJC3IYklzzt+2ZXedftlCZOTrs9DFImD2/UeUoAcAQOb?= =?iso-8859-1?Q?FkQXRhHDjuisdVpktNnS3xqR/ZpKVtKgs6jk73vf4og7j0bGrRg5MLXy4U?= =?iso-8859-1?Q?cM8QiEqLCBwQHAsplbolMIDyv0MujRhwpHh2ekh52WgOm66CwXIMtUUB2u?= =?iso-8859-1?Q?HiDzPg3hEE6s00aXn2257i03U6S7nEyYROwBDWvK3I0wiq14zXZ5b6uUcP?= =?iso-8859-1?Q?rW3wPFq8bPAbmth8QZHrhf+BHqhPah/iUty8IOq1z25Oe1LxnHRW94yONQ?= =?iso-8859-1?Q?uqORf2gu+RtYF723h8tg/SLXXpkHfTUdDKjZSYXg2dwD5lq0Cal3anxAH0?= =?iso-8859-1?Q?FoQY/idYH30Tn/jYr3ZI7AV1G+eLrumIn+UlHQpNDWmSQGBZIGiaKJ3i24?= =?iso-8859-1?Q?IceGGNr1ln1p8AUvUbwu0zGAcGXYiHWhTxkJvW2lCJOW5kBCTgcJL65gLq?= =?iso-8859-1?Q?p/y2HTW45Iavl8/ClebOWMUZfW8PjE8=3D?= Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-OriginatorOrg: epam.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: DU5PR03MB10263.eurprd03.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 588abf02-8138-40f9-2dc5-08de963ef465 X-MS-Exchange-CrossTenant-originalarrivaltime: 09 Apr 2026 13:50:26.5565 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: b41b72d0-4e9f-4c26-8a69-f949f367c91d X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: iYjlsjc5YKfVLkPX7juPbDHSlOjkQg2VEhGt9luVc/PKjLxmUVkFcWHyQPb2mcXOSGAfQzEAzknYtx8brKrAvfMxLGuLgw4k97D/wGSSvb4= X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM0PR03MB6132 X-purgate-ID: tlsNG-c1860d/1775742629-74360497-90379B8F/0/0 X-purgate-type: clean X-purgate-size: 2372 X-ZohoMail-DKIM: pass (identity @epam.com) X-ZM-MESSAGEID: 1775742660265158500 Content-Type: text/plain; charset="utf-8" In pl011_init_postirq(), two code paths could reach the interrupt-unmask write to IMSC without a handler being registered: - When no valid IRQ number was provided (uart->irq <=3D 0), the original positive-condition guard (if uart->irq > 0) skipped the irqaction setup but still fell through to the IMSC write, unmasking RTI|OEI|BEI|PEI|FEI|TXI|RXI with no handler installed. - When setup_irq() returned an error, only an error message was printed and execution continued to the IMSC write, arming all hardware interrupt lines with no handler to service them. On platforms where the GIC receives these asserted lines, the result is either repeated spurious-interrupt warnings or an unhandled interrupt fault. Restructure pl011_init_postirq() to use early returns: return immediately when no valid IRQ is provided, and return after logging the error when setup_irq() fails. The interrupt-enable write to IMSC is only reached when IRQ registration succeeds. Signed-off-by: Oleksii Moisieiev Reviewed-by: Michal Orzel --- xen/drivers/char/pl011.c | 17 +++++++++++------ 1 file changed, 11 insertions(+), 6 deletions(-) diff --git a/xen/drivers/char/pl011.c b/xen/drivers/char/pl011.c index 5f9913367d..918b9d4d4a 100644 --- a/xen/drivers/char/pl011.c +++ b/xen/drivers/char/pl011.c @@ -150,13 +150,18 @@ static void __init pl011_init_postirq(struct serial_p= ort *port) struct pl011 *uart =3D port->uart; int rc; =20 - if ( uart->irq > 0 ) + /* Don't unmask interrupts if no valid irq was provided */ + if ( uart->irq <=3D 0 ) + return; + + uart->irqaction.handler =3D pl011_interrupt; + uart->irqaction.name =3D "pl011"; + uart->irqaction.dev_id =3D port; + if ( (rc =3D setup_irq(uart->irq, 0, &uart->irqaction)) !=3D 0 ) { - uart->irqaction.handler =3D pl011_interrupt; - uart->irqaction.name =3D "pl011"; - uart->irqaction.dev_id =3D port; - if ( (rc =3D setup_irq(uart->irq, 0, &uart->irqaction)) !=3D 0 ) - printk("ERROR: Failed to allocate pl011 IRQ %d\n", uart->irq); + printk("ERROR: Failed to allocate pl011 IRQ %d\n", uart->irq); + /* Do not unmask interrupts if irq handler wasn't set */ + return; } =20 /* Clear pending error interrupts */ --=20 2.43.0 From nobody Sun Apr 12 22:39:08 2026 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) client-ip=192.237.175.120; envelope-from=xen-devel-bounces@lists.xenproject.org; helo=lists.xenproject.org; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1 dmarc=pass fromdomain=epam.com); dmarc=pass(p=quarantine dis=none) header.from=epam.com ARC-Seal: i=2; a=rsa-sha256; t=1775742658; cv=pass; d=zohomail.com; s=zohoarc; b=VKehTnEyLuzyGG7DSBbxAA6SDhEyGhuRJmyVe43Gt+7FDvRbDywMVq9uFcMMijiRSW08cIfs9jOTUnnb/BYIX9is0kJN6419s2dshPfInSYtbmLpVO808D/u84GXPjy7tRCJh/QpmRUEJ43gITO3nEg30xxVHpOrCVvlFYcxMOY= ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1775742658; h=Content-Type:Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=ZwhynW7zmXJxd8AYQ96d1CxfvYTLwAQ0yYVPWGCG7bc=; b=HXyAZkBxT1+yuNl9qkj6Ho6gdqXNeY/uOmVT4wxy4VHW1h6tPd01V8TEEevzQ/VkVe5DZyRxwQkZqHKXP/clJgzVI3aeV0zAPh1tFpEI7NTgA3YQeqCUw63ht+7Ln9PP5KS0Awym7f7fv/SfMS1MT/kXlrlA62GnxeHONNaKX4Q= ARC-Authentication-Results: i=2; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1 dmarc=pass fromdomain=epam.com); dmarc=pass header.from= (p=quarantine dis=none) Return-Path: Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) by mx.zohomail.com with SMTPS id 177574265880996.8050892236455; Thu, 9 Apr 2026 06:50:58 -0700 (PDT) Received: from list by lists.xenproject.org with outflank-mailman.1277415.1562625 (Exim 4.92) (envelope-from ) id 1wApmC-0000Pz-62; Thu, 09 Apr 2026 13:50:32 +0000 Received: by outflank-mailman (output) from mailman id 1277415.1562625; Thu, 09 Apr 2026 13:50:32 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1wApmC-0000OZ-0y; Thu, 09 Apr 2026 13:50:32 +0000 Received: by outflank-mailman (input) for mailman id 1277415; Thu, 09 Apr 2026 13:50:30 +0000 Received: from mx.expurgate.net ([195.190.135.10]) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1wApmA-00007a-3f for xen-devel@lists.xenproject.org; Thu, 09 Apr 2026 13:50:30 +0000 Received: from mx.expurgate.net (helo=localhost) by mx.expurgate.net with esmtp id 1wApm9-005hHg-C5 for xen-devel@lists.xenproject.org; Thu, 09 Apr 2026 15:50:29 +0200 Received: from [10.42.69.8] (helo=localhost) by localhost with ESMTP (eXpurgate MTA 0.9.1) (envelope-from ) id 69d7ae9a-bab6-0a2a0a5309dd-0a2a4508a406-30 for ; Thu, 09 Apr 2026 15:50:29 +0200 Received: from [52.101.65.135] (helo=DU2PR03CU002.outbound.protection.outlook.com) by tlsNG-c1860d.mxtls.expurgate.net with ESMTPS (eXpurgate 4.56.0) (envelope-from ) id 69d7aea3-fab6-0a2a45080019-34654187b92d-6 for ; Thu, 09 Apr 2026 15:50:29 +0200 Received: from DU5PR03MB10263.eurprd03.prod.outlook.com (2603:10a6:10:519::5) by AM0PR03MB6132.eurprd03.prod.outlook.com (2603:10a6:20b:145::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9769.17; Thu, 9 Apr 2026 13:50:27 +0000 Received: from DU5PR03MB10263.eurprd03.prod.outlook.com ([fe80::8c9e:b301:61c0:3908]) by DU5PR03MB10263.eurprd03.prod.outlook.com ([fe80::8c9e:b301:61c0:3908%5]) with mapi id 15.20.9769.020; Thu, 9 Apr 2026 13:50:27 +0000 X-Outflank-Mailman: Message body and most headers restored to incoming version X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" Authentication-Results: eu.smtp.expurgate.cloud; dkim=pass header.s=selector1 header.d=epam.com header.i="@epam.com" header.h="From:Date:Subject:Message-ID:Content-Type:MIME-Version:x-ms-exchange-senderadcheck" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=POqCbvsD7rN2iqWa1gkkj+Ssb5ui9sePq+w3pWZl+dPUU9fUOhRNbYaOOE9G+O8DxZ83JTb/hdXQ64UsLsLJrRYTdhdAatA99ylyhUMf6hHlRpXcHS+EhI2WA/eiDCMxnr+m2HYouAWS2VuI/CPh9qL+/SvL7QwvPtE/1y90cJ3baHcgqjYVTft8dKyYusodB0z1ohXZ2SimQl3Sq6COwUt3qT6/2nXPiEXlBc4+DrC3ktPU97/ifwXVmQLEO+jVwDTHWndo8TbaGmGEPzwgMYC93Lf2YIXRl+zZjytaRUJCnmVhG+oSfnJqitcRnZQ1OnhDM3ZZFiGELqte5ym00g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=ZwhynW7zmXJxd8AYQ96d1CxfvYTLwAQ0yYVPWGCG7bc=; b=btqjX/tAeHyLq2ihsE63HkZlTWrw46lh7u3dg61anVXZIxd3a408JOVAELDG3nKDVC7sBSJWoG38hVygea9yPbRWel5g+kghKHvP5PNbVvxCc2e53zQtyOhSXwkeQ0FB2rPJB9BRcjqpTLh+zkoh+MaFAdhh6w25l9S6HhQaa1lxbf0AytARln2S821jUkOHTyd6OdyCXGb8E1yEBhY/9vN4yNXQxrbQ5ElXUqS3cVe6MYhTCWh0Ut3T87Biw5vPpZPL2G+MDzI7QMfCUuPVmgyim5gGXVzw5hGvxXkqSDWK6ZNuJ4cAhWSB18rdhXHYLQ66++UAgHPRBh0GRs6j8Q== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=epam.com; dmarc=pass action=none header.from=epam.com; dkim=pass header.d=epam.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=epam.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=ZwhynW7zmXJxd8AYQ96d1CxfvYTLwAQ0yYVPWGCG7bc=; b=iXzjGYRAgC4/jdGGy3K8GZtDYrkbqOoATlI30ty119ip8xCk28RlhYWrPfxlcG/vqFxHMri7rO0g+XN/+G8+xWscb7AJ/p97dj3ugXBkMS/Csg5VMlodPEndu7Sm4Wjhmi3dc6wbKGNZ4I4HN1cb9YflZ7tW6Ck/Yk5devRwpQtfOCAnQy7eI7ZcQmja9gvvnY4xBivzSf+ggIhUwrlV1p7JR8o2aZslbxnU/Yn03PAUnCGEHM3XLMwYCPrp4fMIHcvup+lMg5Jmbb5t1tMMtpGSl3rBGPTr6Exs2Tyf0RF0VS+v7Mi0z0t3Wu8iOyRRTP/H+sUvi7+jENRwiKBcmA== From: Oleksii Moisieiev To: "xen-devel@lists.xenproject.org" CC: Bertrand Marquis , Julien Grall , Michal Orzel , Stefano Stabellini , Volodymyr Babchuk , Oleksii Moisieiev Subject: [PATCH v2 3/4] xen/drivers/char/cadence-uart: fix IRQ registration failure propagation Thread-Topic: [PATCH v2 3/4] xen/drivers/char/cadence-uart: fix IRQ registration failure propagation Thread-Index: AQHcyCfShWUf7VBEr0ifQc+UCSxOog== Date: Thu, 9 Apr 2026 13:50:26 +0000 Message-ID: References: In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=epam.com; x-ms-publictraffictype: Email x-ms-traffictypediagnostic: DU5PR03MB10263:EE_|AM0PR03MB6132:EE_ x-ms-office365-filtering-correlation-id: f551e331-802a-4b8e-4425-08de963ef4a9 x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0;ARA:13230040|366016|376014|1800799024|38070700021|22082099003|56012099003|18002099003; x-microsoft-antispam-message-info: WjANiVgJUe2pIJsnNLs//I8+dSW3QElPBXJbAM0Rqc9DcMRLOMotP76XDvYKJ8IPtNcIYfwyAiV+MKNbkUiW0ODDAVAX6kKAnDmxoesQ/6JSS1sfKPpPjuyveuWw7khZY5j0JEBe1TZGlVFbaxIbAgqjbzh31iIktsFVGJZLhS37mtGouzHlW1G3uV1j62+fgnTkHD5xuv81sSyY8kVlbZKHWz7VLJmbLHUHD7exO1m5+T76oiHoUajEvsdanlBCsVuNAesHMyj19xZHY8OMVGFPpkQ+H1HU2fsBAI0I/f2VFyNTQ4u2dp8zrreKR01XAKgzUmeoT3fYIAWi1q8FLfyMEunG7/f1F0buOVZQCRUzl7r07EcOgCr3TJdbmCMRkgdAjOysnQ4kJXlOtAfWKaGr7gTBiDf8v8IjrW2sxBwXjmuzsOYa7bvfHGNBOk4NQ6fLxLsz2mUQ5Fa0rGyCODVPTup5/SzaA5kIfQwxjPViLsR8lIFCt35ly2USCG/eQ7nl/vQUPCqdA3YS7HCQ/oQZf27NXKa2h4YUYIdJHs+wzSn6omCKfe9pVrBIju9+J+4xFup8XHEYSXpTXk/bgOy9J1r+wGLobGGe5wP/lXvfAyP4Yc/QGnypuslJEfsfx52ZyQGvRpSEVXsgK9YFNsI6z8rDd7IXCxT315Px5Kexv2bDiTgbgGWnyg7prKmBDM78KFSUmhCnCxa1SM1kCTR0Sv5wxc5PcEU6DgbiPCWu67AtQMjgY/is8gi49gx3f3p4NBZ7ERQpu/2zBodx6kHfQb1UeltHV9n5bLqidZM= x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DU5PR03MB10263.eurprd03.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(366016)(376014)(1800799024)(38070700021)(22082099003)(56012099003)(18002099003);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?iso-8859-1?Q?ZR9iMLPrxFXbD11Yg4alYgOYIaNr3G5e1vjiPtzI10pQh5mgDPauGqYZem?= =?iso-8859-1?Q?MsiYq1Qbp9BWtBDAYeye0JvRYEgQtOLzuo+M5HNXY1noqwq3uyMDEuUT0/?= =?iso-8859-1?Q?LX8yg1ybOfNFWjg3PWsn+hn+n6tfBLTsyBVj4/EW4rLEYWnRzHuXYkE6fa?= =?iso-8859-1?Q?wrraBUJLVE5MqAHv4s0z8VdcTjo6rFGFj9cQKqF3A2ahRvw5sXzuCDZ+E+?= =?iso-8859-1?Q?KbMEZXob4etBRhJ1PeEQJKHniH6V4kpSDYsDVT39x6iSdlNlI5LoizqwhX?= =?iso-8859-1?Q?+p5TzZkKMOiLxi5qyFNETytAcvYDVimqoinxtUEDcj9zohmrVzLtBbOLYp?= =?iso-8859-1?Q?4inX7hYsAqun+xGd04dh15B1E6/xEsQZEB10hYCluy209LtofHq68uypde?= =?iso-8859-1?Q?wqvJf78F/0l8MGwZYa9ziCeALhdA8OmnzYI+qRaXgd/YeiX7sOA9BTrF38?= =?iso-8859-1?Q?Xq/fG1X8gfdARnCEWYvgLp43kU82FBYenqCC0ZADX7vS6BTtSyT+lCBLOh?= =?iso-8859-1?Q?3R/CWo5nz/ZU3EHL///aq0fFhaQdChqEIkQwV8i3W+b3NgwrwWWnihr1J9?= =?iso-8859-1?Q?x0mf3A8+N3M0bABOI/FRfP8Pr65ipRR/C57UAkGUNgNWvbzURPwtHutK+d?= =?iso-8859-1?Q?Az3FLYxnzK5rDlHlS/K3AwysxY4w9XZobWcAfR4sc2DY18oi4HBAaiCviz?= =?iso-8859-1?Q?KldoB1upt9AgR6TqKb+OLo7jFTms3SBL9A0TRZPIYu8eXjHed4ajaKq7LI?= =?iso-8859-1?Q?JFV10mVhzZswCRaExaXvAnSY7UBd/LevDNeKpRk0gLs5tz4ZwMJJjGoWTW?= =?iso-8859-1?Q?yH6fTJat3J7A07h8feJPAF63HP73pJhI9nVACAwCSTyZzv8Sudo46kZ3DX?= =?iso-8859-1?Q?hk2NAhT9RY5vpwZv+Md59ESuMRAv0T9tI03UTIPlirJiCooKQvRGOOwf4q?= =?iso-8859-1?Q?oKgxjpBtD0+4x4xa9iTpKxgbUFBV1cmXZYoSH9G4VX30ihjhaRubD8FyC3?= =?iso-8859-1?Q?unQwzajaVkmaO70gl7pBERM8oC6ojDKsPR2mxQ4KSv+0CDuTV4yPu65l9A?= =?iso-8859-1?Q?lMnBYTJCWgmJqlrU5P/zEV7Af7cLcHtYAGzyHaG9fglkBELiz0HZ/JWvQB?= =?iso-8859-1?Q?mrHHRWcdzv1sJln7HR6+M3ZAcjfaGOLkaG7XYq43nDfSgMipsKavkeu6H2?= =?iso-8859-1?Q?h7pg/7qfNgohnWnL+E7RtCcXjChqf5Iebs34Cb/2Gc0/41Uf7X+6dfqzBk?= =?iso-8859-1?Q?HpLdlqRt0sx2Q+Huedij78V8By4ta0m8L6cY+FrNGU5KcWzgv0ksZJz+Ko?= =?iso-8859-1?Q?E0w3f8aULzuFnZ2bDrYpzNUHYaya9/42v8GZn7v3ixVgkZ8F/PNj86UxNP?= =?iso-8859-1?Q?JVVeA87cWbr9JecktqjkPnbyRnq11wyadTFuFKFIlGWCJYeVee091dLDD1?= =?iso-8859-1?Q?JdM/s0/Cu0GhJnTrRAGTznA34sMkesxHRGBUqHtkkpBOB6tJ12UiXxKJ36?= =?iso-8859-1?Q?G6hTPPDE0K22XHftqgPC+ATgMW3nD1F6eJKXDDRBx4mfVvK4C1KWEM8TgM?= =?iso-8859-1?Q?sch/7FmbZkL6z/K88a8/fg/nylg3/L5jhRBOL4TtZ3FtixZnVPul9l7Bpl?= =?iso-8859-1?Q?w4OH4XM64vvq72eNiRWRZbANTC7s/3wYpyXHzCtfqlww9mncc2ROExjBp2?= =?iso-8859-1?Q?qznvA2RI3zFcdFuzyiXPH2eJxM3XUDi1MlhDoW+H78q37tpyx28jjKstsO?= =?iso-8859-1?Q?N/NusGZqajhE2CaaWDeZhUT8vRbg9Oixxbkr8zJmvWR7oXRTudIhEzelYp?= =?iso-8859-1?Q?sHB0ZIoalI+IhnevS15ZGIjRGnPW554=3D?= Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-OriginatorOrg: epam.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: DU5PR03MB10263.eurprd03.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: f551e331-802a-4b8e-4425-08de963ef4a9 X-MS-Exchange-CrossTenant-originalarrivaltime: 09 Apr 2026 13:50:26.9497 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: b41b72d0-4e9f-4c26-8a69-f949f367c91d X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: uwtj2AKzoHIeHY2WJbd197sf1xEFRTqCvUTeRI4LBag/9Ab//IKD6raq3Y9D9+7OU6fJFbVvQWNbxP+C6+CXAGOYIxCcCwuCyuKxfjVkw0o= X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM0PR03MB6132 X-purgate-ID: tlsNG-c1860d/1775742629-F674E497-EB7A424C/0/0 X-purgate-type: clean X-purgate-size: 2266 X-ZohoMail-DKIM: pass (identity @epam.com) X-ZM-MESSAGEID: 1775742660271158500 Content-Type: text/plain; charset="utf-8" In cuart_init_postirq(), two code paths could reach the interrupt-enable write to IER without a handler being registered: - When no valid IRQ number was provided (uart->irq <=3D 0), the original positive-condition guard (if uart->irq > 0) skipped the irqaction setup but still fell through to the IER write, enabling the receive data interrupt with no handler installed. - When setup_irq() returned an error, only an error message was printed and execution continued to the IER write, arming the receive hardware interrupt line with no handler to service it. On platforms where the GIC receives this asserted line, the result is either repeated spurious-interrupt warnings or an unhandled interrupt fault. Restructure cuart_init_postirq() to use early returns in both error paths. Signed-off-by: Oleksii Moisieiev --- xen/drivers/char/cadence-uart.c | 17 +++++++++++------ 1 file changed, 11 insertions(+), 6 deletions(-) diff --git a/xen/drivers/char/cadence-uart.c b/xen/drivers/char/cadence-uar= t.c index b2f379833f..a63dc4adb2 100644 --- a/xen/drivers/char/cadence-uart.c +++ b/xen/drivers/char/cadence-uart.c @@ -72,13 +72,18 @@ static void __init cuart_init_postirq(struct serial_por= t *port) struct cuart *uart =3D port->uart; int rc; =20 - if ( uart->irq > 0 ) + /* Don't unmask interrupts if no valid irq was provided */ + if ( uart->irq <=3D 0 ) + return; + + uart->irqaction.handler =3D cuart_interrupt; + uart->irqaction.name =3D "cadence-uart"; + uart->irqaction.dev_id =3D port; + if ( (rc =3D setup_irq(uart->irq, 0, &uart->irqaction)) !=3D 0 ) { - uart->irqaction.handler =3D cuart_interrupt; - uart->irqaction.name =3D "cadence-uart"; - uart->irqaction.dev_id =3D port; - if ( (rc =3D setup_irq(uart->irq, 0, &uart->irqaction)) !=3D 0 ) - printk("ERROR: Failed to allocate cadence-uart IRQ %d\n", uart= ->irq); + printk("ERROR: Failed to allocate cadence-uart IRQ %d\n", uart->ir= q); + /* Do not unmask interrupts if irq handler wasn't set */ + return; } =20 /* Clear pending error interrupts */ --=20 2.43.0 From nobody Sun Apr 12 22:39:08 2026 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) client-ip=192.237.175.120; envelope-from=xen-devel-bounces@lists.xenproject.org; helo=lists.xenproject.org; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1 dmarc=pass fromdomain=epam.com); dmarc=pass(p=quarantine dis=none) header.from=epam.com ARC-Seal: i=2; a=rsa-sha256; t=1775742660; cv=pass; d=zohomail.com; s=zohoarc; b=APfju786ODNBw8vYyrskjf9PgSgMAuf8vnX+Oz51gBpJUr7pspomCY5lJ7Cc6enFFThuAWpSZ9gF2c6BeNV6L101mZEDfDhuY0a8BkNsbNO/VXMBiTw1v8HE51inGyBI9w96zEwqOlErbmFJCiul1cpNDt4Rg1V3AxvyjKvaO0U= ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1775742660; h=Content-Type:Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=jHTUPc+0Yn1eQpeQBKN/rncMUFe7rtRQF5uHYfz3Cvs=; b=QAtPTNiQc9rFdkLgctvlajM96Jr0ICcGeCm0AQilMrxGTPJKUmAGJxMUXdMfThcATryZoIKxX4c5JXptzNnbO389rKH9F4sH8c9qvcf/BcPtWCF1zxO5zMZbujmkutULuyDEUNvZI8+0+diNBsM/1cGne9/tPAZo9rqed5qGY7w= ARC-Authentication-Results: i=2; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1 dmarc=pass fromdomain=epam.com); dmarc=pass header.from= (p=quarantine dis=none) Return-Path: Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) by mx.zohomail.com with SMTPS id 1775742660301418.6213224124581; Thu, 9 Apr 2026 06:51:00 -0700 (PDT) Received: from list by lists.xenproject.org with outflank-mailman.1277416.1562634 (Exim 4.92) (envelope-from ) id 1wApmC-0000b0-Jo; Thu, 09 Apr 2026 13:50:32 +0000 Received: by outflank-mailman (output) from mailman id 1277416.1562634; Thu, 09 Apr 2026 13:50:32 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1wApmC-0000Zo-FZ; Thu, 09 Apr 2026 13:50:32 +0000 Received: by outflank-mailman (input) for mailman id 1277416; Thu, 09 Apr 2026 13:50:30 +0000 Received: from mx.expurgate.net ([195.190.135.10]) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1wApmA-00007h-Lj for xen-devel@lists.xenproject.org; Thu, 09 Apr 2026 13:50:30 +0000 Received: from mx.expurgate.net (helo=localhost) by mx.expurgate.net with esmtp id 1wApmA-005hLD-1m for xen-devel@lists.xenproject.org; Thu, 09 Apr 2026 15:50:30 +0200 Received: from [10.42.69.8] (helo=localhost) by localhost with ESMTP (eXpurgate MTA 0.9.1) (envelope-from ) id 69d7ae9a-bab6-0a2a0a5309dd-0a2a4508a406-34 for ; Thu, 09 Apr 2026 15:50:30 +0200 Received: from [52.101.65.135] (helo=DU2PR03CU002.outbound.protection.outlook.com) by tlsNG-c1860d.mxtls.expurgate.net with ESMTPS (eXpurgate 4.56.0) (envelope-from ) id 69d7aea3-fab6-0a2a45080019-34654187b92d-7 for ; Thu, 09 Apr 2026 15:50:29 +0200 Received: from DU5PR03MB10263.eurprd03.prod.outlook.com (2603:10a6:10:519::5) by AM0PR03MB6132.eurprd03.prod.outlook.com (2603:10a6:20b:145::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9769.17; Thu, 9 Apr 2026 13:50:27 +0000 Received: from DU5PR03MB10263.eurprd03.prod.outlook.com ([fe80::8c9e:b301:61c0:3908]) by DU5PR03MB10263.eurprd03.prod.outlook.com ([fe80::8c9e:b301:61c0:3908%5]) with mapi id 15.20.9769.020; Thu, 9 Apr 2026 13:50:27 +0000 X-Outflank-Mailman: Message body and most headers restored to incoming version X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" Authentication-Results: eu.smtp.expurgate.cloud; dkim=pass header.s=selector1 header.d=epam.com header.i="@epam.com" header.h="From:Date:Subject:Message-ID:Content-Type:MIME-Version:x-ms-exchange-senderadcheck" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=BcKRM+P9oMVXFy3WKaA1/GSsutKdusNeW7iAL6lnL8YOw1GBGINqhf0MSo3ZE9rzAt4ivtra/DITzWXqtN99CctI0qQZZOlahBckEFtf05XCuAbughxibt8EULXrxmMtW3dOFTKEzetv7yMyCy8/LQrEIF3IJvdZhCgSRBNz5nBkcZTtuqKfA1ZGIIyxqYMy7xV7xo6tcUatHKZ4771bWh2WxP+UZvrnX7oThb1MBhHX0fbHiaPnwYtTwY9y013vY/uADvz732bs8imGRe1VwF0lWSW24LZSfrsJHTF/KgO5FBOmYLjzaa5FWFeWVfyhVWjK8ccKd7DjBVsKpJdtEg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=jHTUPc+0Yn1eQpeQBKN/rncMUFe7rtRQF5uHYfz3Cvs=; b=g05rZbdPBwmdNTxDpkV+mmWjgMo15ZxhtG9sSjDMyt6+5nFz2GSqcEoDKIMr5jHkDY0iMzgDX8aWyVnjKGsl5iWI2zfzxThWxvJVmTar+2OJw2bdPjTtGDuM1ugFwzbuZwl9eODC9r3H7fHvfi7KHKHH1KIMRSczaCbZ++ChpjuvqBdDfnt6+FikjpxBgAmlekt2N1gM/XDIxhBWxCf5HM3nSJANlFhOJ1F2JCruytoYu3d2l689pHZnQf0tVbLinHKRCM4gpRZJc+/Ir7sTAkp2UOvtre99Y6e39R8FxSPeA6OpKHuMIWjcupx2mfDi8AtHyqXMkG9zQDyl0RBB+g== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=epam.com; dmarc=pass action=none header.from=epam.com; dkim=pass header.d=epam.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=epam.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=jHTUPc+0Yn1eQpeQBKN/rncMUFe7rtRQF5uHYfz3Cvs=; b=nexaRASvtgArsh3DTWMFf4YGozkkWQ/dpP1eba7XKdfxJO7eLtHEOsYHoZNHxu6z4YBOP/AOOdJs0ZpBgbODThx90KA5239QSlci7SSmBfu0Azn3hIo59tEyjyqOlC/SUkmY+gdyxgyaDfsVloL1DZZ3qUxOnR6y1wctL3FPAwB+jAhA+N/NarFybIV+1Rs7vMqsyxPqBdxlBWloS2oZvijM+hYGvx4Vs/cMYtnWUIEgRgKvi6X21bOpezMYksZu4o8Yfy6jkjeqREQo+sLLZ9TBjtJ5HQ5FAxVokfjw6yfbS+qT0b1TFVyRIKfFTGfFJs6TDEeMcOf2TGkb5z7lgw== From: Oleksii Moisieiev To: "xen-devel@lists.xenproject.org" CC: Bertrand Marquis , Julien Grall , Michal Orzel , Stefano Stabellini , Volodymyr Babchuk , Oleksii Moisieiev Subject: [PATCH v2 4/4] xen/drivers/char: fix exynos4210 IRQ registration failure propagation Thread-Topic: [PATCH v2 4/4] xen/drivers/char: fix exynos4210 IRQ registration failure propagation Thread-Index: AQHcyCfSPi7W/0ovn0ufnBNUSA+61g== Date: Thu, 9 Apr 2026 13:50:27 +0000 Message-ID: <043b73edeee02b477f210cfd82f0b2940de2ef21.1775742562.git.oleksii_moisieiev@epam.com> References: In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=epam.com; x-ms-publictraffictype: Email x-ms-traffictypediagnostic: DU5PR03MB10263:EE_|AM0PR03MB6132:EE_ x-ms-office365-filtering-correlation-id: f7fe5ff6-5a33-4375-4c6e-08de963ef4dc x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0;ARA:13230040|366016|376014|1800799024|38070700021|22082099003|56012099003|18002099003; x-microsoft-antispam-message-info: b8RsrXgJbzOn84cba5UZX6I2brJhrYycwDAj52g43yr3jf9sd+LbJCl6H/l91IISkzmu9h1vw5DiqrNLvROKlqD+PDKW1BDaCiU42G197mz1OCsOZ5rVbXiL6epU4DcLORply4Lfkgisj6Gu9tpKFqfU6+ZTuaU5NXA5UBVvPVmBzFMzGnrCknafRwmJi0EdMTllYCBouGdUoq3LaW0NM5DmjG7QrPpv5XxoHo0EAJDDCpuAD3qUkVtiBLT4LU2J7+W8ASsw7ajztRMjO6+Oo0wnpVwJyPgDcI3ah8EzEARWIAJpGEySh4RFVgyrJDBkD4t348189VoFpH4v31URdj9CKh9a9/j47HJ5jCIi+r8/UK4VEh+aEaNGllRG1PDYOQLkJZgVYDF/9Kc7cqBSzFhyHlBm8rxpRQJUhJrdv5QK1hs0VutTdC6TyNihjMPMupUdHpuTi98NNfLGyxRDNpqYzEpMTKyPcWYQSqcJZmI1YO4q377NBic6AWz7H0YUiHyrJOiPtuMdQjvGGt1BG1SgXWRFRt78qcI5q8a2vR+irYidPrae/aFjEfsdIioZp6p9+wGfBI8ggTBxNsEALJ3XXjFhOfI6hZwCpqNpLV77y5CQo/o7nxRnsN7CrD9YYj34/0V3iLmOM67mxIN3UeW6nlA9ngaDng2VnnbyYPLHfBiG4kAC2VFZHeRKqW84qgeq6XTyWLYA+QELV+to4juIZC8t0TssopvoAQAdNRRm+wXYA1qQuSALhcKyASWXHswzwRi1erCDnvOORfbKxB8aWYuLz/selBXUK8y/TUw= x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DU5PR03MB10263.eurprd03.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(366016)(376014)(1800799024)(38070700021)(22082099003)(56012099003)(18002099003);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?iso-8859-1?Q?GWgOb9fvA5tdLryWjCV8g3A0WNHo3WkvrJ0uNH3s9TSDqtfFkpCt5om41x?= =?iso-8859-1?Q?JMYOej7utaLYCxJdmEMHT3UtYljY8NRGEgXeBF/eOEz7mfCV1vBXrMD5TF?= =?iso-8859-1?Q?k8nONeIsvYMma+CCe6zdn1c7+daQ727ViUt/4Dl8yi/zzlk72Xe/d9h9b6?= =?iso-8859-1?Q?kjpG6CK2N473QFDfaLUOvznNvZSYz1toGq3hL/nph0v69RUKrLkJDTLROb?= =?iso-8859-1?Q?7PqSmKrQjA8JvVwH6DtWQhFiTJhhO9OZkz8AvvqTl5Y5D7V8sCuE8HatQX?= =?iso-8859-1?Q?sJ26LK5YlT7d8oKg6tFqbjmmA4NVBOMKcOE8XhRgS2PqswCF1JHPPxeZIo?= =?iso-8859-1?Q?U/nkvSvSIDTbWECgeXpsAb2M1UvqjuqT03e/6adIGFMO94FgQYQPsTbW5p?= =?iso-8859-1?Q?kcFX3/+UVN14KuCHJy8Aq96gXgAHnfo8AE/jiaGwDBx11P+7for534jKOk?= =?iso-8859-1?Q?Q0owhQlVvRPR+b1VH/lc5LulBdIkOwFMJkhpChtlex7szPG3vn80DrFveZ?= =?iso-8859-1?Q?plDI6PEcuNMV5VeIUv9XCXqdZ45HMFvUgw1x9WCs1OQH8LM6KqC9zHlXPf?= =?iso-8859-1?Q?6vQ9FaU3dOBE0P0o/0DMiTiejBPxzRn4qim8PSrMdRIyfReifVYn7P3/WN?= =?iso-8859-1?Q?qPFUn64D5J/blHZoIA45/0bqPFhwrjCDd0Enrj8OVc6zdW51wYk3MYFUjJ?= =?iso-8859-1?Q?avXR/gFsgv+lMEzEN/jL8BbbHTFcmBObDEm51mQLONLySo0URSIEmxL4qt?= =?iso-8859-1?Q?T7tKBGMpuoOajIoOddcLQ5Ue8Kci+08tnwESefRAhQsrZVQHXk1okPwC6A?= =?iso-8859-1?Q?jX/XYj6oguKZrpimBx2bGFexsjj/lbBQvOT6eiCEuXmp6YHQN/Rb1LywON?= =?iso-8859-1?Q?/s3fZ+KpKewyHK7w3LzFYX2GKuerlZ3bwURjn0Wj3J1ISbwyuzlXVMQh+v?= =?iso-8859-1?Q?68s3dbyQvIBL9+XJyzMimnqQYoMY/YUCeutDW/Pq6A5zcET6U9vmLJN6nB?= =?iso-8859-1?Q?JtZA2mUWWQMy6lv/eBGhNVpaF5N+XZJortYObvaOV0L5wH9I3vS6IAYdki?= =?iso-8859-1?Q?NIHXqDsqgYcakpRwFpz9l+JBJBDAcCLr2RytRDWpT8M8TDsbngN9NIu/cx?= =?iso-8859-1?Q?tpbvmzsDfW0y9fLZv8RK+zfKyTVmXEORcdzTtbYkolY3cHilpZHm+KyjLA?= =?iso-8859-1?Q?BeNeKXJdqstxHGG53flCazd+yidgAv+ysdW/0Z88R5T0AKy7gFvjqhPvBY?= =?iso-8859-1?Q?HS/8ByxSawUZiWSc5kHtpagxcJhSOl00NMPEltSz3bbAsnMqv/hGKy4wCv?= =?iso-8859-1?Q?OulZH3iXX90vuVQAyNAYeNOKyJopxWhoUiV2pv8mds5qiEsNU2RQk1bbnT?= =?iso-8859-1?Q?LrTClku99Thulv6CInqGaPulhez6yOZp+DzIOGWFB6ioh8CsWDePkq3Xa+?= =?iso-8859-1?Q?Kr5t32MuhSw6sefG2jag3a2UtlRuK4OwPA7WvchGILZAWgVcgyJQ+TTj14?= =?iso-8859-1?Q?t4XrRF4PRlnAWhmCfZzyu16s3TIKiDkUTeFu/J8WfNXVdJdGOhMwRBS/dS?= =?iso-8859-1?Q?4CAbfMvG1DGSdmmOY3OxgOwpn3BGH+pssjiB6u3Q8aUGQgXSGIK9/1mkEm?= =?iso-8859-1?Q?8PH16W+c1eUupHFPhUvaIiL/vvjeKo4fMTF2GA+X42NDPOJ0XOVuJPyGkB?= =?iso-8859-1?Q?p4J25uztxqINVFKwsckhDRs5CjLAe/rq8o+imkA7GIhOKFbZeoyPU2q1js?= =?iso-8859-1?Q?VRvB+gsOgLLmZoFBQOTL2CYkJFRG3ME3jgH541BOzt+XGTk+KI5AyKZ6L8?= =?iso-8859-1?Q?UcrTUlcuFf6qrDEFc4+a9xXy5CR0Wlo=3D?= Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-OriginatorOrg: epam.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: DU5PR03MB10263.eurprd03.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: f7fe5ff6-5a33-4375-4c6e-08de963ef4dc X-MS-Exchange-CrossTenant-originalarrivaltime: 09 Apr 2026 13:50:27.3167 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: b41b72d0-4e9f-4c26-8a69-f949f367c91d X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: NTdgNFySvviM6r2x80ywyNq7qBgyjqw2w+GXnnJHc7oo7wEbv0O7pzRMtVI4eFgNxDUQNYxW5XOC0Vp9UFFZn+v+yeSyXRc2swI0CIENuwY= X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM0PR03MB6132 X-purgate-ID: tlsNG-c1860d/1775742629-75358497-AF44ED15/0/0 X-purgate-type: clean X-purgate-size: 1472 X-ZohoMail-DKIM: pass (identity @epam.com) X-ZM-MESSAGEID: 1775742662552154100 Content-Type: text/plain; charset="utf-8" In exynos4210_uart_init_postirq(), when setup_irq() returns an error the failure was only logged via dprintk() and execution continued, unconditionally clearing UINTM and setting UMCON_INT_EN. This enabled receive and transmit interrupt lines with no handler registered. On platforms where the GIC receives these asserted lines, the result is either repeated spurious-interrupt warnings or an unhandled interrupt fault. Add an early return in the setup_irq() error branch so that the interrupt-enable writes to UINTM and UMCON are skipped when IRQ registration fails. Signed-off-by: Oleksii Moisieiev Reviewed-by: Michal Orzel --- xen/drivers/char/exynos4210-uart.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/xen/drivers/char/exynos4210-uart.c b/xen/drivers/char/exynos42= 10-uart.c index 58901df554..2e5860ea2b 100644 --- a/xen/drivers/char/exynos4210-uart.c +++ b/xen/drivers/char/exynos4210-uart.c @@ -199,8 +199,12 @@ static void __init exynos4210_uart_init_postirq(struct= serial_port *port) uart->irqaction.dev_id =3D port; =20 if ( (rc =3D setup_irq(uart->irq, 0, &uart->irqaction)) !=3D 0 ) + { dprintk(XENLOG_ERR, "Failed to allocated exynos4210_uart IRQ %d\n", uart->irq); + /* Don't enable interrupts if irq setup was failed */ + return; + } =20 /* Unmask interrupts */ exynos4210_write(uart, UINTM, ~UINTM_ALLI); --=20 2.43.0