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 F2BDAC83039 for ; Mon, 30 Jun 2025 18:14:41 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Subject:Cc:To:From:Message-ID:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=nKradABHZILoYTzzU6NYdMfKFrdMm81bSyaUpgJa5VI=; b=xaOnR53/8Br1zQ QVbdTi3WnWq99rqWzeKYdxdH9ttpb2bMmNI+qjJx3ebaXLm5J4cuT4iChrofuzyKHyjzGE7mMfcXp tyccGm6gsnvBBWupCDkFtiEBU3M6JLTZtF4KgxyEVyiPpjeb4w++CFzkB4NX6DoYkI7b7TdFDZEQx 1OWDLnsQ9bZsUFwJQ7ZtPt9DKXgWyRVlDrCZYYfy0BZFPOmUS/BEBCr5Ji2IqPliz8atWxNaUEgK8 XO5Neirw7ZfuchL30KEcM8dho6azDHNtxjrSXnW5lf6KCRcXtVQVLKMHja17MW/KgLOd8UmKRJMfl Ih76QHwHHCxeQMNiya4Q==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1uWJ1Z-000000039gT-3C3O; Mon, 30 Jun 2025 18:14:37 +0000 Received: from dfw.source.kernel.org ([2604:1380:4641:c500::1]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1uWIDg-0000000345T-0P8G; Mon, 30 Jun 2025 17:23:05 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id 907AA5C643F; Mon, 30 Jun 2025 17:23:03 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 31786C4CEE3; Mon, 30 Jun 2025 17:23:03 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1751304183; bh=w15NJ/7DEmXwWaHvvIBuUtaQ5bXCQ49qvYa4CYdrJ3w=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=ovAGC3m9hCEDf7z8XhoEMS+86eLdhSN8xKRvinXQZXM+UKnJ9ru4EA7hvpRwrmxn7 QkoZEGSzoM8Far17Ebs4XO/8Guft6TtUul/p/djwqAv8+PUtpZSd97RfK89Jnag32a ooFhe9orMaCf5hNHkw17dbpEO8dwTAiBoaLvj871pAVTNrYf0yLp/tguJau3+/2QND 7eG6NNlCBkMOXdRaWVZOwcwUi8uRJtvz4UzP61FoMVUakRWkTfaEffkKHUeqNpDVP+ wib2pk/PQ1v9mNr+LTNw89F6+nxG+vsPKXSq0ce7hcJWoORTAPD25/bKuBH/izLsJe sT8a6wtyFsuIg== 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 1uWIDc-00BK7Y-Ty; Mon, 30 Jun 2025 18:23:00 +0100 Date: Mon, 30 Jun 2025 18:23:00 +0100 Message-ID: <86ikkdb0uj.wl-maz@kernel.org> From: Marc Zyngier To: Bjorn Helgaas Cc: Bjorn Helgaas , Alyssa Rosenzweig , Rob Herring , Manivannan Sadhasivam , Lorenzo Pieralisi , Krzysztof =?UTF-8?B?V2lsY3p5xYRza2k=?= , Janne Grunau , Geert Uytterhoeven , linux-pci@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-riscv@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 0/3] PCI: host-generic: Fix driver_data overwriting bugs In-Reply-To: <20250630170601.GA1785925@bhelgaas> References: <20250625111806.4153773-1-maz@kernel.org> <20250630170601.GA1785925@bhelgaas> 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/30.1 (aarch64-unknown-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") X-SA-Exim-Connect-IP: 185.219.108.64 X-SA-Exim-Rcpt-To: helgaas@kernel.org, bhelgaas@google.com, alyssa@rosenzweig.io, robh@kernel.org, mani@kernel.org, lpieralisi@kernel.org, kwilczynski@kernel.org, j@jannau.net, geert+renesas@glider.be, linux-pci@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-riscv@lists.infradead.org, linux-kernel@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-20250630_102304_220289_6FB2C67D X-CRM114-Status: GOOD ( 25.58 ) X-BeenThere: linux-riscv@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-riscv" Errors-To: linux-riscv-bounces+linux-riscv=archiver.kernel.org@lists.infradead.org On Mon, 30 Jun 2025 18:06:01 +0100, Bjorn Helgaas wrote: > > On Wed, Jun 25, 2025 at 12:18:03PM +0100, Marc Zyngier wrote: > > Geert reports that some drivers do rely on the device driver_data > > field containing a pointer to the bridge structure at the point of > > initialising the root port, while this has been recently changed to > > contain some other data for the benefit of the Apple PCIe driver. > > > > This small series builds on top of Geert previously posted (and > > included as a prefix for reference) fix for the Microchip driver, > > which breaks the Apple driver. This is basically swapping a regression > > for another, which isn't a massive deal at this stage, as the > > follow-up patch fixes things for the Apple driver by adding extra > > tracking. > > Is there a bisection hole between patches 1 and 2? > > 1: PCI: host-generic: Set driver_data before calling gen_pci_init() > 2: PCI: apple: Add tracking of probed root ports > > If so, would it be practical to avoid the hole by reordering those > patches? Sure, but you said you already had queued patch #1, and what is in -rc1 already breaks Geert's box. So no matter the order, we break something at some point. If you want to only break one thing, then yes, swapping these two patches is the correct thing to do. Thanks, M. -- Without deviation from the norm, progress is not possible. _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv