Re: [RFC PATCH 0/3] single-binary: make QAPI generated files common

2025-04-29 Thread Thomas Huth via Devel
On 29/04/2025 21.48, Pierrick Bouvier wrote: ... I'm not keen to have a default target set, but it's a personal opinion based on fear of "implicit smart choice hurts", so I'll be happy to change my mind with a good argument for it. No default target, please! We've seen this with the default ma

Re: [PATCH v4 4/5] cpu_map: Add POWER11 cpu model support

2025-04-29 Thread Jiří Denemark via Devel
On Tue, Apr 29, 2025 at 13:03:12 +0100, Daniel P. Berrangé wrote: > On Tue, Apr 29, 2025 at 01:42:22PM +0200, Peter Krempa wrote: > > On Tue, Apr 29, 2025 at 12:36:40 +0100, Daniel P. Berrangé wrote: > > > On Tue, Apr 15, 2025 at 04:13:08AM -0400, Narayana Murty N wrote: > > > > Add POWER11 as a su

Re: [PATCH v2 3/5] docs/about/deprecated: auto-generate a note for versioned machine types

2025-04-29 Thread Thomas Huth via Devel
On 29/04/2025 15.15, Daniel P. Berrangé wrote: We deprecate versioned machine types on a fixed schedule. This allows us to auto-generate a paragraph in the deprecated.rst document that always has accurate version info. Signed-off-by: Daniel P. Berrangé --- docs/about/deprecated.rst | 7 +

Re: [PATCH v2 2/5] include/hw/boards: cope with dev/rc versions in deprecation checks

2025-04-29 Thread Thomas Huth via Devel
On 29/04/2025 15.15, Daniel P. Berrangé wrote: When VERSION is set to a development snapshot (micro >= 50), or a release candidate (micro >= 90) we have an off-by-1 in determining deprecation and deletion thresholds for versioned machine types. In such cases we need to use the next major/minor ve

Re: [PATCH v2 1/5] Revert "include/hw: temporarily disable deletion of versioned machine types"

2025-04-29 Thread Daniel P . Berrangé via Devel
On Tue, Apr 29, 2025 at 04:30:56PM +0200, Thomas Huth wrote: > On 29/04/2025 15.15, Daniel P. Berrangé wrote: > > This reverts commit c9fd2d9a48ee3c195cf83cc611b87b09f02f0013. > > > > When we introduced the specialized machine type deprecation policy, we > > allow automatic deprecation to take eff

Re: [PATCH v2 1/5] Revert "include/hw: temporarily disable deletion of versioned machine types"

2025-04-29 Thread Thomas Huth via Devel
On 29/04/2025 15.15, Daniel P. Berrangé wrote: This reverts commit c9fd2d9a48ee3c195cf83cc611b87b09f02f0013. When we introduced the specialized machine type deprecation policy, we allow automatic deprecation to take effect immediately, but blocked the automatic deletion of machine types for 2 re

Re: [PATCH 15/15] cpu_x86: Fix algorithm for computing CPU model weight

2025-04-29 Thread Peter Krempa via Devel
On Tue, Apr 29, 2025 at 12:19:50 +0200, Jiri Denemark via Devel wrote: > From: Jiri Denemark > > This patch is effectively a NOP, but it fixes a logic bug and makes the > heuristics more visible and easier to change should there be a need to > do so in the future. > > We decide which CPU model i

Re: [PATCH 00/20] Finish the work to stop hardcoding paths at build time

2025-04-29 Thread Daniel P . Berrangé via Devel
On Tue, Apr 29, 2025 at 02:57:13PM +0200, Ján Tomko wrote: > On a Tuesday in 2025, Daniel P. Berrangé via Devel wrote: > > Over the years we've made various changes to stop hardcoding paths at > > build time, but the work is incomplete. > > > > This bit us significantly in Fedora 42 where they hav

Re: [PATCH 14/15] cpu_x86: Refactor virCPUx86CompareCandidateFeatureList

2025-04-29 Thread Peter Krempa via Devel
On Tue, Apr 29, 2025 at 15:27:44 +0200, Peter Krempa via Devel wrote: > On Tue, Apr 29, 2025 at 12:19:49 +0200, Jiri Denemark via Devel wrote: > > From: Jiri Denemark > > > > Refactor weight calculation to a separate virCPUx86WeightFeatures > > function to avoid code duplication. > > > > Signed-

Re: [PATCH 14/15] cpu_x86: Refactor virCPUx86CompareCandidateFeatureList

2025-04-29 Thread Peter Krempa via Devel
On Tue, Apr 29, 2025 at 12:19:49 +0200, Jiri Denemark via Devel wrote: > From: Jiri Denemark > > Refactor weight calculation to a separate virCPUx86WeightFeatures > function to avoid code duplication. > > Signed-off-by: Jiri Denemark > --- > src/cpu/cpu_x86.c | 83 ++---

[PATCH v2 3/5] docs/about/deprecated: auto-generate a note for versioned machine types

2025-04-29 Thread Daniel P . Berrangé via Devel
We deprecate versioned machine types on a fixed schedule. This allows us to auto-generate a paragraph in the deprecated.rst document that always has accurate version info. Signed-off-by: Daniel P. Berrangé --- docs/about/deprecated.rst | 7 +++ docs/conf.py | 33 +++

[PATCH v2 5/5] include/hw/boards: add warning about changing deprecation logic

2025-04-29 Thread Daniel P . Berrangé via Devel
If we change the deprecation logic in include/hw/boards.h, we must make a corresponding change to docs/conf.py and docs/about/deprecated.rst. Add comments to these files as a warning to future maintainers to keep these files in sync. Reviewed-by: Thomas Huth Signed-off-by: Daniel P. Berrangé ---

[PATCH v2 4/5] docs/about/removed-features: auto-generate a note for versioned machine types

2025-04-29 Thread Daniel P . Berrangé via Devel
We remove versioned machine types on a fixed schedule. This allows us to auto-generate a paragraph in the removed-features.rst document that always has accurate version info. Signed-off-by: Daniel P. Berrangé --- docs/about/removed-features.rst | 10 ++ docs/conf.py|

