Re: [PATCH 8/8] NEWS: Mention guest agent error code improvements

2025-03-20 Thread Ján Tomko via Devel
On a Thursday in 2025, Peter Krempa via Devel wrote: From: Peter Krempa Signed-off-by: Peter Krempa --- NEWS.rst | 10 ++ 1 file changed, 10 insertions(+) diff --git a/NEWS.rst b/NEWS.rst index 98ca838642..b2f3415001 100644 --- a/NEWS.rst +++ b/NEWS.rst @@ -37,6 +37,16 @@ v11.2.0 (unre

Re: [PATCH] qemu/dbus: Allow connections from root to the dbus-daemon

2025-03-20 Thread Ján Tomko via Devel
On a Thursday in 2025, Martin Kletzander via Devel wrote: From: Martin Kletzander In commit dbfb96d18c04 libvirt started connecting to the daemon to set RDP credentials, but our configuration file did not allow connections from the root user, so the connection failed and the VM failed to start.

Re: [PATCH] Add NEWS item for auto-shutdown feature

2025-03-20 Thread Laine Stump via Devel
On 3/20/25 10:57 AM, Daniel P. Berrangé via Devel wrote: From: Daniel P. Berrangé Signed-off-by: Daniel P. Berrangé --- NEWS.rst | 17 + 1 file changed, 17 insertions(+) diff --git a/NEWS.rst b/NEWS.rst index 98ca838642..45d48c1485 100644 --- a/NEWS.rst +++ b/NEWS.rst @@ -3

Re: [PATCH] Add NEWS item for auto-shutdown feature

2025-03-20 Thread Daniel P . Berrangé via Devel
On Thu, Mar 20, 2025 at 01:44:41PM -0400, Laine Stump wrote: > On 3/20/25 10:57 AM, Daniel P. Berrangé via Devel wrote: > > From: Daniel P. Berrangé > > > > Signed-off-by: Daniel P. Berrangé > > --- > > NEWS.rst | 17 + > > 1 file changed, 17 insertions(+) > > > > diff --git

Re: [PATCH] Add NEWS item for auto-shutdown feature

2025-03-20 Thread Peter Krempa via Devel
On Thu, Mar 20, 2025 at 14:57:48 +, Daniel P. Berrangé via Devel wrote: > From: Daniel P. Berrangé > > Signed-off-by: Daniel P. Berrangé > --- > NEWS.rst | 17 + > 1 file changed, 17 insertions(+) Reviewed-by: Peter Krempa

Re: Questions regarding potential resize implementation in libvirt

2025-03-20 Thread Maximilian Immanuel Brandtner
On Thu, 2025-03-20 at 10:28 +, Daniel P. Berrangé wrote: > On Thu, Mar 20, 2025 at 11:09:41AM +0100, Maximilian Immanuel > Brandtner wrote: > > On Thu, 2025-03-20 at 09:58 +, Daniel P. Berrangé wrote: > > > On Thu, Mar 20, 2025 at 08:56:04AM +0100, Maximilian Immanuel > > > Brandtner wrote:

Re: [libvirt PATCH 0/6] cleanup and fixes of the mapped-ram feature

2025-03-20 Thread Jim Fehlig via Devel
On 3/20/25 17:07, Pavel Hrdina via Devel wrote: Pavel Hrdina (6): tools: remove --parallel from virsh restore command tools: remote --parallel from virsh save command qemu: remove VIR_DOMAIN_SAVE_PARALLEL flag Heh, I'm having one of those "why did I not realize that" moments :-).

[libvirt PATCH 2/6] tools: remote --parallel from virsh save command

2025-03-20 Thread Pavel Hrdina via Devel
From: Pavel Hrdina There is no need to have --parallel and --parallel-channels especially when --parallel on its own is the same as not used at all. In both cases libvirt will default to single channel. Signed-off-by: Pavel Hrdina --- docs/manpages/virsh.rst | 11 +-- tools/virsh-domai

[PATCH 2/2] qemu: Fix build on 32-bit platforms

2025-03-20 Thread Jim Fehlig via Devel
Commits c2518f7bc7 and 28a0621528 introduced build failures on 32-bit platforms by using incorrect format specifiers with g_strdup_printf. In one case, an 'unsigned long' format specifier is used with a 'long long int' variable. Fix by changing the format specifier to 'uintmax_t', and casting the

[PATCH 1/2] qemu: Fix CLang build

2025-03-20 Thread Jim Fehlig via Devel
Commit f0169f4d caused a build failure with CLang due to potentially uninitialized variable ../src/qemu/qemu_migration_params.c:825:17: error: variable 'nchannels' is used uninitialized whenever '&&' condition is false [-Werror,-Wsometimes-uninitialized] 825 | if (params && virTypedP

[libvirt PATCH 1/6] tools: remove --parallel from virsh restore command

2025-03-20 Thread Pavel Hrdina via Devel
From: Pavel Hrdina There is no need to have --parallel and --parallel-channels especially when --parallel on its own is the same as not used at all. In both cases libvirt will default to single channel. Signed-off-by: Pavel Hrdina --- docs/manpages/virsh.rst | 8 tools/virsh-domain.c

[PATCH 0/2] qemu: Build fixes following merge of mapped-ram

2025-03-20 Thread Jim Fehlig via Devel
Patches to fix the recent build failures caused by merging the mapped-ram series. They get a green light from a CI run in my fork https://gitlab.com/jfehlig/libvirt/-/pipelines/1727301916 I would push them as build-breakers, but the first one does raise a question that should be answered beforeha

[libvirt PATCH 5/6] tools: use virDomainSaveParams only when necessary

2025-03-20 Thread Pavel Hrdina via Devel
From: Pavel Hrdina We should use the newest API only when user sets parallel-channels. Signed-off-by: Pavel Hrdina --- tools/virsh-domain.c | 22 +- 1 file changed, 13 insertions(+), 9 deletions(-) diff --git a/tools/virsh-domain.c b/tools/virsh-domain.c index c21cf847c8..

[libvirt PATCH 3/6] qemu: remove VIR_DOMAIN_SAVE_PARALLEL flag

2025-03-20 Thread Pavel Hrdina via Devel
From: Pavel Hrdina There is no need to use extra flag in addition to the new "parallel.channels" param. Using the flag without param would result in using uninitialized variable. Fixing it would result in error that parallel channels cannot be less then 1 or setting 1 as default. Using the para

Re: Questions regarding potential resize implementation in libvirt

2025-03-20 Thread Daniel P . Berrangé via Devel
On Thu, Mar 20, 2025 at 08:56:04AM +0100, Maximilian Immanuel Brandtner wrote: > I've been working on an RFC patch-set to implement resizing for > consoles in QEMU. Now that the patch-set is in review I've turned my > attention to bringing this feature to libvirt. Can you point to the QEMU patchse

Re: Questions regarding potential resize implementation in libvirt

2025-03-20 Thread Daniel P . Berrangé via Devel
On Thu, Mar 20, 2025 at 11:05:04AM +0100, Peter Krempa via Devel wrote: > > Libvirt supports console also for unix-socket backed chardevs. I presume > that would require the QMP command approach instead. That will likely > reuqire more plumbing because on the first glance we don't seem to > have t

Questions regarding potential resize implementation in libvirt

2025-03-20 Thread Maximilian Immanuel Brandtner
I've been working on an RFC patch-set to implement resizing for consoles in QEMU. Now that the patch-set is in review I've turned my attention to bringing this feature to libvirt. With the QEMU patch-set there are two ways to resize a pty chardev. 1. send a TIOCSWINSZ ioctl 2. send a QMP msg The

Re: [PATCH V4 08/18] qemu: Add helper function for creating save image fd

2025-03-20 Thread Daniel P . Berrangé via Devel
On Wed, Mar 19, 2025 at 05:09:01PM -0600, Jim Fehlig wrote: > On 3/19/25 06:49, Daniel P. Berrangé wrote: > > On Wed, Mar 05, 2025 at 03:48:17PM -0700, Jim Fehlig via Devel wrote: > > > Move the code in qemuSaveImageCreate that opens, labels, and wraps the > > > save image fd to a helper function,

Re: Questions regarding potential resize implementation in libvirt

2025-03-20 Thread Daniel P . Berrangé via Devel
On Thu, Mar 20, 2025 at 11:09:41AM +0100, Maximilian Immanuel Brandtner wrote: > On Thu, 2025-03-20 at 09:58 +, Daniel P. Berrangé wrote: > > On Thu, Mar 20, 2025 at 08:56:04AM +0100, Maximilian Immanuel > > Brandtner wrote: > > > I've been working on an RFC patch-set to implement resizing for

[PATCH] qemu/dbus: Allow connections from root to the dbus-daemon

2025-03-20 Thread Martin Kletzander via Devel
From: Martin Kletzander In commit dbfb96d18c04 libvirt started connecting to the daemon to set RDP credentials, but our configuration file did not allow connections from the root user, so the connection failed and the VM failed to start. In order to avoid such issue allow root to connect if the

Re: [PATCH v2] virsh: Introduce new hypervisor-cpu-models command

2025-03-20 Thread Ján Tomko via Devel
On a Thursday in 2025, Collin Walling wrote: From: David Judkovics Add new virsh command 'hypervisor-cpu-models'. Command pulls from the existing domcapabilities XML and uses xpath to parse CPU model strings. By default, only models reported as usable by the hypervisor on the host system are p

[PATCH 3/8] qemuAgentCommandFull: Use VIR_ERR_AGENT_COMMAND_TIMEOUT when agent disappears

2025-03-20 Thread Peter Krempa via Devel
From: Peter Krempa When the agent disappears after geting a proper command we ought to report the same error code as if we timed out as it's uncertain whether the guest agent did anything. Signed-off-by: Peter Krempa --- src/qemu/qemu_agent.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(

[PATCH 2/8] qemu: agent: Differentiate timeouts when syncing from command timeout

2025-03-20 Thread Peter Krempa via Devel
From: Peter Krempa As the guest agent code uses timeouts it is possible that we stop waiting before the guest agent replies. If this happens while syncing everything is okay because we didn't send any state-changing command. In case when the timeout happens after a real command was transmitted i

Re: [PATCH v9 reviewed 00/17] qemu: block: Support block disk along with throttle filters

2025-03-20 Thread Peter Krempa via Devel
On Wed, Mar 19, 2025 at 03:00:47 +0100, Ján Tomko wrote: > On a Tuesday in 2025, Peter Krempa via Devel wrote: > > v9 of the throttle filtering series with my reviews and R-b tags > > applied. > > > > Requires > > > > [PATCH 0/5] qemu: Two block job fixes > > https://lists.libvirt.org/archives/li

[PATCH] Add NEWS item for auto-shutdown feature

2025-03-20 Thread Daniel P . Berrangé via Devel
From: Daniel P. Berrangé Signed-off-by: Daniel P. Berrangé --- NEWS.rst | 17 + 1 file changed, 17 insertions(+) diff --git a/NEWS.rst b/NEWS.rst index 98ca838642..45d48c1485 100644 --- a/NEWS.rst +++ b/NEWS.rst @@ -35,6 +35,23 @@ v11.2.0 (unreleased) or even map them to s

[PATCH 1/8] lib: error: Introduce 'VIR_ERR_AGENT_COMMAND_TIMEOUT'

2025-03-20 Thread Peter Krempa via Devel
From: Peter Krempa Introduce a new special error code for guest agent commands. The error code will be specifically reported only when an actual command (not a sync) was issued to the guest agent and the timeout time was reached. This will allow users and management applications to differentiat

[PATCH 8/8] NEWS: Mention guest agent error code improvements

2025-03-20 Thread Peter Krempa via Devel
From: Peter Krempa Signed-off-by: Peter Krempa --- NEWS.rst | 10 ++ 1 file changed, 10 insertions(+) diff --git a/NEWS.rst b/NEWS.rst index 98ca838642..b2f3415001 100644 --- a/NEWS.rst +++ b/NEWS.rst @@ -37,6 +37,16 @@ v11.2.0 (unreleased) * **Improvements** + * qemu: Improved gue

[PATCH 4/8] docs: Point to VIR_ERR_AGENT_COMMAND_TIMEOUT when setting timeout

2025-03-20 Thread Peter Krempa via Devel
From: Peter Krempa In addition to the error constant appearing add docs hinting that this new error code can be produced on timeouts. The most relevant place is to do it when setting the timeout. Signed-off-by: Peter Krempa --- docs/manpages/virsh.rst | 3 +++ src/libvirt-domain.c| 4

[PATCH 5/8] lib: error: Introduce 'VIR_ERR_AGENT_COMMAND_FAILED'

2025-03-20 Thread Peter Krempa via Devel
From: Peter Krempa Add a special error code for when the guest agent returned a failure message. Allow management applications to deterministically detect failure of the guest agent command. Signed-off-by: Peter Krempa --- include/libvirt/virterror.h | 2 ++ src/util/virerror.c | 3 ++