Ie. use a mask based on smr_mask_bits to map all stream IDs of an SMMU
to one context.

This behaviour is controlled per SMMU node with DT property
"arm,smmu-mask-stream-ids" and is only allowed if just a single master
is attached to an SMMU. If the option is specified, all stream-ids
that are provided in DT for the single master have no relevance.

This is useful/convenient if a master has more than 8 stream-ids or if
not all stream-ids are known for a master device.

Signed-off-by: Andreas Herrmann <[email protected]>
---
 drivers/iommu/arm-smmu.c |   44 ++++++++++++++++++++++++++++++++------------
 1 file changed, 32 insertions(+), 12 deletions(-)

diff --git a/drivers/iommu/arm-smmu.c b/drivers/iommu/arm-smmu.c
index 87239e8..f877d02 100644
--- a/drivers/iommu/arm-smmu.c
+++ b/drivers/iommu/arm-smmu.c
@@ -50,6 +50,7 @@
 
 /* Driver options */
 #define ARM_SMMU_OPT_ISOLATE_DEVICES           (1 << 0)
+#define ARM_SMMU_OPT_MASK_STREAM_IDS           (1 << 1)
 
 /* Maximum number of stream IDs assigned to a single device */
 #define MAX_MASTER_STREAMIDS           8
@@ -362,6 +363,7 @@ struct arm_smmu_device {
 
        u32                             num_mapping_groups;
        DECLARE_BITMAP(smr_map, ARM_SMMU_MAX_SMRS);
+       u32                             smr_mask_bits;
 
        unsigned long                   input_size;
        unsigned long                   s1_output_size;
@@ -373,6 +375,7 @@ struct arm_smmu_device {
 
        struct list_head                list;
        struct rb_root                  masters;
+       u32                             num_masters;
 };
 
 struct arm_smmu_cfg {
@@ -410,6 +413,7 @@ struct arm_smmu_option_prop {
 
 static struct arm_smmu_option_prop arm_smmu_options [] = {
        { ARM_SMMU_OPT_ISOLATE_DEVICES, "arm,smmu-isolate-devices" },
+       { ARM_SMMU_OPT_MASK_STREAM_IDS, "arm,smmu-mask-stream-ids" },
        { 0, NULL},
 };
 
@@ -475,6 +479,7 @@ static int insert_smmu_master(struct arm_smmu_device *smmu,
 
        rb_link_node(&master->node, parent, new);
        rb_insert_color(&master->node, &smmu->masters);
+       smmu->num_masters++;
        return 0;
 }
 
@@ -507,8 +512,20 @@ static int register_smmu_master(struct arm_smmu_device 
*smmu,
        master->of_node         = masterspec->np;
        master->num_streamids   = masterspec->args_count;
 
-       for (i = 0; i < master->num_streamids; ++i)
-               master->streamids[i] = masterspec->args[i];
+       if (smmu->options & ARM_SMMU_OPT_MASK_STREAM_IDS) {
+               if (smmu->num_masters) {
+                       dev_err(dev, "%s not supported with multiple masters\n",
+                               get_driver_option_property(
+                                       ARM_SMMU_OPT_MASK_STREAM_IDS));
+                       return -EINVAL;
+               }
+               /* set fixed streamid (0) that will be used for masking */
+               master->num_streamids = 1;
+               master->streamids[0] = 0;
+       } else {
+               for (i = 0; i < master->num_streamids; ++i)
+                       master->streamids[i] = masterspec->args[i];
+       }
 
        return insert_smmu_master(smmu, master);
 }
@@ -1050,17 +1067,20 @@ static int arm_smmu_master_configure_smrs(struct 
arm_smmu_device *smmu,
                        goto err_free_smrs;
                }
 
-               smrs[i] = (struct arm_smmu_smr) {
-                       .idx    = idx,
-                       .mask   = 0, /* We don't currently share SMRs */
-                       .id     = master->streamids[i],
-               };
+               smrs[i].idx = idx;
+               smrs[i].id = master->streamids[i];
+
+               if (smmu->options & ARM_SMMU_OPT_MASK_STREAM_IDS)
+                       smrs[i].mask = smmu->smr_mask_bits;
+               else
+                       smrs[i].mask = 0;
        }
 
        /* It worked! Now, poke the actual hardware */
        for (i = 0; i < master->num_streamids; ++i) {
                u32 reg = SMR_VALID | smrs[i].id << SMR_ID_SHIFT |
                          smrs[i].mask << SMR_MASK_SHIFT;
+               dev_dbg(smmu->dev, "SMR%d: 0x%x\n", smrs[i].idx, reg);
                writel_relaxed(reg, gr0_base + ARM_SMMU_GR0_SMR(smrs[i].idx));
        }
 
@@ -1713,7 +1733,7 @@ static int arm_smmu_device_cfg_probe(struct 
arm_smmu_device *smmu)
        }
 
        if (id & ID0_SMS) {
-               u32 smr, sid, mask;
+               u32 smr, sid;
 
                smmu->features |= ARM_SMMU_FEAT_STREAM_MATCH;
                smmu->num_mapping_groups = (id >> ID0_NUMSMRG_SHIFT) &
@@ -1729,18 +1749,18 @@ static int arm_smmu_device_cfg_probe(struct 
arm_smmu_device *smmu)
                writel_relaxed(smr, gr0_base + ARM_SMMU_GR0_SMR(0));
                smr = readl_relaxed(gr0_base + ARM_SMMU_GR0_SMR(0));
 
-               mask = (smr >> SMR_MASK_SHIFT) & SMR_MASK_MASK;
+               smmu->smr_mask_bits = (smr >> SMR_MASK_SHIFT) & SMR_MASK_MASK;
                sid = (smr >> SMR_ID_SHIFT) & SMR_ID_MASK;
-               if ((mask & sid) != sid) {
+               if ((smmu->smr_mask_bits & sid) != sid) {
                        dev_err(smmu->dev,
                                "SMR mask bits (0x%x) insufficient for ID field 
(0x%x)\n",
-                               mask, sid);
+                               smmu->smr_mask_bits, sid);
                        return -ENODEV;
                }
 
                dev_notice(smmu->dev,
                           "\tstream matching with %u register groups, mask 
0x%x",
-                          smmu->num_mapping_groups, mask);
+                          smmu->num_mapping_groups, smmu->smr_mask_bits);
        }
 
        /* ID1 */
-- 
1.7.9.5

_______________________________________________
iommu mailing list
[email protected]
https://lists.linuxfoundation.org/mailman/listinfo/iommu

Reply via email to