* [PATCH v2] drivers/vfio: Allow type-1 IOMMU instantiation with all ARM/ARM64 IOMMUs
@ 2018-09-13 13:15 Geert Uytterhoeven
[not found] ` <20180913131529.11837-1-geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
0 siblings, 1 reply; 4+ messages in thread
From: Geert Uytterhoeven @ 2018-09-13 13:15 UTC (permalink / raw)
To: Alex Williamson, Robin Murphy, Jean-Philippe Brucker
Cc: Stephen Rothwell, Laurent Pinchart, Geert Uytterhoeven,
kvm-u79uwXL29TY76Z2rM5mHXA, Magnus Damm,
linux-renesas-soc-u79uwXL29TY76Z2rM5mHXA, David Howells,
iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA, Al Viro
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 <geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
---
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
<robin.murphy-5wv7dgnIgG8@public.gmane.org>.
---
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.
--
2.17.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v2] drivers/vfio: Allow type-1 IOMMU instantiation with all ARM/ARM64 IOMMUs
[not found] ` <20180913131529.11837-1-geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
@ 2018-09-13 13:24 ` Robin Murphy
2018-09-17 7:26 ` Simon Horman
2018-09-25 19:43 ` Alex Williamson
2 siblings, 0 replies; 4+ messages in thread
From: Robin Murphy @ 2018-09-13 13:24 UTC (permalink / raw)
To: Geert Uytterhoeven, Alex Williamson, Jean-Philippe Brucker
Cc: Stephen Rothwell, Laurent Pinchart, kvm-u79uwXL29TY76Z2rM5mHXA,
Magnus Damm, linux-renesas-soc-u79uwXL29TY76Z2rM5mHXA,
David Howells, iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA,
Al Viro
On 13/09/18 14:15, 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.
Reviewed-by: Robin Murphy <robin.murphy-5wv7dgnIgG8@public.gmane.org>
> Signed-off-by: Geert Uytterhoeven <geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
> ---
> 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
> <robin.murphy-5wv7dgnIgG8@public.gmane.org>.
> ---
> 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.
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2] drivers/vfio: Allow type-1 IOMMU instantiation with all ARM/ARM64 IOMMUs
[not found] ` <20180913131529.11837-1-geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
2018-09-13 13:24 ` Robin Murphy
@ 2018-09-17 7:26 ` Simon Horman
2018-09-25 19:43 ` Alex Williamson
2 siblings, 0 replies; 4+ messages in thread
From: Simon Horman @ 2018-09-17 7:26 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Stephen Rothwell, Laurent Pinchart, kvm-u79uwXL29TY76Z2rM5mHXA,
Jean-Philippe Brucker, Magnus Damm,
iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA, David Howells,
Alex Williamson, Al Viro,
linux-renesas-soc-u79uwXL29TY76Z2rM5mHXA, Robin Murphy
On Thu, Sep 13, 2018 at 03:15:29PM +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 <geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
Reviewed-by: Simon Horman <horms+renesas-/R6kz+dDXgpPR4JQBCEnsQ@public.gmane.org>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2] drivers/vfio: Allow type-1 IOMMU instantiation with all ARM/ARM64 IOMMUs
[not found] ` <20180913131529.11837-1-geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
2018-09-13 13:24 ` Robin Murphy
2018-09-17 7:26 ` Simon Horman
@ 2018-09-25 19:43 ` Alex Williamson
2 siblings, 0 replies; 4+ messages in thread
From: Alex Williamson @ 2018-09-25 19:43 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Stephen Rothwell, Laurent Pinchart, kvm-u79uwXL29TY76Z2rM5mHXA,
Jean-Philippe Brucker, Magnus Damm,
linux-renesas-soc-u79uwXL29TY76Z2rM5mHXA, David Howells,
iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA, Robin Murphy
On Thu, 13 Sep 2018 15:15:29 +0200
Geert Uytterhoeven <geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org> 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 <geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
> ---
> 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
> <robin.murphy-5wv7dgnIgG8@public.gmane.org>.
> ---
> 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
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-09-25 19:43 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-09-13 13:15 [PATCH v2] drivers/vfio: Allow type-1 IOMMU instantiation with all ARM/ARM64 IOMMUs Geert Uytterhoeven
[not found] ` <20180913131529.11837-1-geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
2018-09-13 13:24 ` Robin Murphy
2018-09-17 7:26 ` Simon Horman
2018-09-25 19:43 ` Alex Williamson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).