From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from esa.microchip.iphmx.com (esa.microchip.iphmx.com [68.232.153.233]) (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 51365107B9 for ; Tue, 16 May 2023 07:12:41 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=microchip.com; i=@microchip.com; q=dns/txt; s=mchp; t=1684221161; x=1715757161; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=t8CijbnvNDIu5h7DZKaUt0LbJeKMTc2o7hsFHeeJoGY=; b=c4euYUl6qC5aZv1ANsjmJdP3bmpranfjEYwsAa+VT37E+zZfTUtunJwR vYSp5Gv5bFTpN4eWrDFCHjnez1u6/4X413adUl4HmX5u22HZHgBt87fV2 GlwkstPe01yAYQjGMQsb3eB3ot6DUuTdFnbfowX3/g2tBicRrUKCw6O4a irVWUMc+jrTOVj4grL7dcyrURXpGe2FIcKKh+Q3P+CKp0PvZNnQOB7S8a 7G8dx6nprliYy734VQ0CSisG45YO8LtgbyjjtrsB7gtBrHyZo0PHdPfkV OkUQ21ipqUyUqxC/ubGhbiO9i3DJt8WP6/slWvoMoXB7cTdX/IAsJ9qA5 g==; X-IronPort-AV: E=Sophos;i="5.99,278,1677567600"; d="asc'?scan'208";a="213481929" X-Amp-Result: UNKNOWN X-Amp-Original-Verdict: FILE UNKNOWN Received: from unknown (HELO email.microchip.com) ([170.129.1.10]) by esa3.microchip.iphmx.com with ESMTP/TLS/AES256-SHA256; 16 May 2023 00:12:33 -0700 Received: from chn-vm-ex02.mchp-main.com (10.10.87.72) by chn-vm-ex02.mchp-main.com (10.10.87.72) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.21; Tue, 16 May 2023 00:12:33 -0700 Received: from wendy (10.10.115.15) by chn-vm-ex02.mchp-main.com (10.10.85.144) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.21 via Frontend Transport; Tue, 16 May 2023 00:12:31 -0700 Date: Tue, 16 May 2023 08:12:10 +0100 From: Conor Dooley To: Alexandre Ghiti CC: Randy Dunlap , Robin Murphy , , Joerg Roedel , Will Deacon , , Conor Dooley , Subject: Re: [PATCH] iommu: PGTABLE_LPAE is also for RISCV Message-ID: <20230516-gristle-spill-e27c58770a47@wendy> References: <20230330060105.29460-1-rdunlap@infradead.org> <2529dd93-2cb2-6ed8-20c0-c424e6613717@arm.com> <833203b0-aee5-eb23-afb4-ff71a3024258@infradead.org> <280c0994-f7cd-bb0e-8c68-62f72a4a38ca@ghiti.fr> Precedence: bulk X-Mailing-List: iommu@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="kEpcCnhkORFemaUM" Content-Disposition: inline In-Reply-To: <280c0994-f7cd-bb0e-8c68-62f72a4a38ca@ghiti.fr> --kEpcCnhkORFemaUM Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, May 16, 2023 at 09:02:50AM +0200, Alexandre Ghiti wrote: > Hi Randy, >=20 >=20 > On 3/30/23 18:49, Randy Dunlap wrote: > >=20 > > On 3/30/23 09:34, Robin Murphy wrote: > > > On 2023-03-30 07:01, Randy Dunlap wrote: > > > > On riscv64, linux-next-20233030 (and for several days earlier), > > > > there is a kconfig warning: > > > >=20 > > > > WARNING: unmet direct dependencies detected for IOMMU_IO_PGTABLE_LP= AE > > > > =A0=A0 Depends on [n]: IOMMU_SUPPORT [=3Dy] && (ARM || ARM64 || CO= MPILE_TEST [=3Dn]) && !GENERIC_ATOMIC64 [=3Dn] > > > > =A0=A0 Selected by [y]: > > > > =A0=A0 - IPMMU_VMSA [=3Dy] && IOMMU_SUPPORT [=3Dy] && (ARCH_RENESA= S [=3Dy] || COMPILE_TEST [=3Dn]) && !GENERIC_ATOMIC64 [=3Dn] > > > >=20 > > > > and build errors: > > > >=20 > > > > riscv64-linux-ld: drivers/iommu/io-pgtable-arm.o: in function `.L14= 0': > > > > io-pgtable-arm.c:(.init.text+0x1e8): undefined reference to `alloc_= io_pgtable_ops' > > > > riscv64-linux-ld: drivers/iommu/io-pgtable-arm.o: in function `.L16= 8': > > > > io-pgtable-arm.c:(.init.text+0xab0): undefined reference to `free_i= o_pgtable_ops' > > > > riscv64-linux-ld: drivers/iommu/ipmmu-vmsa.o: in function `.L140': > > > > ipmmu-vmsa.c:(.text+0xbc4): undefined reference to `free_io_pgtable= _ops' > > > > riscv64-linux-ld: drivers/iommu/ipmmu-vmsa.o: in function `.L0 ': > > > > ipmmu-vmsa.c:(.text+0x145e): undefined reference to `alloc_io_pgtab= le_ops' > > > >=20 > > > > Add RISCV as an allowed ARCH dependency to fix these problems. > > > >=20 > > > > Fixes: d286a58bc8f4 ("iommu: Tidy up io-pgtable dependencies") > > > BTW that doesn't look like the right fix target - this presumably dat= es back as far as when ARCH_RENESAS was added to RISCV, such that it was po= ssible to start selecting IPMMU_VMSA without COMPILE_TEST? > > OK, I can go with that. Thanks for your help. > >=20 >=20 > Any updates on this fix? Superseeded by https://lore.kernel.org/linux-iommu/20230330165817.21920-1-rdunlap@infradea= d.org/ I guess that never went anywhere? --kEpcCnhkORFemaUM Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZGMsygAKCRB4tDGHoIJi 0vOKAQCLR9m9tYztuGYjhMQ0trXZPKE2Y8xVwNtfr5jJy14lRgD/WEmttGELpMve zitK847mqPuq6YE2XnjnZvsdGcD6cAc= =WJ81 -----END PGP SIGNATURE----- --kEpcCnhkORFemaUM--