Re: [libvirt PATCH 19/21] qemu: Generate command line for virtio-iommu

2021-10-11 Thread Peter Krempa
On Fri, Oct 08, 2021 at 18:51:54 +0200, Andrea Bolognani wrote: > https://bugzilla.redhat.com/show_bug.cgi?id=1653327 > > Signed-off-by: Andrea Bolognani > --- > src/qemu/qemu_command.c | 17 ++--- > .../virtio-iommu-aarch64.aarch64-latest.args| 1 + > ..

[PATCH] qemu: capabilities: Remove QEMU_CAPS_PIIX4_ACPI_HOTPLUG_BRIDGE

2021-10-11 Thread Peter Krempa
Commit 58ba0f6a3d7342fba29edbbf2bb9cb5497c870e5 added a capability which is supported by all qemu versions we support. Remove it and the associated dead code. Since the capability isn't present in any upstream release we can delete it completely. Specifically the commit itself states that it was i

Re: [PATCH 001/103] virJSONValueObjectAddVArgs: Add 'k' convertor for formatting non-negative integers

2021-10-11 Thread Peter Krempa
On Fri, Oct 08, 2021 at 13:20:54 +0200, Ján Tomko wrote: > On a Friday in 2021, Kristina Hanicova wrote: > > On Thu, Oct 7, 2021 at 6:43 PM Ján Tomko wrote: > > > > > On a Thursday in 2021, Peter Krempa wrote: > > > >In many cases we use a signed value, but use the sign to note that it > > > >was

Re: [libvirt PATCH 19/21] qemu: Generate command line for virtio-iommu

