From nobody Tue Feb 10 02:08:43 2026 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 1754312828691799.2926835248334; Mon, 4 Aug 2025 06:07:08 -0700 (PDT) Received: from list by lists.xenproject.org with outflank-mailman.1069354.1433215 (Exim 4.92) (envelope-from ) id 1uiutz-0005PG-0A; Mon, 04 Aug 2025 13:06:55 +0000 Received: by outflank-mailman (output) from mailman id 1069354.1433215; Mon, 04 Aug 2025 13:06: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 1uiuty-0005P9-TU; Mon, 04 Aug 2025 13:06:54 +0000 Received: by outflank-mailman (input) for mailman id 1069354; Mon, 04 Aug 2025 13:06:53 +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 1uiutx-0005P3-Hr for xen-devel@lists.xenproject.org; Mon, 04 Aug 2025 13:06:53 +0000 Received: from rein-hpcbdc09 (unknown [1.6.89.6]) by se1-gles-sth1.inumbo.com (Halon) with ESMTPS id e180481f-7133-11f0-a321-13f23c93f187; Mon, 04 Aug 2025 15:06:50 +0200 (CEST) Received: by rein-hpcbdc09 (Postfix, from userid 1000) id 6AF1380C0979; Mon, 4 Aug 2025 18:36:46 +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: e180481f-7133-11f0-a321-13f23c93f187 From: Jahan Murudi To: xen-devel@lists.xenproject.org Cc: Jahan Murudi , Anthony PERARD Subject: [RFC PATCH v3] tools/xentop: Add physical CPU statistics support Date: Mon, 4 Aug 2025 18:36:43 +0530 Message-Id: <20250804130643.1046157-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: 1754312830832116600 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 hos= t 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 xentop -d 1 -p # With different delays 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 | 5 +- tools/xentop/pcpu.c | 152 ++++++++++++++++++++++++++++++++++++++++++ tools/xentop/pcpu.h | 15 +++++ tools/xentop/xentop.c | 57 ++++++++++------ 4 files changed, 207 insertions(+), 22 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..f514a6f7a8 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,8 @@ 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) =20 .PHONY: install install: all diff --git a/tools/xentop/pcpu.c b/tools/xentop/pcpu.c new file mode 100644 index 0000000000..53d6b9c30c --- /dev/null +++ b/tools/xentop/pcpu.c @@ -0,0 +1,152 @@ +/* SPDX-License-Identifier: GPL-2.0-only */ +/* + * Copyright (C) 2025 Renesas Electronics Corporation + */ + +#include +#include +#include +#include +#include +#include +#include "pcpu.h" + +#define MAX_PCPUS 128 + +typedef struct { + float usage_pct; +} pcpu_stat_t; + +static pcpu_stat_t *pcpu_stats =3D NULL; +static uint64_t *prev_idle =3D NULL; +static uint64_t *prev_time =3D NULL; +static int allocated_pcpus =3D 0; +static xc_interface *xc_handle =3D NULL; + +static void report_pcpu_error(const char *context) +{ + fprintf(stderr, "PCPU error: %s (%s)\n", context, strerror(errno)); +} + +int update_pcpu_stats(const struct timeval *now, unsigned int delay_sec) +{ + struct xen_sysctl_cpuinfo info[MAX_PCPUS]; + int detected_cpus =3D 0; + int ret, i; + uint64_t current_time =3D (uint64_t)now->tv_sec * 1000000ULL + now->tv= _usec; + + if (!xc_handle) { + xc_handle =3D xc_interface_open(NULL, NULL, 0); + if (!xc_handle) { + report_pcpu_error("xc_interface_open failed"); + return -1; + } + } + + ret =3D xc_getcpuinfo(xc_handle, MAX_PCPUS, info, &detected_cpus); + if (ret < 0) { + report_pcpu_error("xc_getcpuinfo failed"); + return -1; + } + + /* Allocate/reallocate memory if needed */ + if (!pcpu_stats || detected_cpus > allocated_pcpus) { + pcpu_stat_t *new_stats =3D realloc(pcpu_stats, + detected_cpus * sizeof(*pcpu_stats)); + if (!new_stats) goto alloc_error; + + uint64_t *new_prev_idle =3D realloc(prev_idle, + detected_cpus * sizeof(*prev_idle)); + if (!new_prev_idle) { + free(new_stats); + goto alloc_error; + } + + uint64_t *new_prev_time =3D realloc(prev_time, + detected_cpus * sizeof(*prev_time)); + if (!new_prev_time) { + free(new_stats); + free(new_prev_idle); + goto alloc_error; + } + + pcpu_stats =3D new_stats; + prev_idle =3D new_prev_idle; + prev_time =3D new_prev_time; + allocated_pcpus =3D detected_cpus; + + /* Initialize new entries */ + for (i =3D 0; i < detected_cpus; i++) { + prev_idle[i] =3D info[i].idletime / 1000; /* ns->us */ + prev_time[i] =3D current_time; + pcpu_stats[i].usage_pct =3D 0.0; + } + return 0; + } + + /* Calculate CPU usage with delay normalization */ + for (i =3D 0; i < detected_cpus; i++) { + uint64_t current_idle =3D info[i].idletime / 1000; + uint64_t idle_diff =3D current_idle - prev_idle[i]; + uint64_t time_diff =3D current_time - prev_time[i]; + + /* Use configured delay when actual interval is too small */ + if (time_diff < 100000) { + time_diff =3D delay_sec * 1000000ULL; + } + + if (time_diff > 0) { + double usage =3D 100.0 * (1.0 - ((double)idle_diff / time_diff= )); + /* Clamp between 0-100% */ + pcpu_stats[i].usage_pct =3D (usage < 0) ? 0.0 : + (usage > 100) ? 100.0 : usage; + } else { + pcpu_stats[i].usage_pct =3D 0.0; + } + + prev_idle[i] =3D current_idle; + prev_time[i] =3D current_time; + } + + return 0; + +alloc_error: + free_pcpu_stats(); + errno =3D ENOMEM; + report_pcpu_error("memory allocation failed"); + return -1; +} + +void print_pcpu_stats(void) +{ + if (!pcpu_stats || allocated_pcpus =3D=3D 0) { + printf("\r\nNo PCPU data available\r\n"); + return; + } + + printf("\r\nPhysical CPU Usage:\r\n"); + printf("+-------+--------+\r\n"); + printf("| Core | Usage |\r\n"); + printf("+-------+--------+\r\n"); + + for (int i =3D 0; i < allocated_pcpus; i++) { + printf("| %-5d | %5.1f%% |\r\n", i, pcpu_stats[i].usage_pct); + } + + printf("+-------+--------+\r\n"); +} + +void free_pcpu_stats(void) +{ + if (xc_handle) { + xc_interface_close(xc_handle); + xc_handle =3D NULL; + } + free(pcpu_stats); + free(prev_idle); + free(prev_time); + pcpu_stats =3D NULL; + prev_idle =3D NULL; + prev_time =3D NULL; + allocated_pcpus =3D 0; +} diff --git a/tools/xentop/pcpu.h b/tools/xentop/pcpu.h new file mode 100644 index 0000000000..2ce86292a6 --- /dev/null +++ b/tools/xentop/pcpu.h @@ -0,0 +1,15 @@ +/* SPDX-License-Identifier: GPL-2.0-only */ +/* + * Copyright (C) 2025 Renesas Electronics Corporation + */ + +#ifndef __XENTOP_PCPU_H__ +#define __XENTOP_PCPU_H__ + +#include + +int update_pcpu_stats(const struct timeval *now, unsigned int delay); +void print_pcpu_stats(void); +void free_pcpu_stats(void); + +#endif /* __XENTOP_PCPU_H__ */ diff --git a/tools/xentop/xentop.c b/tools/xentop/xentop.c index f5d6c19cf9..dcf2f69b59 100644 --- a/tools/xentop/xentop.c +++ b/tools/xentop/xentop.c @@ -37,6 +37,7 @@ #endif =20 #include +#include "pcpu.h" =20 #define XENTOP_VERSION "1.0" =20 @@ -205,6 +206,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; @@ -230,22 +232,23 @@ static WINDOW *cwin; /* Print usage message, using given program name */ static void usage(const char *program) { - printf("Usage: %s [OPTION]\n" - "Displays ongoing information about xen vm resources \n\n" - "-h, --help display this help and exit\n" - "-V, --version output version information and exit\n" - "-d, --delay=3DSECONDS seconds between updates (default 3)\n" - "-n, --networks output vif network data\n" - "-x, --vbds output vbd block device data\n" - "-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" - "-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" - "\n" XENTOP_BUGSTO, - program); - return; + printf("Usage: %s [OPTION]\n" + "Displays ongoing information about xen vm resources \n\n" + "-h, --help display this help and exit\n" + "-V, --version output version information and exit\n" + "-d, --delay=3DSECONDS seconds between updates (default 3)\n" + "-n, --networks output vif network data\n" + "-x, --vbds output vbd block device data\n" + "-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 accep= ted\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 truncate= d)\n" + "-z, --dom0-first display dom0 first (ignore sorting)\n" + "\n" XENTOP_BUGSTO, + program); + return; } =20 /* Print program version information */ @@ -267,6 +270,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 +1194,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 +1250,17 @@ static void top(void) do_vbd(domains[i]); } =20 - if (!batch) + if (!batch && !show_pcpus) do_bottom_line(); =20 + if (show_pcpus) { + if (update_pcpu_stats(&curtime, delay) =3D=3D 0) { + print_pcpu_stats(); + } else { + fail("Error getting PCPU stats\n"); + } + } + free(domains); } =20 @@ -1271,13 +1284,14 @@ int main(int argc, char **argv) { "repeat-header", no_argument, NULL, 'r' }, { "vcpus", no_argument, NULL, 'v' }, { "delay", required_argument, NULL, 'd' }, - { "batch", no_argument, NULL, 'b' }, + { "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 +1326,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