[PATCH v2 2/5] include/hw/boards: cope with dev/rc versions in deprecation checks

2025-04-29 Thread Daniel P . Berrangé via Devel
When VERSION is set to a development snapshot (micro >= 50), or a release candidate (micro >= 90) we have an off-by-1 in determining deprecation and deletion thresholds for versioned machine types. In such cases we need to use the next major/minor version in threshold checks. This adapts the depre

[PATCH v2 1/5] Revert "include/hw: temporarily disable deletion of versioned machine types"

2025-04-29 Thread Daniel P . Berrangé via Devel
This reverts commit c9fd2d9a48ee3c195cf83cc611b87b09f02f0013. When we introduced the specialized machine type deprecation policy, we allow automatic deprecation to take effect immediately, but blocked the automatic deletion of machine types for 2 releases. This ensured we complied with the histori

[PATCH v2 0/5] docs: automated info about machine deprecation/removal info

2025-04-29 Thread Daniel P . Berrangé via Devel
Since we deprecate and remove versioned machine types on a fixed schedule, we can automatically ensure that the docs reflect the latest version info, rather than requiring manual updates on each dev cycle. The first patch in this series removes the hack which postponed automatic removal of version

Re: [PATCH 00/20] Finish the work to stop hardcoding paths at build time

2025-04-29 Thread Ján Tomko via Devel
On a Tuesday in 2025, Daniel P. Berrangé via Devel wrote: Over the years we've made various changes to stop hardcoding paths at build time, but the work is incomplete. This bit us significantly in Fedora 42 where they have merged the 'sbin' and 'bin' dirs together. In the build environment we ha

Re: [PATCH 13/15] cputest: Add data for a newer version of Intel Atom(R) P5362 CPU

2025-04-29 Thread Peter Krempa via Devel
On Tue, Apr 29, 2025 at 12:19:48 +0200, Jiri Denemark via Devel wrote: > From: Jiri Denemark > > Signed-off-by: Jiri Denemark > --- > tests/cputest.c |1 + > .../x86_64-cpuid-Atom-P5362-2-disabled.xml|9 + > .../x86_64-cpuid-Atom-P5362-2-enabled.xml

Re: [PATCH 16/20] util: remove use hardcoded TC path

2025-04-29 Thread Peter Krempa via Devel
On Tue, Apr 29, 2025 at 13:40:31 +0100, Daniel P. Berrangé wrote: > On Tue, Apr 29, 2025 at 02:38:45PM +0200, Peter Krempa wrote: > > On Tue, Apr 29, 2025 at 12:14:16 +0100, Daniel P. Berrangé via Devel wrote: > > > From: Daniel P. Berrangé > > > > > > Allow virCommand to find 'tc' in $PATH. This

Re: [PATCH 20/20] openvz: stop hardcoding vzlist/vzctl/vzmigrate paths

2025-04-29 Thread Peter Krempa via Devel
On Tue, Apr 29, 2025 at 12:14:20 +0100, Daniel P. Berrangé via Devel wrote: > From: Daniel P. Berrangé > > Allow virCommand to find them in $PATH, which will always include 'sbin' > since the openvz driver only operates in system mode. > > Signed-off-by: Daniel P. Berrangé > --- > src/openvz/o

Re: [PATCH 19/20] util: stop hardcoding pkttyagent path

2025-04-29 Thread Peter Krempa via Devel
On Tue, Apr 29, 2025 at 12:14:19 +0100, Daniel P. Berrangé via Devel wrote: > From: Daniel P. Berrangé > > Allow virCommand to find it in $PATH > > Signed-off-by: Daniel P. Berrangé > --- Now we can suggest a workaround in https://gitlab.com/libvirt/libvirt/-/issues/757 defining a fake agent

Re: [PATCH 18/20] storage: stop hardcoding LVM tool paths

2025-04-29 Thread Peter Krempa via Devel
On Tue, Apr 29, 2025 at 12:14:18 +0100, Daniel P. Berrangé via Devel wrote: > From: Daniel P. Berrangé > > Change the mesonm rules to always enable the LVM driver if on a s/mesonm/meson/ > Linux host, unless the meson options say not to. > > The virCommand APIs will return suitable runtime err

Re: [PATCH 16/20] util: remove use hardcoded TC path

2025-04-29 Thread Daniel P . Berrangé via Devel
On Tue, Apr 29, 2025 at 02:38:45PM +0200, Peter Krempa wrote: > On Tue, Apr 29, 2025 at 12:14:16 +0100, Daniel P. Berrangé via Devel wrote: > > From: Daniel P. Berrangé > > > > Allow virCommand to find 'tc' in $PATH. This command is only used > > when running privileged in which case both 'bin' a

Re: [PATCH 17/20] meson: stop setting conf var for optional programs

2025-04-29 Thread Peter Krempa via Devel
On Tue, Apr 29, 2025 at 12:14:17 +0100, Daniel P. Berrangé via Devel wrote: > From: Daniel P. Berrangé > > There is no reference to AUGPARSE, BLACK, FLAKE8, PDWTAGS or PYTEST conf > variables anywhere, only the ${name}_prog meson variables are used. > > Signed-off-by: Daniel P. Berrangé > --- >

Re: [PATCH 16/20] util: remove use hardcoded TC path

2025-04-29 Thread Peter Krempa via Devel
On Tue, Apr 29, 2025 at 12:14:16 +0100, Daniel P. Berrangé via Devel wrote: > From: Daniel P. Berrangé > > Allow virCommand to find 'tc' in $PATH. This command is only used > when running privileged in which case both 'bin' and 'sbin' dirs will > be in $PATH, so virFindFileInPath will do the righ

Re: [PATCH 15/20] util: remove use hardcoded OVS_VSCTL path

