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