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 2FB09C021B8 for ; Sat, 1 Mar 2025 11:53:26 +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:Content-Type:MIME-Version: References:In-Reply-To:Subject:Cc:To:From:Message-ID:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=CfHLGCWuyrsPC+guXnb1mMUQVeO4fDvE/CwDUg4znsI=; b=dUg3ko+sowA05RUhjjtH0LUnnC AhmfmyxHi4JPRC8ISnIByQGKqEilvkNbZ69t1ONDkewTA8KXlrp8cRQpdw2WIEn4MpNgVVgW4G7Jt ky8RgrCNfBk9bFO3+YmGsRzYYFbHVrF48d6Hx5L6bUAwbzW9xFn0EeqJRhrMHbr8Kl0+bQa0aRu1J Ok6gyTyx0ILUcp494SUNyJ8CCqQOoDjjD5WPxLzhj9GuRcKJHmWTMwIo1yydK4weVY1OLTK4TJeG/ neRTW+1XuhQ2cbrhRrwv9MQLtVSzr9IN9QlsoLojs9+h/fTdSFm72Du5b6SpnSAyU4H8McCmsU3/e ABZjMUYA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1toLPA-0000000DsG5-3CzQ; Sat, 01 Mar 2025 11:53:16 +0000 Received: from dfw.source.kernel.org ([139.178.84.217]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1toLH4-0000000DrW3-1cXQ for linux-arm-kernel@lists.infradead.org; Sat, 01 Mar 2025 11:44:55 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id D71705C2953; Sat, 1 Mar 2025 11:42:36 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 54273C4CEDD; Sat, 1 Mar 2025 11:44:53 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1740829493; bh=bDBsRUkAcZO02Q5u156/Y9EgZnHcH5+wGnYr5yxlUMU=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=NUgeLJyJUL439NXL4ufMy1ioW0tAbo0T0VAbBijQxH6qFMe9Hdycj0x5Do5Ihra/a lVW/ZgjOZqFjB8vJOtr4FEvujsIkMMFhWrvIcXfOwBbW1zIZv6xFTdkIttZCvTbIcz tvFWeLbGRt9RoW8heZ8vsJ34B7M9ZztwVn7oRocCmXI6GSWCceEMcw35C/CrugcwUf IksoF9wjBBHOLPsoZCruczrLaXunZSM0y2/RAQqw2VzimDskW5i1VPeLnWmw2bs3MZ t0FuTYVTxewXwlB3B55tnd6tiM4Y3PsbqpUQsY5+bcnlPhQrhxEy2Ca/Ot+R9iNJmP G0h+OXk3yk64Q== Received: from sofa.misterjones.org ([185.219.108.64] helo=goblin-girl.misterjones.org) by disco-boy.misterjones.org with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.95) (envelope-from ) id 1toLH1-009Leu-3R; Sat, 01 Mar 2025 11:44:51 +0000 Date: Sat, 01 Mar 2025 11:44:50 +0000 Message-ID: <86mse5otzh.wl-maz@kernel.org> From: Marc Zyngier To: Frank Li Cc: Kishon Vijay Abraham I , "Rafael J. Wysocki" , Thomas Gleixner , Anup Patel , Greg Kroah-Hartman , Danilo Krummrich , Manivannan Sadhasivam , Krzysztof =?UTF-8?B?V2lsY3p5xYRza2k=?= , Bjorn Helgaas , Arnd Bergmann , Shuah Khan , Richard Zhu , Lucas Stach , Lorenzo Pieralisi , Rob Herring , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , Krzysztof Kozlowski , Conor Dooley , Niklas Cassel , dlemoal@kernel.org, jdmason@kudzu.us, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-pci@vger.kernel.org, linux-kselftest@vger.kernel.org, imx@lists.linux.dev, devicetree@vger.kernel.org Subject: Re: [PATCH v15 07/15] PCI: endpoint: pci-ep-msi: Add MSI address/data pair mutable check In-Reply-To: <20250211-ep-msi-v15-7-bcacc1f2b1a9@nxp.com> References: <20250211-ep-msi-v15-0-bcacc1f2b1a9@nxp.com> <20250211-ep-msi-v15-7-bcacc1f2b1a9@nxp.com> User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI-EPG/1.14.7 (Harue) FLIM-LB/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL-LB/10.8 EasyPG/1.0.0 Emacs/29.4 (aarch64-unknown-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") Content-Type: text/plain; charset=US-ASCII X-SA-Exim-Connect-IP: 185.219.108.64 X-SA-Exim-Rcpt-To: Frank.Li@nxp.com, kishon@kernel.org, rafael@kernel.org, tglx@linutronix.de, apatel@ventanamicro.com, gregkh@linuxfoundation.org, dakr@kernel.org, manivannan.sadhasivam@linaro.org, kw@linux.com, bhelgaas@google.com, arnd@arndb.de, shuah@kernel.org, hongxing.zhu@nxp.com, l.stach@pengutronix.de, lpieralisi@kernel.org, robh@kernel.org, shawnguo@kernel.org, s.hauer@pengutronix.de, kernel@pengutronix.de, festevam@gmail.com, krzk+dt@kernel.org, conor+dt@kernel.org, cassel@kernel.org, dlemoal@kernel.org, jdmason@kudzu.us, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-pci@vger.kernel.org, linux-kselftest@vger.kernel.org, imx@lists.linux.dev, devicetree@vger.kernel.org X-SA-Exim-Mail-From: maz@kernel.org X-SA-Exim-Scanned: No (on disco-boy.misterjones.org); SAEximRunCond expanded to false X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250301_034454_510387_E7559E2C X-CRM114-Status: GOOD ( 23.09 ) 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 Tue, 11 Feb 2025 19:22:00 +0000, Frank Li wrote: > > Some MSI controller change address/data pair when irq_set_affinity(). > Current PCI endpoint can't support this type MSI controller. So add flag > MSI_FLAG_MUTABLE in include/linux/msi.h and check it when allocate > doorbell. > > Signed-off-by: Frank Li > --- > change from v14 to v15 > - none > > change from v13 to v14 > - bring v10 back > > Change from v9 to v10 > - new patch > --- > drivers/pci/endpoint/pci-ep-msi.c | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/drivers/pci/endpoint/pci-ep-msi.c b/drivers/pci/endpoint/pci-ep-msi.c > index 549b55b864d0e..c0e2d806ee658 100644 > --- a/drivers/pci/endpoint/pci-ep-msi.c > +++ b/drivers/pci/endpoint/pci-ep-msi.c > @@ -44,6 +44,14 @@ int pci_epf_alloc_doorbell(struct pci_epf *epf, u16 num_db) > > dev_set_msi_domain(dev, dom); > > + if (!irq_domain_is_msi_parent(dom)) > + return -EINVAL; > + > + if (!irq_domain_is_msi_immutable(dom)) { > + dev_err(dev, "Can't support mutable address/data pair MSI controller\n"); > + return -EINVAL; > + } > + > msg = kcalloc(num_db, sizeof(struct pci_epf_doorbell_msg), GFP_KERNEL); > if (!msg) > return -ENOMEM; > I really don't think this brings much to the table. These systems are not built by picking up random bits that can be put together by hand. They are integrated in an SoC, and I can't imagine that the MSI controller is picked randomly. So my conclusion is that this is either *designed* to work, or it doesn't exist, and that this code is just dead code. Thanks, M. -- Without deviation from the norm, progress is not possible.