2021-10-11 Thread Andrea Bolognani
On Mon, Oct 11, 2021 at 09:04:43AM +0200, Peter Krempa wrote: > On Fri, Oct 08, 2021 at 18:51:54 +0200, Andrea Bolognani wrote: > > case VIR_DOMAIN_IOMMU_MODEL_VIRTIO: > > +if (qemuBuildVirtioDevStr(&opts, "virtio-iommu", qemuCaps, > > + VIR_DOMAIN_DEVI

Re: set custom loglevel for external libraries

2021-10-11 Thread Olaf Hering
This question is still open: where should the libxl specific setting be specified? Olaf Am Sun, 25 Jul 2021 10:17:49 +0200 schrieb Olaf Hering : > I think it is wrong to use internal libvirt values for internal libxentoollog > values. > > There has to be a separate configuration setting to sp

[libvirt PATCH 1/5] cpu_arm: No need to protect macro definitions

2021-10-11 Thread Tim Wiederhake
Signed-off-by: Tim Wiederhake --- src/cpu/cpu_arm.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/src/cpu/cpu_arm.c b/src/cpu/cpu_arm.c index 09ade1d422..ac174891b7 100644 --- a/src/cpu/cpu_arm.c +++ b/src/cpu/cpu_arm.c @@ -39,12 +39,11 @@ #include "virxml.h" #def

[libvirt PATCH 0/5] Add virCPUDataIsIdentical

2021-10-11 Thread Tim Wiederhake
A function to compare two `virCPUData` objects. This is a prerequisite for a later series, that will add the host's cpuid information to the cpu flag cache as per [1], to solve BZ 1953389 [2]. [1] https://listman.redhat.com/archives/libvir-list/2021-August/msg00293.html [2] https://bugzilla.redhat

[libvirt PATCH 2/5] cpu: Add virCPUDataIsIdentical

2021-10-11 Thread Tim Wiederhake
Signed-off-by: Tim Wiederhake --- src/cpu/cpu.c| 29 + src/cpu/cpu.h| 9 + src/libvirt_private.syms | 1 + 3 files changed, 39 insertions(+) diff --git a/src/cpu/cpu.c b/src/cpu/cpu.c index b4965f3ee0..a3531dd665 100644 --- a/src/cpu/

[libvirt PATCH 5/5] cpu_ppc64: Implement virCPUDataIsIdentical for ppc64

2021-10-11 Thread Tim Wiederhake
Signed-off-by: Tim Wiederhake --- src/cpu/cpu_ppc64.c | 26 ++ 1 file changed, 26 insertions(+) diff --git a/src/cpu/cpu_ppc64.c b/src/cpu/cpu_ppc64.c index 4909f61ff1..fcd68c8a7c 100644 --- a/src/cpu/cpu_ppc64.c +++ b/src/cpu/cpu_ppc64.c @@ -578,6 +578,31 @@ ppc64DriverD

[libvirt PATCH 4/5] cpu_arm: Implement virCPUDataIsIdentical for arm

2021-10-11 Thread Tim Wiederhake
Signed-off-by: Tim Wiederhake --- src/cpu/cpu_arm.c | 27 +++ 1 file changed, 27 insertions(+) diff --git a/src/cpu/cpu_arm.c b/src/cpu/cpu_arm.c index ac174891b7..c9114d53bf 100644 --- a/src/cpu/cpu_arm.c +++ b/src/cpu/cpu_arm.c @@ -124,6 +124,32 @@ virCPUarmDataClear(vi

[libvirt PATCH 3/5] cpu_x86: Implement virCPUDataIsIdentical for x86

2021-10-11 Thread Tim Wiederhake
Signed-off-by: Tim Wiederhake --- src/cpu/cpu_x86.c | 69 +++ 1 file changed, 69 insertions(+) diff --git a/src/cpu/cpu_x86.c b/src/cpu/cpu_x86.c index 1b829e5658..55da407ec4 100644 --- a/src/cpu/cpu_x86.c +++ b/src/cpu/cpu_x86.c @@ -3369,6 +3369,74 @@

Re: [PATCH] qemu: capabilities: Remove QEMU_CAPS_PIIX4_ACPI_HOTPLUG_BRIDGE

2021-10-11 Thread Ján Tomko
On a Monday in 2021, Peter Krempa wrote: Commit 58ba0f6a3d7342fba29edbbf2bb9cb5497c870e5 added a capability which is supported by all qemu versions we support. Remove it and the associated dead code. Since the capability isn't present in any upstream release we can delete it completely. Specific

Re: [PATCH] qemu: capabilities: Remove QEMU_CAPS_PIIX4_ACPI_HOTPLUG_BRIDGE

2021-10-11 Thread Peter Krempa
On Mon, Oct 11, 2021 at 13:26:14 +0200, Ján Tomko wrote: > On a Monday in 2021, Peter Krempa wrote: > > Commit 58ba0f6a3d7342fba29edbbf2bb9cb5497c870e5 added a capability which > > is supported by all qemu versions we support. Remove it and the > > associated dead code. Since the capability isn't p

[PATCH] spec: change the ownership of cache directory

2021-10-11 Thread Peng Liang
In a50c473ad6c9 ("qemu: move temp file of screenshot and memorypeek to per-domain dir") and c4f3c955d578 ("qemu: don't change ownership of cache directory"), I move the temporary files of screenshot and memorypeek from the cache directory to per-domain directory, and the only user of the cache dire

[PATCH v2 00/10] qemu: Fix image labels lost after migration with shared fs

2021-10-11 Thread Peng Liang
This is v2 of: https://listman.redhat.com/archives/libvir-list/2021-August/msg00698.html When migrating using shared fs, the dst host will remember the image labels again before launching the VM and the src host will remove the image labels after migration. It leads to that the image labels lost

[PATCH v2 02/10] security: add virSecurityManagerUpdateImageLabel

2021-10-11 Thread Peng Liang
After migration, some labels of images need to be updated. So add virSecurityManagerUpdateImageLabel to do it. Signed-off-by: Peng Liang --- src/libvirt_private.syms| 1 + src/security/security_driver.h | 5 + src/security/security_manager.c | 29 + sr

[PATCH v2 01/10] security: add virSecurityUpdateTimestampIfexists

2021-10-11 Thread Peng Liang
Signed-off-by: Peng Liang --- src/security/security_util.c | 32 src/security/security_util.h | 4 2 files changed, 36 insertions(+) diff --git a/src/security/security_util.c b/src/security/security_util.c index 26a7861e2935..07960b577e1a 100644 --- a/src/s

[PATCH v2 07/10] security: rename 2 functions in DAC and SELinux

2021-10-11 Thread Peng Liang
virSecurity{DAC,SELinux}SetImageLabelInt will be added in the next patch, so rename virSecurity{DAC,SELinux}SetImageLabelInternal to virSecurity{DAC,SELinux}SetImageLabelSingle to avoid confusion and keep consistent with virSecurity{DAC,SELinux}RestoreImageLabelInt and virSecurity{DAC,SELinux}Resto

[PATCH v2 10/10] migration: update image labels in dst after migration

2021-10-11 Thread Peng Liang
Bacause the timestamp XATTR (the uptime of the host) is used to validate the remembered labels, it need to update after migration. Signed-off-by: Peng Liang --- src/qemu/qemu_migration.c | 12 1 file changed, 12 insertions(+) diff --git a/src/qemu/qemu_migration.c b/src/qemu/qemu_m

[PATCH v2 05/10] security: implement domainUpdateSecurityImageLabel for SELinux

2021-10-11 Thread Peng Liang
Signed-off-by: Peng Liang --- src/security/security_selinux.c | 18 ++ 1 file changed, 18 insertions(+) diff --git a/src/security/security_selinux.c b/src/security/security_selinux.c index cc7245332980..5c491fc131ea 100644 --- a/src/security/security_selinux.c +++ b/src/security/

[PATCH v2 03/10] security: implement domainUpdateSecurityImageLabel for stack

2021-10-11 Thread Peng Liang
Signed-off-by: Peng Liang --- src/security/security_stack.c | 20 1 file changed, 20 insertions(+) diff --git a/src/security/security_stack.c b/src/security/security_stack.c index 3c2239910aa5..7712cac3b542 100644 --- a/src/security/security_stack.c +++ b/src/security/securi

[PATCH v2 04/10] security: implement domainUpdateSecurityImageLabel for DAC

2021-10-11 Thread Peng Liang
Signed-off-by: Peng Liang --- src/security/security_dac.c | 18 ++ 1 file changed, 18 insertions(+) diff --git a/src/security/security_dac.c b/src/security/security_dac.c index 1733d63410b3..d1e1552bb683 100644 --- a/src/security/security_dac.c +++ b/src/security/security_dac.c @

[PATCH v2 09/10] migration: don't remove image labels after migration

2021-10-11 Thread Peng Liang
After migration, the image labels will be removed in the src host (on success) or the dst host (on failure). However, if we migrate using shared fs and remove image labels after migration in one host, the image labels will also lost in another host, which leads to that the ownership of the image w

[PATCH v2 08/10] security: don't remember image labels when migrating with shared fs

2021-10-11 Thread Peng Liang
When migrating with shared fs, the image labels has been remembered in the src host. If the dst host trys to remember image labels again, then the origin labels remembered in the src host will lost. Signed-off-by: Peng Liang --- src/security/security_dac.c | 32 +++--

[PATCH v2 06/10] qemu: add qemuSecurityUpdateImageLabel

2021-10-11 Thread Peng Liang
Signed-off-by: Peng Liang --- src/qemu/qemu_security.c | 10 ++ src/qemu/qemu_security.h | 5 + 2 files changed, 15 insertions(+) diff --git a/src/qemu/qemu_security.c b/src/qemu/qemu_security.c index 19d957dd4b96..d8f2049e0d46 100644 --- a/src/qemu/qemu_security.c +++ b/src/qemu/qe

[PATCH 1/2] qemu: Move pid file of pr-helper to stateDir

2021-10-11 Thread Peng Liang
Libvirt will put the pid file of pr-helper to per-domain directory. However, the ownership of the per-domain directory is the user to run the QEMU process and the user has the write permission of the directory. If VM escape occurs, the attacker can 1. write arbitrary content to the pid file (if run

[PATCH 0/2] qemu: Move 2 pid files to stateDir

2021-10-11 Thread Peng Liang
Please see the commit messages for detail. Peng Liang (2): qemu: Move pid file of pr-helper to stateDir qemu: Move pid file of virtiofsd to stateDir src/qemu/qemu_process.c | 6 -- src/qemu/qemu_virtiofs.c | 8 2 files changed, 8 insertions(+), 6 deletions(-) -- 2.31.1

[PATCH 2/2] qemu: Move pid file of virtiofsd to stateDir

2021-10-11 Thread Peng Liang
Libvirt will put the pid file of virtiofsd to per-domain directory. However, the ownership of the per-domain directory is the user to run the QEMU process and the user has the write permission of the directory. If VM escape occurs, the attacker can 1. write arbitrary content to the pid file (if run

[PATCH v2 0/7] qemu: capabilities: Remove QEMU_CAPS_PIIX4_ACPI_HOTPLUG_BRIDGE

2021-10-11 Thread Peter Krempa
After a review from Jano I set out to fix also the test invocations to use real capabilities. This has shown that there was a bug in the validation logic which was pointed out by Jano in the output error file change. This version fixes the bug, improves testing and then removes the useless capabil

[PATCH v2 2/7] qemuValidateDomainDefPCIFeature: Fix validation logic

2021-10-11 Thread Peter Krempa
The error that "acpi-bridge-hotplug" is not supported would be triggered only if both the ICH9 and PIIX don't support the capability and the machine is q35. This makes no sense. We want to check that the appropriate platform supports the appropriate feature. Fixes: 7300ccc9b3e Signed-off-by: Pete

[PATCH v2 1/7] qemuValidateDomainDefPCIFeature: un-break error messages

2021-10-11 Thread Peter Krempa
https://www.libvirt.org/coding-style.html#error-message-format Signed-off-by: Peter Krempa --- src/qemu/qemu_validate.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/qemu/qemu_validate.c b/src/qemu/qemu_validate.c index be609c9d39..e530368cb3 100644 --- a/src/qemu

[PATCH v2 3/7] qemuxml2argvtest: Use real-caps testing for 'pc-i440fx-acpi-root-hotplug-disable'

2021-10-11 Thread Peter Krempa
We can use two real example configs to prove the support without the need for using fake capabilities. Fix the recently added test cases. Fixes: 133d7983d6c Signed-off-by: Peter Krempa --- ...fx-acpi-root-hotplug-disable.x86_64-5.1.0.err} | 0 ...-acpi-root-hotplug-disable.x86_64-latest.args} |

[PATCH v2 4/7] qemuxml2argvtest: Use real-caps testing for 'acpi-hotplug-bridge-disable'

2021-10-11 Thread Peter Krempa
We can use real example configs to prove the support without the need for using fake capabilities. Fix the recently added test cases. The negative case for 'pc-i440fx-acpi-hotplug-bridge-disable' is removed completely as there is no real qemu libvirt supports which wouldn't have the capability. F

[PATCH v2 6/7] qemuxml2xmltest: Convert all acpi-hotplug control related tests to DO_TEST_CAPS_LATEST

2021-10-11 Thread Peter Krempa
Signed-off-by: Peter Krempa --- ...i-hotplug-bridge-disable.x86_64-latest.xml | 36 + .../pc-i440fx-acpi-hotplug-bridge-disable.xml | 1 - ...pi-hotplug-bridge-enable.x86_64-latest.xml | 36 + .../pc-i440fx-acpi-hotplug-bridge-enable.xml | 1 - ...cpi-root-hotplug-disab

[PATCH v2 7/7] qemu: capabilities: Remove QEMU_CAPS_PIIX4_ACPI_HOTPLUG_BRIDGE

2021-10-11 Thread Peter Krempa
Commit 58ba0f6a3d7342fba29edbbf2bb9cb5497c870e5 added a capability which is supported by all qemu versions we support. Remove it and the associated dead code. Since the capability isn't present in any upstream release we can delete it completely. Specifically the commit itself states that it was i

[PATCH v2 5/7] qemuxml2argvtest: Add '-enable' variants for ACPI-hotplug related cases

2021-10-11 Thread Peter Krempa
We have input files for those, provide also xml2argv testing since we have them. Signed-off-by: Peter Krempa --- ...i-hotplug-bridge-enable.x86_64-latest.args | 34 + ...-acpi-root-hotplug-enable.x86_64-5.1.0.err | 1 + ...cpi-root-hotplug-enable.x86_64-latest.args | 34

[libvirt PATCH v6 1/5] Add a PCI/PCIe device VPD Parser

2021-10-11 Thread Dmitrii Shcherbakov
Add support for deserializing the binary PCI/PCIe VPD format and storing results in memory. The VPD format is specified in "I.3. VPD Definitions" in PCI specs (2.2+) and "6.28.1 VPD Format" PCIe 4.0. As section 6.28 in PCIe 4.0 notes, the PCI Local Bus and PCIe VPD formats are binary compatible an

[libvirt PATCH v6 4/5] Add PCI VPD Capability Documentation

2021-10-11 Thread Dmitrii Shcherbakov
Describes the format of the newly added VPD capability and gives and example for a real-world device. Signed-off-by: Dmitrii Shcherbakov --- docs/drvnodedev.html.in | 69 + docs/formatnode.html.in | 63 - 2 files changed

[libvirt PATCH v6 3/5] Add PCI VPD Capability Support

2021-10-11 Thread Dmitrii Shcherbakov
* XML serialization and deserialization of PCI VPD; * PCI VPD capability flags added and used in relevant places; * XML to XML tests for the added capability. Signed-off-by: Dmitrii Shcherbakov --- docs/schemas/nodedev.rng | 96 ++ include/libvirt/libvirt-nodedev.h

[libvirt PATCH v6 5/5] news: Add PCI VPD parser & capability notes

2021-10-11 Thread Dmitrii Shcherbakov
Signed-off-by: Dmitrii Shcherbakov --- NEWS.rst | 22 ++ 1 file changed, 22 insertions(+) diff --git a/NEWS.rst b/NEWS.rst index ae678bffc4..7d4244e938 100644 --- a/NEWS.rst +++ b/NEWS.rst @@ -40,6 +40,28 @@ v7.9.0 (unreleased) domain definition when the guest was first

[libvirt PATCH v6 2/5] Add PCI VPD-related helper functions to virpci

2021-10-11 Thread Dmitrii Shcherbakov
Add helper functions to virpci to provide means of checking for a VPD file presence and for VPD resource retrieval using the PCI VPD parser. The added test assesses the basic functionality of VPD retrieval while the full parser is tested by virpcivpdtest. Signed-off-by: Dmitrii Shcherbakov ---

[libvirt PATCH v6 0/5] Add a PCI/PCIe device VPD Capability

2021-10-11 Thread Dmitrii Shcherbakov
Add support for deserializing the binary PCI/PCIe VPD format and exposing VPD resources as XML elements in a new nested capability of PCI/PCIe devices called 'vpd'. The series contains the following incremental changes: * The PCI VPD parser module, in-memory resource representation and tests; *

Re: [PATCH v2 4/7] qemuxml2argvtest: Use real-caps testing for 'acpi-hotplug-bridge-disable'

2021-10-11 Thread Ján Tomko
On a Monday in 2021, Peter Krempa wrote: We can use real example configs to prove the support without the need for using fake capabilities. Fix the recently added test cases. The negative case for 'pc-i440fx-acpi-hotplug-bridge-disable' is removed completely as there is no real qemu libvirt supp

Re: [PATCH v2 0/7] qemu: capabilities: Remove QEMU_CAPS_PIIX4_ACPI_HOTPLUG_BRIDGE

2021-10-11 Thread Ján Tomko
On a Monday in 2021, Peter Krempa wrote: After a review from Jano I set out to fix also the test invocations to use real capabilities. This has shown that there was a bug in the validation logic which was pointed out by Jano in the output error file change. The cover letter mentions my name tw

Re: [PATCH 071/103] qemuBuildDeviceAddressProps: Format also 'drive' addresses

2021-10-11 Thread Ján Tomko
On a Thursday in 2021, Peter Krempa wrote: Introduce infrastructure to format 'drive' addresses via the standard helper rather than hand-rolled generators used inline. The code needs to know the disk bus to format the correct address which is passed in via an internal field in virDomainDeviceDri

Re: [PATCH v2 04/15] qom: Reduce use of error_propagate()

2021-10-11 Thread Markus Armbruster
Kevin Wolf writes: > ERRP_GUARD() makes debugging easier by making sure that &error_abort > still fails at the real origin of the error instead of > error_propagate(). > > Signed-off-by: Kevin Wolf Yes, please! Reviewed-by: Markus Armbruster

Re: [PATCH 072/103] qemu: validate: Move disk address validation code

2021-10-11 Thread Ján Tomko
On a Thursday in 2021, Peter Krempa wrote: Move the code from 'qemuValidateDomainDeviceDefDiskFrontend' into 'qemuValidateDomainDeviceDefAddressDrive' which is called from 'qemuValidateDomainDeviceDefAddress' so that we have all address validation code together. This also allows us to remove the

Re: [PATCH 073/103] qemuBuildHostdevSCSICommandLine: Generate via JSON

2021-10-11 Thread Ján Tomko
On a Thursday in 2021, Peter Krempa wrote: Since 'qemuBuildDeviceAddressProps' now also builds 'drive' addresses the generator is way simpler and doesn't use any special fields. Signed-off-by: Peter Krempa --- src/qemu/qemu_command.c | 56 - src/qemu/qemu_

Re: [PATCH 074/103] qemuBuildSCSIHostdevDevProps: Reorder 'drive', 'id' and 'bootindex' after address

2021-10-11 Thread Ján Tomko
On a Thursday in 2021, Peter Krempa wrote: Simplify the generator by shuffling around few fields. Signed-off-by: Peter Krempa --- src/qemu/qemu_command.c| 10 +++--- .../hostdev-scsi-lsi.x86_64-4.1.0.args | 12 ++-- .../hostdev-scsi-lsi.x86_64-l

Re: [PATCH 075/103] qemuBuildIoEventFdStr: Always assume support for QEMU_CAPS_VIRTIO_IOEVENTFD

2021-10-11 Thread Ján Tomko
On a Thursday in 2021, Peter Krempa wrote: The support for the 'ioeventfd' knob of virtio devices was introduced by QEMU commit 25db9ebe15125 contained in v0.14.0-rc0 and it can't be compiled out. Thus libvirt can assume it's support and remove its support it's supported conditional code. Si

Re: [PATCH 076/103] qemu: capabilities: Retire QEMU_CAPS_VIRTIO_IOEVENTFD

2021-10-11 Thread Ján Tomko
On a Thursday in 2021, Peter Krempa wrote: It's not used since last commit. Signed-off-by: Peter Krempa --- src/qemu/qemu_capabilities.c | 3 +-- src/qemu/qemu_capabilities.h | 2 +- [...] 38 files changed, 2 insertions(+), 39 deletions(-) Review

Re: [PATCH 077/103] qemu: Always assume presence of QEMU_CAPS_VIRTIO_(BLK|NET)_EVENT_IDX

2021-10-11 Thread Ján Tomko
On a Thursday in 2021, Peter Krempa wrote: The 'event_idx' option for virtio devices was introduced by QEMU commit bcbabae8f which is contained in v0.15.0-rc0 and can't be compiled out, thus we don't need to conditionally enable it. Signed-off-by: Peter Krempa --- src/qemu/qemu_command.c | 6 ++

Re: [PATCH 078/103] qemu: capabilities: Retire QEMU_CAPS_VIRTIO_(BLK|NET)_EVENT_IDX

2021-10-11 Thread Ján Tomko
On a Thursday in 2021, Peter Krempa wrote: Both are assumed as always present since last commit. Signed-off-by: Peter Krempa --- src/qemu/qemu_capabilities.c | 6 ++ src/qemu/qemu_capabilities.h | 4 ++-- tests/qemucapabilitiesdata/caps_2.11.0.s390x

Re: [PATCH 079/103] qemuBuildDiskFrontendAttributeErrorPolicy: Extract logic

2021-10-11 Thread Ján Tomko
On a Thursday in 2021, Peter Krempa wrote: Extract the logic to determine the actual settings into 'qemuBuildDiskGetErrorPolicy' so that it can be reused when we'll convert the disk -device formatter to JSON. Signed-off-by: Peter Krempa --- src/qemu/qemu_command.c | 29 +++--

Re: [PATCH 080/103] qemuBuildDiskCommandLine: Generate via JSON

2021-10-11 Thread Ján Tomko
On a Thursday in 2021, Peter Krempa wrote: The types for the special fields of the 'virtio-blk-pci' according to QEMU are: iothread=> ioeventfd= - on/off (default: true) event_idx= - on/off (default: true) scsi=- on/off (default: false) num-queues=- (default: 655

Re: [PATCH 081/103] qemuBuildInterfaceCommandLine: Generate via JSON

2021-10-11 Thread Ján Tomko
On a Thursday in 2021, Peter Krempa wrote: virtio-net-pci specific properties and their types according to QEMU: tx= ioeventfd= - on/off (default: true) event_idx= - on/off (default: true) csum=- on/off (default: true) gso= - on/off (default: true) host_t

Re: [PATCH 082/103] qemuBuildNicDevProps: Move formatting of bootindex

2021-10-11 Thread Ján Tomko
On a Thursday in 2021, Peter Krempa wrote: Move the bootindex before the address so that the code is simpler. Signed-off-by: Peter Krempa --- src/qemu/qemu_command.c | 6 +- tests/qemuxml2argvdata/aarch64-video-virtio-gpu-pci.args| 2 +- tests/qemuxml2a

Re: [PATCH 083/103] qemu: command: Remove unused qemuBuildRomStr

2021-10-11 Thread Ján Tomko
On a Thursday in 2021, Peter Krempa wrote: Now that all users were converted to qemuBuildRomProps we can remove the old code and un-mark qemuBuildRomProps as unused. Signed-off-by: Peter Krempa --- src/qemu/qemu_command.c | 34 +- 1 file changed, 1 insertion(+), 3

Re: [PATCH 084/103] qemuBuildControllerDevStr: Split out formatting of SCSI controller

2021-10-11 Thread Ján Tomko
On a Thursday in 2021, Peter Krempa wrote: Move the code into a new function called qemuBuildControllerSCSIDevStr so that the code is self contained and the original function easier to follow. This patch also moves the formatting of the properties relevant only for the 'virtio-scsi' controller t

Re: [PATCH 085/103] qemuBuildControllerDevStr: Split out formatting of PCI controller

2021-10-11 Thread Ján Tomko
On a Thursday in 2021, Peter Krempa wrote: Move the code into a new function called qemuBuildControllerPCIDevStr so that the code is self contained and the original function easier to follow. Signed-off-by: Peter Krempa --- src/qemu/qemu_command.c | 157 ++--

Re: [PATCH 086/103] qemuBuildControllerSCSIDevStr: Format via JSON properties

2021-10-11 Thread Ján Tomko
On a Thursday in 2021, Peter Krempa wrote: Internally format the SCSI controller properties into JSON, but convert it back to a string so that we for now change just the SCSI controller. The change in tests is expected as the 'reg' field for a spapr-vio address is expected to be a number: $ qe

Re: [PATCH 087/103] qemuBuildControllerPCIDevStr: Format via JSON properties

2021-10-11 Thread Ján Tomko
On a Thursday in 2021, Peter Krempa wrote: Internally format the PCI controller properties into JSON, but convert it back to a string so that we for now change just the SCSI controller. The change in tests is expected as the 'port' field for various PCI controllers is expected to be a number and

Re: [PATCH 088/103] qemuBuildUSBControllerDevStr: Split out validation of USB controller

2021-10-11 Thread Ján Tomko
On a Thursday in 2021, Peter Krempa wrote: Move the validation code into a separate function. For now the validation is still kept in the commandline format step as simply just moving it to the validator causes failures in the test suite, which will need to be investigated deeper. Signed-off-by:

Re: [PATCH 089/103] qemuBuildUSBControllerDevStr: Format via JSON properties.

2021-10-11 Thread Ján Tomko
On a Thursday in 2021, Peter Krempa wrote: Internally format the PCI controller properties into JSON, but convert it back to a string as preparation for upcoming refactors. The following types are declared for the properties we use by QEMU: 'nec-usb-xhci' p2=- (default: 4) p3=

Re: [PATCH 090/103] qemuBuildControllersByTypeCommandLine: Generate via JSON

2021-10-11 Thread Ján Tomko
On a Thursday in 2021, Peter Krempa wrote: The handlers for PCI, SCSI and USB controllers already use JSON internally. This patch converts 'virtio-serial', 'ccid' and 'sata' to do the same and passes out the JSON directly so that it can be used in monitor code to avoid conversion. From the cont

Re: [PATCH 091/103] qemuBuildChrDeviceCommandLine: Generate via JSON

2021-10-11 Thread Ján Tomko
On a Thursday in 2021, Peter Krempa wrote: Build commandlines for character devices via JSON. For devices using 'VIR_DOMAIN_DEVICE_ADDRESS_TYPE_VIRTIO_SERIAL' address type 'qemuBuildDeviceAddressProps' will now generate the address. The only special property is 'nr'. QEMU declares it as: nr=

Re: [PATCH 092/103] qemuBuildHostdevCommandLine: Generate via JSON

2021-10-11 Thread Ján Tomko
On a Thursday in 2021, Peter Krempa wrote: Build the properties of 'vhost-scsi' device via JSON. In comparison to previous similar refactors this also modifies the hotplug code to attach the vhost fd handle explicitly rather than using 'qemuMonitorAddDeviceWithFd'. The 'vhost-scsi' device doesn'

Re: [PATCH 093/103] qemuBuildVsockCommandLine: Generate via JSON

2021-10-11 Thread Ján Tomko
On a Thursday in 2021, Peter Krempa wrote: Build the properties of 'vhost-vsock' device via JSON. In comparison to previous similar refactors this also modifies the hotplug code to attach the vhost fd handle explicitly rather than using 'qemuMonitorAddDeviceWithFd'. The properties of vhost-vsock

Re: [PATCH 094/103] qemu: monitor: Remove legacy 'device_add' infrastrcture

2021-10-11 Thread Ján Tomko
On a Thursday in 2021, Peter Krempa wrote: Remove the old-style 'device_add' helpers which parse the commandline arguments to JSON since we now coverted all usage to use JSON directly. Signed-off-by: Peter Krempa --- src/qemu/qemu_monitor.c | 34 src/qemu/qemu_monitor.h | 7

Re: [PATCH 095/103] qemuBuildVHostUserFsCommandLine: Generate via JSON

2021-10-11 Thread Ján Tomko
On a Thursday in 2021, Peter Krempa wrote: The 'vhost-user-fs-pci' has following properties we control: chardev= - ID of a chardev to use as a backend queue-size=- (default: 128) tag= bootindex= Signed-off-by: Peter Krempa --- src/qemu/qemu_command.c | 43 -

Re: [PATCH 096/103] qemuBuildFSDevCommandLine: Generate via JSON

2021-10-11 Thread Ján Tomko
On a Thursday in 2021, Peter Krempa wrote: Signed-off-by: Peter Krempa --- src/qemu/qemu_command.c | 37 - 1 file changed, 20 insertions(+), 17 deletions(-) Reviewed-by: Ján Tomko Jano signature.asc Description: PGP signature

Re: [PATCH 097/103] qemuBuildVideoCommandLine: Generate via JSON

2021-10-11 Thread Ján Tomko
On a Thursday in 2021, Peter Krempa wrote: We control the following properties of the devices in question: 'virtio-gpu' virgl= - on/off (default: true) 'qxl' ram_size= - (default: 67108864) vram_size= - (default: 67108864) vram64_size_mb= - (default: 4294967295) vgame

Re: [PATCH 099/103] qemuBuildSoundCommandLine: Generate via JSON

2021-10-11 Thread Ján Tomko
On a Thursday in 2021, Peter Krempa wrote: The sound devices have only the 'audiodev' property which is a string. Signed-off-by: Peter Krempa --- src/qemu/qemu_command.c | 40 +++- 1 file changed, 23 insertions(+), 17 deletions(-) diff --git a/src/qemu/qemu_c

Re: [PATCH 100/103] qemuBuildSoundCommandLine: Generate codecs via JSON

2021-10-11 Thread Ján Tomko
On a Thursday in 2021, Peter Krempa wrote: The codec devices have the following properties we control: cad= - (default: 4294967295) audiodev= - ID of an audiodev to use as a backend Signed-off-by: Peter Krempa --- src/qemu/qemu_command.c | 64 ++-

Re: [PATCH 098/103] qemu: Command remove unused 'qemuBuildVirtioDevStr'

2021-10-11 Thread Ján Tomko
On a Thursday in 2021, Peter Krempa wrote: All virtio devices were converted to the new JSON formatter so we can remove the old one. Signed-off-by: Peter Krempa --- src/qemu/qemu_command.c | 65 + 1 file changed, 1 insertion(+), 64 deletions(-) Reviewed

Re: [PATCH 101/103] qemuBuildHubCommandLine: Generate via JSON

2021-10-11 Thread Ján Tomko
On a Thursday in 2021, Peter Krempa wrote: The 'usb-hub' device doesn't have any special properties. Signed-off-by: Peter Krempa --- src/qemu/qemu_command.c | 33 ++--- 1 file changed, 18 insertions(+), 15 deletions(-) Reviewed-by: Ján Tomko Jano signature.asc

Re: [PATCH 102/103] qemuBuildTPMCommandLine: Generate via JSON

2021-10-11 Thread Ján Tomko
On a Thursday in 2021, Peter Krempa wrote: We control only the 'tpmdev' property of TPM devices which is a string. Signed-off-by: Peter Krempa --- src/qemu/qemu_command.c | 32 +++ .../ppc64-tpmproxy-with-tpm.ppc64-latest.args | 2 +- .../tpm-emulator-spapr.

Re: [PATCH 103/103] qemu: command: Remove unused 'qemuBuildDeviceAddressStr'

2021-10-11 Thread Ján Tomko
On a Thursday in 2021, Peter Krempa wrote: All users were converted to the JSON counterpart. Signed-off-by: Peter Krempa --- src/qemu/qemu_command.c | 99 + 1 file changed, 1 insertion(+), 98 deletions(-) Reviewed-by: Ján Tomko Jano signature.asc De

Re: [PATCH 102/103] qemuBuildTPMCommandLine: Generate via JSON

2021-10-11 Thread Ján Tomko
On a Monday in 2021, Ján Tomko wrote: On a Thursday in 2021, Peter Krempa wrote: We control only the 'tpmdev' property of TPM devices which is a string. Signed-off-by: Peter Krempa --- src/qemu/qemu_command.c | 32 +++ .../ppc64-tpmproxy-with-tpm.ppc64-late

Re: [PATCH v2 4/7] qemuxml2argvtest: Use real-caps testing for 'acpi-hotplug-bridge-disable'

2021-10-11 Thread Andrea Bolognani
On Mon, Oct 11, 2021 at 04:51:40PM +0200, Ján Tomko wrote: > On a Monday in 2021, Peter Krempa wrote: > > +++ > > b/tests/qemuxml2argvdata/aarch64-acpi-hotplug-bridge-disable.aarch64-latest.err > > @@ -0,0 +1 @@ > > +XML error: The PCI controller with index='0' must be model='pcie-root' for > > t

Re: [PATCH v2 1/5] qapi: Enable enum member introspection to show more than name

2021-10-11 Thread Eric Blake
On Sat, Oct 09, 2021 at 02:09:40PM +0200, Markus Armbruster wrote: > The next commit will add feature flags to enum members. There's a > problem, though: query-qmp-schema shows an enum type's members as an > array of member names (SchemaInfoEnum member @values). If it showed > an array of objects

Re: [PATCH v2 4/5] qapi: Implement deprecated-input={reject,crash} for enum values

2021-10-11 Thread Eric Blake
On Sat, Oct 09, 2021 at 02:09:43PM +0200, Markus Armbruster wrote: > This copies the code implementing the policy from qapi/qmp-dispatch.c > to qapi/qobject-input-visitor.c. Tolerable, but if we acquire more > copes, we should look into factoring them out. copies > > Signed-off-by: Markus Armbr

Re: [PATCH RFC v2 5/5] block: Deprecate transaction type drive-backup

2021-10-11 Thread Eric Blake
On Sat, Oct 09, 2021 at 02:09:44PM +0200, Markus Armbruster wrote: > Several moons ago, Vladimir posted > > Subject: [PATCH v2 3/3] qapi: deprecate drive-backup > Date: Wed, 5 May 2021 16:58:03 +0300 > Message-Id: <20210505135803.67896-4-vsement...@virtuozzo.com> > https://lists.g

Re: set custom loglevel for external libraries

2021-10-11 Thread Olaf Hering
Am Mon, 11 Oct 2021 13:25:20 -0600 schrieb Jim Fehlig : > >> I think it is wrong to use internal libvirt values for internal > >> libxentoollog values. > What is gained over the sparse mapping? Are values not covered actually > useful > in practice? Also, how are the values specified when usi

Re: [PATCH v2 09/15] softmmu/qdev-monitor: add error handling in qdev_set_id

2021-10-11 Thread Eric Blake
On Fri, Oct 08, 2021 at 03:34:36PM +0200, Kevin Wolf wrote: > From: Damien Hedde > > qdev_set_id() is mostly used when the user adds a device (using > -device cli option or device_add qmp command). This commit adds > an error parameter to handle the case where the given id is > already taken. >

[PATCH] gitdm: Add other email into Red Hat company list

2021-10-11 Thread Julio Faracco
Some people from Red Hat does not use 'redhat.com' domain emails. They use personal or other domains. Signed-off-by: Julio Faracco --- docs/gitdm/companies/ibm| 1 - docs/gitdm/companies/redhat | 1 + 2 files changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/gitdm/companies/ibm b/doc

Re: set custom loglevel for external libraries

2021-10-11 Thread Jim Fehlig
On 10/11/21 04:30, Olaf Hering wrote: This question is still open: where should the libxl specific setting be specified? Sorry for not responding to your earlier mail. Am Sun, 25 Jul 2021 10:17:49 +0200 schrieb Olaf Hering : I think it is wrong to use internal libvirt values for internal l

Re: set custom loglevel for external libraries

2021-10-11 Thread Jim Fehlig
On 10/11/21 14:38, Olaf Hering wrote: Am Mon, 11 Oct 2021 13:25:20 -0600 schrieb Jim Fehlig : I think it is wrong to use internal libvirt values for internal libxentoollog values. What is gained over the sparse mapping? Are values not covered actually useful in practice? Also, how are the val