x86/resctrl: Remove set_mba_sc()s control array re-initialisation
authorJames Morse <james.morse@arm.com>
Fri, 2 Sep 2022 15:48:14 +0000 (15:48 +0000)
committerBorislav Petkov <bp@suse.de>
Thu, 22 Sep 2022 14:08:20 +0000 (16:08 +0200)
set_mba_sc() enables the 'software controller' to regulate the bandwidth
based on the byte counters. This can be managed entirely in the parts
of resctrl that move to /fs/, without any extra support from the
architecture specific code. set_mba_sc() is called by rdt_enable_ctx()
during mount and unmount. It currently resets the arch code's ctrl_val[]
and mbps_val[] arrays.

The ctrl_val[] was already reset when the domain was created, and by
reset_all_ctrls() when the filesystem was last unmounted. Doing the work
in set_mba_sc() is not necessary as the values are already at their
defaults due to the creation of the domain, or were previously reset
during umount(), or are about to reset during umount().

Add a reset of the mbps_val[] in reset_all_ctrls(), allowing the code in
set_mba_sc() that reaches in to the architecture specific structures to
be removed.

Signed-off-by: James Morse <james.morse@arm.com>
Signed-off-by: Borislav Petkov <bp@suse.de>
Reviewed-by: Jamie Iles <quic_jiles@quicinc.com>
Reviewed-by: Shaopeng Tan <tan.shaopeng@fujitsu.com>
Reviewed-by: Reinette Chatre <reinette.chatre@intel.com>
Tested-by: Xin Hao <xhao@linux.alibaba.com>
Tested-by: Shaopeng Tan <tan.shaopeng@fujitsu.com>
Tested-by: Cristian Marussi <cristian.marussi@arm.com>
Link: https://lore.kernel.org/r/20220902154829.30399-7-james.morse@arm.com
arch/x86/kernel/cpu/resctrl/rdtgroup.c

index 5830905a92d2c2ab401ad20188b53781def9c6e9..b32ceff8325a983002e932939c141ef177be7562 100644 (file)
@@ -1898,18 +1898,12 @@ void rdt_domain_reconfigure_cdp(struct rdt_resource *r)
 static int set_mba_sc(bool mba_sc)
 {
        struct rdt_resource *r = &rdt_resources_all[RDT_RESOURCE_MBA].r_resctrl;
-       struct rdt_hw_domain *hw_dom;
-       struct rdt_domain *d;
 
        if (!is_mbm_enabled() || !is_mba_linear() ||
            mba_sc == is_mba_sc(r))
                return -EINVAL;
 
        r->membw.mba_sc = mba_sc;
-       list_for_each_entry(d, &r->domains, list) {
-               hw_dom = resctrl_to_arch_dom(d);
-               setup_default_ctrlval(r, hw_dom->ctrl_val, hw_dom->mbps_val);
-       }
 
        return 0;
 }
@@ -2327,8 +2321,10 @@ static int reset_all_ctrls(struct rdt_resource *r)
                hw_dom = resctrl_to_arch_dom(d);
                cpumask_set_cpu(cpumask_any(&d->cpu_mask), cpu_mask);
 
-               for (i = 0; i < hw_res->num_closid; i++)
+               for (i = 0; i < hw_res->num_closid; i++) {
                        hw_dom->ctrl_val[i] = r->default_ctrl;
+                       hw_dom->mbps_val[i] = MBA_MAX_MBPS;
+               }
        }
        cpu = get_cpu();
        /* Update CBM on this cpu if it's in cpu_mask. */