[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20251031181722.250851-2-atomlin@atomlin.com>
Date: Fri, 31 Oct 2025 14:17:21 -0400
From: Aaron Tomlin <atomlin@...mlin.com>
To: tony.luck@...el.com,
reinette.chatre@...el.com,
Dave.Martin@....com,
james.morse@....com,
babu.moger@....com,
tglx@...utronix.de,
mingo@...hat.com,
bp@...en8.de,
dave.hansen@...ux.intel.com
Cc: linux-kernel@...r.kernel.org
Subject: [PATCH 1/1] fs/resctrl: Show domain CPU list in schema output
The resctrl schemata output currently displays only the domain ID and
its configured value e.g., "0=ffff". This format lacks visibility into
the underlying topology, making it difficult for users to quickly verify
which logical CPUs belong to the listed resource domain.
This patch improves visibility and aids in validation when configuring
or monitoring resource groupings, changing the output format from
"0=ffff" to "0=ffff,0-15".
Signed-off-by: Aaron Tomlin <atomlin@...mlin.com>
---
fs/resctrl/ctrlmondata.c | 4 +++-
fs/resctrl/rdtgroup.c | 4 ++--
2 files changed, 5 insertions(+), 3 deletions(-)
diff --git a/fs/resctrl/ctrlmondata.c b/fs/resctrl/ctrlmondata.c
index 0d0ef54fc4de..5ad12d15b350 100644
--- a/fs/resctrl/ctrlmondata.c
+++ b/fs/resctrl/ctrlmondata.c
@@ -402,7 +402,9 @@ static void show_doms(struct seq_file *s, struct resctrl_schema *schema, int clo
ctrl_val = resctrl_arch_get_config(r, dom, closid,
schema->conf_type);
- seq_printf(s, schema->fmt_str, dom->hdr.id, ctrl_val);
+ seq_printf(s, schema->fmt_str, dom->hdr.id, ctrl_val,
+ cpumask_pr_args(&dom->hdr.cpu_mask));
+
sep = true;
}
seq_puts(s, "\n");
diff --git a/fs/resctrl/rdtgroup.c b/fs/resctrl/rdtgroup.c
index 0320360cd7a6..cec89ae549e0 100644
--- a/fs/resctrl/rdtgroup.c
+++ b/fs/resctrl/rdtgroup.c
@@ -2661,10 +2661,10 @@ static int schemata_list_add(struct rdt_resource *r, enum resctrl_conf_type type
switch (r->schema_fmt) {
case RESCTRL_SCHEMA_BITMAP:
- s->fmt_str = "%d=%x";
+ s->fmt_str = "%d=%x,%*pbl";
break;
case RESCTRL_SCHEMA_RANGE:
- s->fmt_str = "%d=%u";
+ s->fmt_str = "%d=%u,%*pbl";
break;
}
--
2.51.0
Powered by blists - more mailing lists