Currently, its_send_vmapp operates on all ITSs. As we're about
to try and limit the amount of commands we send to ITSs that are
not involved in dealing with a given VM, let's redefine that
primitive so that it takes a target ITS as a parameter.

Signed-off-by: Marc Zyngier <marc.zyng...@arm.com>
---
 drivers/irqchip/irq-gic-v3-its.c | 23 ++++++++++++-----------
 1 file changed, 12 insertions(+), 11 deletions(-)

diff --git a/drivers/irqchip/irq-gic-v3-its.c b/drivers/irqchip/irq-gic-v3-its.c
index 202f43b30bd3..4583af37785b 100644
--- a/drivers/irqchip/irq-gic-v3-its.c
+++ b/drivers/irqchip/irq-gic-v3-its.c
@@ -918,21 +918,16 @@ static void its_send_vmovi(struct its_device *dev, u32 id)
        its_send_single_vcommand(dev->its, its_build_vmovi_cmd, &desc);
 }
 
-static void its_send_vmapp(struct its_vpe *vpe, bool valid)
+static void its_send_vmapp(struct its_node *its,
+                          struct its_vpe *vpe, bool valid)
 {
        struct its_cmd_desc desc;
-       struct its_node *its;
 
        desc.its_vmapp_cmd.vpe = vpe;
        desc.its_vmapp_cmd.valid = valid;
+       desc.its_vmapp_cmd.col = &its->collections[vpe->col_idx];
 
-       list_for_each_entry(its, &its_nodes, entry) {
-               if (!its->is_v4)
-                       continue;
-
-               desc.its_vmapp_cmd.col = &its->collections[vpe->col_idx];
-               its_send_single_vcommand(its, its_build_vmapp_cmd, &desc);
-       }
+       its_send_single_vcommand(its, its_build_vmapp_cmd, &desc);
 }
 
 static void its_send_vmovp(struct its_vpe *vpe)
@@ -2715,12 +2710,12 @@ static void its_vpe_irq_domain_activate(struct 
irq_domain *domain,
 
        /* Map the VPE to the first possible CPU */
        vpe->col_idx = cpumask_first(cpu_online_mask);
-       its_send_vmapp(vpe, true);
 
        list_for_each_entry(its, &its_nodes, entry) {
                if (!its->is_v4)
                        continue;
 
+               its_send_vmapp(its, vpe, true);
                its_send_vinvall(its, vpe);
        }
 }
@@ -2729,8 +2724,14 @@ static void its_vpe_irq_domain_deactivate(struct 
irq_domain *domain,
                                          struct irq_data *d)
 {
        struct its_vpe *vpe = irq_data_get_irq_chip_data(d);
+       struct its_node *its;
+
+       list_for_each_entry(its, &its_nodes, entry) {
+               if (!its->is_v4)
+                       continue;
 
-       its_send_vmapp(vpe, false);
+               its_send_vmapp(its, vpe, false);
+       }
 }
 
 static const struct irq_domain_ops its_vpe_domain_ops = {
-- 
2.11.0

Reply via email to