From: Nicholas Piggin
s390x with TCG is more stable now. Enable it.
Signed-off-by: Nicholas Piggin
Message-Id: <20240525131241.378473-3-npig...@gmail.com>
Reviewed-by: Prasad Pandit
[thuth: Added "with TCG" to the commit message]
Signed-off-by: Thomas Huth
---
tests/qtest/
The -Wl,-pie and -nostdlib flags are added to LDFLAGS twice. Merge
the two lines to get rid of the duplicates.
Message-ID: <20240621082422.136217-2-th...@redhat.com>
Signed-off-by: Thomas Huth
---
pc-bios/s390-ccw/Makefile | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff
From: Akihiko Odaki
These GThreads are never referenced.
Signed-off-by: Akihiko Odaki
Reviewed-by: Peter Maydell
Reviewed-by: Michael S. Tsirkin
Message-ID: <20240627-san-v2-15-750bb0946...@daynix.com>
Signed-off-by: Thomas Huth
---
tests/qtest/vhost-user-test.c | 6 +++---
1 file c
: <20240620135731.977377-1-erdn...@crans.org>
Signed-off-by: Thomas Huth
---
docs/devel/tcg-plugins.rst | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/docs/devel/tcg-plugins.rst b/docs/devel/tcg-plugins.rst
index 9cc09d8c3d..f7d7b9e3a4 100644
--- a/docs/devel/tcg-plugins.rst
+++ b/docs
From: Akihiko Odaki
This fixes LeakSanitizer warnings.
Signed-off-by: Akihiko Odaki
Reviewed-by: Michael S. Tsirkin
Message-ID: <20240627-san-v2-14-750bb0946...@daynix.com>
Signed-off-by: Thomas Huth
---
tests/qtest/qos-test.c | 16
1 file changed, 12 insertions
e, so s390_cpu_exec_interrupt falls through to halting
again."
Thus let's finally migrate the pending state, and to be on the safe
side, also the other state variables of the QEMUS390FLICState structure.
Message-ID: <20240619144421.261342-1-th...@redhat.com>
Signed-off-by: Thomas Huth
---
):
tests/qtest/migration-test: enable on s390x with TCG
Thomas Huth (4):
hw/intc/s390_flic: Fix interrupt controller migration on s390x with TCG
hw/s390x: Attach default virtio-net devices to the
/machine/virtual-css-bridge
.travis.yml: Install python3-tomli in all build jobs
pc
' followed by an endless string of 'B's
I think the better fix would be to call bootfile_create() only once from
main() since we don't have to create the bootfile multiple times, do we?
Thomas
he qtest patches through my tree.
Thomas
-introspect-test.c | 7 +++
1 file changed, 3 insertions(+), 4 deletions(-)
Reviewed-by: Thomas Huth
The initial virtio-net-ccw devices currently do not have a proper parent
in the QOM tree, so they show up under /machine/unattached - which is
somewhat ugly. Let's attach them to /machine/virtual-css-bridge/virtual-css
instead.
Signed-off-by: Thomas Huth
---
hw/s390x/s390-virtio-ccw.c | 3
On 01/07/2024 17.06, Michael S. Tsirkin wrote:
On Mon, Jul 01, 2024 at 04:07:56PM +0200, Thomas Huth wrote:
On 18/06/2024 14.19, Thomas Huth wrote:
The unrealize functions of the various vhost-user devices are
calling the corresponding vhost_*_set_status() functions with a
status of 0 to shut
On 18/06/2024 14.19, Thomas Huth wrote:
The unrealize functions of the various vhost-user devices are
calling the corresponding vhost_*_set_status() functions with a
status of 0 to shut down the device correctly.
Now these vhost_*_set_status() functions all follow this scheme:
bool
hw/hw.h only contains the prototype of hw_error() nowadays, so
files that don't use this function don't need to include this
header.
Signed-off-by: Thomas Huth
---
include/hw/misc/xlnx-cfi-if.h | 1 -
hw/misc/edu.c | 1 -
hw/vfio/container.c | 1 -
3 files changed, 3
6/pc_q35.c | 215 ---
1 file changed, 90 insertions(+), 125 deletions(-)
Reviewed-by: Thomas Huth
thrice in three different formats in the calls
to DEFINE_I440FX_MACHINE.
Signed-off-by: Daniel P. Berrangé
---
hw/i386/pc_piix.c| 219 +++
include/hw/i386/pc.h | 26 +
2 files changed, 122 insertions(+), 123 deletions(-)
Reviewed-by: Thomas Huth
On 28/06/2024 20.01, Jared Rossi wrote:
On 6/24/24 1:55 AM, Thomas Huth wrote:
[...]
I think it should be fine, both functions are basically just a wrapper
around the write() function in sclp.c, with sclp_print() being rather dumb
while printf() is doing the usual string formatting before
This g_test_skip here does not make too much sense (since you're doing it in
the caller site, too) ... could you please replace it with a proper comment
why this code needs to be reworked? Thanks!
Thomas
sdhci_cmd_regs(qts, NPCM7XX_MMC_BA, 0,
We don't ship a binary that is simply called "qemu", so we should
avoid this in the documentation. Use the configurable binary name
via "|qemu_system|" instead.
Signed-off-by: Thomas Huth
---
docs/system/devices/usb.rst | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-
uint64_t res = 0;
+
+res = qtest_csr_call(qts, name, cpu, csrno, val);
+
+return res;
+}
This wrapper function looks completely unnecessary, just call
qtest_csr_call() everywhere directly instead?
Thomas
+static int qcsr_get_csr(QTestState *qts, uint64_t cpu,
+int csrno,
name,
+ uint64_t cpu, int csr,
+ unsigned long *val);
+
/**
* qtest_bufread:
* @s: #QTestState instance to operate on.
For the tests/qtest part:
Acked-by: Thomas Huth
('-device', 'pcie-pci-bridge,id=pci.1,bus=pcie.0')
+
+self.launch_and_wait()
+self.blockdev_add()
+
+self.plug()
+self.unplug()
Reviewed-by: Thomas Huth
to re-introduce this job.
Partially revert but using the debian-i686 cross build
images this time as fedora has deprecated the 32 bit stuff.
Reported-by: Richard Henderson
Suggested-by: Thomas Huth
Signed-off-by: Alex Bennée
---
.gitlab-ci.d/crossbuilds.yml | 10 ++
1 file changed
to go a step backwards, then basically just
revert commit 4f9a8315e65561bafa03651518aa5d22af09bdee and use the
i686-debian-cross-container image instead of the removed
i386-fedora-cross-container image.
HTH,
Thomas
++-enabled downstream repository.
Thomas
On 24/06/2024 12.09, Alex Bennée wrote:
Thomas Huth writes:
Since commit 1f97715c83 ('Revert "python: use vendored tomli"')
this package is a hard requirement for compiling QEMU, so install
it now in all Travis jobs, too.
AFAICT the only repo currently running these tests is y
uot;scsi-disk-base" class instead? Or should hw_compat_9_0 rather list the
devices that have the property instead of using the parent class? Could you
please have a look?
Thanks,
Thomas
Since commit 1f97715c83 ('Revert "python: use vendored tomli"')
this package is a hard requirement for compiling QEMU, so install
it now in all Travis jobs, too.
Signed-off-by: Thomas Huth
---
.travis.yml | 6 ++
1 file changed, 6 insertions(+)
diff --git a/.travis.yml b/.travis
Goater
Reviewed-by: Zhenzhong Duan
Reviewed-by: Anthony Krowiak
Reviewed-by: Eric Farman
Reviewed-by: Thomas Huth
Message-ID: <20240522170107.289532-6-...@redhat.com>
Signed-off-by: Thomas Huth
---
hw/vfio/ccw.c | 12 +---
1 file changed, 5 insertions(+), 7 deletions(-)
diff --gi
Reviewed-by: Anthony Krowiak
Reviewed-by: Eric Farman
Reviewed-by: Thomas Huth
Message-ID: <20240522170107.289532-5-...@redhat.com>
Signed-off-by: Thomas Huth
---
include/hw/s390x/s390-ccw.h | 2 +-
hw/s390x/s390-ccw.c | 7 ---
hw/vfio/ccw.c | 3 +--
3 files chan
From: Dmitry Frolov
Found with fuzzing for qemu-8.2, but also relevant for master
Signed-off-by: Dmitry Frolov
Reviewed-by: Thomas Huth
Reviewed-by: Alexander Bulekov
Message-ID: <20240521103106.119021-3-fro...@swemel.ru>
Signed-off-by: Thomas Huth
---
tests/qtest/fuzz/qos_fuzz.c |
-by: Zhenzhong Duan
Reviewed-by: Anthony Krowiak
Reviewed-by: Eric Farman
Reviewed-by: Thomas Huth
Message-ID: <20240522170107.289532-8-...@redhat.com>
Signed-off-by: Thomas Huth
---
hw/vfio/ap.c | 2 +-
hw/vfio/ccw.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/hw/vfi
From: Omar Sandoval
The pid field of prstatus needs to be big endian like all of the other
fields.
Fixes: f738f296eaae ("s390x/arch_dump: pass cpuid into notes sections")
Signed-off-by: Omar Sandoval
Reviewed-by: Thomas Huth
Message-ID:
<5929f76d536d355afd04af51bf293695a10651
used to disable them (and old machine types that use them by default).
Message-Id: <20240614125019.588928-1-th...@redhat.com>
Signed-off-by: Thomas Huth
---
hw/s390x/s390-virtio-ccw.c | 5 +
target/s390x/cpu_models.c | 9 +
target/s390x/Kconfig | 5 +
3 files chang
registration errors
Dmitry Frolov (1):
tests/qtest/fuzz: fix memleak in qos_fuzz.c
Omar Sandoval (1):
target/s390x/arch_dump: use correct byte order for pid
Thomas Huth (2):
MAINTAINERS: Cover all tests/qtest/migration-* files
target/s390x: Add a CONFIG switch to disable legacy CPUs
ric Farman
Reviewed-by: Thomas Huth
Message-ID: <20240522170107.289532-7-...@redhat.com>
Signed-off-by: Thomas Huth
---
hw/vfio/ccw.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/hw/vfio/ccw.c b/hw/vfio/ccw.c
index a468fa2342..36f2677a44 100644
--- a/hw/vfio/ccw.c
+++
From: Cédric Le Goater
Use the 'Error **errp' argument of s390_ccw_realize() instead and
remove the error_propagate() call.
Signed-off-by: Cédric Le Goater
Reviewed-by: Zhenzhong Duan
Reviewed-by: Anthony Krowiak
Reviewed-by: Eric Farman
Reviewed-by: Thomas Huth
Message-ID
-Daudé
Acked-by: Fabiano Rosas
Reviewed-by: Peter Xu
Signed-off-by: Thomas Huth
---
MAINTAINERS | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index cef54de759..f144b5af44 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -3312,6 +3312,7 @@ F: tests/qte
Krowiak
Reviewed-by: Eric Farman
Reviewed-by: Thomas Huth
Message-ID: <20240522170107.289532-2-...@redhat.com>
Signed-off-by: Thomas Huth
---
hw/s390x/s390-ccw.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/hw/s390x/s390-ccw.c b/hw/s390x/s390-ccw.c
Reviewed-by: Anthony Krowiak
Reviewed-by: Eric Farman
Reviewed-by: Thomas Huth
Message-ID: <20240522170107.289532-3-...@redhat.com>
Signed-off-by: Thomas Huth
---
hw/s390x/ccw-device.h | 2 +-
hw/s390x/ccw-device.c | 3 ++-
hw/s390x/s390-ccw.c | 3 +--
3 files changed, 4 insertions(+), 4 del
Patch 06b12970174 ("virtio-net: fix network stall under load")
added double-check to test whether the available buffer size
can satisfy the request or not, in case the guest has added
some buffers to the avail ring simultaneously after the first
check.
It will be lucky if the available buffer
deletions(-)
I'm going to pick up patch 1 for my s390x tree ... for the second patch, it
would be good if someone of the ppc guys could have a look at it first.
Thomas
On 21/06/2024 22.51, Eric Farman wrote:
On Fri, 2024-06-21 at 10:24 +0200, Thomas Huth wrote:
We originally built a separate binary for the netboot code since it
was considered as experimental and we could not be sure that the
necessary SLOF module had been checked out. Time passed, the netboot
On 2024-06-21 06:26:19+, Michael S. Tsirkin wrote:
> On Mon, May 27, 2024 at 08:27:49AM +0200, Thomas Weißschuh wrote:
> > diff --git a/hw/misc/pvpanic.c b/hw/misc/pvpanic.c
> > index 1540e9091a45..a4982cc5928e 100644
> > --- a/hw/misc/pvpanic.c
> > +++ b/hw/m
ure whether we want to commit this separately
or wait 'til Jared finished his series, too). Sorry, I should have mentioned
it in the cover letter.
Thomas
and/or possibly "you didn't
tell the kernel to output on the serial port".
By the way, it seems like we don't even have an avocado test for that
machine available. Peter, do you know whether there is a kernel for this
machine available somewhere that we could use for testing?
Thomas
Remove the information about the separate s390-netboot.img from
the documentation.
Signed-off-by: Thomas Huth
---
docs/system/s390x/bootdevices.rst | 20 +++-
1 file changed, 7 insertions(+), 13 deletions(-)
diff --git a/docs/system/s390x/bootdevices.rst
b/docs/system/s390x
Since the netboot code has now been merged into the main s390-ccw.img
binary, we don't need the separate s390-netboot.img anymore. Remove
it and the code that was responsible for loading it.
Signed-off-by: Thomas Huth
---
hw/s390x/ipl.h | 12 +++-
hw/s390x/ipl.c
the s390-ccw bios (see commit bf6903f6944f), we can drop
the simple implementation and use the SLOF libc for the s390-ccw.img
binary, too.
Signed-off-by: Thomas Huth
---
pc-bios/s390-ccw/netboot.mak | 3 --
pc-bios/s390-ccw/libc.h | 89
pc-bios/s390-ccw
device.
Signed-off-by: Thomas Huth
---
pc-bios/s390-ccw/netboot.mak | 14 --
pc-bios/s390-ccw/cio.h | 2 ++
pc-bios/s390-ccw/iplb.h | 4 ++--
pc-bios/s390-ccw/s390-ccw.h | 3 +++
pc-bios/s390-ccw/virtio.h| 1 -
pc-bios/s390-ccw/bootmap.c | 2 +-
pc-bios/s390-ccw
assigned to the machine before blindly
using it.
Signed-off-by: Thomas Huth
---
hw/s390x/ipl.c | 10 +-
1 file changed, 9 insertions(+), 1 deletion(-)
diff --git a/hw/s390x/ipl.c b/hw/s390x/ipl.c
index e934bf89d1..9362de0b6f 100644
--- a/hw/s390x/ipl.c
+++ b/hw/s390x/ipl.c
@@ -45,6 +45,7
Now that the netboot code has been merged into the main s390-ccw.img,
it also does not make sense to keep the build rules in a separate
file. Thus let's merge netboot.mak into the main Makefile.
Signed-off-by: Thomas Huth
---
pc-bios/s390-ccw/netboot.mak | 45
The -Wl,-pie and -nostdlib flags are added to LDFLAGS twice. Merge
the two lines to get rid of the duplicates.
Signed-off-by: Thomas Huth
---
pc-bios/s390-ccw/Makefile | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/pc-bios/s390-ccw/Makefile b/pc-bios/s390-ccw/Makefile
much sense anymore to keep the netboot code
in a separate binary. To make it easier to support a more flexible
boot process soon that supports more than one boot device via the
bootindex properties, let's finally merge the netboot code into the
main s390-ccw.img binary now.
Thomas Huth (7):
pc-bio
e machine.
Can you please help me to use designware-root-host/port devices ?
It seems like the i.MX7 SABRE machine is using this device, so instead of
"-M virt", you could have a try with "-M mcimx7d-sabre" (and a kernel that
supports this machine) instead.
HTH,
Thomas
rn; \
+} \
} \
} while (0)
Reviewed-by: Thomas Huth
ime control does not support migration");
+migrate_add_blocker(_blocker, NULL);
+#endif
return _control;
}
return NULL;
Reviewed-by: Thomas Huth
On 17/06/2024 16.49, Christian Borntraeger wrote:
Am 05.06.24 um 15:37 schrieb Thomas Huth:
On 29/05/2024 17.43, jro...@linux.ibm.com wrote:
From: Jared Rossi
On a panic during IPL (i.e. a device failed to boot) check for another
device
to boot from, as indicated by the presence
On 17/06/2024 01.44, Jared Rossi wrote:
On 6/7/24 1:57 AM, Thomas Huth wrote:
On 05/06/2024 16.48, Jared Rossi wrote:
diff --git a/pc-bios/s390-ccw/s390-ccw.h b/pc-bios/s390-ccw/s390-ccw.h
index c977a52b50..de3d1f0d5a 100644
--- a/pc-bios/s390-ccw/s390-ccw.h
+++ b/pc-bios/s390-ccw/s390
On 19/06/2024 15.34, Paolo Bonzini wrote:
On Wed, Jun 19, 2024 at 2:49 PM Thomas Huth wrote:
On 19/06/2024 13.46, Paolo Bonzini wrote:
Treat it as a MIPS64 machine.
...
diff --git a/configure b/configure
index d0703ea279d..3669eec86e5 100755
--- a/configure
+++ b/configure
@@ -452,7 +452,7
able to me...
... but in case somebody else wants to merge this, FWIW:
s390x parts
Acked-by: Thomas Huth
On Mon, May 27, 2024 at 09:18:37PM +0800, Zhao Liu wrote:
Date: Mon, 27 May 2024 21:18:37 +0800
From: Zhao Liu
Subject: [PATCH] hw/core: Rename CpuTopology to CPUTopology
X-Mailer: git-
e, so s390_cpu_exec_interrupt falls through to halting
again."
Thus let's finally migrate the pending state, and to be on the safe
side, also the other state variables of the QEMUS390FLICState structure.
Signed-off-by: Thomas Huth
---
Once this has been merged, we can enable the migration-
+++ b/configure
@@ -452,7 +452,7 @@ case "$cpu" in
linux_arch=loongarch
;;
- mips64*)
+ mips64*|mipsisa64*)
Maybe simply switch to mips*64*) ?
cpu=mips64
host_arch=mips
linux_arch=mips
Thomas
.note) - sizeof(arg.note.contents) + nf->contents_size;
Reviewed-by: Thomas Huth
gs->acrs[i] = cpu_to_be32(cpu->env.aregs[i]);
regs->gprs[i] = cpu_to_be64(cpu->env.regs[i]);
}
-note->contents.prstatus.pid = id;
+note->contents.prstatus.pid = cpu_to_be32(id);
}
static void s390x_write_elf64_fpregset(Note *note, S390CPU *cpu, int id)
Reviewed-by: Thomas Huth
. Since the test is under very active development, it was causing
a lot of distraction to the generic qtest maintainers with regards to
the patches that need to be reviewed by the migration maintainers anyway.
Signed-off-by: Thomas Huth
---
MAINTAINERS | 3 ++-
1 file changed, 2 insertions(+), 1
| 4
target/s390x/trace-events | 4
6 files changed, 47 insertions(+), 25 deletions(-)
Series
Reviewed-by: Thomas Huth
the same purpose, it should be fine to fix it in this
central place there without any risk to change the behavior of other
code.
Fixes: 9f6bcfd99f ("hw/virtio: move vm_running check to virtio_device_started")
Buglink: https://issues.redhat.com/browse/RHEL-40708
Signed-off-by: Thomas Huth
---
inclu
%"PRIx64" %d 0x%"PRIx64"\n",
+name, cpu, csr, *val);
+
+qtest_rsp_csr(s, val);
Just a matter of taste, but I think I'd rather inline the contents of
qtest_rsp_csr() here since both functions are not very big yet.
(unless you need qtest_rsp_csr() in another function later, then it's of
course better to keep it separate)
+return 0;
+}
Thomas
If guest has made some buffers available during double check,
but the total buffer size available is lower than @bufsize,
notify the guest with the latest available idx(event idx)
seen by the host.
Fixes: 06b12970174 ("virtio-net: fix network stall under load")
Signed-off-by: wencheng Yang
---
used to disable them (and old machine types that use them by default).
Signed-off-by: Thomas Huth
---
v2:
- Add comment in source code
- Only consider z9 and older as legacy
hw/s390x/s390-virtio-ccw.c | 5 +
target/s390x/cpu_models.c | 9 +
target/s390x/Kconfig | 5 +
On 14/06/2024 10.17, Christian Borntraeger wrote:
Am 14.06.24 um 09:15 schrieb Thomas Huth:
On 14/06/2024 08.07, Christian Borntraeger wrote:
Am 13.06.24 um 19:07 schrieb Thomas Huth:
Old CPU models are not officially supported anymore by IBM, and for
downstream builds of QEMU, we would
On 14/06/2024 08.07, Christian Borntraeger wrote:
Am 13.06.24 um 19:07 schrieb Thomas Huth:
Old CPU models are not officially supported anymore by IBM, and for
downstream builds of QEMU, we would like to be able to disable these
CPUs in the build. Thus add a CONFIG switch that can be used
On 13/06/2024 19.17, Philippe Mathieu-Daudé wrote:
Hi Thomas,
On 13/6/24 19:07, Thomas Huth wrote:
Old CPU models are not officially supported anymore by IBM, and for
downstream builds of QEMU, we would like to be able to disable these
CPUs in the build. Thus add a CONFIG switch that can
On 13/06/2024 13.59, Дмитрий Фролов wrote:
On 13.06.2024 13:08, Thomas Huth wrote:
On 23/05/2024 12.28, Dmitry Frolov wrote:
If QTestState was already CLOSED due to error, calling qtest_clock_step()
afterwards makes no sense and only raises false-crash with message:
"assertion timer !=
Old CPU models are not officially supported anymore by IBM, and for
downstream builds of QEMU, we would like to be able to disable these
CPUs in the build. Thus add a CONFIG switch that can be used to
disable these CPUs (and old machine types that use them by default).
Signed-off-by: Thomas Huth
clock_step() ? ... or where does the QTestState get
closed? During flush_events() ?
Thomas
(val, ==, 0x100);
+}
+
+int main(int argc, char **argv)
+{
+g_test_init(, , NULL);
+
+qtest_add_func("/cpu/csr", run_test_csr);
+
+qtest_start("--nographic -machine virt -cpu any,mvendorid=0x100");
You don't need --nographic, it's been taken care off by the libqtest
framework already.
+g_test_run();
+
+qtest_quit(global_qtest);
+
+return 0;
You should return the result of g_test_run() here, otherwise your test will
look like it always succeeds.
+}
Thomas
);
}
Reviewed-by: Thomas Huth
as intended).
Maybe it could be included in the stable releases before 9.0, too?
CC:-ing qemu-stable for this now.
Thomas
Fixes: 06b12970174 ("virtio-net: fix network stall under load")
If guest has made some buffers available during double check,
but the total buffer size available is lower than @bufsize,
notify the guest with the latest available idx(event idx)
seen by the host.
---
hw/net/virtio-net.c | 1 +
1
25.350323-8-zhao1@intel.com>
Signed-off-by: Thomas Huth
---
tests/unit/test-smp-parse.c | 103
1 file changed, 103 insertions(+)
diff --git a/tests/unit/test-smp-parse.c b/tests/unit/test-smp-parse.c
index 01832e5eda..2ca8530e93 100644
--- a/tests/u
older cpu models after
checking if QEMU actually supported these models.
Signed-off-by: Ani Sinha
Reviewed-by: Reviewed-by: Daniel P. Berrangé
Message-ID: <20240610155303.7933-3-anisi...@redhat.com>
Signed-off-by: Thomas Huth
---
tests/qtest/libqtest.h | 8
tests/qtest/libqtest.
consistent with the
other architectures.
Signed-off-by: Ilya Leoshkevich
Reviewed-by: Thomas Huth
Message-ID: <20240522184116.35975-1-...@linux.ibm.com>
Signed-off-by: Thomas Huth
---
tests/tcg/s390x/Makefile.softmmu-target | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a
isi...@redhat.com>
Signed-off-by: Thomas Huth
---
tests/qtest/test-x86-cpuid-compat.c | 170 ++--
1 file changed, 108 insertions(+), 62 deletions(-)
diff --git a/tests/qtest/test-x86-cpuid-compat.c
b/tests/qtest/test-x86-cpuid-compat.c
index 6a39454fce..b9e7e5ef7b 100644
--- a
From: Zhao Liu
With module level, QEMU now support 8-levels topology hierarchy.
Cover "modules" in SMP_CONFIG_WITH_FULL_TOPO related cases.
Signed-off-by: Zhao Liu
Tested-by: Yongwei Ma
Message-ID: <20240529061925.350323-9-zhao1@intel.com>
Signed-off-by: Thomas Huth
---
From: Zhao Liu
Cover the module cases in test-smp-parse.
Signed-off-by: Zhao Liu
Tested-by: Yongwei Ma
Message-ID: <20240529061925.350323-7-zhao1@intel.com>
Signed-off-by: Thomas Huth
---
tests/unit/test-smp-parse.c | 112 +---
1 file change
The following changes since commit 80e8f0602168f451a93e71cbb1d59e93d745e62e:
Merge tag 'bsd-user-misc-2024q2-pull-request' of gitlab.com:bsdimp/qemu into
staging (2024-06-09 11:21:55 -0700)
are available in the Git repository at:
https://gitlab.com/thuth/qemu.git
From: Zhao Liu
SMP_CONFIG_WITH_FULL_TOPO hasn't support module level, so the parameter
should indicate the "clusters".
Additionally, reorder the parameters of -smp to match the topology
hierarchy order.
Signed-off-by: Zhao Liu
Reviewed-by: Thomas Huth
Tested-by: Yongwei Ma
From: Ani Sinha
'pentium' cpu is old and obsolete and should be avoided for running tests if
its not strictly needed. Use 'max' cpu instead for generic non-cpu specific
numa test.
Reviewed-by: Thomas Huth
Reviewed-by: Igor Mammedov
Tested-by: Mario Casquero
Signed-off-by: Ani Sinha
Message
From: Zhao Liu
Currently, -smp supports module level.
It is necessary to consider the effects of module in the test cases to
ensure that the calculations are correct. This is also the preparation
to add module test cases.
Signed-off-by: Zhao Liu
Reviewed-by: Thomas Huth
Tested-by: Yongwei Ma
From: Zhao Liu
Fix the comments to match the actual configurations.
Signed-off-by: Zhao Liu
Reviewed-by: Thomas Huth
Tested-by: Yongwei Ma
Message-ID: <20240529061925.350323-2-zhao1@intel.com>
Signed-off-by: Thomas Huth
---
tests/unit/test-smp-parse.c | 8
1 file chan
From: Song Gao
The VM uses old bios to boot up only 1 cpu, causing the test case to fail.
Update the bios to solve this problem.
Reported-by: Thomas Huth
Signed-off-by: Song Gao
Message-ID: <20240604030058.2327145-1-gaos...@loongson.cn>
Signed-off-by: Thomas Huth
---
tests/a
From: Zhao Liu
Adjust the "cpus" parameter to match the comment configuration.
Signed-off-by: Zhao Liu
Reviewed-by: Thomas Huth
Tested-by: Yongwei Ma
Message-ID: <20240529061925.350323-4-zhao1@intel.com>
Signed-off-by: Thomas Huth
---
tests/unit/test-smp-parse.c | 2 +-
: Philippe Mathieu-Daudé
Signed-off-by: zhenwei pi
Reviewed-by: Philippe Mathieu-Daudé
Message-ID: <20240611105427.61395-2-pizhen...@bytedance.com>
Signed-off-by: Thomas Huth
---
meson.build | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/meson.build b/meson.build
From: Zhao Liu
Since -smp allows parameters=1 whether the level is supported by
machine, to avoid the test scenarios where the parameter defaults to 1
cause some errors to be masked, explicitly set undesired parameters to
0.
Signed-off-by: Zhao Liu
Reviewed-by: Thomas Huth
Tested-by: Yongwei
From: zhenwei pi
Remove libibumad dependence from the test environment.
Suggested-by: Philippe Mathieu-Daudé
Signed-off-by: zhenwei pi
Reviewed-by: Philippe Mathieu-Daudé
Message-ID: <20240611105427.61395-3-pizhen...@bytedance.com>
Signed-off-by: Thomas Huth
---
scripts/ci/setup/
-parse.c | 13 +++--
1 file changed, 11 insertions(+), 2 deletions(-)
Reviewed-by: Thomas Huth
-smp-parse.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
Reviewed-by: Thomas Huth
On 29/05/2024 08.19, Zhao Liu wrote:
Adjust the "cpus" parameter to match the comment configuration.
Signed-off-by: Zhao Liu
---
tests/unit/test-smp-parse.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Thomas Huth
101 - 200 of 18468 matches
Mail list logo