xenpm doesn't provide any interesting usable features outside of x86,
skip building it if we are not x86.
Suggested-by: Jan Beulich <jbeulich@suse.com>
Signed-off-by: Teddy Astie <teddy.astie@vates.tech>
---
CHANGELOG.md | 3 +++
tools/misc/Makefile | 2 +-
2 files changed, 4 insertions(+), 1 deletion(-)
diff --git a/CHANGELOG.md b/CHANGELOG.md
index 3aaf598623..1fa58ce848 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -16,6 +16,9 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
deprecated in Xen 4.7 and noted not to work correctly with AMD CPUs from
2011 onwards.
+ - Removed xenpm on non-x86 platforms as it doesn't actually provide anything
+ useful outside of x86.
+
## [4.21.0](https://xenbits.xenproject.org/gitweb/?p=xen.git;a=shortlog;h=RELEASE-4.21.0) - 2025-11-19
### Changed
diff --git a/tools/misc/Makefile b/tools/misc/Makefile
index c26e544e83..672df02c3b 100644
--- a/tools/misc/Makefile
+++ b/tools/misc/Makefile
@@ -23,13 +23,13 @@ INSTALL_SBIN-$(CONFIG_X86) += xen-lowmemd
INSTALL_SBIN-$(CONFIG_X86) += xen-mceinj
INSTALL_SBIN-$(CONFIG_X86) += xen-memshare
INSTALL_SBIN-$(CONFIG_X86) += xen-mfndump
+INSTALL_SBIN-$(CONFIG_X86) += xenpm
INSTALL_SBIN-$(CONFIG_X86) += xen-ucode
INSTALL_SBIN-$(CONFIG_X86) += xen-vmtrace
INSTALL_SBIN += xencov
INSTALL_SBIN += xenhypfs
INSTALL_SBIN += xenlockprof
INSTALL_SBIN += xenperf
-INSTALL_SBIN += xenpm
INSTALL_SBIN += xenwatchdogd
INSTALL_SBIN += xen-access
INSTALL_SBIN += xen-livepatch
--
2.51.2
--
Teddy Astie | Vates XCP-ng Developer
XCP-ng & Xen Orchestra - Vates solutions
web: https://vates.techOn Fri, Dec 19, 2025 at 03:42:17PM +0000, Teddy Astie wrote: > xenpm doesn't provide any interesting usable features outside of x86, > skip building it if we are not x86. > > Suggested-by: Jan Beulich <jbeulich@suse.com> > Signed-off-by: Teddy Astie <teddy.astie@vates.tech> Happy with that on the Arm side? Julien suggested so on #XenDevel. For me, this seems fine: Acked-by: Anthony PERARD <anthony.perard@vates.tech> > --- > CHANGELOG.md | 3 +++ > tools/misc/Makefile | 2 +- > 2 files changed, 4 insertions(+), 1 deletion(-) > > diff --git a/CHANGELOG.md b/CHANGELOG.md > index 3aaf598623..1fa58ce848 100644 > --- a/CHANGELOG.md > +++ b/CHANGELOG.md > @@ -16,6 +16,9 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/) > deprecated in Xen 4.7 and noted not to work correctly with AMD CPUs from > 2011 onwards. > > + - Removed xenpm on non-x86 platforms as it doesn't actually provide anything > + useful outside of x86. > + > ## [4.21.0](https://xenbits.xenproject.org/gitweb/?p=xen.git;a=shortlog;h=RELEASE-4.21.0) - 2025-11-19 > > ### Changed > diff --git a/tools/misc/Makefile b/tools/misc/Makefile > index c26e544e83..672df02c3b 100644 > --- a/tools/misc/Makefile > +++ b/tools/misc/Makefile > @@ -23,13 +23,13 @@ INSTALL_SBIN-$(CONFIG_X86) += xen-lowmemd > INSTALL_SBIN-$(CONFIG_X86) += xen-mceinj > INSTALL_SBIN-$(CONFIG_X86) += xen-memshare > INSTALL_SBIN-$(CONFIG_X86) += xen-mfndump > +INSTALL_SBIN-$(CONFIG_X86) += xenpm Nit: I would sort this by taking the dash `-` into account since we do so for the arch-common list, so xenpm would go after xen-vmtrace. > INSTALL_SBIN-$(CONFIG_X86) += xen-ucode > INSTALL_SBIN-$(CONFIG_X86) += xen-vmtrace > INSTALL_SBIN += xencov > INSTALL_SBIN += xenhypfs > INSTALL_SBIN += xenlockprof > INSTALL_SBIN += xenperf > -INSTALL_SBIN += xenpm > INSTALL_SBIN += xenwatchdogd > INSTALL_SBIN += xen-access > INSTALL_SBIN += xen-livepatch -- Anthony PERARD
On 07.01.2026 16:58, Anthony PERARD wrote: > On Fri, Dec 19, 2025 at 03:42:17PM +0000, Teddy Astie wrote: >> --- a/tools/misc/Makefile >> +++ b/tools/misc/Makefile >> @@ -23,13 +23,13 @@ INSTALL_SBIN-$(CONFIG_X86) += xen-lowmemd >> INSTALL_SBIN-$(CONFIG_X86) += xen-mceinj >> INSTALL_SBIN-$(CONFIG_X86) += xen-memshare >> INSTALL_SBIN-$(CONFIG_X86) += xen-mfndump >> +INSTALL_SBIN-$(CONFIG_X86) += xenpm > > Nit: I would sort this by taking the dash `-` into account since we do > so for the arch-common list, so xenpm would go after xen-vmtrace. Not really; imo it's coincidental that all present x86-only ones have a dash. I think this simply wants to ... >> INSTALL_SBIN-$(CONFIG_X86) += xen-ucode >> INSTALL_SBIN-$(CONFIG_X86) += xen-vmtrace >> INSTALL_SBIN += xencov >> INSTALL_SBIN += xenhypfs >> INSTALL_SBIN += xenlockprof >> INSTALL_SBIN += xenperf >> -INSTALL_SBIN += xenpm ... stay in its present place, merely becoming conditional. >> INSTALL_SBIN += xenwatchdogd >> INSTALL_SBIN += xen-access >> INSTALL_SBIN += xen-livepatch The three entries with dashes quite likely were blindly added to the tail, without paying attention to sorting. Imo they want moving up into their respective positions. Jan
On 19.12.2025 16:42, Teddy Astie wrote: > xenpm doesn't provide any interesting usable features outside of x86, > skip building it if we are not x86. > > Suggested-by: Jan Beulich <jbeulich@suse.com> > Signed-off-by: Teddy Astie <teddy.astie@vates.tech> > --- > CHANGELOG.md | 3 +++ > tools/misc/Makefile | 2 +- > 2 files changed, 4 insertions(+), 1 deletion(-) Arm being what is mainly affected, I think you may have wanted to Cc Arm folks, for them to ack this (alongside Anthony). > --- a/CHANGELOG.md > +++ b/CHANGELOG.md > @@ -16,6 +16,9 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/) > deprecated in Xen 4.7 and noted not to work correctly with AMD CPUs from > 2011 onwards. > > + - Removed xenpm on non-x86 platforms as it doesn't actually provide anything > + useful outside of x86. Maybe add the word "tool" for those who don't recognize "xenpm" at the first glance? Jan
© 2016 - 2026 Red Hat, Inc.