From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b="YHHsfnmj" Received: from mgamail.intel.com (mgamail.intel.com [134.134.136.20]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 87C27C4; Tue, 28 Nov 2023 19:11:41 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1701227501; x=1732763501; h=message-id:date:mime-version:cc:subject:to:references: from:in-reply-to:content-transfer-encoding; bh=AFgY7sxn3CNPZK+1LW7PN8hahu/sESx2ATRSHU4pdQk=; b=YHHsfnmjrutA6rqxIvt5fcrwsUDJSRZJzEw3pJnO8LKM2IXO6yygaK0K 09MGURNSkX4Aj3YVf1wzSgvFKkAsaWrrsstdJ0v4NLTog29GaFuphLBj/ iHCuZHGWoehP5jjka3IJHeLHcfE/fviJZ2yGJ9YvmAMqVVSDRaOfzMJD+ 2Cs4MvaLHs6qgiUS9sNL4eOgfRU9z7EkJA7/nB/oGdSJ3nzGd7XsX7qp+ vlaikpKU5LY6h06p0x5VNrMT2Nz7Qhz6wx/L1jFxAGOSSDVQeVjb4AzCy IwAKbbhUTxpsWn/+JhmJYojpYU6bywzYF07TmLCLxbIKvYh2gcDouqwU9 A==; X-IronPort-AV: E=McAfee;i="6600,9927,10908"; a="383476185" X-IronPort-AV: E=Sophos;i="6.04,234,1695711600"; d="scan'208";a="383476185" Received: from fmsmga008.fm.intel.com ([10.253.24.58]) by orsmga101.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 28 Nov 2023 19:11:40 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10908"; a="834859960" X-IronPort-AV: E=Sophos;i="6.04,234,1695711600"; d="scan'208";a="834859960" Received: from blu2-mobl.ccr.corp.intel.com (HELO [10.254.211.119]) ([10.254.211.119]) by fmsmga008-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 28 Nov 2023 19:11:27 -0800 Message-ID: <13db755e-682e-4550-9da2-a18628fc041c@linux.intel.com> Date: Wed, 29 Nov 2023 11:11:27 +0800 Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Cc: baolu.lu@linux.intel.com, Christoph Hellwig , Jerry Snitselaar , Hector Martin , Moritz Fischer , patches@lists.linux.dev, "Rafael J. Wysocki" , Rob Herring , Thierry Reding Subject: Re: [PATCH 04/10] iommu: Mark dev_iommu_get() with lockdep Content-Language: en-US To: Jason Gunthorpe , David Airlie , Alyssa Rosenzweig , Albert Ou , asahi@lists.linux.dev, Catalin Marinas , Danilo Krummrich , Daniel Vetter , Dexuan Cui , devicetree@vger.kernel.org, dmaengine@vger.kernel.org, dri-devel@lists.freedesktop.org, David Woodhouse , Frank Rowand , Hanjun Guo , Haiyang Zhang , iommu@lists.linux.dev, Jon Hunter , Joerg Roedel , Karol Herbst , Krzysztof Kozlowski , "K. Y. Srinivasan" , Laxman Dewangan , Len Brown , linux-acpi@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-hyperv@vger.kernel.org, linux-mips@vger.kernel.org, linux-riscv@lists.infradead.org, linux-snps-arc@lists.infradead.org, linux-tegra@vger.kernel.org, Russell King , Lorenzo Pieralisi , Lyude Paul , Marek Szyprowski , nouveau@lists.freedesktop.org, Palmer Dabbelt , Paul Walmsley , "Rafael J. Wysocki" , Rob Herring , Robin Murphy , Sudeep Holla , Suravee Suthikulpanit , Sven Peter , Thomas Bogendoerfer , Vineet Gupta , Vinod Koul , Wei Liu , Will Deacon References: <4-v1-720585788a7d+811b-iommu_fwspec_p1_jgg@nvidia.com> From: Baolu Lu In-Reply-To: <4-v1-720585788a7d+811b-iommu_fwspec_p1_jgg@nvidia.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 2023/11/29 8:48, Jason Gunthorpe wrote: > Allocation of dev->iommu must be done under the > iommu_probe_device_lock. Mark this with lockdep to discourage future > mistakes. > > Reviewed-by: Jerry Snitselaar > Tested-by: Hector Martin > Signed-off-by: Jason Gunthorpe > --- > drivers/iommu/iommu.c | 2 ++ > 1 file changed, 2 insertions(+) Reviewed-by: Lu Baolu Best regards, baolu