On Mon, Jan 02, 2017 at 06:42:36PM +0530, Sricharan R wrote:
> From: Mitchel Humpherys <[email protected]>
> 
> Add the IOMMU_PRIV attribute, which is used to indicate privileged
> mappings.
> 
> Reviewed-by: Robin Murphy <[email protected]>
> Tested-by: Robin Murphy <[email protected]>
> Signed-off-by: Mitchel Humpherys <[email protected]>
> Acked-by: Will Deacon <[email protected]>
> ---
>  include/linux/iommu.h | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/include/linux/iommu.h b/include/linux/iommu.h
> index 0ff5111..8c15ada 100644
> --- a/include/linux/iommu.h
> +++ b/include/linux/iommu.h
> @@ -31,6 +31,7 @@
>  #define IOMMU_CACHE  (1 << 2) /* DMA cache coherency */
>  #define IOMMU_NOEXEC (1 << 3)
>  #define IOMMU_MMIO   (1 << 4) /* e.g. things like MSI doorbells */
> +#define IOMMU_PRIV   (1 << 5) /* privileged */

Since this is a new generic global flag, can you please add a more
verbose comment telling what a 'privileged' mapping is about in the
iommu-case? We should have a clear defined semantic for this flag or we
might end up with different meanings for it with different
iommu-hardware.


        Joerg

Reply via email to