On 08/03/2018 08:22 PM, Paul Durrant wrote: > The name 'need_iommu' is a little confusing as it suggests a domain needs > to use the IOMMU but something might not be set up yet, when in fact it > doesn't become true until IOMMU mappings for the domain have been created. > > Two different meanings are also inferred from it in various places in the > code: > > - Some callers want to test whether a domain has IOMMU mappings > - Some callers want to test whether they need to synchronize the domain's > P2M and IOMMU mappings > > This patch therefore creates two new boolean flags, 'has_iommu_pt' and > 'sync_iommu_pt' to convey those meanings separately and creates the > corresponding macros. All callers of need_iommu() are then modified to > use the macro appropriate to what they are trying to test. > > NOTE: The test of need_iommu(d) in the AMD IOMMU initialization code has > been replaced with a test of iommu_dom0_strict since this more > accurately reflects the meaning of the test and brings it into > line with a similar test in the Intel VT-d code. > > Signed-off-by: Paul Durrant <paul.durr...@citrix.com>
Acked-by: Razvan Cojocaru <rcojoc...@bitdefender.com> Thanks, Razvan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
