[PATCH] remote/stream-event: Fix a memory leak in remoteStreamCallbackFree()

2025-06-24 Thread liu.xuemei1
From: Liu Song The ff callback is never called in remoteStreamCallbackFree() because cbdata->cb can not be NULL. This causes a leak of 'cbdata->opaque'. The leak can be reproduced by attaching and detaching to the console of an VM using `virsh console`. ASAN reports the leak stack as: Direct le

Re: [PATCH v2 09/10] net: Add passt network backend

2025-06-24 Thread Laurent Vivier via Devel
On 24/06/2025 14:03, Daniel P. Berrangé wrote: On Tue, Jun 24, 2025 at 01:55:20PM +0200, Markus Armbruster wrote: Laurent Vivier writes: On 24/06/2025 10:16, Markus Armbruster wrote: Laurent Vivier writes: This commit introduces support for passt as a new network backend. passt is an unpr

Re: [PATCH v2 12/13] qemuxmlconftest: Prepare for proper testing in 'disk-cdrom-usb-empty'

2025-06-24 Thread Jiri Denemark via Devel
On Mon, Jun 23, 2025 at 21:59:17 +0200, Peter Krempa wrote: > From: Peter Krempa > > Modify the validation of empty cdroms to trigger only for > VIR_DOMAIN_DISK_MODEL_USB_STORAGE as with 'usb-bot' we can properly > emulate a cdrom. > > Signed-off-by: Peter Krempa > --- > src/qemu/qemu_validate

[libvirt PATCH] docs: clarify how to build without -Werror

2025-06-24 Thread Ján Tomko via Devel
From: Ján Tomko --werror does not accept any arguments for me and setting git_werror was also needed to disable it with git. Signed-off-by: Ján Tomko --- docs/compiling.rst | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/docs/compiling.rst b/docs/compiling.rst index 0a4

Entering freeze for libvirt-11.5.0

2025-06-24 Thread Jiri Denemark via Devel
I have just tagged v11.5.0-rc1 in the repository and pushed signed tarballs to https://download.libvirt.org/ Please give the release candidate some testing and in case you find a serious issue which should have a fix in the upcoming release, feel free to reply to this thread to make sure the issue

Re: [PATCH v2 13/13] qemu: Replace usb-storage with usb-bot

2025-06-24 Thread Peter Krempa via Devel
On Tue, Jun 24, 2025 at 14:57:13 +0200, Jiri Denemark wrote: > On Mon, Jun 23, 2025 at 21:59:18 +0200, Peter Krempa wrote: > > From: Akihiko Odaki > > > > usb-storage is a compound device that automatically creates a USB mass > > storage device and a SCSI device as its backend. Unfortunately it l

Re: [PATCH v2 13/13] qemu: Replace usb-storage with usb-bot

2025-06-24 Thread Peter Krempa via Devel
On Mon, Jun 23, 2025 at 21:59:18 +0200, Peter Krempa via Devel wrote: > From: Akihiko Odaki > > usb-storage is a compound device that automatically creates a USB mass > storage device and a SCSI device as its backend. Unfortunately it lacks > some configuration options that are usually present wi

Re: [PATCH v2 08/13] conf: introduce usb disk models 'usb-storage' and 'usb-bot'

2025-06-24 Thread Jiří Denemark via Devel
On Tue, Jun 24, 2025 at 12:51:57 +0200, Peter Krempa wrote: > On Tue, Jun 24, 2025 at 11:25:03 +0200, Jiri Denemark wrote: > > On Mon, Jun 23, 2025 at 21:59:13 +0200, Peter Krempa wrote: > > > From: Peter Krempa > > > > > > Historically libvirt specified 'usb-storage' as driver for USB disks. > >

Re: [PATCH v2 13/13] qemu: Replace usb-storage with usb-bot

2025-06-24 Thread Jiri Denemark via Devel
On Mon, Jun 23, 2025 at 21:59:18 +0200, Peter Krempa wrote: > From: Akihiko Odaki > > usb-storage is a compound device that automatically creates a USB mass > storage device and a SCSI device as its backend. Unfortunately it lacks > some configuration options that are usually present with a SCSI

[PATCH] qemu_monitor_json: Use proper initializer in qemuMonitorJSONGetBlockInfo()

2025-06-24 Thread Michal Privoznik via Devel
From: Michal Privoznik Inside of qemuMonitorJSONGetBlockInfo() there's a for loop in which a variable of struct qemuDomainDiskInfo type is declared and initialized as { false }. This works only because stdbool.h declares 'false' as a macro, so preprocessor expands initializer to proper form of {

Re: [PATCH v2 09/10] net: Add passt network backend

2025-06-24 Thread Markus Armbruster via Devel
Laurent Vivier writes: > On 24/06/2025 10:16, Markus Armbruster wrote: >> Laurent Vivier writes: >> >>> This commit introduces support for passt as a new network backend. >>> passt is an unprivileged, user-mode networking solution that provides >>> connectivity for virtual machines by launching

Re: [PATCH v2 11/13] qemu: monitor: Introduce 'qemuMonitorSetUSBDiskAttached'

