Hi Stephen,
> > +/* IOMMU types we support */
> > +static const struct vfio_iommu_type iommu_types[] = {
> > + /* x86 IOMMU, otherwise known as type 1 */
> > + { VFIO_TYPE1_IOMMU, "Type 1",
> &vfio_iommu_type1_dma_map},
> > + /* IOMMU-less mode */
> > + { VFIO_NOIOMMU_IOMMU, "No-IOMMU",
> &vfio_iommu_noiommu_dma_map},
> > +};
> > +
>
> Nit.. Why full-tab indent here?Readability mainly... at least it's more readable to me that way. I can change that if necessary. Thanks, Anatoly

