From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alex Williamson Subject: Re: [PATCH v2] drivers/vfio: Allow type-1 IOMMU instantiation with all ARM/ARM64 IOMMUs Date: Tue, 25 Sep 2018 13:43:22 -0600 Message-ID: <20180925134322.405ccae2@t450s.home> References: <20180913131529.11837-1-geert+renesas@glider.be> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20180913131529.11837-1-geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: iommu-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org Errors-To: iommu-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org To: Geert Uytterhoeven Cc: Stephen Rothwell , Laurent Pinchart , kvm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Jean-Philippe Brucker , Magnus Damm , linux-renesas-soc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, David Howells , iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org, Robin Murphy List-Id: iommu@lists.linux-foundation.org On Thu, 13 Sep 2018 15:15:29 +0200 Geert Uytterhoeven wrote: > Currently the type-1 IOMMU instantiation depends on "ARM_SMMU || > ARM_SMMU_V3", while it applies to other ARM/ARM64 platforms with an > IOMMU (e.g. Renesas VMSA-compatible IPMMUs). > > Instead of extending the list of IOMMU types on ARM platforms, replace > the list by "ARM || ARM64", like other architectures do. The feature is > still restricted to ARM/ARM64 platforms with an IOMMU by the dependency > on IOMMU_API. > > Signed-off-by: Geert Uytterhoeven > --- > Tested with sata_rcar on Renesas R-Car H3 ES2.0. > > This causes a trivial merge conflict with commit c01eaa95ad30897b ("Make > anon_inodes unconditional") in vfs/for-next. > > v2: > - Make the feature just depend on ARM || ARM64, instead of adding yet > another IPMMU_VMSA dependency, as suggested by Robin Murphy > . > --- > drivers/vfio/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/vfio/Kconfig b/drivers/vfio/Kconfig > index c84333eb5eb59bef..9de5ed38da830a91 100644 > --- a/drivers/vfio/Kconfig > +++ b/drivers/vfio/Kconfig > @@ -21,7 +21,7 @@ config VFIO_VIRQFD > menuconfig VFIO > tristate "VFIO Non-Privileged userspace driver framework" > depends on IOMMU_API > - select VFIO_IOMMU_TYPE1 if (X86 || S390 || ARM_SMMU || ARM_SMMU_V3) > + select VFIO_IOMMU_TYPE1 if (X86 || S390 || ARM || ARM64) > select ANON_INODES > help > VFIO provides a framework for secure userspace device drivers. Applied to vfio next branch for v4.20 with Robin and Simon's R-b. Thanks, Alex