2025-04-29 Thread Peter Krempa via Devel
On Tue, Apr 29, 2025 at 12:14:15 +0100, Daniel P. Berrangé via Devel wrote: > From: Daniel P. Berrangé > > Allow virCommand to find 'ovs-vsctl' in $PATH. This command is only used > when running privileged in which case both 'bin' and 'sbin' dirs will > be in $PATH, so virFindFileInPath will do t

Re: [PATCH 14/20] util: remove use hardcoded MODPROBE/RMMOD paths

2025-04-29 Thread Peter Krempa via Devel
On Tue, Apr 29, 2025 at 12:14:14 +0100, Daniel P. Berrangé via Devel wrote: > From: Daniel P. Berrangé > > Allow virCommand to find 'modprobe' & 'rmmod' in $PATH. These commands > are only used when running privileged in which case both 'bin' and > 'sbin' dirs will be in $PATH, so virFindFileInPa

Re: [PATCH 13/20] util: remove use hardcoded MM_CTL path

2025-04-29 Thread Peter Krempa via Devel
On Tue, Apr 29, 2025 at 12:14:13 +0100, Daniel P. Berrangé via Devel wrote: > From: Daniel P. Berrangé > > Allow virCommand to find 'mm-ctl' in $PATH. This command is only used > when running privileged in which case both 'bin' and 'sbin' dirs will > be in $PATH, so virFindFileInPath will do the

Re: [PATCH 12/20] nodedev: remove use hardcoded MDEVCTL path

2025-04-29 Thread Peter Krempa via Devel
On Tue, Apr 29, 2025 at 12:14:12 +0100, Daniel P. Berrangé via Devel wrote: > From: Daniel P. Berrangé > > Allow virCommand to find 'mdevctl' in $PATH. This command is only used > when running privileged in which case both 'bin' and 'sbin' dirs will > be in $PATH, so virFindFileInPath will do the

Re: [PATCH 11/20] util: remove hardcoded ISCSIADM command path

2025-04-29 Thread Peter Krempa via Devel
On Tue, Apr 29, 2025 at 12:14:11 +0100, Daniel P. Berrangé via Devel wrote: > From: Daniel P. Berrangé > > Allow virCommand to find 'dmidecode' in $PATH. This command is only > usable when running privileged since it relies on reading from a > privileged kernel file. Thus we can assume both 'bin'

Re: [PATCH 10/20] meson: remove check for 'ip' program

2025-04-29 Thread Peter Krempa via Devel
On Tue, Apr 29, 2025 at 12:14:10 +0100, Daniel P. Berrangé via Devel wrote: > From: Daniel P. Berrangé > > There are no references to an "IP" conf variable nor a meson > 'ip_prog' variable so the check serves no purpose. > > Signed-off-by: Daniel P. Berrangé > --- > meson.build| 1

Re: [PATCH 09/20] util: remove use hardcoded DMIDECODE path

2025-04-29 Thread Peter Krempa via Devel
On Tue, Apr 29, 2025 at 12:14:09 +0100, Daniel P. Berrangé via Devel wrote: > From: Daniel P. Berrangé > > Allow virCommand to find 'dmidecode' in $PATH. This command is only > usable when running privileged since it relies on reading from a > privileged kernel file. Thus we can assume both 'bin'

Re: [PATCH v4 4/5] cpu_map: Add POWER11 cpu model support

2025-04-29 Thread Peter Krempa via Devel
On Tue, Apr 29, 2025 at 13:03:12 +0100, Daniel P. Berrangé wrote: > On Tue, Apr 29, 2025 at 01:42:22PM +0200, Peter Krempa wrote: > > On Tue, Apr 29, 2025 at 12:36:40 +0100, Daniel P. Berrangé wrote: > > > On Tue, Apr 15, 2025 at 04:13:08AM -0400, Narayana Murty N wrote: > > > > Add POWER11 as a su

Re: [RFC PATCH 0/3] single-binary: make QAPI generated files common

2025-04-29 Thread BALATON Zoltan
On Tue, 29 Apr 2025, Markus Armbruster wrote: Pierrick Bouvier writes: On 4/28/25 4:07 AM, Markus Armbruster wrote: Peter Krempa writes: So what should libvirt do once multiple targets are supported? How do we query CPUs for each of the supported targets? It's kind of a similar question

Re: [PATCH v4 4/5] cpu_map: Add POWER11 cpu model support

2025-04-29 Thread Daniel P . Berrangé via Devel
On Tue, Apr 29, 2025 at 01:42:22PM +0200, Peter Krempa wrote: > On Tue, Apr 29, 2025 at 12:36:40 +0100, Daniel P. Berrangé wrote: > > On Tue, Apr 15, 2025 at 04:13:08AM -0400, Narayana Murty N wrote: > > > Add POWER11 as a supported cpu model for ppc64. > > > > > > Signed-off-by: Narayana Murty N

Re: [PATCH 08/20] meson: stop setting conf var for required programs

2025-04-29 Thread Peter Krempa via Devel
On Tue, Apr 29, 2025 at 12:14:08 +0100, Daniel P. Berrangé via Devel wrote: > From: Daniel P. Berrangé > > There is no reference to PERL, PYTHON3, XMLLINT or XSLTPROC conf > variables anywhere, only the ${name}_prog meson variables are > used. > > Signed-off-by: Daniel P. Berrangé > --- > meso

Re: [PATCH 07/20] build-aux: add missing definition of PERL variable

2025-04-29 Thread Peter Krempa via Devel
On Tue, Apr 29, 2025 at 12:14:07 +0100, Daniel P. Berrangé via Devel wrote: > From: Daniel P. Berrangé > > Currently $(PERL) is expanding to nothing, but the spacing-check.pl test > luckily still works via the shebang. > > Signed-off-by: Daniel P. Berrangé > --- > build-aux/Makefile.in | 1 + >

Re: [PATCH 06/20] docs: stop setting vars for docs tools

