From mboxrd@z Thu Jan 1 00:00:00 1970 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 E8A3D1DBF3A; Sun, 24 Mar 2024 22:51:39 +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=1711320701; cv=none; b=LwMFCDHDj3O6n5c7KTieE6g5Rv6OCVKIsHPyyNwxulOcNJXf7JStUHKEQ+Pw4HmCSBDH5JwiVnzx78tm5wkZ+DEoXTimnH1s69M7AeJEdMxvR8kC/ObOXTs07P4IgYy58Nku5wVEc64lff87vAUhHidMNaZ3TezwGYAoZffC8FA= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1711320701; c=relaxed/simple; bh=Zb0MMyK9GPqZrc9DkGXJxkLXDfwEcBT5Gn5m/Ks8+fw=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=ZeLdqyfVo3NRHPCuyq3bFT/ZZ1oeRxJEcGrBupZBw55ZdtogDh7/LrAoQiQCn0r+ENvCcrsAE6nl4NLvislI2RUcPDl+L7T5iglJBd9ObKPAXZShPQDGAW4ZedwdIrKtahYXD1ZGfYgPaR1PIItm8BvSBd7zXDNlGJpYxrHpoWM= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=RBmvrhY9; 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="RBmvrhY9" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 1AAFCC43330; Sun, 24 Mar 2024 22:51:39 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1711320699; bh=Zb0MMyK9GPqZrc9DkGXJxkLXDfwEcBT5Gn5m/Ks8+fw=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=RBmvrhY93cvZNbznUtpB7zQUvptFnmHpEcDeDY3Wq6qniPMsoY4FUckOaryZPOCa2 ScuP0/e4efgsOgOZYSPxgSbVQ8tgvAJ+kDPqR2XWDd7Y91sGc5Q9q85U7cEkxSv/zT 77Xr2TL96ahOz9+X4Sf4zLI3v9ieNfvTash+kVXaipdexDCEH/GvSjBcK8RRDN7JG2 nspyGDImbFzU0k7WAQV7MhFdFR2+a1ocPw0ZHAelr3UHyNhe2/yri1T+/EUjdak2ee E/sXXlXF15ZyX3dZIwJonYh76twfQcxtlo9mhZFS043qIQGVkUmsjEIL73ERoMSRJi LlN/8M9/XcNCw== From: Sasha Levin To: linux-kernel@vger.kernel.org, stable@vger.kernel.org Cc: Mario Limonciello , Vasant Hegde , Joerg Roedel , Sasha Levin Subject: [PATCH 6.7 262/713] iommu/amd: Mark interrupt as managed Date: Sun, 24 Mar 2024 18:39:48 -0400 Message-ID: <20240324224720.1345309-263-sashal@kernel.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240324224720.1345309-1-sashal@kernel.org> References: <20240324224720.1345309-1-sashal@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-stable: review X-Patchwork-Hint: Ignore Content-Transfer-Encoding: 8bit From: Mario Limonciello [ Upstream commit 0feda94c868d396fac3b3cb14089d2d989a07c72 ] On many systems that have an AMD IOMMU the following sequence of warnings is observed during bootup. ``` pci 0000:00:00.2 can't derive routing for PCI INT A pci 0000:00:00.2: PCI INT A: not connected ``` This series of events happens because of the IOMMU initialization sequence order and the lack of _PRT entries for the IOMMU. During initialization the IOMMU driver first enables the PCI device using pci_enable_device(). This will call acpi_pci_irq_enable() which will check if the interrupt is declared in a PCI routing table (_PRT) entry. According to the PCI spec [1] these routing entries are only required under PCI root bridges: The _PRT object is required under all PCI root bridges The IOMMU is directly connected to the root complex, so there is no parent bridge to look for a _PRT entry. The first warning is emitted since no entry could be found in the hierarchy. The second warning is then emitted because the interrupt hasn't yet been configured to any value. The pin was configured in pci_read_irq() but the byte in PCI_INTERRUPT_LINE return 0xff which means "Unknown". After that sequence of events pci_enable_msi() is called and this will allocate an interrupt. That is both of these warnings are totally harmless because the IOMMU uses MSI for interrupts. To avoid even trying to probe for a _PRT entry mark the IOMMU as IRQ managed. This avoids both warnings. Link: https://uefi.org/htmlspecs/ACPI_Spec_6_4_html/06_Device_Configuration/Device_Configuration.html?highlight=_prt#prt-pci-routing-table [1] Signed-off-by: Mario Limonciello Fixes: cffe0a2b5a34 ("x86, irq: Keep balance of IOAPIC pin reference count") Reviewed-by: Vasant Hegde Link: https://lore.kernel.org/r/20240122233400.1802-1-mario.limonciello@amd.com Signed-off-by: Joerg Roedel Signed-off-by: Sasha Levin --- drivers/iommu/amd/init.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/iommu/amd/init.c b/drivers/iommu/amd/init.c index 64bcf3df37ee5..7f65f3ecb231e 100644 --- a/drivers/iommu/amd/init.c +++ b/drivers/iommu/amd/init.c @@ -2068,6 +2068,9 @@ static int __init iommu_init_pci(struct amd_iommu *iommu) /* Prevent binding other PCI device drivers to IOMMU devices */ iommu->dev->match_driver = false; + /* ACPI _PRT won't have an IRQ for IOMMU */ + iommu->dev->irq_managed = 1; + pci_read_config_dword(iommu->dev, cap_ptr + MMIO_CAP_HDR_OFFSET, &iommu->cap); -- 2.43.0