From nobody Sat Feb 7 06:13:36 2026 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 640C7207A0B; Wed, 31 Dec 2025 09:26:19 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1767173179; cv=none; b=J2UzXPnYEOFDiU/zHv9uL++odoXgsSk3Q3ez0xnXUwYAzdjCMo8iXHe4oF+U+7YVPvdlgv1mZVv00mlZhqCPtt3FPyPb4sJNWPJ8D07gBVyUlLE4xMQxZM3yFG3ezXKjCw6zQw9WVps7kHaf8TpPh6bn/BS2XuVY9CBsxNI8XTw= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1767173179; c=relaxed/simple; bh=g1Co/HPqGgkfcaq62wPDtixMTJAutCdlb40s5TN3O/I=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=VvSmgVPqp3METy8IXFoXW/0dd1uQPhWfGmeY9kfGUszBlMg1ITpBAbd6lRfjoTxpH3CN10Iqvl8RwlsBQRCnuHZ5rgJdGP9LnlVtMkwW2LNXhWJQ/Si2TmNrg94NnmQ/OAeNQuJYOoGRkpAinRIH80Zdqf6DHAufqhxvNL8mJbk= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=Y2qoWycC; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="Y2qoWycC" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 63975C113D0; Wed, 31 Dec 2025 09:26:17 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1767173178; bh=g1Co/HPqGgkfcaq62wPDtixMTJAutCdlb40s5TN3O/I=; h=From:To:Cc:Subject:Date:From; b=Y2qoWycCNXhS8aZzgbHmgTZGgohfkq6PklJAiLJmYXdX8RWzTYsag5ydjnhhk7dzi KF7sZup3V3bN5aWvICAilrHekY/6I6NoYajusuEnFIqT26A/b0DtoUnCrRMD+ld1Jo hCdOm5KNkUM+0Ujh/ToerwH518hN8lhVdXIgjEJ8KdF0h3k//CfqrJSM/CWNEt8kyZ F74cr1SRV4kNFuv1qvUEWyM1YaespKL0Hy3ABc7OztBKgpWw3uBQZ99b+It0RZOggk l/QkzHTFdmAaUPIVcV4hyRwkFgwkjAgmjGWGMDMofdgKeJGA44kXB6wzIAHytTlYZi RLEoknSHF29DQ== From: Lorenzo Pieralisi To: linux-kernel@vger.kernel.org Cc: linux-acpi@vger.kernel.org, linux-pci@vger.kernel.org, Len Brown , Bjorn Helgaas , "Rafael J. Wysocki" Subject: [PATCH] ACPI: PCI: IRQ: Handle INTx GSIs as u32 values not int Date: Wed, 31 Dec 2025 10:26:15 +0100 Message-ID: <20251231092615.3014761-1-lpieralisi@kernel.org> X-Mailer: git-send-email 2.50.1 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" In ACPI Global System Interrupts (GSIs) are described using a 32-bit value. ACPI/PCI legacy interrupts (INTx) parsing code treats GSIs as 'int', which poses issues if the GSI interrupt value is a 32-bit value with the MSB set (as required in some interrupt configurations - eg ARM64 GICv5 systems). Fix ACPI/PCI legacy INTx parsing by converting variables representing GSIs from 'int' to 'u32' bringing the code in line with the ACPI specification. Signed-off-by: Lorenzo Pieralisi Cc: Len Brown Cc: Bjorn Helgaas Cc: "Rafael J. Wysocki" Reviewed-by: Bjorn Helgaas --- drivers/acpi/pci_irq.c | 19 ++++++++++-------- drivers/acpi/pci_link.c | 39 ++++++++++++++++++++++++------------- drivers/xen/acpi.c | 13 +++++++------ include/acpi/acpi_drivers.h | 2 +- 4 files changed, 44 insertions(+), 29 deletions(-) diff --git a/drivers/acpi/pci_irq.c b/drivers/acpi/pci_irq.c index ad81aa03fe2f..c416942ff3e2 100644 --- a/drivers/acpi/pci_irq.c +++ b/drivers/acpi/pci_irq.c @@ -188,7 +188,7 @@ static int acpi_pci_irq_check_entry(acpi_handle handle,= struct pci_dev *dev, * the IRQ value, which is hardwired to specific interrupt inputs on * the interrupt controller. */ - pr_debug("%04x:%02x:%02x[%c] -> %s[%d]\n", + pr_debug("%04x:%02x:%02x[%c] -> %s[%u]\n", entry->id.segment, entry->id.bus, entry->id.device, pin_name(entry->pin), prt->source, entry->index); =20 @@ -384,7 +384,7 @@ static inline bool acpi_pci_irq_valid(struct pci_dev *d= ev, u8 pin) int acpi_pci_irq_enable(struct pci_dev *dev) { struct acpi_prt_entry *entry; - int gsi; + u32 gsi; u8 pin; int triggering =3D ACPI_LEVEL_SENSITIVE; /* @@ -422,18 +422,21 @@ int acpi_pci_irq_enable(struct pci_dev *dev) return 0; } =20 + rc =3D -ENODEV; + if (entry) { if (entry->link) - gsi =3D acpi_pci_link_allocate_irq(entry->link, + rc =3D acpi_pci_link_allocate_irq(entry->link, entry->index, &triggering, &polarity, - &link); - else + &link, &gsi); + else { gsi =3D entry->index; - } else - gsi =3D -1; + rc =3D 0; + } + } =20 - if (gsi < 0) { + if (rc < 0) { /* * No IRQ known to the ACPI subsystem - maybe the BIOS / * driver reported one, then use it. Exit in any case. diff --git a/drivers/acpi/pci_link.c b/drivers/acpi/pci_link.c index bed7dc85612e..b91b039a3d20 100644 --- a/drivers/acpi/pci_link.c +++ b/drivers/acpi/pci_link.c @@ -448,7 +448,7 @@ static int acpi_isa_irq_penalty[ACPI_MAX_ISA_IRQS] =3D { /* >IRQ15 */ }; =20 -static int acpi_irq_pci_sharing_penalty(int irq) +static int acpi_irq_pci_sharing_penalty(u32 irq) { struct acpi_pci_link *link; int penalty =3D 0; @@ -474,7 +474,7 @@ static int acpi_irq_pci_sharing_penalty(int irq) return penalty; } =20 -static int acpi_irq_get_penalty(int irq) +static int acpi_irq_get_penalty(u32 irq) { int penalty =3D 0; =20 @@ -528,7 +528,7 @@ static int acpi_irq_balance =3D -1; /* 0: static, 1: ba= lance */ static int acpi_pci_link_allocate(struct acpi_pci_link *link) { acpi_handle handle =3D link->device->handle; - int irq; + u32 irq; int i; =20 if (link->irq.initialized) { @@ -598,44 +598,53 @@ static int acpi_pci_link_allocate(struct acpi_pci_lin= k *link) return 0; } =20 -/* - * acpi_pci_link_allocate_irq - * success: return IRQ >=3D 0 - * failure: return -1 +/** + * acpi_pci_link_allocate_irq(): Retrieve a link device GSI + * + * @handle: Handle for the link device + * @index: GSI index + * @triggering: pointer to store the GSI trigger + * @polarity: pointer to store GSI polarity + * @name: pointer to store link device name + * @gsi: pointer to store GSI number + * + * Returns: + * 0 on success with @triggering, @polarity, @name, @gsi initialized. + * -ENODEV on failure */ int acpi_pci_link_allocate_irq(acpi_handle handle, int index, int *trigger= ing, - int *polarity, char **name) + int *polarity, char **name, u32 *gsi) { struct acpi_device *device =3D acpi_fetch_acpi_dev(handle); struct acpi_pci_link *link; =20 if (!device) { acpi_handle_err(handle, "Invalid link device\n"); - return -1; + return -ENODEV; } =20 link =3D acpi_driver_data(device); if (!link) { acpi_handle_err(handle, "Invalid link context\n"); - return -1; + return -ENODEV; } =20 /* TBD: Support multiple index (IRQ) entries per Link Device */ if (index) { acpi_handle_err(handle, "Invalid index %d\n", index); - return -1; + return -ENODEV; } =20 mutex_lock(&acpi_link_lock); if (acpi_pci_link_allocate(link)) { mutex_unlock(&acpi_link_lock); - return -1; + return -ENODEV; } =20 if (!link->irq.active) { mutex_unlock(&acpi_link_lock); acpi_handle_err(handle, "Link active IRQ is 0!\n"); - return -1; + return -ENODEV; } link->refcnt++; mutex_unlock(&acpi_link_lock); @@ -647,7 +656,9 @@ int acpi_pci_link_allocate_irq(acpi_handle handle, int = index, int *triggering, if (name) *name =3D acpi_device_bid(link->device); acpi_handle_debug(handle, "Link is referenced\n"); - return link->irq.active; + *gsi =3D link->irq.active; + + return 0; } =20 /* diff --git a/drivers/xen/acpi.c b/drivers/xen/acpi.c index d2ee605c5ca1..eab28cfe9939 100644 --- a/drivers/xen/acpi.c +++ b/drivers/xen/acpi.c @@ -89,11 +89,11 @@ int xen_acpi_get_gsi_info(struct pci_dev *dev, int *trigger_out, int *polarity_out) { - int gsi; + u32 gsi; u8 pin; struct acpi_prt_entry *entry; int trigger =3D ACPI_LEVEL_SENSITIVE; - int polarity =3D acpi_irq_model =3D=3D ACPI_IRQ_MODEL_GIC ? + int ret, polarity =3D acpi_irq_model =3D=3D ACPI_IRQ_MODEL_GIC ? ACPI_ACTIVE_HIGH : ACPI_ACTIVE_LOW; =20 if (!dev || !gsi_out || !trigger_out || !polarity_out) @@ -105,17 +105,18 @@ int xen_acpi_get_gsi_info(struct pci_dev *dev, =20 entry =3D acpi_pci_irq_lookup(dev, pin); if (entry) { + ret =3D 0; if (entry->link) - gsi =3D acpi_pci_link_allocate_irq(entry->link, + ret =3D acpi_pci_link_allocate_irq(entry->link, entry->index, &trigger, &polarity, - NULL); + NULL, &gsi); else gsi =3D entry->index; } else - gsi =3D -1; + ret =3D -ENODEV; =20 - if (gsi < 0) + if (ret < 0) return -EINVAL; =20 *gsi_out =3D gsi; diff --git a/include/acpi/acpi_drivers.h b/include/acpi/acpi_drivers.h index b14d165632e7..402b97d12138 100644 --- a/include/acpi/acpi_drivers.h +++ b/include/acpi/acpi_drivers.h @@ -51,7 +51,7 @@ =20 int acpi_irq_penalty_init(void); int acpi_pci_link_allocate_irq(acpi_handle handle, int index, int *trigger= ing, - int *polarity, char **name); + int *polarity, char **name, u32 *gsi); int acpi_pci_link_free_irq(acpi_handle handle); =20 /* ACPI PCI Device Binding */ --=20 2.50.1