x86/resctrl: Add domain offline callback for resctrl work

Because domains are exposed to user-space via resctrl, the filesystem
must update its state when CPU hotplug callbacks are triggered.

Some of this work is common to any architecture that would support
resctrl, but the work is tied up with the architecture code to
free the memory.

Move the monitor subdir removal and the cancelling of the mbm/limbo
works into a new resctrl_offline_domain() call. These bits are not
specific to the architecture. Grouping them in one function allows
that code to be moved to /fs/ and re-used by another architecture.

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-6-james.morse@arm.com

diff --git a/arch/x86/kernel/cpu/resctrl/core.c b/arch/x86/kernel/cpu/resctrl/core.c
index e37889f7a1a540cfb3d29ce3eb33cfcb7fdbef18..f69182973175e1b550538cea0abcab0f54707271 100644 (file)
--- a/arch/x86/kernel/cpu/resctrl/core.c
+++ b/arch/x86/kernel/cpu/resctrl/core.c
@@ -523,27 +523,8 @@ static void domain_remove_cpu(int cpu, struct rdt_resource *r)
 
        cpumask_clear_cpu(cpu, &d->cpu_mask);
        if (cpumask_empty(&d->cpu_mask)) {
-               /*
-                * If resctrl is mounted, remove all the
-                * per domain monitor data directories.
-                */
-               if (static_branch_unlikely(&rdt_mon_enable_key))
-                       rmdir_mondata_subdir_allrdtgrp(r, d->id);
+               resctrl_offline_domain(r, d);
                list_del(&d->list);
-               if (r->mon_capable && is_mbm_enabled())
-                       cancel_delayed_work(&d->mbm_over);
-               if (is_llc_occupancy_enabled() &&  has_busy_rmid(r, d)) {
-                       /*
-                        * When a package is going down, forcefully
-                        * decrement rmid->ebusy. There is no way to know
-                        * that the L3 was flushed and hence may lead to
-                        * incorrect counts in rare scenarios, but leaving
-                        * the RMID as busy creates RMID leaks if the
-                        * package never comes back.
-                        */
-                       __check_limbo(d, true);
-                       cancel_delayed_work(&d->cqm_limbo);
-               }
 
                /*
                 * rdt_domain "d" is going to be freed below, so clear
@@ -551,11 +532,8 @@ static void domain_remove_cpu(int cpu, struct rdt_resource *r)
                 */
                if (d->plr)
                        d->plr->d = NULL;
-
-               bitmap_free(d->rmid_busy_llc);
-               kfree(d->mbm_total);
-               kfree(d->mbm_local);
                domain_free(hw_dom);
+
                return;
        }
 

diff --git a/arch/x86/kernel/cpu/resctrl/internal.h b/arch/x86/kernel/cpu/resctrl/internal.h
index be48a682dbdbe3182866e716b78c0c3b80e245eb..e12b55f815bfda5a963596d20b3ec05215675500 100644 (file)
--- a/arch/x86/kernel/cpu/resctrl/internal.h
+++ b/arch/x86/kernel/cpu/resctrl/internal.h
@@ -522,8 +522,6 @@ void free_rmid(u32 rmid);
 int rdt_get_mon_l3_config(struct rdt_resource *r);
 void mon_event_count(void *info);
 int rdtgroup_mondata_show(struct seq_file *m, void *arg);
-void rmdir_mondata_subdir_allrdtgrp(struct rdt_resource *r,
-                                   unsigned int dom_id);
 void mon_event_read(struct rmid_read *rr, struct rdt_resource *r,
                    struct rdt_domain *d, struct rdtgroup *rdtgrp,
                    int evtid, int first);

diff --git a/arch/x86/kernel/cpu/resctrl/rdtgroup.c b/arch/x86/kernel/cpu/resctrl/rdtgroup.c
index 030a70326ccc76a28b099d107df8b6099f40808b..5830905a92d2c2ab401ad20188b53781def9c6e9 100644 (file)
--- a/arch/x86/kernel/cpu/resctrl/rdtgroup.c
+++ b/arch/x86/kernel/cpu/resctrl/rdtgroup.c
@@ -2499,14 +2499,12 @@ static int mon_addfile(struct kernfs_node *parent_kn, const char *name,
  * Remove all subdirectories of mon_data of ctrl_mon groups
  * and monitor groups with given domain id.
  */
-void rmdir_mondata_subdir_allrdtgrp(struct rdt_resource *r, unsigned int dom_id)
+static void rmdir_mondata_subdir_allrdtgrp(struct rdt_resource *r,
+                                          unsigned int dom_id)
 {
        struct rdtgroup *prgrp, *crgrp;
        char name[32];
 
-       if (!r->mon_capable)
-               return;
-
        list_for_each_entry(prgrp, &rdt_all_groups, rdtgroup_list) {
                sprintf(name, "mon_%s_%02d", r->name, dom_id);
                kernfs_remove_by_name(prgrp->mon.mon_data_kn, name);
@@ -3233,6 +3231,45 @@ out:
        return ret;
 }
 
+static void domain_destroy_mon_state(struct rdt_domain *d)
+{
+       bitmap_free(d->rmid_busy_llc);
+       kfree(d->mbm_total);
+       kfree(d->mbm_local);
+}
+
+void resctrl_offline_domain(struct rdt_resource *r, struct rdt_domain *d)
+{
+       lockdep_assert_held(&rdtgroup_mutex);
+
+       if (!r->mon_capable)
+               return;
+
+       /*
+        * If resctrl is mounted, remove all the
+        * per domain monitor data directories.
+        */
+       if (static_branch_unlikely(&rdt_mon_enable_key))
+               rmdir_mondata_subdir_allrdtgrp(r, d->id);
+
+       if (is_mbm_enabled())
+               cancel_delayed_work(&d->mbm_over);
+       if (is_llc_occupancy_enabled() && has_busy_rmid(r, d)) {
+               /*
+                * When a package is going down, forcefully
+                * decrement rmid->ebusy. There is no way to know
+                * that the L3 was flushed and hence may lead to
+                * incorrect counts in rare scenarios, but leaving
+                * the RMID as busy creates RMID leaks if the
+                * package never comes back.
+                */
+               __check_limbo(d, true);
+               cancel_delayed_work(&d->cqm_limbo);
+       }
+
+       domain_destroy_mon_state(d);
+}
+
 static int domain_setup_mon_state(struct rdt_resource *r, struct rdt_domain *d)
 {
        size_t tsize;

diff --git a/include/linux/resctrl.h b/include/linux/resctrl.h
index d512455b4c3a4750de546da909a8dace309b25c9..5d283bdd616254a7085848e7c87adbb29157eed9 100644 (file)
--- a/include/linux/resctrl.h
+++ b/include/linux/resctrl.h
@@ -193,5 +193,6 @@ int resctrl_arch_update_domains(struct rdt_resource *r, u32 closid);
 u32 resctrl_arch_get_config(struct rdt_resource *r, struct rdt_domain *d,
                            u32 closid, enum resctrl_conf_type type);
 int resctrl_online_domain(struct rdt_resource *r, struct rdt_domain *d);
+void resctrl_offline_domain(struct rdt_resource *r, struct rdt_domain *d);
 
 #endif /* _RESCTRL_H */