[PATCH 3/3] qemu_command: do use host-nodes for system memory

2021-12-15 Thread Michal Privoznik
After previous commit, it's no longer possible to change nodeset for strict numatune. Therefore, we can start generating host-nodes onto command line again. This partially reverts d73265af6ec41104c20633b5c0a23688a62105e6. Signed-off-by: Michal Privoznik --- src/qemu/qemu_command.c

[PATCH 1/3] qemu: Allow VIR_DOMAIN_NUMATUNE_MEM_RESTRICTIVE in qemuDomainSetNumaParamsLive()

2021-12-15 Thread Michal Privoznik
The whole idea of VIR_DOMAIN_NUMATUNE_MEM_RESTRICTIVE is that the memory location is restricted only via CGroups and thus can be changed on the fly (which is exactly what qemuDomainSetNumaParamsLive() does. Allow this mode there then. Signed-off-by: Michal Privoznik --- src/qemu/qemu_driver.c |

[PATCH 0/3] qemu: Explicitly forbid changing nodeset for strict numatune

2021-12-15 Thread Michal Privoznik
See 2/3 for explanation. Michal Prívozník (3): qemu: Allow VIR_DOMAIN_NUMATUNE_MEM_RESTRICTIVE in qemuDomainSetNumaParamsLive() qemu: Explicitly forbid changing nodeset for strict numatune qemu_command: do use host-nodes for system memory src/qemu/qemu_command.c

[PATCH 2/3] qemu: Explicitly forbid changing nodeset for strict numatune

2021-12-15 Thread Michal Privoznik
Let's imagine a guest that's configured with strict numatune: For guests with NUMA: Depending on machine type used (see commit v6.4.0-rc1~75) we generate either: 1) -object '{"qom-type":"memory-backend-ram","id":"ram-node0",\

Re: [PATCH] qemu: fix missing cleanup on error in qemuSaveImageStartVM

2021-12-15 Thread Jiri Denemark
On Wed, Dec 15, 2021 at 18:37:21 +0530, Ani Sinha wrote: > Commit 52521de8332c2323bd ("qemu: Use qemuDomainSaveStatus") replaced a call > to virDomainObjSave() with qemuDomainSaveStatus() as a part of cleanup. Since > qemuDomainSaveStatus() does not indicate any failure through its return code, >

[libvirt PATCH] qemu: qemu_command: clean up includes

2021-12-15 Thread Ján Tomko
Over time, the code using them got split into other files. (Mostly qemu_interface.c and qemu_process.c) Signed-off-by: Ján Tomko --- src/qemu/qemu_command.c | 10 -- 1 file changed, 10 deletions(-) diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c index

Re: [libvirt PATCH 1/2] qemu: Use qemuDomainSaveStatus

2021-12-15 Thread Ani Sinha
On Tue, Dec 14, 2021 at 9:52 PM Jiri Denemark wrote: > > It is a nice wrapper around virDomainObjSave which logs a warning, but > otherwise ignores the error. Let's use it where appropriate. > > Signed-off-by: Jiri Denemark > --- > src/qemu/qemu_domain.c| 11 ++--- >

[PATCH] qemu: fix missing cleanup on error in qemuSaveImageStartVM

2021-12-15 Thread Ani Sinha
Commit 52521de8332c2323bd ("qemu: Use qemuDomainSaveStatus") replaced a call to virDomainObjSave() with qemuDomainSaveStatus() as a part of cleanup. Since qemuDomainSaveStatus() does not indicate any failure through its return code, the error handling cleanup code got eliminated in the process.

Re: [PATCH] qemucapstest: Update qemu capabilities probe dump for qemu-6.2 release

2021-12-15 Thread Ján Tomko
On a Wednesday in 2021, Peter Krempa wrote: qemu-6.2 is out, update the caps dump for the final time. Notable changes: - 'unstable' feature flag for various QMP schema entries - 'aio-max-batch' iothread property - 'kernel-hashes' knob for the 'sev-guest' object - 'native-hotplug' of

Re: [PATCH v1 0/7] Fix build without XDR library