2025-06-24 Thread Jiri Denemark via Devel
On Mon, Jun 23, 2025 at 21:59:16 +0200, Peter Krempa wrote: > From: Peter Krempa > > The helper sets the 'attached' property of the 'usb-bot' device to true, > which will be used on the hotplug code path. > > Signed-off-by: Peter Krempa > --- > src/qemu/qemu_monitor.c | 12 >

Re: [PATCH v2 07/13] qemuxmlconftest: Invoke "disk-usb-device" case also without QEMU_CAPS_DEVICE_USB_BOT and with ABI_UPDATE

2025-06-24 Thread Jiri Denemark via Devel
On Mon, Jun 23, 2025 at 21:59:12 +0200, Peter Krempa wrote: > From: Peter Krempa > > The QEMU_CAPS_DEVICE_USB_BOT device can be compiled out but > realistically it makes no sense to do it thus also makes no sense to > have another variant of input data for it. > > Add another invocation of "disk

Re: [PATCH v2 06/13] qemu_capabilities: Introduce QEMU_CAPS_DEVICE_USB_BOT

2025-06-24 Thread Jiri Denemark via Devel
On Mon, Jun 23, 2025 at 21:59:11 +0200, Peter Krempa wrote: > From: Akihiko Odaki > > usb-bot is supported by all supported QEMU versions; it is present since > 1.4.0 and libvirt supports 4.2.0 or later. > > Add a capability just in case USB_STORAGE_BOT is disabled when building > QEMU. > > Sig

Re: [PATCH v2 05/13] qemuxmlconftest: Distribute testing of 'removable' disk property

2025-06-24 Thread Jiri Denemark via Devel
On Mon, Jun 23, 2025 at 21:59:10 +0200, Peter Krempa wrote: > From: Peter Krempa > > The 'removable' property is tested for 'usb' and 'scsi' disks. The test > case for 'usb' disks already has another test case for this, so add > testing of 'removable' SCSI disks into the 'disk-scsi' case and remo

Re: [PATCH v2 04/13] qemuxmlconftest: Drop 'disk-cdrom-bus-other'

2025-06-24 Thread Jiri Denemark via Devel
On Mon, Jun 23, 2025 at 21:59:09 +0200, Peter Krempa wrote: > From: Peter Krempa > > The test case is now redundant since 'disk-usb-device' case also tests > all cdrom configurations. Remove it. > > Signed-off-by: Peter Krempa > --- > .../disk-cdrom-bus-other.x86_64-latest.args | 34

Re: [PATCH v2 03/13] qemusecuritytest: Use 'disk-usb-device' case instead of 'disk-cdrom-bus-other'

2025-06-24 Thread Jiri Denemark via Devel
On Mon, Jun 23, 2025 at 21:59:08 +0200, Peter Krempa wrote: > From: Peter Krempa > > Signed-off-by: Peter Krempa > --- > tests/qemusecuritytest.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Reviewed-by: Jiri Denemark

Re: [PATCH v2 02/13] qemuxmlconftest: Test various combinations of config

2025-06-24 Thread Jiri Denemark via Devel
On Mon, Jun 23, 2025 at 21:59:07 +0200, Peter Krempa wrote: > From: Peter Krempa > > Add multiple USB disks to the definition testing a matrix of 'disk' and > 'cdrom' elements with user-aliases, 'serial' and 'removable' > properties configured. > > This patch also removes the 'ide' disk which i

Re: [PATCH v2 01/13] qemuhotplugtest: Use VIR_DOMAIN_DEF_PARSE_ABI_UPDATE for virDomainDeviceDefParse

2025-06-24 Thread Jiri Denemark via Devel
On Mon, Jun 23, 2025 at 21:59:06 +0200, Peter Krempa wrote: > From: Peter Krempa > > The qemu hotplug code parses the device with ABI updates enabled so > qemuhotplugtest ought to do the same. > > Signed-off-by: Peter Krempa > --- > tests/qemuhotplugtest.c | 4 ++-- > 1 file changed, 2 inserti

Re: [PATCH v2 09/13] qemu: Fill in model of 'usb' disks to preserve ABI compatibility

2025-06-24 Thread Jiri Denemark via Devel
On Mon, Jun 23, 2025 at 21:59:14 +0200, Peter Krempa wrote: > From: Peter Krempa > > While 'usb-bot' and 'usb-storage' are ABI and migration compatible for > disks it's not the case for cdroms. When migrating from a new config > using 'usb-bot' to an older daemon which would use 'usb-storage' the

Re: [PATCH v2 08/13] conf: introduce usb disk models 'usb-storage' and 'usb-bot'

2025-06-24 Thread Jiri Denemark via Devel
On Mon, Jun 23, 2025 at 21:59:13 +0200, Peter Krempa wrote: > From: Peter Krempa > > Historically libvirt specified 'usb-storage' as driver for USB disks. > This though combined with '-blockdev' doesn't properly configure the > device to look like CDROM for . > > 'usb-bot' acts like a controler

Re: virsh migrate fails when --copy-storage-all option is given!

2025-06-24 Thread Anushree Mathur
CC: libvirt devel list Hi Kevin/Peter, Thank you so much for addressing this issue. I tried out few more things and here is my analysis: Even when I removed the readonly option from guest xml I was still seeing the issue in migration,In the qemu-commandline I could still see auto-read-only opti