nt chip-id.
Signed-off-by: Daniel Henrique Barboza
---
src/conf/domain_validate.c| 5 ++-
src/qemu/qemu_domain.c| 2 +-
src/qemu/qemu_validate.c | 5 +++
.../powernv8-two-sockets.ppc64-latest.args| 35 +
.../qemuxm
These devices must have unique targetIndex/chip-id pairs.
Signed-off-by: Daniel Henrique Barboza
---
src/conf/domain_conf.c| 35 +++
tests/qemuxml2argvdata/powernv8-dupPHBs.err | 1 +
.../powernv8-dupPHBs.ppc64-latest.err | 1 +
tests
available since 5.0.0, meaning that we
need to check QEMU version and arch manually before setting it.
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_capabilities.c| 7 ++-
src/qemu/qemu_capabilities.h| 1 +
tests/qemucapabilitiesdata/caps_7.0.0
As done with the 'chip-id' attribute, use zero as a default
targetIndex value for pnv-phb3 devices in case it's absent
from the controller definition.
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_domain_address.c| 2 ++
src/qemu/
roller has index = 0 this scenario will occur
every time. This patch aims to alleaviate this behavior by adding a new
virDomainDefFormatFlags that will allow an empty address to be formatted
in the XML. This flag is then used only when formatting PowerNV root
ports.
Signed-off-by: Daniel Henriq
This device is an implementation of pcie-root-port, similar to its
sibling pnv-phb3-root-port. Since it's a new model name that Libvirt
automatically sets, we refrain from documenting it to users.
Signed-off-by: Daniel Henrique Barboza
---
docs/schemas/domaincommon.rng | 1 +
src
If ommited from the controller definition, chip-id defaults to zero.
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_domain_address.c | 16 +++-
src/qemu/qemu_validate.c | 5 +
.../powernv8-basic.ppc64-latest.xml
We're going to use the 'targetIndex' element for PowerNV PHBs. Clarify
that the same attribute will have a different meaning in this context.
Signed-off-by: Daniel Henrique Barboza
---
docs/formatdomain.rst | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
di
Update the virDomainControllerIsPowerNVPHB() helper to make the pnv-phb4
device receive the same handling as the existing pnv-phb3.
Signed-off-by: Daniel Henrique Barboza
---
src/conf/domain_conf.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/conf/domain_conf.c b
The 'chip-id' attribute indicates which chip/socket that owns the
PowerNV pcie-root controller.
Reviewed-by: Peter Krempa
Signed-off-by: Daniel Henrique Barboza
---
docs/formatdomain.rst | 6 ++
docs/schemas/domaincommon.rng | 5 +
src/conf/domain_conf.c
The command line for the pnv-phb3 device is similar to the
spapr-pci-host-bridge command line but adding the extra 'chip-id'
attribute.
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_command.c | 21 +--
.../powernv8-basic.ppc64-l
. This means that probing it as
default will be misleading for users. Instead, let's use
virQEMUCapsInitQMPVersionCaps() to check for the adequate QEMU version
and arch and set it manually.
Suggested-by: Peter Krempa
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_capabilit
nging.
Signed-off-by: Daniel Henrique Barboza
---
docs/schemas/domaincommon.rng | 1 +
src/conf/domain_conf.c | 1 +
src/conf/domain_conf.h | 1 +
src/qemu/qemu_domain_address.c | 5 +
src/qemu/qemu_validate.c | 12 +++-
5 files changed, 19 insertions(
o false since the balloon
driver wasn't really tested with the PowerNV kernel.
Reviewed-by: Peter Krempa
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_domain.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c
index 1b4b29338
Add support for the pcie-root implementation that PowerNV8 domains uses,
pnv-phb3.
It consists of a PCI model name that isn't supposed to be changed by
users, so no doc changes in formatdomain.rst were made.
Signed-off-by: Daniel Henrique Barboza
---
docs/schemas/domaincommo
Both PowerNV and pSeries machines don't support parallel ports.
Reviewed-by: Peter Krempa
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_validate.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/qemu/qemu_validate.c b/src/qemu/qemu_validate.c
index 0a879
ewed-by: Peter Krempa
Signed-off-by: Daniel Henrique Barboza
---
src/conf/domain_conf.c | 56 +-
1 file changed, 28 insertions(+), 28 deletions(-)
diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index 57ab5b19dd..72ee46026a 100644
--- a/src/
Add 'virt type' to allow for an easier time debugging.
Reviewed-by: Peter Krempa
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_validate.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/qemu/qemu_validate.c b/src/qemu/qemu_validate.c
index
Both pSeries and PowerNV machines don't have floppy device support.
Reviewed-by: Peter Krempa
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_capabilities.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/qemu/qemu_capabilities.c b/src
The PowerNV machines uses ISA as the default serial type.
Reviewed-by: Peter Krempa
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_domain.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c
index 98a6a2657d..1b4b293388 100644
--- a
We're now able to boot a simple PowerNV8 domain in Libvirt.
Signed-off-by: Daniel Henrique Barboza
---
.../powernv8-basic.ppc64-latest.args | 33 +++
tests/qemuxml2argvdata/powernv8-basic.xml | 16 +
tests/qemuxml2argvtest.c
The function is now unused outside of qemu_domain.c.
Reviewed-by: Peter Krempa
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_domain.c | 2 +-
src/qemu/qemu_domain.h | 2 --
2 files changed, 1 insertion(+), 3 deletions(-)
diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c
ing a 'qemuDomainIsPowerPC' helper that will be used in those
instances.
[1] https://open-power.github.io/skiboot/doc/overview.html
Reviewed-by: Peter Krempa
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_domain.c | 41 +
src/qemu/qemu_domain.h |
will be handled on QEMU side
- patch 14 (new):
* added documentation of the different semantics 'targetIndex' will have
for PowerNV PHBs
- several other minor changes suggested by Peter
- v1 link:
https://listman.redhat.com/archives/libvir-list/2022-January/msg00902.html
Daniel Henrique Bar
On 1/21/22 11:17, Peter Krempa wrote:
On Thu, Jan 20, 2022 at 10:52:14 -0300, Daniel Henrique Barboza wrote:
Apart from being usable only with pnv-phb3 PCIE host bridges (to be
added soon), this device acts as a regular pcie-root-port but with a
specific model name.
Signed-off-by: Daniel
On 1/21/22 11:21, Peter Krempa wrote:
On Thu, Jan 20, 2022 at 10:52:16 -0300, Daniel Henrique Barboza wrote:
The pnv-phb3 device is the pcie-root controller for PowerNV8 domains.
Add its QEMU capability so Libvirt can recognize it.
Signed-off-by: Daniel Henrique Barboza
---
src/qemu
On 1/21/22 11:33, Daniel P. Berrangé wrote:
On Thu, Jan 20, 2022 at 10:52:12AM -0300, Daniel Henrique Barboza wrote:
The PowerNV machine does not implement any form of migration.
What do you mean by that ?
Migration is a general feature in QEMU, not typically something
that a machine
On 1/24/22 09:04, Peter Krempa wrote:
On Mon, Jan 24, 2022 at 08:57:02 -0300, Daniel Henrique Barboza wrote:
On 1/21/22 11:08, Peter Krempa wrote:
On Thu, Jan 20, 2022 at 10:52:11 -0300, Daniel Henrique Barboza wrote:
We're now able to boot a simple PowerNV8 domain in Libvirt.
S
On 1/21/22 11:08, Peter Krempa wrote:
On Thu, Jan 20, 2022 at 10:52:11 -0300, Daniel Henrique Barboza wrote:
We're now able to boot a simple PowerNV8 domain in Libvirt.
Signed-off-by: Daniel Henrique Barboza
---
tests/qemuxml2argvdata/powernv8-basic.args
On 1/24/22 08:43, Peter Krempa wrote:
On Mon, Jan 24, 2022 at 08:41:38 -0300, Daniel Henrique Barboza wrote:
On 1/21/22 11:02, Peter Krempa wrote:
On Thu, Jan 20, 2022 at 10:52:09 -0300, Daniel Henrique Barboza wrote:
Add 'virt type' to allow for an easier time debugging.
Sig
On 1/21/22 11:02, Peter Krempa wrote:
On Thu, Jan 20, 2022 at 10:52:09 -0300, Daniel Henrique Barboza wrote:
Add 'virt type' to allow for an easier time debugging.
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_validate.c | 6 --
1 file changed, 4 insert
On 1/21/22 10:44, Peter Krempa wrote:
On Thu, Jan 20, 2022 at 10:52:06 -0300, Daniel Henrique Barboza wrote:
Both pSeries and PowerNV machines don't have floppy device support.
qemuDomainMachineIsPSeries() can be turned into static since it's
now unused outside of qem
On 1/20/22 13:30, Andrea Bolognani wrote:
On Mon, Jan 17, 2022 at 02:45:28PM -0300, Daniel Henrique Barboza wrote:
Changes in all 'ppc64-latest.ags' files were needed due to the
JSONification of command line devices.
Signed-off-by: Daniel Henrique Barboza
---
tests/doma
On 1/20/22 10:58, Peter Krempa wrote:
On Thu, Jan 20, 2022 at 10:52:08 -0300, Daniel Henrique Barboza wrote:
The PowerNV machines uses ISA as the default serial type.
ISA? Are you sure?
ISA is very legacy x86 bus.
Yes, these machines uses ISA buses. I believe that since it's also
ort support.
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_capabilities.c| 2 ++
src/qemu/qemu_capabilities.h| 1 +
tests/qemucapabilitiesdata/caps_5.0.0.ppc64.xml | 1 +
tests/qemucapabilitiesdata/caps_5.2.0.ppc64.xml | 1 +
tests/qemucapabilitiesd
nt chip-id.
Signed-off-by: Daniel Henrique Barboza
---
src/conf/domain_validate.c| 5 ++-
src/qemu/qemu_domain.c| 2 +-
src/qemu/qemu_validate.c | 5 +++
.../qemuxml2argvdata/powernv8-two-sockets.xml | 26 +
.../po
As done with the 'chip-id' attribute, use zero as a default
targetIndex value for pnv-phb3 devices in case it's absent
from the controller definition.
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_domain_address.c | 2 ++
src/qemu/qemu_validate.c
The pnv-phb3 device is the pcie-root controller for PowerNV8 domains.
Add its QEMU capability so Libvirt can recognize it.
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_capabilities.c| 2 ++
src/qemu/qemu_capabilities.h| 1 +
tests
Use the 'modelName' attribute for the pnv-phb3-root-port device.
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_domain_address.c | 5 +
src/qemu/qemu_validate.c | 10 +-
2 files changed, 14 insertions(+), 1 deletion(-)
diff --git a/src/qemu/qemu_domain_a
We're now able to boot a simple PowerNV8 domain in Libvirt.
Signed-off-by: Daniel Henrique Barboza
---
tests/qemuxml2argvdata/powernv8-basic.args | 31 +
tests/qemuxml2argvdata/powernv8-basic.xml | 16 +++
tests/qemuxml2argvtest.c| 3 ++
ll free to give this a ride and let us know how it goes.
[1] https://listman.redhat.com/archives/libvir-list/2022-January/msg00766.html
Daniel Henrique Barboza (33):
tests: update QEMU and domain ppc64 capabilities for qemu 7.0
qemu_domain.c: add PowerNV machine helpers
qemu_capabi
nts of pnv-phb3-root-port also
applies for the pnv-phb4-root-port device.
Signed-off-by: Daniel Henrique Barboza
---
src/conf/domain_conf.c | 3 ++-
src/qemu/qemu_domain_address.c | 14 +-
src/qemu/qemu_validate.c | 6 --
3 files changed, 19 insertions(+), 4 deletions
Similar to the existing pnv-phb3 device, pnv-phb4 is also an
implementation of pcie-root.
Signed-off-by: Daniel Henrique Barboza
---
docs/schemas/domaincommon.rng | 1 +
src/conf/domain_conf.c| 1 +
src/conf/domain_conf.h| 1 +
src/qemu/qemu_validate.c | 2 ++
4 files
Since the nuances of PowerNV PHBs and root ports were already handled
when adding support for pnv-phb3* devices, we're already set to support
PowerNV9 PHBs and root ports as well.
Signed-off-by: Daniel Henrique Barboza
---
tests/qemuxml2argvdata/powernv9-dupPHBs.err | 1 +
Use the 'modelName' attribute for the pnv-phb4 pcie-root device. Update
the virDomainControllerIsPowerNVPHB() helper as well so this device
receives the same handling the existing pnv-phb3 has.
Signed-off-by: Daniel Henrique Barboza
---
src/conf/domain_conf.c | 4 ++--
The pnv-phb4 device is the pcie-root controller for PowerNV9 domains.
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_capabilities.c| 2 ++
src/qemu/qemu_capabilities.h| 1 +
tests/qemucapabilitiesdata/caps_5.0.0.ppc64.xml | 1 +
tests
This device is an implementation of pcie-root-port, similar to its
sibling pnv-phb3-root-port.
Signed-off-by: Daniel Henrique Barboza
---
docs/schemas/domaincommon.rng | 1 +
src/conf/domain_conf.c| 1 +
src/conf/domain_conf.h| 1 +
src/qemu/qemu_validate.c | 2 ++
4 files
ight before virDomainDeviceInfoIterate(). This is
enough to allow for a root port to be added in slot 0 of a pnv-phb3
bus while not being intrusive with existing devices.
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_domain_address.c | 14 ++
1 file changed, 1
These devices must have unique targetIndex/chip-id pairs.
Signed-off-by: Daniel Henrique Barboza
---
src/conf/domain_conf.c | 35 +
tests/qemuxml2argvdata/powernv8-dupPHBs.err | 1 +
tests/qemuxml2argvdata/powernv8-dupPHBs.xml | 27
roller has index = 0 this scenario will occur
every time. This patch aims to alleaviate this behavior by adding a new
virDomainDefFormatFlags that will allow an empty address to be formatted
in the XML. This flag is then used only when formatting PowerNV root
ports.
Signed-off-by: Daniel Henriq
Signed-off-by: Daniel Henrique Barboza
---
.../qemuxml2argvdata/powernv8-root-port.args | 33 +
tests/qemuxml2argvdata/powernv8-root-port.xml | 17 +
tests/qemuxml2argvtest.c | 4 +++
.../qemuxml2xmloutdata/powernv8-root-port.xml | 36
The command line for the pnv-phb3 device is similar to the
spapr-pci-host-bridge command line but adding the extra 'chip-id'
attribute.
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_command.c| 21 +++--
tests/qemuxml2argvdata/powernv8-
The 'chip-id' attribute indicates which chip/socket that owns the
PowerNV pcie-root controller.
Signed-off-by: Daniel Henrique Barboza
---
docs/formatdomain.rst | 4
docs/schemas/domaincommon.rng | 5 +
src/conf/domain_conf.c| 15 +++
If ommited from the controller definition, chip-id defaults to zero.
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_domain_address.c | 16 +++-
src/qemu/qemu_validate.c| 5 +
tests/qemuxml2xmloutdata/powernv8-basic.xml | 1 +
3 files
Signed-off-by: Daniel Henrique Barboza
---
src/conf/domain_conf.c | 21 +
src/conf/domain_conf.h | 1 +
src/libvirt_private.syms | 1 +
3 files changed, 23 insertions(+)
diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index d1cda8c3f5..1815308a8c 100644
Use the 'modelName' attribute for the pnv-phb3 pcie-root device.
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_domain_address.c | 3 +++
src/qemu/qemu_validate.c| 6 --
tests/qemuxml2argvtest.c| 3 ++-
tests/qemuxml2
Signed-off-by: Daniel Henrique Barboza
---
docs/schemas/domaincommon.rng | 2 ++
src/conf/domain_conf.c| 1 +
src/conf/domain_conf.h| 1 +
src/qemu/qemu_validate.c | 2 ++
4 files changed, 6 insertions(+)
diff --git a/docs/schemas/domaincommon.rng b/docs/schemas
The identation of VIR_DOMAIN_CONTROLLER_TYPE_PCI elements are in the
same level as the parent 'if (def->type == ...TYPE_PCI)' clause,
and the closing bracket of this 'if' looks like a misplaced bracket
of the 'targetIndex' clause that comes right before it.
Signe
The PowerNV machine does not implement any form of migration.
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_migration.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/src/qemu/qemu_migration.c b/src/qemu/qemu_migration.c
index 2635ef1162..dc2fe92e9b 100644
--- a/src/qemu
The pnv-phb3-root-port device is the pcie-root-port device for PowerNV8
domains.
Add its QEMU capability so Libvirt can recognize it.
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_capabilities.c| 2 ++
src/qemu/qemu_capabilities.h| 1 +
tests
The PowerNV machines uses ISA as the default serial type.
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_domain.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c
index bf418c8e80..88487c6293 100644
--- a/src/qemu/qemu_domain.c
Apart from being usable only with pnv-phb3 PCIE host bridges (to be
added soon), this device acts as a regular pcie-root-port but with a
specific model name.
Signed-off-by: Daniel Henrique Barboza
---
docs/schemas/domaincommon.rng | 1 +
src/conf/domain_conf.c| 1 +
src/conf
Add 'virt type' to allow for an easier time debugging.
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_validate.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/src/qemu/qemu_validate.c b/src/qemu/qemu_validate.c
index da41848761..95af0ecf3b 100644
Both pSeries and PowerNV machines don't have floppy device support.
qemuDomainMachineIsPSeries() can be turned into static since it's
now unused outside of qemu_domain.c.
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_capabilities.c | 4 ++--
src/qemu/qemu_domain.c |
o false since the balloon
driver wasn't really tested with the PowerNV kernel.
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_domain.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c
index 88487c6293..d0bb4938ec 100644
---
ing a 'qemuDomainIsPowerPC' helper that will be used in those
instances.
[1] https://open-power.github.io/skiboot/doc/overview.html
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_domain.c | 41 +
src/qemu/qemu_domain.h | 4
2 files changed
Both PowerNV and pSeries machines don't support parallel ports.
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_validate.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/qemu/qemu_validate.c b/src/qemu/qemu_validate.c
index ae0ee4e744..da41848761 100644
Create extra g_autofree strings and use them in an adequate scope to
eliminate the need for VIR_FREE() calls and the 'cleanup' label.
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_capabilities.c | 43 +++-
1 file changed, 18 insertions(+), 25
Create a new helper to remove the emulator validation logic from the
body of virQEMUCapsLoadCache().
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_capabilities.c | 35 ---
1 file changed, 24 insertions(+), 11 deletions(-)
diff --git a/src/qemu
Create a new helper to remove the parse capabilities flag logic from the
body of virQEMUCapsLoadCache().
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_capabilities.c | 61 +++-
1 file changed, 39 insertions(+), 22 deletions(-)
diff --git a/src/qemu
Create a new helper to remove the arch validation logic from the
body of virQEMUCapsLoadCache().
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_capabilities.c | 31 +--
1 file changed, 21 insertions(+), 10 deletions(-)
diff --git a/src/qemu
'str' is no longer being used and the 'cleanup' label can be removed
together with the 'ret' variable.
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_capabilities.c | 42
1 file changed, 18 insertions(+), 24 delet
ppc64Compute() but also remove the ppc64MakeCPUData() entirely.
Signed-off-by: Daniel Henrique Barboza
---
src/cpu/cpu_ppc64.c | 26 +-
1 file changed, 1 insertion(+), 25 deletions(-)
diff --git a/src/cpu/cpu_ppc64.c b/src/cpu/cpu_ppc64.c
index c7caaa9608..314d2f7c86 100
The 'nodes' var is not being used.
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_capabilities.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/src/qemu/qemu_capabilities.c b/src/qemu/qemu_capabilities.c
index 38558a9ee0..2523a369d4 100644
--- a/src/qemu/qemu_capa
- v1 link:
https://listman.redhat.com/archives/libvir-list/2021-November/msg00497.html
Daniel Henrique Barboza (8):
qemu_capabilities.c: add virQEMUCapsParseFlags()
qemu_capabilities.c: add virQEMUCapsParseGIC()
qemu_capabilities.c: del 'nodes' var from virQEMUCap
Create a new helper to remove the GIC parse logic from the body of
virQEMUCapsLoadCache().
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_capabilities.c | 142 +++
1 file changed, 77 insertions(+), 65 deletions(-)
diff --git a/src/qemu
On 11/18/21 11:21, Peter Krempa wrote:
On Thu, Nov 18, 2021 at 11:20:11 -0300, Daniel Henrique Barboza wrote:
I'll resend this series because I somehow messed with my own email in the CC.
Sorry for the span.
Is that really necessary? You'll still get the replies from the mailing
Hi,
Here are some cleanups I ended up making while investigating a
test issue with ppc64 hosts recently.
Daniel Henrique Barboza (7):
qemu_capabilities.c: use g_autofree in virQEMUCapsLoadCache()
cpu_x86.c: remove 'guest' param from x86Compute()
cpu_ppc64.c: remove 'gu
All 'virCPUDef' pointers can be auto-freed and the 'cleanup' label
removed.
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_capabilities.c | 21 -
1 file changed, 8 insertions(+), 13 deletions(-)
diff --git a/src/qemu/qemu_capab
Use 'g_autofree' in the 'str' parameter and remove the 'cleanup'
label.
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_capabilities.c | 29 +++--
1 file changed, 11 insertions(+), 18 deletions(-)
diff --git a/src/qemu/
The 'xmlopt' parameter can be auto-unref by using g_autoptr().
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_process.c | 15 +--
1 file changed, 5 insertions(+), 10 deletions(-)
diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c
index b6c81dd23a..
This will enable code cleanups on code that still relies on
virObjectUnref() this pointer manually.
Signed-off-by: Daniel Henrique Barboza
---
src/conf/domain_conf.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/conf/domain_conf.h b/src/conf/domain_conf.h
index ab9a7d66f8..0ecd3803db
c0c0cb2 a few years ago. Since then all callers of x86Compute()
uses it with 'guest' = NULL.
Removing the 'guest' parameter allow us to remove a good chunk of
logic that isn't being used for awhile.
Signed-off-by: Daniel Henrique Barboza
---
src/cpu/cpu_x86.c | 38 +-
cb2.
Removing the 'guest' parameter will not only remove unused code from
ppc64Compute() but also remove the ppc64MakeCPUData() entirely.
Signed-off-by: Daniel Henrique Barboza
---
src/cpu/cpu_ppc64.c | 26 +-
1 file changed, 1 insertion(+), 25 deletions(-)
diff --
Use autofree with 'str' and 'nodes' to get rid of the 'cleanup'
label.
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_capabilities.c | 66
1 file changed, 30 insertions(+), 36 deletions(-)
diff --git a/src/qemu/
I'll resend this series because I somehow messed with my own email in the CC.
Sorry for the span.
Daniel
On 11/18/21 10:59, Daniel Henrique Barboza wrote:
Hi,
Here are some cleanups I ended up making while investigating a
test issue with ppc64 hosts recently.
Daniel Henrique Barbo
All 'virCPUDef' pointers can be auto-freed and the 'cleanup' label
removed.
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_capabilities.c | 21 -
1 file changed, 8 insertions(+), 13 deletions(-)
diff --git a/src/qemu/qemu_capab
The 'xmlopt' parameter can be auto-unref by using g_autoptr().
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_process.c | 15 +--
1 file changed, 5 insertions(+), 10 deletions(-)
diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c
index b6c81dd23a..
Use autofree with 'str' and 'nodes' to get rid of the 'cleanup'
label.
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_capabilities.c | 66
1 file changed, 30 insertions(+), 36 deletions(-)
diff --git a/src/qemu/
This will enable code cleanups on code that still relies on
virObjectUnref() this pointer manually.
Signed-off-by: Daniel Henrique Barboza
---
src/conf/domain_conf.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/conf/domain_conf.h b/src/conf/domain_conf.h
index ab9a7d66f8..0ecd3803db
Use 'g_autofree' in the 'str' parameter and remove the 'cleanup'
label.
Signed-off-by: Daniel Henrique Barboza
---
src/qemu/qemu_capabilities.c | 29 +++--
1 file changed, 11 insertions(+), 18 deletions(-)
diff --git a/src/qemu/
Hi,
Here are some cleanups I ended up making while investigating a
test issue with ppc64 hosts recently.
Daniel Henrique Barboza (7):
qemu_capabilities.c: use g_autofree in virQEMUCapsLoadCache()
cpu_x86.c: remove 'guest' param from x86Compute()
cpu_ppc64.c: remove 'gu
c0c0cb2 a few years ago. Since then all callers of x86Compute()
uses it with 'guest' = NULL.
Removing the 'guest' parameter allow us to remove a good chunk of
logic that isn't being used for awhile.
Signed-off-by: Daniel Henrique Barboza
---
src/cpu/cpu_x86.c | 38 +-
cb2.
Removing the 'guest' parameter will not only remove unused code from
ppc64Compute() but also remove the ppc64MakeCPUData() entirely.
Signed-off-by: Daniel Henrique Barboza
---
src/cpu/cpu_ppc64.c | 26 +-
1 file changed, 1 insertion(+), 25 deletions(-)
diff --
check='none' and avoid the virCPUCompare()
that causes the problem for ppc64 and s390x hosts. Note that this is a
build fix: a proper investigation of why we're defaulting to an
incompatible x86_64 CPU in the circunstances mentioned above is still
warranted.
Reported-by: Boris Fi
Hi,
Re-sending the patch with the commit msg change proposed by
Peter (i.e. mentioning that this is a build fix, root cause is
still under investigation) and with Boris r-b.
v1 link:
https://listman.redhat.com/archives/libvir-list/2021-November/msg00457.html
Daniel Henrique Barboza (1
On 11/17/21 08:26, Peter Krempa wrote:
On Wed, Nov 17, 2021 at 08:16:05 -0300, Daniel Henrique Barboza wrote:
On 11/17/21 08:02, Boris Fiuczynski wrote:
On 11/17/21 1:11 AM, Daniel Henrique Barboza wrote:
[...]
fix works for you?
The proposed patch works on s390x as well.
Reviewed-by
On 11/17/21 08:02, Boris Fiuczynski wrote:
On 11/17/21 1:11 AM, Daniel Henrique Barboza wrote:
Commit 65b0b746b516 changed spice tests to use latest caps. Before this
change, "FLAG_REAL_CAPS" wasn't being set in testQemuInfoInitArgs(). The
absence of this flag triggered the c
On 11/17/21 07:51, Peter Krempa wrote:
On Tue, Nov 16, 2021 at 21:11:50 -0300, Daniel Henrique Barboza wrote:
Commit 65b0b746b516 changed spice tests to use latest caps. Before this
change, "FLAG_REAL_CAPS" wasn't being set in testQemuInfoInitArgs(). The
absence of this fla
check='none' and avoid the virCPUCompare()
that causes the problem for ppc64 and s390x hosts.
Reported-by: Boris Fiuczynski
Signed-off-by: Daniel Henrique Barboza
---
Sending as a RFC because I'm not sure if this patch fixes the
problem for s390x. Boris, can you please te
101 - 200 of 1918 matches
Mail list logo