all pins are high
- Connected GPIO pin tests of P10 PCA9552 device. Tests
output of pins 0-4 affect input of pins 5-9 respectively.
- PCA9554 GPIO pins test. Tests input and ouput functionality.
Reviewed-by: Cédric Le Goater
Signed-off-by: Glenn Miles
---
Changes from previous
On 1/25/24 15:42, Cédric Le Goater wrote:
When doing device assignment of a physical device, MSI-X can be
enabled with no vectors enabled and this sets the IRQ index to
VFIO_PCI_MSIX_IRQ_INDEX. However, when MSI-X is disabled, the IRQ
index is left untouched if no vectors are in use. Then, when
MSI-X is
disabled.
Buglink: https://issues.redhat.com/browse/RHEL-21293
Fixes: 5ebffa4e87e7 ("vfio/pci: use an invalid fd to enable MSI-X")
Cc: Jing Liu
Cc: Alex Williamson
Signed-off-by: Cédric Le Goater
---
hw/vfio/pci.c | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
will require more work, a new CPU PVR, may be with
a machine "fused=on/off" option, and various adjustements in the models.
I hope you find time to upstream these changes at least for OPAL/P10.
Reviewed-by: Cédric Le Goater
C.
---
include/hw/ppc/pnv_chip.h | 2 +-
include/hw/ppc/
TLB elided flushes
Before37s 508238 7680722
After 34s 73 1143
Signed-off-by: Nicholas Piggin
Acked-by: Cédric Le Goater
Thanks,
C.
---
target/ppc/mmu_helper.c | 43 +++--
1
On 1/17/24 16:12, Nicholas Piggin wrote:
Rather than tlbwe_lo always flushing all TCG TLBs, have it flush just
those corresponding to the old software TLB, and only if it was valid.
Signed-off-by: Nicholas Piggin
Acked-by: Cédric Le Goater
Thanks,
C.
---
target/ppc/mmu_helper.c | 10
On 1/17/24 16:12, Nicholas Piggin wrote:
Have 440 tlbwe flush only the range corresponding to the addresses
covered by the software TLB entry being modified rather than the
entire TLB. This matches what 4xx does.
Signed-off-by: Nicholas Piggin
Acked-by: Cédric Le Goater
Thanks,
C
.
Signed-off-by: Nicholas Piggin
Acked-by: Cédric Le Goater
Thanks,
C.
---
target/ppc/mmu_helper.c | 7 ---
1 file changed, 7 deletions(-)
diff --git a/target/ppc/mmu_helper.c b/target/ppc/mmu_helper.c
index 949ae87f4f..68632bf54e 100644
--- a/target/ppc/mmu_helper.c
+++ b/target
On 1/17/24 16:12, Nicholas Piggin wrote:
Flushing the TCG TLB pages that cache a software TLB is a common
operation, factor it into its own function.
Signed-off-by: Nicholas Piggin
Acked-by: Cédric Le Goater
Thanks,
C.
---
target/ppc/mmu_helper.c | 24 +---
1
.
- If the TLB attr bit 0 (translation address space) changes.
- If low prot (access control) bits change.
Fix this by removing tricks to avoid TLB flushes, and just invalidate
the TLB if any valid entry is being changed, similarly to 4xx.
Signed-off-by: Nicholas Piggin
Acked-by: Cédric Le Goater
On 1/17/24 15:12, Nicholas Piggin wrote:
ppc has no avocado tests for the KVM backend. Add a KVM boot_linux.py
test for pseries.
Signed-off-by: Nicholas Piggin
Reviewed-by: Cédric Le Goater
Thanks,
C.
---
tests/avocado/boot_linux.py | 8
1 file changed, 8 insertions
On 1/17/24 15:12, Nicholas Piggin wrote:
POWER CPUs support hash and radix MMU modes. Linux supports running in
either mode, but defaults to radix. To keep up testing of QEMU's hash
MMU implementation, add some Linux hash boot tests.
Signed-off-by: Nicholas Piggin
Reviewed-by: Cédric Le
is sometimes cut off from console
+# output, suspect S14silence-console init script.
def test_ppc_powernv8(self):
"""
Reviewed-by: Cédric Le Goater
Thanks,
C.
-by: Cédric Le Goater
Thanks,
C.
---
tests/avocado/boot_linux.py | 16 +++-
1 file changed, 7 insertions(+), 9 deletions(-)
diff --git a/tests/avocado/boot_linux.py b/tests/avocado/boot_linux.py
index de4c8805f7..7c9cf6ae15 100644
--- a/tests/avocado/boot_linux.py
+++ b/tests
On 1/25/24 06:55, Philippe Mathieu-Daudé wrote:
Series fully reviewed.
There is not much in the aspeed queue apart from [1]. I will consider
sending a PR after a review (Reviewers please ?). Anyhow, if you need
this series soon, feel free to take it.
Thanks,
C.
[1]
dth and using it to override the
phys_bits value as shown in this patch.
Previous attempt at solving this issue in OVMF:
https://edk2.groups.io/g/devel/topic/102359124
Cc: Gerd Hoffmann
Cc: Philippe Mathieu-Daudé
Cc: Alex Williamson
Cc: Cédric Le Goater
Cc: Laszlo Ersek
Cc: Dongwon Kim
Ac
is invalid (see commit
e702cbc19e ("machine: Improve is_cpu_type_supported()").
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Cédric Le Goater
Thanks,
C.
---
include/hw/arm/aspeed_soc.h | 3 ++-
hw/arm/aspeed.c | 1 +
hw/arm/aspeed_ast10x0.c | 6 +
On 1/23/24 23:48, Philippe Mathieu-Daudé wrote:
In order to alter AspeedSoCClass::cpu_type in the next
commit, introduce the aspeed_soc_cpu_type() helper to
retrieve the per-SoC CPU type from AspeedSoCClass.
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Cédric Le Goater
Thanks,
C
_name);
+aspeed_machine_class_init_cpus_defaults(mc);
}
static void fby35_reset(MachineState *state, ShutdownCause reason)
@@ -1515,6 +1502,7 @@ static void aspeed_machine_fby35_class_init(ObjectClass
*oc, void *data)
amc->i2c_init = fby35_i2c_init;
/* FIXME: Replace this macro with somethin
oc:
Add AST1030 support") and supermicrox11-bmc (commit 40a38df55e
"hw/arm/aspeed: Add board model for Supermicro X11 BMC") machines.
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Cédric Le Goater
Thanks,
C.
---
hw/arm/aspeed.c | 5 -
1 file changed, 4 insertions(+),
On 1/23/24 23:48, Philippe Mathieu-Daudé wrote:
Remove copy/paste typo from commit 6c323aba40 ("hw/arm/aspeed:
Adding new machine Tiogapass in QEMU").
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Cédric Le Goater
Thanks,
C.
---
hw/arm/aspeed.c | 1 -
1 file
On 1/7/24 18:05, Nicholas Piggin wrote:
is_prefix_insn_excp() loads the first word of the instruction address
which caused an exception, to determine whether or not it was prefixed
so the prefix bit can be set in [H]SRR1.
In case it was the instruction fetch itself that caused the exception,
On 1/23/24 10:46, Duan, Zhenzhong wrote:
-Original Message-
From: Cédric Le Goater
Subject: Re: [PATCH rfcv1 4/6] vfio: initialize IOMMUFDDevice and pass to
vIOMMU
On 1/15/24 11:13, Zhenzhong Duan wrote:
Initialize IOMMUFDDevice in vfio and pass to vIOMMU, so that vIOMMU
could get
e raven MR
to correctly mark it as handling unaligned accesses itself.
Tested with 'make check' and 'make check-avocado' only.
It doesn't affect the prep machine boot with OpenBIOS and a
"Debian GNU/Linux 3.0 6015" image.
Tested-by: Cédric Le Goater
Reviewed-by: Cédric Le Goater
Thanks,
55a7fa34f89 ("target/ppc: Machine check on invalid real address access on
POWER9/10")
Fixes: 5a5d3b23cb2 ("target/ppc: Add SRR1 prefix indication to interrupt
handlers")
Signed-off-by: Nicholas Piggin
Ack-by: Cédric Le Goater
Thanks,
C.
---
target/ppc/excp_helper.c |
On 1/22/24 07:40, Zhenzhong Duan wrote:
IOMMUPciBus pointer cache is indexed by bus number, bus number
may not always be a fixed value, i.e., guest reboot to different
kernel which set bus number with different algorithm.
This could lead to endpoint binding to wrong iommu MR in
On 1/23/24 10:27, Cédric Le Goater wrote:
On 1/23/24 07:38, Philippe Mathieu-Daudé wrote:
Restrict MachineClass::valid_cpu_types[] to the single
valid CPU type.
Signed-off-by: Philippe Mathieu-Daudé
---
hw/arm/aspeed.c | 12
1 file changed, 12 insertions(+)
diff --git a/hw
On 1/23/24 07:38, Philippe Mathieu-Daudé wrote:
Restrict MachineClass::valid_cpu_types[] to the single
valid CPU type.
Signed-off-by: Philippe Mathieu-Daudé
---
hw/arm/aspeed.c | 12
1 file changed, 12 insertions(+)
diff --git a/hw/arm/aspeed.c b/hw/arm/aspeed.c
index
On 1/23/24 07:38, Philippe Mathieu-Daudé wrote:
Restrict MachineClass::valid_cpu_types[] to the single
valid CPU type.
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Cédric Le Goater
Thanks,
C.
---
hw/arm/aspeed.c | 8
1 file changed, 8 insertions(+)
diff --git a/hw
On 1/23/24 07:38, Philippe Mathieu-Daudé wrote:
Restrict MachineClass::valid_cpu_types[] to the single
valid CPU type.
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Cédric Le Goater
Thanks,
C.
---
hw/arm/aspeed.c | 14 ++
1 file changed, 14 insertions(+)
diff
On 1/15/24 11:13, Zhenzhong Duan wrote:
From: Yi Liu
Add a framework to check and synchronize host IOMMU cap/ecap with
vIOMMU cap/ecap.
Currently only stage-2 translation is supported which is backed by
shadow page table on host side. So we don't need exact matching of
each bit of cap/ecap
On 1/23/24 07:37, Duan, Zhenzhong wrote:
-Original Message-
From: Cédric Le Goater
Subject: Re: [PATCH rfcv1 2/6] hw/pci: introduce
pci_device_set/unset_iommu_device()
On 1/15/24 11:13, Zhenzhong Duan wrote:
From: Yi Liu
This adds pci_device_set/unset_iommu_device() to set/unset
(e.g., DEC, TB, CFAR), so risk
of breaking users should be low.
QEMU should follow the ISA for register name convention (where there is
no established GDB name).
Signed-off-by: Nicholas Piggin
Acked-by: Cédric Le Goater
Thanks,
C.
---
There is never a great time to change user interface
-by: Cédric Le Goater
No change from previous version
tests/qtest/meson.build | 1 +
tests/qtest/pnv-host-i2c-test.c | 650
2 files changed, 651 insertions(+)
create mode 100644 tests/qtest/pnv-host-i2c-test.c
diff --git a/tests/qtest/meson.build b
not right, but it is enough for skiboot
and certain other proprietary firmware.
Signed-off-by: Nicholas Piggin
Acked-by: Cédric Le Goater
Thanks,
C.
---
target/ppc/timebase_helper.c | 105 ---
target/ppc/translate.c | 42 +-
2 files
On 12/1/23 13:16, Nicholas Piggin wrote:
Wire the ChipTOD model to powernv9 and powernv10 machines.
Suggested-by-by: Cédric Le Goater
Signed-off-by: Nicholas Piggin
Reviewed-by: Cédric Le Goater
Thanks,
C.
---
include/hw/ppc/pnv_chip.h | 3 +++
hw/ppc/pnv.c | 30
On 1/15/24 11:13, Zhenzhong Duan wrote:
Initialize IOMMUFDDevice in vfio and pass to vIOMMU, so that vIOMMU
could get hw IOMMU information.
In VFIO legacy backend mode, we still pass a NULL IOMMUFDDevice to vIOMMU,
in case vIOMMU needs some processing for VFIO legacy backend mode.
On 1/15/24 11:13, Zhenzhong Duan wrote:
From: Yi Liu
This adds set/unset_iommu_device() implementation in Intel vIOMMU.
In set call, IOMMUFDDevice is recorded in hash table indexed by
PCI BDF.
Signed-off-by: Yi Liu
Signed-off-by: Yi Sun
Signed-off-by: Zhenzhong Duan
---
On 1/15/24 11:13, Zhenzhong Duan wrote:
From: Yi Liu
This adds pci_device_set/unset_iommu_device() to set/unset
IOMMUFDDevice for a given PCIe device. Caller of set
should fail if set operation fails.
Extract out pci_device_get_iommu_bus_devfn() to facilitate
implementation of
On 1/19/24 08:31, Duan, Zhenzhong wrote:
-Original Message-
From: Eric Auger
Subject: Re: [PATCH rfcv1 1/6] backends/iommufd_device: introduce
IOMMUFDDevice
On 1/15/24 11:13, Zhenzhong Duan wrote:
IOMMUFDDevice represents a device in iommufd and can be used as
a communication
Hello,
On 1/22/24 03:06, Peter Xu wrote:
Hi, Peter,
On Fri, Jan 19, 2024 at 04:35:07PM +, Peter Maydell wrote:
I wrote this ages ago and recently picked it back up because of a
recent PCI related reset ordering problem noted by Peter Xu. I'm not
sure if this patchset is necessary as a
Signed-off-by: Nicholas Piggin
May be we could drop support for power10_v1.0 and power9_v1.0 ? These
were bringup CPUs.
Anyhow,
Reviewed-by: Cédric Le Goater
Thanks,
C.
---
hw/ppc/pnv.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/hw/ppc/pnv.c b/hw/ppc/pnv.c
index
Reviewed-by: Cédric Le Goater
Thanks,
C.
---> [blob omitted, patch is just for comments]
pc-bios/skiboot.lid | Bin 2527240 -> 2527328 bytes
roms/skiboot| 2 +-
2 files changed, 1 insertion(+), 1 deletion(-)
diff --git a/roms/skiboot b/roms/skiboot
index 24a7
Hello Nick,
On 1/18/24 16:06, Nicholas Piggin wrote:
This is mostly TCG core emulation improvements and fixes. I
got the chiptod model in there because it's intertwined with
TFMR SPR.
Other non-TCG patches are spapr MSR entry point change which
goes together with the other machine check /
that went unnoticed because the kernel previously did not detect and
issue a warning for this scenario.
Changes for v2:
- Fold a typo fix from patch 2 into patch 1 where it belongs
- Add block comment re: timing of ISM reset
- Add review tags
Reviewed-by: Cédric Le Goater
Thanks,
C.
Hello Nick,
On 1/18/24 15:34, Nicholas Piggin wrote:
I'd like to resurrect this and get it merged, because it's quite
useful for testing (has caught several Linux bugs in the past).
IIRC the main concern David had last time it came up was that it
should have QMP commands too. Which is valid.
On 1/18/24 06:09, Peter Xu wrote:
On Wed, Jan 17, 2024 at 05:03:44PM +0100, Paolo Bonzini wrote:
Do not use uint64_t for the type of the declaration and __u64 when
computing the number of elements in the array.
Signed-off-by: Paolo Bonzini
---
hw/vfio/common.c | 2 +-
1 file changed, 1
On 1/17/24 22:11, Matthew Rosato wrote:
On 1/17/24 10:19 AM, Matthew Rosato wrote:
On 1/17/24 6:01 AM, Cédric Le Goater wrote:
Adding Alex,
On 1/16/24 23:31, Matthew Rosato wrote:
ISM devices are sensitive to manipulation of the IOMMU, so the ISM device
needs to be reset before the vfio-pci
On 1/17/24 15:10, Thomas Huth wrote:
For consistency we should drop the names with a "+" in it in the
long run.
Signed-off-by: Thomas Huth
Reviewed-by: Cédric Le Goater
Thanks,
C.
---
docs/about/deprecated.rst | 9 +
1 file changed, 9 insertions(+)
diff --git a/
a hack in type_name_is_valid() to still allow them for
compatibility reasons. However, there is a much nicer solution for this:
Simply use aliases! This way we can still support the old names without
the need for the ugly hack in type_name_is_valid().
Signed-off-by: Thomas Huth
Reviewed-by: Cédric
On 1/17/24 11:28, Eric Auger wrote:
Hi Peter,
On 1/17/24 10:15, pet...@redhat.com wrote:
From: Peter Xu
QEMU resets do not have a way to order reset hooks. Add one coarse grained
reset stage so that some devices can be reset later than some others.
I would precise that the lowest stage has
eboot")
Fixes: 03451953c7 ("s390x/pci: reset ISM passthrough devices on shutdown and system
reset")
Reported-by: Cédric Le Goater
Signed-off-by: Matthew Rosato
---
hw/s390x/s390-pci-bus.c | 26 +-
hw/s390x/s390-virtio-ccw.c | 2 ++
inclu
? Doesn't KVM handle multiple calls
to KVM_S390_ZPCIOP_DEREG_AEN cleanly ? Just asking, I am no expert there.
Thanks,
C.
Fixes: d0bc7091c2 ("s390x/pci: enable adapter event notification for interpreted
devices")
Reported-by: Cédric Le Goater
Signed-off-by: Matthew Rosato
---
hw/
aif.
Fixes: 03451953c7 ("s390x/pci: reset ISM passthrough devices on shutdown and system
reset")
Reported-by: Cédric Le Goater
Signed-off-by: Matthew Rosato
---
hw/s390x/s390-pci-kvm.c | 11 ++-
1 file changed, 10 insertions(+), 1 deletion(-)
diff --git a/hw/s390x/s390-pci-
before disabling aif.
Fixes: 03451953c7 ("s390x/pci: reset ISM passthrough devices on shutdown and system
reset")
Reported-by: Cédric Le Goater
Signed-off-by: Matthew Rosato
---
hw/s390x/s390-pci-kvm.c | 11 ++-
1 file changed, 10 insertions(+), 1 deletion(-)
diff --git a/hw/
VICE(>soc->ftgmac100[i]),
+ true, NULL)) {
+ break; /* No configs left; stop asking */
}
}
Acked-by: Cédric Le Goater
Thanks,
C.
On 1/12/24 20:54, Fabiano Rosas wrote:
Cédric Le Goater writes:
On 1/10/24 14:19, Fabiano Rosas wrote:
Markus Armbruster writes:
Peter Xu writes:
On Tue, Jan 09, 2024 at 10:22:31PM +0100, Philippe Mathieu-Daudé wrote:
Hi Fabiano,
On 9/1/24 21:21, Fabiano Rosas wrote:
Cédric Le
Adding Aspeed Engineers. This reminds me of a discussion a while ago.
On 1/11/24 18:38, Stephen Longfield wrote:
We’ve noticed inconsistent behavior when running a large number of aspeed
ast2600 executions, that seems to be tied to a race condition in the smp boot
when executing on TCG-QEMU,
On 1/11/24 00:15, Ninad Palsule wrote:
Added maintainer for IBM FSI model
Signed-off-by: Cédric Le Goater
Please remove my S-o-b.
Signed-off-by: Ninad Palsule
---
MAINTAINERS | 8
1 file changed, 8 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 00ec1f7eca
On 1/11/24 00:15, Ninad Palsule wrote:
Documentation for IBM FSI model.
Signed-off-by: Cédric Le Goater
Please remove mu S-o-b.
Thanks,
C.
Signed-off-by: Ninad Palsule
---
docs/specs/fsi.rst | 138 +++
docs/specs/index.rst | 1 +
2 files
On 1/11/24 00:15, Ninad Palsule wrote:
Added basic qtests for FSI model.
Acked-by: Thomas Huth
Signed-off-by: Cédric Le Goater
Please remove mu S-o-b.
Thanks,
C.
Signed-off-by: Ninad Palsule
---
tests/qtest/aspeed-fsi-test.c | 205 ++
tests/qtest
fsi_aspeed_apb2opb_realize()
- removed FSIMasterState object and fsi_opb_realize()
- simplified OPBus ]
Signed-off-by: Andrew Jeffery
Signed-off-by: Cédric Le Goater
Signed-off-by: Ninad Palsule
Reviewed-by: Joel Stanley
Nah. Joel you should re-review next respin :)
---
v9:
- Removed unused
()
- dropped FSIBus definition ]
Move the list down before my S-o-b.
Signed-off-by: Andrew Jeffery
Reviewed-by: Joel Stanley
I think you can drop Joel's reviews, the code has changed a lot.
Signed-off-by: Cédric Le Goater
Signed-off-by: Ninad Palsule
---
v9:
- Initialized
- dropped hw/fsi/trace-events changes ]
Move the list down before my S-o-b.
Signed-off-by: Andrew Jeffery
Signed-off-by: Cédric Le Goater
Signed-off-by: Ninad Palsule
---
include/hw/fsi/fsi.h | 38 +++
hw/fsi/fsi.c | 111
- reworked fsi_cfam_realize() ]
Move the list down before my S-o-b.
Signed-off-by: Andrew Jeffery
Signed-off-by: Cédric Le Goater
Signed-off-by: Ninad Palsule
---
v9:
- Added more registers to scratchpad
- Removed unnecessary address space
- Removed unnecessary header file
- Defi
ifications should be before my S-o-b.
Signed-off-by: Andrew Jeffery
Signed-off-by: Cédric Le Goater
Signed-off-by: Ninad Palsule
---
v9:
- Changed LBUS memory region to 1MB.
---
meson.build | 1 +
hw/fsi/trace.h| 1 +
include/hw/fsi/lbus.h | 52 +
Hello Ninad,
This is looking much better. I have a couple more comments.
Please wait a little before sending a respin ! :) and please
run make check and script/checkpatch.pl.
On 1/11/24 00:15, Ninad Palsule wrote:
Hello,
Please review the patch-set version 10.
I have incorporated review
On 1/12/24 11:55, Thomas Huth wrote:
On 12/01/2024 06.21, Harsh Prateek Bora wrote:
On 1/12/24 10:42, Thomas Huth wrote:
On 12/01/2024 05.57, Harsh Prateek Bora wrote:
On 1/11/24 22:16, Thomas Huth wrote:
The character "+" is now forbidden in QOM device names (see commit
b447378e1217 -
On 1/10/24 14:19, Fabiano Rosas wrote:
Markus Armbruster writes:
Peter Xu writes:
On Tue, Jan 09, 2024 at 10:22:31PM +0100, Philippe Mathieu-Daudé wrote:
Hi Fabiano,
On 9/1/24 21:21, Fabiano Rosas wrote:
Cédric Le Goater writes:
On 1/9/24 18:40, Fabiano Rosas wrote:
Cédric Le Goater
On 1/10/24 07:26, Peter Xu wrote:
On Wed, Jan 10, 2024 at 07:03:06AM +0100, Markus Armbruster wrote:
Peter Xu writes:
On Tue, Jan 09, 2024 at 10:22:31PM +0100, Philippe Mathieu-Daudé wrote:
Hi Fabiano,
On 9/1/24 21:21, Fabiano Rosas wrote:
Cédric Le Goater writes:
On 1/9/24 18:40
On 1/10/24 07:03, Markus Armbruster wrote:
Peter Xu writes:
On Tue, Jan 09, 2024 at 10:22:31PM +0100, Philippe Mathieu-Daudé wrote:
Hi Fabiano,
On 9/1/24 21:21, Fabiano Rosas wrote:
Cédric Le Goater writes:
On 1/9/24 18:40, Fabiano Rosas wrote:
Cédric Le Goater writes:
On 1/3/24 20
On 1/9/24 22:09, Philippe Mathieu-Daudé wrote:
On 9/1/24 22:07, Philippe Mathieu-Daudé wrote:
Hi Cédric,
On 9/1/24 19:06, Cédric Le Goater wrote:
On 1/9/24 18:40, Fabiano Rosas wrote:
Cédric Le Goater writes:
On 1/3/24 20:53, Fabiano Rosas wrote:
Philippe Mathieu-Daudé writes:
+Peter
Regarding pseries, migration compat broke because of 5bc8d26de20c
("spapr: allocate the ICPState object from under sPAPRCPUCore") which
is similar to the changes proposed by this series, it impacts the QOM
hierarchy. Here is the workaround/fix from Greg : 46f7afa37096
("spapr: fix migration of
ported via
+an alias, but for consistency these will get removed in a future
+release, too. Use ``power5plus_v2.1`` and ``power7plus_v2.1`` instead.
+
System emulator machines
----
Reviewed-by: Cédric Le Goater
Thanks,
C.
a hack in type_name_is_valid() to still allow them for
compatibility reasons. However, there is a much nicer solution for this:
Simply use aliases! This way we can still support the old names without
the need for the ugly hack in type_name_is_valid().
Signed-off-by: Thomas Huth
Reviewed-by: Cédric Le Goater
Thanks,
C.
have gone wrong.
Buglink: https://issues.redhat.com/browse/RHEL-18212
Signed-off-by: Thomas Huth
---
v2: Print the error in s390_machine_protect() instead of doing it
in s390_pv_set_sec_parms(), report the text via Error **errp
Reviewed-by: Cédric Le Goater
s390_pv_
On 1/10/24 21:28, Matthew Rosato wrote:
On 1/10/24 1:30 PM, Cédric Le Goater wrote:
On 9/12/23 13:41, Thomas Huth wrote:
From: Janosch Frank
Bound APQNs have to be reset before tearing down the secure config via
s390_machine_unprotect(). Otherwise the Ultravisor will return a error
code.
So
On 9/12/23 13:41, Thomas Huth wrote:
From: Janosch Frank
Bound APQNs have to be reset before tearing down the secure config via
s390_machine_unprotect(). Otherwise the Ultravisor will return a error
code.
So let's do a subsystem_reset() which includes a AP reset before the
unprotect call.
On 1/10/24 03:37, Peter Xu wrote:
On Tue, Jan 09, 2024 at 02:21:26PM +0100, Cédric Le Goater wrote:
A few things I'd like to mention alongside, because it's documentation
relevant too, and I'd like to collect if there's any comment.
I just mostly rewrote two wiki pages completely:
https
Hello Ninad,
Here are comments on the file organization and configs.
On 11/29/23 00:56, Ninad Palsule wrote:
Hello,
Please review the patch-set version 8.
I have incorporated review comments from Cedric.
- Fixed checkpatch failures.
- Fixed commit messages.
- Fixed LBUS memory map
On 1/5/24 19:04, Fabiano Rosas wrote:
The migration tests have support for being passed two QEMU binaries to
test migration compatibility.
Add a CI job that builds the lastest release of QEMU and another job
that uses that version plus an already present build of the current
version and run the
On 1/9/24 18:40, Fabiano Rosas wrote:
Cédric Le Goater writes:
On 1/3/24 20:53, Fabiano Rosas wrote:
Philippe Mathieu-Daudé writes:
+Peter/Fabiano
On 2/1/24 17:41, Cédric Le Goater wrote:
On 1/2/24 17:15, Philippe Mathieu-Daudé wrote:
Hi Cédric,
On 2/1/24 15:55, Cédric Le Goater wrote
/torvalds/linux.git/commit/?id=9deb10cf160e
Signed-off-by: Cédric Le Goater
---
tests/avocado/machine_aspeed.py | 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/tests/avocado/machine_aspeed.py b/tests/avocado/machine_aspeed.py
index 6fa5459a07bf..cec018142453
On 1/9/24 15:30, Thomas Huth wrote:
It's a common scenario to copy guest images from one host to another
to run the guest on the other machine. This (of course) does not work
with "secure exection" guests since they are encrypted with one certain
host key. However, if you still (accidentally)
On 1/3/24 20:53, Fabiano Rosas wrote:
Philippe Mathieu-Daudé writes:
+Peter/Fabiano
On 2/1/24 17:41, Cédric Le Goater wrote:
On 1/2/24 17:15, Philippe Mathieu-Daudé wrote:
Hi Cédric,
On 2/1/24 15:55, Cédric Le Goater wrote:
On 12/12/23 17:29, Philippe Mathieu-Daudé wrote:
Hi,
When
A few things I'd like to mention alongside, because it's documentation
relevant too, and I'd like to collect if there's any comment.
I just mostly rewrote two wiki pages completely:
https://wiki.qemu.org/ToDo/LiveMigration
https://wiki.qemu.org/Features/Migration>
I merged all the TODO
Hello Ninad,
+static void fsi_aspeed_apb2opb_realize(DeviceState *dev, Error **errp)
+{
+ SysBusDevice *sbd = SYS_BUS_DEVICE(dev);
+ AspeedAPB2OPBState *s = ASPEED_APB2OPB(dev);
+ int i;
+
+ sysbus_init_irq(sbd, >irq);
+
+ memory_region_init_io(>iomem, OBJECT(s), _apb2opb_ops, s,
On 1/9/24 07:46, pet...@redhat.com wrote:
From: Peter Xu
Move it one layer down, so taking Virtio-migration as a feature for
migration.
Cc: Michael S. Tsirkin
Cc: Jason Wang
Signed-off-by: Peter Xu
Reviewed-by: Cédric Le Goater
Thanks,
C.
---
docs/devel/migration/features.rst
On 1/9/24 07:46, pet...@redhat.com wrote:
From: Peter Xu
Move it one layer down, so taking VFIO-migration as a feature for
migration.
Cc: Alex Williamson
Cc: Cédric Le Goater
Signed-off-by: Peter Xu
Reviewed-by: Cédric Le Goater
Thanks,
C.
---
docs/devel/migration/features.rst
+
Postcopy
+.. contents::
+
'Postcopy' migration is a way to deal with migrations that refuse to converge
The quote character is used in a few places to emphasize words
which should be reworked. The rest looks good, so
Reviewed-by: Cédric Le Goater
Thanks,
C.
(or take too l
On 1/9/24 07:46, pet...@redhat.com wrote:
From: Peter Xu
Split that into a separate file, put under "features".
Cc: Yong Huang
Signed-off-by: Peter Xu
Reviewed-by: Cédric Le Goater
Thanks,
C.
---
docs/devel/migration/dirty-limit.rst | 71
On 1/9/24 07:46, pet...@redhat.com wrote:
From: Peter Xu
Split postcopy into a separate file. Introduce a head page "features.rst"
to keep all the features on top of migration framework.
Signed-off-by: Peter Xu
Reviewed-by: Cédric Le Goater
Thanks,
C.
---
docs/devel
On 1/9/24 07:46, pet...@redhat.com wrote:
From: Peter Xu
Move the two sections into a separate file called "best-practises.rst".
Add the entry into index.
Signed-off-by: Peter Xu
Reviewed-by: Cédric Le Goater
Thanks,
C.
---
docs/devel/migration/best-practise
On 1/9/24 07:46, pet...@redhat.com wrote:
From: Peter Xu
Split the section from main.rst into a separate file. Reference it in the
index.rst.
Signed-off-by: Peter Xu
Reviewed-by: Cédric Le Goater
Thanks,
C.
On 1/9/24 07:46, pet...@redhat.com wrote:
From: Peter Xu
Convert the plain old .txt into .rst, add it into migration/index.rst.
Signed-off-by: Peter Xu
Reviewed-by: Cédric Le Goater
Thanks,
C.
---
docs/devel/migration/index.rst | 1 +
docs/devel/migration/virtio.rst | 115
"migration framework".
Cc: Alex Williamson
Cc: Cédric Le Goater
Signed-off-by: Peter Xu
Reviewed-by: Cédric Le Goater
Thanks,
C.
---
docs/devel/index-internals.rst | 3 +--
docs/devel/migration/index.rst | 11 +++
docs/devel/migration/main.rst | 6 +++---
docs/deve
is still not yet converted to rST. That is a job
for later.
Cc: Michael S. Tsirkin
Cc: Jason Wang
Cc: Alex Williamson
Cc: Cédric Le Goater
Signed-off-by: Peter Xu
Reviewed-by: Cédric Le Goater
Thanks,
C.
---
docs/devel/index-internals.rst| 2 +-
docs/devel
-by: Samuel Tardieu
Reviewed-by: Cédric Le Goater
Thanks,
C.
---
docs/about/deprecated.rst | 2 +-
hw/block/tc58128.c| 1 +
2 files changed, 2 insertions(+), 1 deletion(-)
diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
index e6a12c9077..15e39f8bbb 100644
to deprecate it.
Signed-off-by: Samuel Tardieu
Reviewed-by: Cédric Le Goater
Thanks,
C.
---
docs/about/deprecated.rst | 5 +
hw/sh4/shix.c | 1 +
2 files changed, 6 insertions(+)
diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
index 2e15040246
/rtas*
F: tests/qtest/libqos/rtas*
F: tests/avocado/ppc_pseries.py
+F: tests/avocado/ppc_hv_tests.py
PowerNV (Non-Virtualized)
M: Cédric Le Goater
diff --git a/tests/avocado/ppc_hv_tests.py b/tests/avocado/ppc_hv_tests.py
new file mode 100644
index 00..2162d6bd68
--- /dev/null
701 - 800 of 9428 matches
Mail list logo
