On 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 <[email protected]> > Signed-off-by: Teddy Astie <[email protected]>
Happy with that on the Arm side? Julien suggested so on #XenDevel. For me, this seems fine: Acked-by: Anthony PERARD <[email protected]> > --- > 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