2025-04-29 Thread Peter Krempa via Devel
On Tue, Apr 29, 2025 at 12:14:06 +0100, Daniel P. Berrangé via Devel wrote: > From: Daniel P. Berrangé > > Nothing in the tree references RST2MAN or RST2HTML5 variables, only > rst2man_prog & rst2html5_prog, so the former can be removed. > > Signed-off-by: Daniel P. Berrangé > --- > docs/meson

Re: [PATCH 05/20] util: stop hardcoding 'ifconfig' path

2025-04-29 Thread Peter Krempa via Devel
On Tue, Apr 29, 2025 at 12:14:05 +0100, Daniel P. Berrangé via Devel wrote: > From: Daniel P. Berrangé > > Change the source to assume use of 'ifconfig' on FreeBSD builds, > allowing virCommand to report missing commands at runtime. > > Signed-off-by: Daniel P. Berrangé > --- > meson.build

Re: [PATCH 04/20] util: stop hardcoding bhyve, bhyvectl, bhyveload paths

2025-04-29 Thread Peter Krempa via Devel
On Tue, Apr 29, 2025 at 12:14:04 +0100, Daniel P. Berrangé via Devel wrote: > From: Daniel P. Berrangé > > Change the meson rules to always enable numad if on a FreeBSD host, > unless the meson options say not to. > > Signed-off-by: Daniel P. Berrangé > --- > meson.build | 11 +--

Re: [PATCH 03/20] util: stop hardcoding numad path

2025-04-29 Thread Peter Krempa via Devel
On Tue, Apr 29, 2025 at 12:14:03 +0100, Daniel P. Berrangé via Devel wrote: > From: Daniel P. Berrangé > > Change the meson rules to always enable numad if on a Linux host, unless > the meson options say not to. > > Signed-off-by: Daniel P. Berrangé > --- > meson.build| 10 +++--- >

Re: [PATCH v4 4/5] cpu_map: Add POWER11 cpu model support

2025-04-29 Thread Peter Krempa via Devel
On Tue, Apr 29, 2025 at 12:36:40 +0100, Daniel P. Berrangé wrote: > On Tue, Apr 15, 2025 at 04:13:08AM -0400, Narayana Murty N wrote: > > Add POWER11 as a supported cpu model for ppc64. > > > > Signed-off-by: Narayana Murty N > > --- > > src/cpu_map/index.xml | 1 + > > src/

Re: [PATCH v4 5/5] cpu_ppc64: Add POWER11 hostmodel support and accept case insensitive

2025-04-29 Thread Daniel P . Berrangé via Devel
On Tue, Apr 15, 2025 at 04:13:09AM -0400, Narayana Murty N wrote: > QEMU has historically used uppercase CPU model names like "POWER8", > "POWER9", etc. However, starting with commit c5354f54aa60 ("ppc: make > cpu_model translation to type consistent"), QEMU began using lowercase > CPU model names

Re: [PATCH v4 4/5] cpu_map: Add POWER11 cpu model support

2025-04-29 Thread Daniel P . Berrangé via Devel
On Tue, Apr 15, 2025 at 04:13:08AM -0400, Narayana Murty N wrote: > Add POWER11 as a supported cpu model for ppc64. > > Signed-off-by: Narayana Murty N > --- > src/cpu_map/index.xml | 1 + > src/cpu_map/meson.build| 1 + > src/cpu_map/ppc64_POWER11.xml

Re: [PATCH 02/20] tests: storage hardcoding paths for mount & vgchange

2025-04-29 Thread Peter Krempa via Devel
On Tue, Apr 29, 2025 at 12:14:02 +0100, Daniel P. Berrangé via Devel wrote: > From: Daniel P. Berrangé > > This is redundant since the tests will strip any path component from the > binary name before comparison. > > Signed-off-by: Daniel P. Berrangé > --- > tests/storagepoolxml2argvtest.c | 1

Re: [PATCH 01/20] storage: stop hardcoding paths for mkfs, mount, umount

2025-04-29 Thread Peter Krempa via Devel
On Tue, Apr 29, 2025 at 12:14:01 +0100, Daniel P. Berrangé via Devel wrote: > From: Daniel P. Berrangé > > This was always undesirable but now causes problems on Fedora 42 > where at build time we detect a /sbin path but at runtime this > will only exist on upgraded machines, not fresh installs.

Re: [PATCH v4 5/5] cpu_ppc64: Add POWER11 hostmodel support and accept case insensitive

