Now that the L2/L2CODE/L2DATA resources are merged together, alloc_enabled
doesn't mean anything, its the same as alloc_capable which indicates
CAT is supported by this cache.

Take the opportunity to kill of alloc_enabled and its helpers.

Signed-off-by: James Morse <james.mo...@arm.com>
---
 arch/x86/kernel/cpu/intel_rdt.c             | 3 ---
 arch/x86/kernel/cpu/intel_rdt.h             | 5 -----
 arch/x86/kernel/cpu/intel_rdt_ctrlmondata.c | 2 +-
 arch/x86/kernel/cpu/intel_rdt_rdtgroup.c    | 6 +++---
 include/linux/resctrl.h                     | 2 --
 5 files changed, 4 insertions(+), 14 deletions(-)

diff --git a/arch/x86/kernel/cpu/intel_rdt.c b/arch/x86/kernel/cpu/intel_rdt.c
index f6f1eceb366f..982897839711 100644
--- a/arch/x86/kernel/cpu/intel_rdt.c
+++ b/arch/x86/kernel/cpu/intel_rdt.c
@@ -157,7 +157,6 @@ static inline void cache_alloc_hsw_probe(void)
        r->cache.shareable_bits = 0xc0000;
        r->cache.min_cbm_bits = 2;
        r->alloc_capable = true;
-       r->alloc_enabled = true;
 
        rdt_alloc_capable = true;
 }
@@ -214,7 +213,6 @@ static bool rdt_get_mem_config(struct rdt_resource *r)
        r->data_width = 3;
 
        r->alloc_capable = true;
-       r->alloc_enabled = true;
 
        return true;
 }
@@ -234,7 +232,6 @@ static void rdt_get_cache_alloc_cfg(int idx, struct 
rdt_resource *r)
        r->cache.shareable_bits = ebx & r->default_ctrl;
        r->data_width = (r->cache.cbm_len + 3) / 4;
        r->alloc_capable = true;
-       r->alloc_enabled = true;
 }
 
 static int get_cache_id(int cpu, int level)
diff --git a/arch/x86/kernel/cpu/intel_rdt.h b/arch/x86/kernel/cpu/intel_rdt.h
index a4aba005cfea..735d7bd4bcd9 100644
--- a/arch/x86/kernel/cpu/intel_rdt.h
+++ b/arch/x86/kernel/cpu/intel_rdt.h
@@ -342,11 +342,6 @@ static inline struct rdt_resource *resctrl_inc(struct 
rdt_resource *res)
             r = resctrl_inc(r))                                                
        \
                if (r->mon_capable)
 
-#define for_each_alloc_enabled_rdt_resource(r)                               \
-       for (r = &rdt_resources_all[0].resctrl; r < 
&rdt_resources_all[RDT_NUM_RESOURCES].resctrl;\
-            r = resctrl_inc(r))                                                
        \
-               if (r->alloc_enabled)
-
 #define for_each_mon_enabled_rdt_resource(r)                                 \
        for (r = &rdt_resources_all[0].resctrl; r < 
&rdt_resources_all[RDT_NUM_RESOURCES].resctrl;\
             r = resctrl_inc(r))                                                
\
diff --git a/arch/x86/kernel/cpu/intel_rdt_ctrlmondata.c 
b/arch/x86/kernel/cpu/intel_rdt_ctrlmondata.c
index e8264637a4d3..ab8bc97dee0b 100644
--- a/arch/x86/kernel/cpu/intel_rdt_ctrlmondata.c
+++ b/arch/x86/kernel/cpu/intel_rdt_ctrlmondata.c
@@ -329,7 +329,7 @@ ssize_t rdtgroup_schemata_write(struct kernfs_open_file *of,
                        goto out;
        }
 
-       for_each_alloc_enabled_rdt_resource(r) {
+       for_each_alloc_capable_rdt_resource(r) {
                ret = resctrl_arch_update_domains(r);
                if (ret)
                        goto out;
diff --git a/arch/x86/kernel/cpu/intel_rdt_rdtgroup.c 
b/arch/x86/kernel/cpu/intel_rdt_rdtgroup.c
index 39038bdfa7d6..38cd463443e8 100644
--- a/arch/x86/kernel/cpu/intel_rdt_rdtgroup.c
+++ b/arch/x86/kernel/cpu/intel_rdt_rdtgroup.c
@@ -103,7 +103,7 @@ static void closid_init(void)
        int rdt_min_closid = 32;
 
        /* Compute rdt_min_closid across all resources */
-       for_each_alloc_enabled_rdt_resource(r)
+       for_each_alloc_capable_rdt_resource(r)
                rdt_min_closid = min(rdt_min_closid, r->num_closid);
 
        closid_free_map = BIT_MASK(rdt_min_closid) - 1;
@@ -1307,7 +1307,7 @@ static int create_schemata_list(void)
        int ret = 0;
        struct rdt_resource *r;
 
-       for_each_alloc_enabled_rdt_resource(r) {
+       for_each_alloc_capable_rdt_resource(r) {
                if (r->cdp_enabled) {
                        ret = add_schema(CDP_CODE, r);
                        ret |= add_schema(CDP_DATA, r);
@@ -1586,7 +1586,7 @@ static void rdt_kill_sb(struct super_block *sb)
        set_mba_sc(false);
 
        /*Put everything back to default values. */
-       for_each_alloc_enabled_rdt_resource(r)
+       for_each_capable_rdt_resource(r)
                reset_all_ctrls(r);
        cdp_disable_all();
        rmdir_all_sub();
diff --git a/include/linux/resctrl.h b/include/linux/resctrl.h
index 8b06ed8e7407..7a955ce93988 100644
--- a/include/linux/resctrl.h
+++ b/include/linux/resctrl.h
@@ -86,7 +86,6 @@ struct resctrl_membw {
 };
 
 /**
- * @alloc_enabled:     Is allocation enabled on this machine
  * @mon_enabled:       Is monitoring enabled for this feature
  * @cdp_enabled                Is CDP enabled for this resource
  * @alloc_capable:     Is allocation available on this machine
@@ -113,7 +112,6 @@ struct resctrl_membw {
  * @fflags:            flags to choose base and info files
  */
 struct rdt_resource {
-       bool                            alloc_enabled;
        bool                            mon_enabled;
        bool                            cdp_enabled;
        bool                            alloc_capable;
-- 
2.18.0

Reply via email to