From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 71022C02180 for ; Wed, 15 Jan 2025 21:01:58 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To: Content-Transfer-Encoding:Content-Type:MIME-Version:Message-ID:Subject:Cc:To: From:Date:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:References:List-Owner; bh=l5qD6HofcM5BLpVMOAdbesnzquI5O51xyRmuV4KjVtE=; b=eI++DDNggUrjda23zwuOgFYHCP swIevH9QeqLtMMn4uFg9MwBuMJv5G9diixX7uYQ0dKWeys4BdFBE5EN+ULlFEqezECt3tgmaPHVju QBJn+xofEoOYiP+uzxAElPV6zRqez0iMtg8iBM6c/1H4Q7qD0TKyCNxrOgMjquk25xuPkl0TBKXkf /YOjlNfMeuSjbeR/Ggskj2iwMA9BJPBSITf6W9vC1T1asf3xCnyjya2xSoXuTDB0Fb7yPpo/JQiGM 8WyF1cNxwi9awwgi5prn0rgez2kY3nKX5NoNZQfn/q8jIVnPGGIbOZ/3rRl7eRMf8TgEeSv8Izrcs g+Rlqz9Q==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1tYAWI-0000000D4wB-2idh; Wed, 15 Jan 2025 21:01:46 +0000 Received: from nyc.source.kernel.org ([2604:1380:45d1:ec00::3]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1tYAUn-0000000D4XO-0IxU for linux-arm-kernel@lists.infradead.org; Wed, 15 Jan 2025 21:00:14 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by nyc.source.kernel.org (Postfix) with ESMTP id 19667A42503; Wed, 15 Jan 2025 20:58:24 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 9DDFEC4CED1; Wed, 15 Jan 2025 21:00:11 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1736974811; bh=RlPD0kUWIPW5R0PKSVU27JaRoL2dMi9vvo/1Qhhlx7Q=; h=Date:From:To:Cc:Subject:In-Reply-To:From; b=JLrz2I58zheY19rQK47qVvp+t77dq2WMCZsCyasfRIy09JD1KERUCk98ZXRnBnnvG o7i50NxFKCdsWe97sGCjZhlF6XwW1zEDJRH0wrr0gFCuGRAWnkC65twQLcDXMPuFCP dUsqKh2CRhaPyNAWNMp+Nqi/XqvCIcJ8FWTpAKMeCg5a3W6E+pLdwuwFwOi77PL86R hqoA1YQBuH6l7LcEbZ5tawJpXZ4cBt72pebRmdZ1982K6PkrkOMgDLAOnQg+B40PJC 6aj1IEJuKx5Ry8+YO4jwPUQZk+LO8utjRG6HdPnM2cq+Y53kNwPOAoch6Mlx9YCGqg uQwV5KiA3sbsQ== Date: Wed, 15 Jan 2025 15:00:10 -0600 From: Bjorn Helgaas To: Marc Zyngier Cc: Frank Li , Bjorn Helgaas , Richard Zhu , Lucas Stach , Lorenzo Pieralisi , Krzysztof =?utf-8?Q?Wilczy=C5=84ski?= , Manivannan Sadhasivam , Rob Herring , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, imx@lists.linux.dev, alyssa@rosenzweig.io, bpf@vger.kernel.org, broonie@kernel.org, jgg@ziepe.ca, joro@8bytes.org, lgirdwood@gmail.com, p.zabel@pengutronix.de, robin.murphy@arm.com, will@kernel.org Subject: Re: [PATCH v9 0/2] PCI: add enabe(disable)_device() hook for bridge Message-ID: <20250115210010.GA551375@bhelgaas> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <86bjw8wj05.wl-maz@kernel.org> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250115_130013_241867_5EBC2F4B X-CRM114-Status: GOOD ( 21.29 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Wed, Jan 15, 2025 at 08:58:50AM +0000, Marc Zyngier wrote: > On Tue, 14 Jan 2025 22:33:41 +0000, > Bjorn Helgaas wrote: > > > > On Tue, Jan 14, 2025 at 03:37:07PM -0500, Frank Li wrote: > > > Some system's IOMMU stream(master) ID bits(such as 6bits) less than > > > pci_device_id (16bit). It needs add hardware configuration to enable > > > pci_device_id to stream ID convert. > > > > > > https://lore.kernel.org/imx/20240622173849.GA1432357@bhelgaas/ > > > This ways use pcie bus notifier (like apple pci controller), when new PCIe > > > device added, bus notifier will call register specific callback to handle > > > look up table (LUT) configuration. > > > > > > https://lore.kernel.org/imx/20240429150842.GC1709920-robh@kernel.org/ > > > which parse dt's 'msi-map' and 'iommu-map' property to static config LUT > > > table (qcom use this way). This way is rejected by DT maintainer Rob. > > > > > > Above ways can resolve LUT take or stream id out of usage the problem. If > > > there are not enough stream id resource, not error return, EP hardware > > > still issue DMA to do transfer, which may transfer to wrong possition. > > > > > > Add enable(disable)_device() hook for bridge can return error when not > > > enough resource, and PCI device can't enabled. > > > > > > Basicallly this version can match Bjorn's requirement: > > > 1: simple, because it is rare that there are no LUT resource. > > > 2: EP driver probe failure when no LUT, but lspci can see such device. > > > > > > [ 2.164415] nvme nvme0: pci function 0000:01:00.0 > > > [ 2.169142] pci 0000:00:00.0: Error enabling bridge (-1), continuing > > > [ 2.175654] nvme 0000:01:00.0: probe with driver nvme failed with error -12 > > > > > > > lspci > > > 0000:00:00.0 PCI bridge: Philips Semiconductors Device 0000 > > > 0000:01:00.0 Non-Volatile memory controller: Micron Technology Inc 2100AI NVMe SSD [Nitro] (rev 03) > > > > > > To: Bjorn Helgaas > > > To: Richard Zhu > > > To: Lucas Stach > > > To: Lorenzo Pieralisi > > > To: Krzysztof Wilczyński > > > To: Manivannan Sadhasivam > > > To: Rob Herring > > > To: Shawn Guo > > > To: Sascha Hauer > > > To: Pengutronix Kernel Team > > > To: Fabio Estevam > > > Cc: linux-pci@vger.kernel.org > > > Cc: linux-kernel@vger.kernel.org > > > Cc: linux-arm-kernel@lists.infradead.org > > > Cc: imx@lists.linux.dev > > > Cc: Frank.li@nxp.com \ > > > Cc: alyssa@rosenzweig.io \ > > > Cc: bpf@vger.kernel.org \ > > > Cc: broonie@kernel.org \ > > > Cc: jgg@ziepe.ca \ > > > Cc: joro@8bytes.org \ > > > Cc: l.stach@pengutronix.de \ > > > Cc: lgirdwood@gmail.com \ > > > Cc: maz@kernel.org \ > > > Cc: p.zabel@pengutronix.de \ > > > Cc: robin.murphy@arm.com \ > > > Cc: will@kernel.org \ > > > Cc: Robin Murphy > > > Cc: Marc Zyngier > > > > > > Signed-off-by: Frank Li > > > > Applied to pci/controller/imx6 for v6.14, thanks! And thanks for your > > patience. > > While you're at it, could you please consider [1], which builds on top > of the same infrastructure to remove the Apple PCIe IOMMU hack? > > [1] https://lore.kernel.org/linux-pci/20241204150145.800408-1-maz@kernel.org/ Done, thanks for the reminder!