2025-04-29 Thread Peter Krempa via Devel
On Tue, Apr 15, 2025 at 04:13:09 -0400, Narayana Murty N wrote: > QEMU has historically used uppercase CPU model names like "POWER8", > "POWER9", etc. However, starting with commit c5354f54aa60 ("ppc: make > cpu_model translation to type consistent"), QEMU began using lowercase > CPU model names (e

Re: [PATCH v4 4/5] cpu_map: Add POWER11 cpu model support

2025-04-29 Thread Peter Krempa via Devel
On Tue, Apr 15, 2025 at 04:13:08 -0400, Narayana Murty N wrote: > Add POWER11 as a supported cpu model for ppc64. uppercase > > Signed-off-by: Narayana Murty N > --- > src/cpu_map/index.xml | 1 + > src/cpu_map/meson.build| 1 + > src/cpu_map/ppc64_POWE

Re: [PATCH v4 2/5] tests: qemuhotplugtest: Set the cpu version at source for PPC64 tests

2025-04-29 Thread Peter Krempa via Devel
On Tue, Apr 15, 2025 at 04:13:06 -0400, Narayana Murty N wrote: > The commit 140ff3c5141 ("tests: qemuhotplugtest: Fix arch-specific > parts of 'ppc64' test XMLs") fixated on the POWER9 in results xml which > actually would be the host-cpu at the moment on the source tree. With updates > to higher

Re: [PATCH v4 3/5] tests: Add capabilities for QEMU 10.0.0 on ppc64

2025-04-29 Thread Peter Krempa via Devel
On Tue, Apr 15, 2025 at 04:13:07 -0400, Narayana Murty N wrote: > Add the qemu test capabilities xml and reply files for > QEMU v10.0.0-rc2 on ppc64. A QEMU v10.0.0-rc2 was used for generating > this data.The tests with the 'latest' suffix, which expect > the latest available CPU version from the c

[PATCH 20/20] openvz: stop hardcoding vzlist/vzctl/vzmigrate paths

2025-04-29 Thread Daniel P . Berrangé via Devel
From: Daniel P. Berrangé Allow virCommand to find them in $PATH, which will always include 'sbin' since the openvz driver only operates in system mode. Signed-off-by: Daniel P. Berrangé --- src/openvz/openvz_conf.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/o

[PATCH 19/20] util: stop hardcoding pkttyagent path

2025-04-29 Thread Daniel P . Berrangé via Devel
From: Daniel P. Berrangé Allow virCommand to find it in $PATH Signed-off-by: Daniel P. Berrangé --- src/util/virpolkit.c | 8 +--- src/util/virpolkit.h | 2 -- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/src/util/virpolkit.c b/src/util/virpolkit.c index 0a637db9ba..81fa5

[PATCH 18/20] storage: stop hardcoding LVM tool paths

2025-04-29 Thread Daniel P . Berrangé via Devel
From: Daniel P. Berrangé Change the mesonm rules to always enable the LVM driver if on a Linux host, unless the meson options say not to. The virCommand APIs will return suitable runtime errors if the tools are not installed. Signed-off-by: Daniel P. Berrangé --- meson.build

[PATCH 16/20] util: remove use hardcoded TC path

2025-04-29 Thread Daniel P . Berrangé via Devel
From: Daniel P. Berrangé Allow virCommand to find 'tc' in $PATH. This command is only used when running privileged in which case both 'bin' and 'sbin' dirs will be in $PATH, so virFindFileInPath will do the right thing to find it. Signed-off-by: Daniel P. Berrangé --- meson.build

[PATCH 17/20] meson: stop setting conf var for optional programs

2025-04-29 Thread Daniel P . Berrangé via Devel
From: Daniel P. Berrangé There is no reference to AUGPARSE, BLACK, FLAKE8, PDWTAGS or PYTEST conf variables anywhere, only the ${name}_prog meson variables are used. Signed-off-by: Daniel P. Berrangé --- meson.build | 1 - 1 file changed, 1 deletion(-) diff --git a/meson.build b/meson.build i

[PATCH 15/20] util: remove use hardcoded OVS_VSCTL path

2025-04-29 Thread Daniel P . Berrangé via Devel
From: Daniel P. Berrangé Allow virCommand to find 'ovs-vsctl' in $PATH. This command is only used when running privileged in which case both 'bin' and 'sbin' dirs will be in $PATH, so virFindFileInPath will do the right thing to find it. Signed-off-by: Daniel P. Berrangé --- meson.build

[PATCH 14/20] util: remove use hardcoded MODPROBE/RMMOD paths

2025-04-29 Thread Daniel P . Berrangé via Devel
From: Daniel P. Berrangé Allow virCommand to find 'modprobe' & 'rmmod' in $PATH. These commands are only used when running privileged in which case both 'bin' and 'sbin' dirs will be in $PATH, so virFindFileInPath will do the right thing to find them. Signed-off-by: Daniel P. Berrangé --- meso

[PATCH 13/20] util: remove use hardcoded MM_CTL path

2025-04-29 Thread Daniel P . Berrangé via Devel
From: Daniel P. Berrangé Allow virCommand to find 'mm-ctl' in $PATH. This command is only used when running privileged in which case both 'bin' and 'sbin' dirs will be in $PATH, so virFindFileInPath will do the right thing to find it. Signed-off-by: Daniel P. Berrangé --- meson.build

[PATCH 12/20] nodedev: remove use hardcoded MDEVCTL path

2025-04-29 Thread Daniel P . Berrangé via Devel
From: Daniel P. Berrangé Allow virCommand to find 'mdevctl' in $PATH. This command is only used when running privileged in which case both 'bin' and 'sbin' dirs will be in $PATH, so virFindFileInPath will do the right thing to find it. Signed-off-by: Daniel P. Berrangé --- src/node_device/node

[PATCH 10/20] meson: remove check for 'ip' program

2025-04-29 Thread Daniel P . Berrangé via Devel
From: Daniel P. Berrangé There are no references to an "IP" conf variable nor a meson 'ip_prog' variable so the check serves no purpose. Signed-off-by: Daniel P. Berrangé --- meson.build| 1 - src/util/virnetdevip.c | 6 +++--- 2 files changed, 3 insertions(+), 4 deletions(-) diff

[PATCH 11/20] util: remove hardcoded ISCSIADM command path

2025-04-29 Thread Daniel P . Berrangé via Devel
From: Daniel P. Berrangé Allow virCommand to find 'dmidecode' in $PATH. This command is only usable when running privileged since it relies on reading from a privileged kernel file. Thus we can assume both 'bin' and 'sbin' dirs will be in $PATH and virFindFileInPath will do the right thing to fin

[PATCH 09/20] util: remove use hardcoded DMIDECODE path

2025-04-29 Thread Daniel P . Berrangé via Devel
From: Daniel P. Berrangé Allow virCommand to find 'dmidecode' in $PATH. This command is only usable when running privileged since it relies on reading from a privileged kernel file. Thus we can assume both 'bin' and 'sbin' dirs will be in $PATH and virFindFileInPath will do the right thing to fin

[PATCH 08/20] meson: stop setting conf var for required programs

2025-04-29 Thread Daniel P . Berrangé via Devel
From: Daniel P. Berrangé There is no reference to PERL, PYTHON3, XMLLINT or XSLTPROC conf variables anywhere, only the ${name}_prog meson variables are used. Signed-off-by: Daniel P. Berrangé --- meson.build | 1 - 1 file changed, 1 deletion(-) diff --git a/meson.build b/meson.build index ea3

[PATCH 07/20] build-aux: add missing definition of PERL variable

2025-04-29 Thread Daniel P . Berrangé via Devel
From: Daniel P. Berrangé Currently $(PERL) is expanding to nothing, but the spacing-check.pl test luckily still works via the shebang. Signed-off-by: Daniel P. Berrangé --- build-aux/Makefile.in | 1 + build-aux/meson.build | 1 + 2 files changed, 2 insertions(+) diff --git a/build-aux/Makefi

[PATCH 06/20] docs: stop setting vars for docs tools

2025-04-29 Thread Daniel P . Berrangé via Devel
From: Daniel P. Berrangé Nothing in the tree references RST2MAN or RST2HTML5 variables, only rst2man_prog & rst2html5_prog, so the former can be removed. Signed-off-by: Daniel P. Berrangé --- docs/meson.build | 1 - 1 file changed, 1 deletion(-) diff --git a/docs/meson.build b/docs/meson.buil

[PATCH 05/20] util: stop hardcoding 'ifconfig' path

2025-04-29 Thread Daniel P . Berrangé via Devel
From: Daniel P. Berrangé Change the source to assume use of 'ifconfig' on FreeBSD builds, allowing virCommand to report missing commands at runtime. Signed-off-by: Daniel P. Berrangé --- meson.build| 4 src/util/virnetdevip.c | 8 2 files changed, 4 insertions(+), 8 d

[PATCH 04/20] util: stop hardcoding bhyve, bhyvectl, bhyveload paths

2025-04-29 Thread Daniel P . Berrangé via Devel
From: Daniel P. Berrangé Change the meson rules to always enable numad if on a FreeBSD host, unless the meson options say not to. Signed-off-by: Daniel P. Berrangé --- meson.build | 11 +-- src/bhyve/bhyve_command.c | 6 +++--- 2 files changed, 4 insertions(+), 13 deleti

[PATCH 03/20] util: stop hardcoding numad path

2025-04-29 Thread Daniel P . Berrangé via Devel
From: Daniel P. Berrangé Change the meson rules to always enable numad if on a Linux host, unless the meson options say not to. Signed-off-by: Daniel P. Berrangé --- meson.build| 10 +++--- src/util/virnuma.c | 2 +- 2 files changed, 4 insertions(+), 8 deletions(-) diff --git a/m

[PATCH 02/20] tests: storage hardcoding paths for mount & vgchange

2025-04-29 Thread Daniel P . Berrangé via Devel
From: Daniel P. Berrangé This is redundant since the tests will strip any path component from the binary name before comparison. Signed-off-by: Daniel P. Berrangé --- tests/storagepoolxml2argvtest.c | 12 ++-- 1 file changed, 2 insertions(+), 10 deletions(-) diff --git a/tests/storage

[PATCH 01/20] storage: stop hardcoding paths for mkfs, mount, umount

2025-04-29 Thread Daniel P . Berrangé via Devel
From: Daniel P. Berrangé This was always undesirable but now causes problems on Fedora 42 where at build time we detect a /sbin path but at runtime this will only exist on upgraded machines, not fresh installs. Signed-off-by: Daniel P. Berrangé --- meson.build | 13 ---

[PATCH 00/20] Finish the work to stop hardcoding paths at build time

2025-04-29 Thread Daniel P . Berrangé via Devel
Over the years we've made various changes to stop hardcoding paths at build time, but the work is incomplete. This bit us significantly in Fedora 42 where they have merged the 'sbin' and 'bin' dirs together. In the build environment we have a clean install where '/sbin' and '/usr/sbin' are simply

Re: [PATCH v4 1/5] tests: Pin pseries-2.7 tests to the version 7.0

2025-04-29 Thread Peter Krempa via Devel
On Tue, Apr 15, 2025 at 04:13:05 -0400, Narayana Murty N wrote: > Support for the pseries-2.7 machine type in QEMU was officially removed in > version 9.2 with qemu commit 445d3facffe8 ("ppc/spapr: remove deprecated > machine pseries-2.7"). Instead of removing related tests, they are now pinned > t

[PATCH 15/15] cpu_x86: Fix algorithm for computing CPU model weight

2025-04-29 Thread Jiri Denemark via Devel
From: Jiri Denemark This patch is effectively a NOP, but it fixes a logic bug and makes the heuristics more visible and easier to change should there be a need to do so in the future. We decide which CPU model is the best match for given CPU data by comparing lists of features that need to be en

[PATCH 13/15] cputest: Add data for a newer version of Intel Atom(R) P5362 CPU

2025-04-29 Thread Jiri Denemark via Devel
From: Jiri Denemark Signed-off-by: Jiri Denemark --- tests/cputest.c |1 + .../x86_64-cpuid-Atom-P5362-2-disabled.xml|9 + .../x86_64-cpuid-Atom-P5362-2-enabled.xml | 19 + .../x86_64-cpuid-Atom-P5362-2-guest.xml | 43 + .../x86_64-cpuid-A

[PATCH 14/15] cpu_x86: Refactor virCPUx86CompareCandidateFeatureList

2025-04-29 Thread Jiri Denemark via Devel
From: Jiri Denemark Refactor weight calculation to a separate virCPUx86WeightFeatures function to avoid code duplication. Signed-off-by: Jiri Denemark --- src/cpu/cpu_x86.c | 83 ++- 1 file changed, 46 insertions(+), 37 deletions(-) diff --git a/src

[PATCH 12/15] cputest: Add data for Intel(R) Xeon(R) w7-3465X CPU

2025-04-29 Thread Jiri Denemark via Devel
From: Jiri Denemark Signed-off-by: Jiri Denemark --- tests/cputest.c |1 + .../x86_64-cpuid-Xeon-w7-3465X-disabled.xml | 10 + .../x86_64-cpuid-Xeon-w7-3465X-enabled.xml| 20 + .../x86_64-cpuid-Xeon-w7-3465X-guest.xml | 55 + .../x86_64-cpuid-X

[PATCH 11/15] cputest: Add data for Intel(R) Xeon(R) Gold 6530 CPU

2025-04-29 Thread Jiri Denemark via Devel
From: Jiri Denemark Signed-off-by: Jiri Denemark --- tests/cputest.c |1 + .../x86_64-cpuid-Xeon-Gold-6530-disabled.xml | 10 + .../x86_64-cpuid-Xeon-Gold-6530-enabled.xml | 20 + .../x86_64-cpuid-Xeon-Gold-6530-guest.xml | 58 + .../x86_64-cpuid-X

[PATCH 10/15] cputest: Add data for Intel(R) Xeon(R) Bronze 3408U CPU

2025-04-29 Thread Jiri Denemark via Devel
From: Jiri Denemark Signed-off-by: Jiri Denemark --- tests/cputest.c |1 + ...86_64-cpuid-Xeon-Bronze-3408U-disabled.xml |9 + ...x86_64-cpuid-Xeon-Bronze-3408U-enabled.xml | 21 + .../x86_64-cpuid-Xeon-Bronze-3408U-guest.xml | 59 + .../x86_64-cpuid-X

[PATCH 08/15] cputest: Add data for AMD EPYC 9334 32-Core CPU

2025-04-29 Thread Jiri Denemark via Devel
From: Jiri Denemark Signed-off-by: Jiri Denemark --- tests/cputest.c |1 + ...86_64-cpuid-EPYC-9334-32-Core-disabled.xml |9 + ...x86_64-cpuid-EPYC-9334-32-Core-enabled.xml | 15 + .../x86_64-cpuid-EPYC-9334-32-Core-guest.xml | 42 + .../x86_64-cpuid-E

[PATCH 09/15] cputest: Add data for Intel(R) Xeon(R) 6731E CPU

2025-04-29 Thread Jiri Denemark via Devel
From: Jiri Denemark Signed-off-by: Jiri Denemark --- tests/cputest.c |1 + .../x86_64-cpuid-Xeon-6731E-disabled.xml | 11 + .../x86_64-cpuid-Xeon-6731E-enabled.xml | 21 + .../x86_64-cpuid-Xeon-6731E-guest.xml | 128 + .../x86_64-cpuid-X

[PATCH 07/15] cputest: Add data for Intel(R) Xeon(R) Gold 6152 CPU

2025-04-29 Thread Jiri Denemark via Devel
From: Jiri Denemark Signed-off-by: Jiri Denemark --- tests/cputest.c |1 + .../x86_64-cpuid-Xeon-Gold-6152-disabled.xml |7 + .../x86_64-cpuid-Xeon-Gold-6152-enabled.xml | 19 + .../x86_64-cpuid-Xeon-Gold-6152-guest.xml | 40 + .../x86_64-cpuid-X

[PATCH 06/15] cputest: Add data for AMD Ryzen 5 5500U CPU

2025-04-29 Thread Jiri Denemark via Devel
From: Jiri Denemark Signed-off-by: Jiri Denemark --- tests/cputest.c |1 + ...-5-5500U-with-Radeon-Graphics-disabled.xml |9 + ...n-5-5500U-with-Radeon-Graphics-enabled.xml | 11 + ...zen-5-5500U-with-Radeon-Graphics-guest.xml | 39 + ...yzen-5-5500U-wi

[PATCH 05/15] cputest: Add data for AMD EPYC 7713 64-Core CPU

2025-04-29 Thread Jiri Denemark via Devel
From: Jiri Denemark Signed-off-by: Jiri Denemark --- tests/cputest.c |1 + ...86_64-cpuid-EPYC-7713-64-Core-disabled.xml | 11 + ...x86_64-cpuid-EPYC-7713-64-Core-enabled.xml | 11 + .../x86_64-cpuid-EPYC-7713-64-Core-guest.xml | 44 + .../x86_64-cpuid-E

[PATCH 04/15] cputest: Add data for Intel(R) Xeon(R) Silver 4214R CPU

2025-04-29 Thread Jiri Denemark via Devel
From: Jiri Denemark Signed-off-by: Jiri Denemark --- tests/cputest.c |1 + ...86_64-cpuid-Xeon-Silver-4214R-disabled.xml |7 + ...x86_64-cpuid-Xeon-Silver-4214R-enabled.xml | 19 + .../x86_64-cpuid-Xeon-Silver-4214R-guest.xml | 46 + .../x86_64-cpuid-X

[PATCH 03/15] cputest: Add data for Intel(R) Xeon(R) CPU E3-1270 v5 CPU

2025-04-29 Thread Jiri Denemark via Devel
From: Jiri Denemark Signed-off-by: Jiri Denemark --- tests/cputest.c |1 + .../x86_64-cpuid-Xeon-E3-1270-v5-disabled.xml |6 + .../x86_64-cpuid-Xeon-E3-1270-v5-enabled.xml | 18 + .../x86_64-cpuid-Xeon-E3-1270-v5-guest.xml| 32 + .../x86_64-cpuid-X

[PATCH 02/15] cputest: Add data for Intel(R) Core(TM) i7-1365U CPU

2025-04-29 Thread Jiri Denemark via Devel
From: Jiri Denemark Signed-off-by: Jiri Denemark --- tests/cputest.c |1 + .../x86_64-cpuid-Core-i7-1365U-disabled.xml |6 + .../x86_64-cpuid-Core-i7-1365U-enabled.xml| 21 + .../x86_64-cpuid-Core-i7-1365U-guest.xml | 71 + .../x86_64-cpuid-C

[PATCH 01/15] cputest: Add data for Intel(R) Core(TM) i7-1270P CPU

2025-04-29 Thread Jiri Denemark via Devel
From: Jiri Denemark Signed-off-by: Jiri Denemark --- tests/cputest.c |1 + .../x86_64-cpuid-Core-i7-1270P-disabled.xml |8 + .../x86_64-cpuid-Core-i7-1270P-enabled.xml| 21 + .../x86_64-cpuid-Core-i7-1270P-guest.xml | 71 + .../x86_64-cpuid-C

[PATCH 00/15] cpu_x86: Fix CPU model comparison heuristics

2025-04-29 Thread Jiri Denemark via Devel
Several CPU tests covering recent-ish CPUs are added to cputest to make sure the fix doesn't break anything. Jiri Denemark (15): cputest: Add data for Intel(R) Core(TM) i7-1270P CPU cputest: Add data for Intel(R) Core(TM) i7-1365U CPU cputest: Add data for Intel(R) Xeon(R) CPU E3-1270 v5 CPU

Re: [RFC PATCH 0/3] single-binary: make QAPI generated files common

2025-04-29 Thread Daniel P . Berrangé via Devel
On Tue, Apr 29, 2025 at 11:35:52AM +0200, Philippe Mathieu-Daudé wrote: > On 29/4/25 10:23, Markus Armbruster wrote: > > Pierrick Bouvier writes: > > > > > On 4/28/25 4:07 AM, Markus Armbruster wrote: > > > > Peter Krempa writes: > > > > > > > > > So what should libvirt do once multiple targets

Re: [RFC PATCH 0/3] single-binary: make QAPI generated files common

2025-04-29 Thread Philippe Mathieu-Daudé
On 29/4/25 11:32, Daniel P. Berrangé wrote: On Tue, Apr 29, 2025 at 11:20:59AM +0200, Thomas Huth via Devel wrote: On 29/04/2025 10.23, Markus Armbruster wrote: ... I don't wish to derail this thread, but we've been dancing around the question of how to best fix the target for some time. I thi

Re: [RFC PATCH 0/3] single-binary: make QAPI generated files common

2025-04-29 Thread Philippe Mathieu-Daudé
On 29/4/25 10:23, Markus Armbruster wrote: Pierrick Bouvier writes: On 4/28/25 4:07 AM, Markus Armbruster wrote: Peter Krempa writes: So what should libvirt do once multiple targets are supported? How do we query CPUs for each of the supported targets? It's kind of a similar question w

Re: [RFC PATCH 0/3] single-binary: make QAPI generated files common

2025-04-29 Thread Daniel P . Berrangé via Devel
On Tue, Apr 29, 2025 at 11:20:59AM +0200, Thomas Huth via Devel wrote: > On 29/04/2025 10.23, Markus Armbruster wrote: > ... > > I don't wish to derail this thread, but we've been dancing around the > > question of how to best fix the target for some time. I think we should > > talk about it for r

Re: [RFC PATCH 0/3] single-binary: make QAPI generated files common

2025-04-29 Thread Thomas Huth via Devel
On 29/04/2025 10.23, Markus Armbruster wrote: ... I don't wish to derail this thread, but we've been dancing around the question of how to best fix the target for some time. I think we should talk about it for real. Mind, this is not an objection to your larger "single binary" idea. It could b

Re: [PATCH v2 3/3] scripts: Adapt mock-noinline.py to ATTRIBUTE_MOCKABLE

2025-04-29 Thread Daniel P . Berrangé via Devel
On Tue, Apr 29, 2025 at 10:31:43AM +0200, Michal Privoznik via Devel wrote: > From: Michal Privoznik > > The script is renamed to mockable-attribute.py and adjusted to > check for the new attribute. > > Signed-off-by: Michal Privoznik > --- > build-aux/syntax-check.mk

Re: [PATCH v2 2/3] src: s/G_NO_INLINE/ATTRIBUTE_MOCKABLE/

2025-04-29 Thread Daniel P . Berrangé via Devel
On Tue, Apr 29, 2025 at 10:31:42AM +0200, Michal Privoznik via Devel wrote: > From: Michal Privoznik > > Per change in coding style done in previous commit, > ATTRIBUTE_MOCKABLE should be used instead of G_NO_INLINE for > functions that are mocked in our test suite. Do the change. > > Signed-off

Re: [PATCH v2 1/3] internal: Introduce ATTRIBUTE_MOCKABLE

2025-04-29 Thread Daniel P . Berrangé via Devel
On Tue, Apr 29, 2025 at 10:31:41AM +0200, Michal Privoznik via Devel wrote: > From: Michal Privoznik > > Currently, if we want to mock a function the noinline attribute > is appended after the function (via G_NO_INLINE macro). This used > to work for non pure functions. But there are some trivial

Re: [RFC PATCH 0/3] single-binary: make QAPI generated files common

2025-04-29 Thread Daniel P . Berrangé via Devel
On Tue, Apr 29, 2025 at 09:43:24AM +0200, Markus Armbruster wrote: > Pierrick Bouvier writes: > > > After looking at the introspection code, I don't see any major blocker. > > We need to keep some of existing "if", as they are based on config-host, > > and should apply. > > We can introduce a ne

[PATCH v2 3/3] scripts: Adapt mock-noinline.py to ATTRIBUTE_MOCKABLE

2025-04-29 Thread Michal Privoznik via Devel
From: Michal Privoznik The script is renamed to mockable-attribute.py and adjusted to check for the new attribute. Signed-off-by: Michal Privoznik --- build-aux/syntax-check.mk | 4 ++-- scripts/meson.build | 2 +- scripts/{mock-noinlin

[PATCH v2 2/3] src: s/G_NO_INLINE/ATTRIBUTE_MOCKABLE/

2025-04-29 Thread Michal Privoznik via Devel
From: Michal Privoznik Per change in coding style done in previous commit, ATTRIBUTE_MOCKABLE should be used instead of G_NO_INLINE for functions that are mocked in our test suite. Do the change. Signed-off-by: Michal Privoznik --- src/cpu/cpu.h | 2 +- src/hypervisor/doma

[PATCH v2 1/3] internal: Introduce ATTRIBUTE_MOCKABLE

2025-04-29 Thread Michal Privoznik via Devel
From: Michal Privoznik Currently, if we want to mock a function the noinline attribute is appended after the function (via G_NO_INLINE macro). This used to work for non pure functions. But there are some trivial functions (for instance virQEMUCapsProbeHVF()) that are pure, i.e. have no side effec

  1   2   >