Commit 3c56e822 authored by Alexey Kardashevskiy's avatar Alexey Kardashevskiy Committed by Michael Ellerman

vfio: powerpc/spapr: Disable DMA mappings on disabled container

At the moment DMA map/unmap requests are handled irrespective to
the container's state. This allows the user space to pin memory which
it might not be allowed to pin.

This adds checks to MAP/UNMAP that the container is enabled, otherwise
-EPERM is returned.
Signed-off-by: default avatarAlexey Kardashevskiy <aik@ozlabs.ru>
[aw: for the vfio related changes]
Acked-by: default avatarAlex Williamson <alex.williamson@redhat.com>
Reviewed-by: default avatarDavid Gibson <david@gibson.dropbear.id.au>
Reviewed-by: default avatarGavin Shan <gwshan@linux.vnet.ibm.com>
Signed-off-by: default avatarMichael Ellerman <mpe@ellerman.id.au>
parent 2d270df8
...@@ -318,6 +318,9 @@ static long tce_iommu_ioctl(void *iommu_data, ...@@ -318,6 +318,9 @@ static long tce_iommu_ioctl(void *iommu_data,
struct iommu_table *tbl = container->tbl; struct iommu_table *tbl = container->tbl;
unsigned long tce; unsigned long tce;
if (!container->enabled)
return -EPERM;
if (!tbl) if (!tbl)
return -ENXIO; return -ENXIO;
...@@ -362,6 +365,9 @@ static long tce_iommu_ioctl(void *iommu_data, ...@@ -362,6 +365,9 @@ static long tce_iommu_ioctl(void *iommu_data,
struct vfio_iommu_type1_dma_unmap param; struct vfio_iommu_type1_dma_unmap param;
struct iommu_table *tbl = container->tbl; struct iommu_table *tbl = container->tbl;
if (!container->enabled)
return -EPERM;
if (WARN_ON(!tbl)) if (WARN_ON(!tbl))
return -ENXIO; return -ENXIO;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment