[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v3 4/4] xen/drivers/char: fix exynos4210 IRQ registration failure propagation
- To: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
- From: Oleksii Moisieiev <Oleksii_Moisieiev@xxxxxxxx>
- Date: Wed, 22 Apr 2026 09:33:14 +0000
- Accept-language: en-US
- 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
- 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=sua2lHyPv7A2Nh5zyXGRUWTpGAlugF2OyFrgHX3AWNk=; b=ZA4BkVc+0/c7uurY0Q4b1JqrKWCnadDCOU0NCEfZ8eSlOZ3DBMWuQ0c+ZmVpyDCeldOQSFk04BgEafGv6RFfRjCn/9daa0kT6ifrZS2MMtLrxe2TT/9eOuerNyVqUz4c76K8DWcB+O9VRk97pUZxQ3iBGAt2Qakp1Q7mqfmOyI9KceucIRSjqpnZviWY+ANRKV9ZabxFa1I2QJ31cwO5qX8r/hRoXUxqVS+zySJO/HStamu3yN0fNXV0Amyg1O5nfbQeIqo52TW+Ua93UYheyAJnhtgw2DBW4GRWA3hoxmxbSwGypel6UFvpm0pde2Jhz5kE46vpnqERMYigD9/1xg==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=rDk0OVqKEV2XzJ8TjXkuBQn6iKAkuNQwhx2wlRxIDH72B6P6w/rpgGHa+5MaJ0Apg9EgzM0osMvRoBXC5YsopDK4nq96ykKDibNcArzppbbNBBgaGXc6/P4H3sbWMOdKVHKSsrFyXMzLTacprgyxm0tsmNcRRAOdhyuppSx0AZ+MnXuKkbUKNVQ1UwqH0mDQCKxI2Vlh9ESm8cShANXG2frckxfAsVglFGgDHDiF2OZKAE4dbdyYtFpaYWt/GvZyb9MwG5nFsZ1hSpqYhz4ML2bbcXzBWUyqPQyNoamFXk9XddKMWbwZhHVDRXcaSvJYbTc/TcQ35zuidOYfclRbzA==
- 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"
- Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=epam.com;
- Cc: Bertrand Marquis <bertrand.marquis@xxxxxxx>, Julien Grall <julien@xxxxxxx>, Michal Orzel <michal.orzel@xxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>, Oleksii Moisieiev <Oleksii_Moisieiev@xxxxxxxx>
- Delivery-date: Wed, 22 Apr 2026 09:33:21 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
- Thread-index: AQHc0jsL+9q0JLBsskmBWrh+PQuj8g==
- Thread-topic: [PATCH v3 4/4] xen/drivers/char: fix exynos4210 IRQ registration failure propagation
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 <oleksii_moisieiev@xxxxxxxx>
---
Changes in v3:
- skip clearing pending interrupts if setup_irq was failed because
according to the 13.4.1.13 of the RM: it must be cleared after
cleaning interrupt pending in INTC.
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/exynos4210-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 = port;
if ( (rc = setup_irq(uart->irq, 0, &uart->irqaction)) != 0 )
+ {
dprintk(XENLOG_ERR, "Failed to allocated exynos4210_uart IRQ %d\n",
uart->irq);
+ /* Don't enable interrupts if irq setup was failed */
+ return;
+ }
/* Unmask interrupts */
exynos4210_write(uart, UINTM, ~UINTM_ALLI);
--
2.43.0
|
