[Xen-changelog] [xen master] xen/iommu: smmu: Advertise when the SMMU support coherent table walk

[patchbot@xxxxxxx]

commit 080dcb781e1bc3bb22f55a9dfdecb830ccbabe88
Author:     Julien Grall <julien.grall@xxxxxxxxxx>
AuthorDate: Mon Mar 2 15:42:43 2015 +0000
Commit:     Ian Campbell <ian.campbell@xxxxxxxxxx>
CommitDate: Mon Mar 2 16:42:56 2015 +0000

    xen/iommu: smmu: Advertise when the SMMU support coherent table walk
    
    When SMMU doesn't support coherent table walk, Xen may need to clean
    updated PT (see commit 4c5f4cb "xen/arm: p2m: Clean cache PT when the
    IOMMU doesn't support coherent walk").
    
    If one SMMU of the platform doesn't support coherent table walk, the
    feature is disabled for the whole platform. This is because device is
    assigned to a domain after the page table are populated.
    
    This could impact performance on domain which doesn't use device
    passthrough. But, as the spec strongly recommends the support of this
    feature for mainstream platform, I expect server will always have SMMUs
    supporting coherent table walk. If not, we may need to enable this feature
    per-domain.
    
    Signed-off-by: Julien Grall <julien.grall@xxxxxxxxxx>
    Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
---
 xen/drivers/passthrough/arm/smmu.c |   35 +++++++++++++++++++++++++++++++++++
 1 files changed, 35 insertions(+), 0 deletions(-)

diff --git a/xen/drivers/passthrough/arm/smmu.c 
b/xen/drivers/passthrough/arm/smmu.c
index 7675767..a7a7da9 100644
--- a/xen/drivers/passthrough/arm/smmu.c
+++ b/xen/drivers/passthrough/arm/smmu.c
@@ -2531,6 +2531,13 @@ MODULE_LICENSE("GPL v2");
 /* Xen only supports stage-2 translation, so force the value to 2. */
 static int force_stage = 2;
 
+/*
+ * Platform features. It indicates the list of features supported by all
+ * SMMUs.
+ * Actually we only care about coherent table walk.
+ */
+static u32 platform_features = ARM_SMMU_FEAT_COHERENT_WALK;
+
 static void arm_smmu_iotlb_flush_all(struct domain *d)
 {
        struct arm_smmu_xen_domain *smmu_domain = 
domain_hvm_iommu(d)->arch.priv;
@@ -2668,6 +2675,10 @@ static int arm_smmu_iommu_domain_init(struct domain *d)
 
        domain_hvm_iommu(d)->arch.priv = xen_domain;
 
+       /* Coherent walk can be enabled only when all SMMUs support it. */
+       if (platform_features & ARM_SMMU_FEAT_COHERENT_WALK)
+               iommu_set_feature(d, IOMMU_FEAT_COHERENT_WALK);
+
        return 0;
 }
 
@@ -2738,10 +2749,28 @@ static const struct iommu_ops arm_smmu_iommu_ops = {
     .unmap_page = arm_smmu_unmap_page,
 };
 
+static __init const struct arm_smmu_device *find_smmu(const struct device *dev)
+{
+       struct arm_smmu_device *smmu;
+       bool found = false;
+
+       spin_lock(&arm_smmu_devices_lock);
+       list_for_each_entry(smmu, &arm_smmu_devices, list) {
+               if (smmu->dev == dev) {
+                       found = true;
+                       break;
+               }
+       }
+       spin_unlock(&arm_smmu_devices_lock);
+
+       return (found) ? smmu : NULL;
+}
+
 static __init int arm_smmu_dt_init(struct dt_device_node *dev,
                                   const void *data)
 {
        int rc;
+       const struct arm_smmu_device *smmu;
 
        /*
         * Even if the device can't be initialized, we don't want to
@@ -2755,6 +2784,12 @@ static __init int arm_smmu_dt_init(struct dt_device_node 
*dev,
 
        iommu_set_ops(&arm_smmu_iommu_ops);
 
+       /* Find the last SMMU added and retrieve its features. */
+       smmu = find_smmu(dt_to_dev(dev));
+       BUG_ON(smmu == NULL);
+
+       platform_features &= smmu->features;
+
        return 0;
 }
 
--
generated by git-patchbot for /home/xen/git/xen.git#master

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxx
http://lists.xensource.com/xen-changelog