From nobody Thu Oct 30 23:25:34 2025 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) client-ip=192.237.175.120; envelope-from=xen-devel-bounces@lists.xenproject.org; helo=lists.xenproject.org; Authentication-Results: mx.zohomail.com; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; dmarc=fail(p=none dis=none) header.from=renesas.com Return-Path: Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) by mx.zohomail.com with SMTPS id 1751975241583146.29577708908323; Tue, 8 Jul 2025 04:47:21 -0700 (PDT) Received: from list by lists.xenproject.org with outflank-mailman.1036375.1408622 (Exim 4.92) (envelope-from ) id 1uZ6mk-0001iw-Td; Tue, 08 Jul 2025 11:46:54 +0000 Received: by outflank-mailman (output) from mailman id 1036375.1408622; Tue, 08 Jul 2025 11:46:54 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1uZ6mk-0001ip-Qr; Tue, 08 Jul 2025 11:46:54 +0000 Received: by outflank-mailman (input) for mailman id 1036375; Tue, 08 Jul 2025 11:46:54 +0000 Received: from se1-gles-sth1-in.inumbo.com ([159.253.27.254] helo=se1-gles-sth1.inumbo.com) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1uZ6mk-0001ij-1d for xen-devel@lists.xenproject.org; Tue, 08 Jul 2025 11:46:54 +0000 Received: from rein-hpcbdc09 (unknown [1.6.89.6]) by se1-gles-sth1.inumbo.com (Halon) with ESMTPS id 3b497c5f-5bf1-11f0-a317-13f23c93f187; Tue, 08 Jul 2025 13:46:51 +0200 (CEST) Received: by rein-hpcbdc09 (Postfix, from userid 1000) id 43C9A80C0306; Tue, 8 Jul 2025 17:16:47 +0530 (IST) X-Outflank-Mailman: Message body and most headers restored to incoming version X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" X-Inumbo-ID: 3b497c5f-5bf1-11f0-a317-13f23c93f187 From: Jahan Murudi To: xen-devel@lists.xenproject.org Cc: Jahan Murudi , Anthony PERARD Subject: [PATCH] tools/xentop: Add physical CPU statistics support Date: Tue, 8 Jul 2025 17:16:32 +0530 Message-Id: <20250708114632.3007693-1-jahan.murudi.zg@renesas.com> X-Mailer: git-send-email 2.34.1 MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-ZM-MESSAGEID: 1751975244591116600 Introduce a new '-p/--pcpus' flag to display physical CPU utilization metri= cs using xc_interface. This provides hypervisor-level CPU usage insights along= side existing domain statistics.This helps correlate VM resource usage with host= CPU load patterns. Changes: - Add pcpu.c/pcpu.h for PCPU stat collection logic - Link against libxenctrl for xc_handle access - Extend CLI with '-p' flag and output formatting - Forward declare xenstat_handle to access xc_handle - Include cleanup for PCPU resources in exit handler Example usage: xentop -p # Shows physical CPU stats xentop -bp # Batch mode Example output with '-p': NAME STATE CPU(%) MEM(%) VCPUS ... Domain-0 -----r 17.0 6.3 8 DomD -----r 767.0 38.1 8 Physical CPU Usage: =E2=94=8C=E2=94=80=E2=94=80=E2=94=80=E2=94=80=E2=94=80=E2=94=80=E2=94=80= =E2=94=AC=E2=94=80=E2=94=80=E2=94=80=E2=94=80=E2=94=80=E2=94=80=E2=94=80=E2= =94=80=E2=94=90 =E2=94=82 Core =E2=94=82 Usage =E2=94=82 =E2=94=9C=E2=94=80=E2=94=80=E2=94=80=E2=94=80=E2=94=80=E2=94=80=E2=94=80= =E2=94=BC=E2=94=80=E2=94=80=E2=94=80=E2=94=80=E2=94=80=E2=94=80=E2=94=80=E2= =94=80=E2=94=A4 =E2=94=82 0 =E2=94=82 98.1% =E2=94=82 =E2=94=82 ... =E2=94=82 ... =E2=94=82 =E2=94=82 7 =E2=94=82 97.3% =E2=94=82 =E2=94=94=E2=94=80=E2=94=80=E2=94=80=E2=94=80=E2=94=80=E2=94=80=E2=94=80= =E2=94=B4=E2=94=80=E2=94=80=E2=94=80=E2=94=80=E2=94=80=E2=94=80=E2=94=80=E2= =94=80=E2=94=98 Signed-off-by: Jahan Murudi --- tools/xentop/Makefile | 11 +++- tools/xentop/pcpu.c | 116 ++++++++++++++++++++++++++++++++++++++++++ tools/xentop/pcpu.h | 18 +++++++ tools/xentop/xentop.c | 30 +++++++++-- 4 files changed, 170 insertions(+), 5 deletions(-) create mode 100644 tools/xentop/pcpu.c create mode 100644 tools/xentop/pcpu.h diff --git a/tools/xentop/Makefile b/tools/xentop/Makefile index 70cc2211c5..91d3ea864a 100644 --- a/tools/xentop/Makefile +++ b/tools/xentop/Makefile @@ -15,6 +15,7 @@ include $(XEN_ROOT)/tools/Rules.mk =20 CFLAGS +=3D -DGCC_PRINTF $(CFLAGS_libxenstat) LDLIBS +=3D $(LDLIBS_libxenstat) $(CURSES_LIBS) $(TINFO_LIBS) $(SOCKET_LIB= S) -lm +LDLIBS +=3D $(LDLIBS_libxenctrl) CFLAGS +=3D -DHOST_$(XEN_OS) =20 # Include configure output (config.h) @@ -25,8 +26,14 @@ TARGETS :=3D xentop .PHONY: all all: $(TARGETS) =20 -xentop: xentop.o - $(CC) $(LDFLAGS) -o $@ $< $(LDLIBS) $(APPEND_LDFLAGS) +xentop: xentop.o pcpu.o + $(CC) $(LDFLAGS) -o $@ $^ $(LDLIBS) $(APPEND_LDFLAGS) + +pcpu.o: pcpu.c pcpu.h + $(CC) $(CFLAGS) -c $< -o $@ + +%.o: %.c + $(CC) $(CFLAGS) -c $< -o $@ =20 .PHONY: install install: all diff --git a/tools/xentop/pcpu.c b/tools/xentop/pcpu.c new file mode 100644 index 0000000000..6b0f2a8d3c --- /dev/null +++ b/tools/xentop/pcpu.c @@ -0,0 +1,116 @@ +#include "pcpu.h" +#include +#include +#include +#include + +#define MAX_PCPUS 128 + +// Convert Xen's idle time (nanoseconds) to microseconds to match gettimeo= fday() +#define XEN_IDLETIME_TO_USEC(idle) ((idle) / 1000) + +// File-scope variables (static for module privacy) +static pcpu_stat_t *pcpu_stats =3D NULL; +static int num_pcpus =3D 0; +static uint64_t *prev_idle =3D NULL; +static uint64_t *prev_total =3D NULL; + +int update_pcpu_stats(xc_interface *xch) +{ + struct xen_sysctl_cpuinfo info[MAX_PCPUS]; + struct timeval now; + int nr_cpus =3D 0; + int i; + + if (!xch || xc_getcpuinfo(xch, MAX_PCPUS, info, &nr_cpus) < 0) { + return -1; + } + + gettimeofday(&now, NULL); + uint64_t current_total =3D (uint64_t)now.tv_sec * 1000000 + now.tv_use= c; + + /* Allocate memory if needed */ + if (!pcpu_stats || nr_cpus > num_pcpus) { + pcpu_stat_t *new_stats =3D realloc(pcpu_stats, nr_cpus * sizeof(pc= pu_stat_t)); + uint64_t *new_prev_idle =3D realloc(prev_idle, nr_cpus * sizeof(ui= nt64_t)); + uint64_t *new_prev_total =3D realloc(prev_total, nr_cpus * sizeof(= uint64_t)); + + if (!new_stats || !new_prev_idle || !new_prev_total) { + free(new_stats); + free(new_prev_idle); + free(new_prev_total); + return -1; + } + + pcpu_stats =3D new_stats; + prev_idle =3D new_prev_idle; + prev_total =3D new_prev_total; + num_pcpus =3D nr_cpus; + + /* Initialize previous values (skip first calculation) */ + for (i =3D 0; i < nr_cpus; i++) { + prev_idle[i] =3D XEN_IDLETIME_TO_USEC(info[i].idletime); + prev_total[i] =3D current_total; + pcpu_stats[i].pcpu_id =3D i; + /* Default to 0% on first run */ + pcpu_stats[i].usage_pct =3D 0.0; + } + return 0; + } + + /* Calculate CPU usage */ + for (i =3D 0; i < nr_cpus; i++) { + uint64_t current_idle =3D XEN_IDLETIME_TO_USEC(info[i].idletime); + uint64_t idle_diff =3D current_idle - prev_idle[i]; + uint64_t total_diff =3D current_total - prev_total[i]; + =20 + if (total_diff > 0) { + double usage =3D 100.0 * (1.0 - ((double)idle_diff / total_dif= f)); + pcpu_stats[i].usage_pct =3D (usage < 0) ? 0 : (usage > 100) ? = 100 : usage; + } else { + pcpu_stats[i].usage_pct =3D 0.0; + } + pcpu_stats[i].pcpu_id =3D i; + /* Update history */ + prev_idle[i] =3D current_idle; + prev_total[i] =3D current_total; + } + + return 0; +} + +void print_pcpu_stats(void) +{ + if (!pcpu_stats || num_pcpus =3D=3D 0) { + printf("No PCPU data available\n"); + return; + } + + printf("\nPhysical CPU Usage:\n"); + =20 + // Print table header + printf("=E2=94=8C=E2=94=80=E2=94=80=E2=94=80=E2=94=80=E2=94=80=E2=94= =80=E2=94=80=E2=94=AC=E2=94=80=E2=94=80=E2=94=80=E2=94=80=E2=94=80=E2=94=80= =E2=94=80=E2=94=80=E2=94=90\n"); + printf("=E2=94=82 Core =E2=94=82 Usage =E2=94=82\n"); + printf("=E2=94=9C=E2=94=80=E2=94=80=E2=94=80=E2=94=80=E2=94=80=E2=94= =80=E2=94=80=E2=94=BC=E2=94=80=E2=94=80=E2=94=80=E2=94=80=E2=94=80=E2=94=80= =E2=94=80=E2=94=80=E2=94=A4\n"); + =20 + // Print each CPU's data + for (int i =3D 0; i < num_pcpus; i++) { + printf("=E2=94=82 %-5d =E2=94=82 %5.1f%% =E2=94=82\n", + pcpu_stats[i].pcpu_id, + pcpu_stats[i].usage_pct); + } + =20 + // Print table footer + printf("=E2=94=94=E2=94=80=E2=94=80=E2=94=80=E2=94=80=E2=94=80=E2=94= =80=E2=94=80=E2=94=B4=E2=94=80=E2=94=80=E2=94=80=E2=94=80=E2=94=80=E2=94=80= =E2=94=80=E2=94=80=E2=94=98\n"); +} + +void free_pcpu_stats(void) +{ + free(pcpu_stats); + free(prev_idle); + free(prev_total); + pcpu_stats =3D NULL; + prev_idle =3D NULL; + prev_total =3D NULL; + num_pcpus =3D 0; +} diff --git a/tools/xentop/pcpu.h b/tools/xentop/pcpu.h new file mode 100644 index 0000000000..889efacf3c --- /dev/null +++ b/tools/xentop/pcpu.h @@ -0,0 +1,18 @@ +#ifndef PCPU_H +#define PCPU_H + +#include +#include +#include + +typedef struct { + int pcpu_id; + float usage_pct; +} pcpu_stat_t; + +/* Public API */ +int update_pcpu_stats(xc_interface *xch); +void print_pcpu_stats(void); +void free_pcpu_stats(void); + +#endif // PCPU_H \ No newline at end of file diff --git a/tools/xentop/xentop.c b/tools/xentop/xentop.c index f5d6c19cf9..477299c883 100644 --- a/tools/xentop/xentop.c +++ b/tools/xentop/xentop.c @@ -24,6 +24,7 @@ #include #include #include +#include "pcpu.h" #include #include #include @@ -69,6 +70,12 @@ =20 #define INT_FIELD_WIDTH(n) ((unsigned int)(log10(n) + 1)) =20 +/* TEMPORARY: Forward declare the internal structure */ +struct xenstat_handle { + xc_interface *xc_handle; + /* Other members don't matter fo now */ +}; + /* * Function prototypes */ @@ -205,6 +212,7 @@ field_id sort_field =3D FIELD_DOMID; unsigned int first_domain_index =3D 0; unsigned int delay =3D 3; unsigned int batch =3D 0; +static unsigned int show_pcpus =3D 0; unsigned int loop =3D 1; unsigned int iterations =3D 0; int show_vcpus =3D 0; @@ -240,6 +248,7 @@ static void usage(const char *program) "-r, --repeat-header repeat table header before each domain\n" "-v, --vcpus output vcpu data\n" "-b, --batch output in batch mode, no user input accepted\n" + "-p, --pcpus show physical CPU stats\n" "-i, --iterations number of iterations before exiting\n" "-f, --full-name output the full domain name (not truncated)\= n" "-z, --dom0-first display dom0 first (ignore sorting)\n" @@ -267,6 +276,8 @@ static void cleanup(void) xenstat_free_node(cur_node); if(xhandle !=3D NULL) xenstat_uninit(xhandle); + + free_pcpu_stats(); } =20 /* Display the given message and gracefully exit */ @@ -1189,7 +1200,7 @@ static void top(void) fail("Failed to retrieve statistics from libxenstat\n"); =20 /* dump summary top information */ - if (!batch) + if (!batch && !show_pcpus) do_summary(); =20 /* Count the number of domains for which to report data */ @@ -1245,9 +1256,18 @@ static void top(void) do_vbd(domains[i]); } =20 - if (!batch) + if (!batch && !show_pcpus ) do_bottom_line(); =20 + if (show_pcpus && xhandle !=3D NULL ) { + if (update_pcpu_stats(xhandle->xc_handle) =3D=3D 0) { + print_pcpu_stats(); + } + else { + print("Error getting PCPU stats\n"); + } + } + free(domains); } =20 @@ -1272,12 +1292,13 @@ int main(int argc, char **argv) { "vcpus", no_argument, NULL, 'v' }, { "delay", required_argument, NULL, 'd' }, { "batch", no_argument, NULL, 'b' }, + { "pcpus", no_argument, NULL, 'p' }, { "iterations", required_argument, NULL, 'i' }, { "full-name", no_argument, NULL, 'f' }, { "dom0-first", no_argument, NULL, 'z' }, { 0, 0, 0, 0 }, }; - const char *sopts =3D "hVnxrvd:bi:fz"; + const char *sopts =3D "hVnxrvd:bpi:fz"; =20 if (atexit(cleanup) !=3D 0) fail("Failed to install cleanup handler.\n"); @@ -1312,6 +1333,9 @@ int main(int argc, char **argv) case 'b': batch =3D 1; break; + case 'p': + show_pcpus =3D 1; + break; case 'i': iterations =3D atoi(optarg); loop =3D 0; --=20 2.34.1