2021-12-15 Thread Ján Tomko
On a Wednesday in 2021, Michal Privoznik wrote: It's not much that's left when no XDR library is found, but some users might want just stateless drivers. BTW: there is still one pending issue - if no XDR library is found then remote driver is turned off which in turn means libvirtd is turned

Re: [PATCH] qemu-options: Remove the deprecated -no-quit option

2021-12-15 Thread Thomas Huth
On 15/12/2021 10.11, Markus Armbruster wrote: Thomas Huth writes: This option was just a wrapper around the -display ...,window-close=off parameter, and the name "no-quit" is rather confusing compared to "window-close" (since there are still other means to quit the emulator), so let's remove

Re: [PATCH] qemu-options: Remove the deprecated -no-quit option

2021-12-15 Thread Markus Armbruster
Thomas Huth writes: > This option was just a wrapper around the -display ...,window-close=off > parameter, and the name "no-quit" is rather confusing compared to > "window-close" (since there are still other means to quit the emulator), > so let's remove this now. > > Signed-off-by: Thomas Huth

Re: [libvirt PATCH 1/2] docs: domain: Clarify on the dangers of migrating with hyperv-passthrough enabled

2021-12-15 Thread Michal Prívozník
On 12/14/21 17:53, Tim Wiederhake wrote: > Signed-off-by: Tim Wiederhake > --- > docs/formatdomain.rst | 9 - > 1 file changed, 8 insertions(+), 1 deletion(-) > Reviewed-by: Michal Privoznik Michal

Re: [PATCH] qemu-options: Remove the deprecated -no-quit option

2021-12-15 Thread Michal Prívozník
On 12/15/21 09:24, Thomas Huth wrote: > This option was just a wrapper around the -display ...,window-close=off > parameter, and the name "no-quit" is rather confusing compared to > "window-close" (since there are still other means to quit the emulator), > so let's remove this now. > >

Re: [libvirt PATCH 00/17] Bump minimum dnsmasq version

2021-12-15 Thread Michal Prívozník
On 12/14/21 21:06, Laine Stump wrote: > On 12/14/21 2:09 PM, Ján Tomko wrote: >> This bumps the minimum dnsmasq version to the point where we do not need >> capability probing, reducing it to a version check (which I will be >> happy to remove on request). >> >> Unless I missed something, this

[PATCH] qemu-options: Remove the deprecated -no-quit option

2021-12-15 Thread Thomas Huth
This option was just a wrapper around the -display ...,window-close=off parameter, and the name "no-quit" is rather confusing compared to "window-close" (since there are still other means to quit the emulator), so let's remove this now. Signed-off-by: Thomas Huth --- docs/about/deprecated.rst

Re: [libvirt PATCH 2/2] virDomainFeaturesHyperVDefParse: Compare hyperv mode

2021-12-15 Thread Michal Prívozník
On 12/14/21 17:53, Tim Wiederhake wrote: > Previous patch neglected the possibility of different modes for hyperv > (e.g. "custom" and "passthrough"). > > Fixes: 6e83fafe331dd0b4fb19aa384c3dd36b3af62933 > Signed-off-by: Tim Wiederhake > --- > src/conf/domain_conf.c | 9 + > 1 file

Re: [libvirt PATCH] Fix some typos

2021-12-15 Thread Michal Prívozník
On 12/14/21 16:01, Tim Wiederhake wrote: > Signed-off-by: Tim Wiederhake > --- > docs/manpages/virsh.rst | 2 +- > src/qemu/qemu_domain.h | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) Reviewed-by: Michal Privoznik Michal

Re: [libvirt PATCH 0/2] qemu: Use qemuDomainSaveStatus

2021-12-15 Thread Michal Prívozník
On 12/14/21 17:22, Jiri Denemark wrote: > Jiri Denemark (2): > qemu: Use qemuDomainSaveStatus > qemu: Drop driver parameter from qemuDomainSetFakeReboot > > src/qemu/qemu_domain.c| 12 ++--- > src/qemu/qemu_domain.h| 3 +- > src/qemu/qemu_driver.c| 14 +++--- >