On 04/12/2023 13.48, Cédric Le Goater wrote:
On 12/4/23 13:43, Thomas Huth wrote:
On 04/12/2023 13.40, Alex Bennée wrote:
Thomas Huth writes:
On 01/12/2023 10.36, Alex Bennée wrote:
We inadvertently built the LE target for BE tests.
Fixes: 78ebc00b06 (gitlab: shuffle some targets
On 04/12/2023 13.40, Alex Bennée wrote:
Thomas Huth writes:
On 01/12/2023 10.36, Alex Bennée wrote:
We inadvertently built the LE target for BE tests.
Fixes: 78ebc00b06 (gitlab: shuffle some targets and reduce avocado
noise)
Signed-off-by: Alex Bennée
---
.gitlab-ci.d/buildtest.yml | 2
On 04/12/2023 09.29, Zhao Liu wrote:
From: Zhao Liu
Add the check for common spelling mistakes for QEMU, which stole
selling.txt from Linux kernel and referenced the Linux kernel's
You need to sellcheck^Wspellcheck the above line ;-)
implementation in checkpatch.pl.
This check covers
On 01/12/2023 10.36, Alex Bennée wrote:
We inadvertently built the LE target for BE tests.
Fixes: 78ebc00b06 (gitlab: shuffle some targets and reduce avocado noise)
Signed-off-by: Alex Bennée
---
.gitlab-ci.d/buildtest.yml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
e big,
so I think it would be better to fix other coding style issues in separate
patches.
This way, you can also verify with "git show -b" that there are no other
unwanted changes in here.
I now also quickly skimmed through the changes and the patch looks fine to
me, so:
Reviewed-by: T
On 23/11/2023 17.03, Philippe Mathieu-Daudé wrote:
On 23/11/23 16:09, Thomas Huth wrote:
On 22/11/2023 19.30, Philippe Mathieu-Daudé wrote:
Add a helper to decide at runtime whether a type can
be registered to the QOM framework or not.
Signed-off-by: Philippe Mathieu-Daudé
---
include/qom
On 22/11/2023 19.30, Philippe Mathieu-Daudé wrote:
Add a helper to decide at runtime whether a type can
be registered to the QOM framework or not.
Signed-off-by: Philippe Mathieu-Daudé
---
include/qom/object.h | 4
qom/object.c | 3 +++
2 files changed, 7 insertions(+)
diff
On 21/11/2023 12.47, Marc-André Lureau wrote:
Hi
On Tue, Nov 21, 2023 at 1:45 PM Thomas Huth wrote:
On 21/11/2023 10.39, Marc-André Lureau wrote:
Hi
On Mon, Nov 20, 2023 at 5:36 PM Nicholas Piggin wrote:
On Mon Nov 20, 2023 at 10:06 PM AEST, Marc-André Lureau wrote:
Hi
On Thu, Nov 16
Let's use 'unsupported_configs' and 'tested_configs' here
instead of non-inclusive words.
Signed-off-by: Thomas Huth
---
v3: Rewording according to the suggestions of Daniel:
- Replaced "cipher not supported" with "config not supported"
- Replaced "not in LUKS_CONF
Let's use 'unsupported_configs' and 'tested_configs' here
instead of non-inclusive words.
Signed-off-by: Thomas Huth
---
v3: Rewording according to the suggestions of Daniel:
- Replaced "cipher not supported" with "config not supported"
- Replaced "not in LUKS_CONF
insertions(+), 61 deletions(-)
Reviewed-by: Thomas Huth
that is not worthwhile, but since the cost is negative...
Signed-off-by: Markus Armbruster
---
tests/unit/test-qmp-event.c | 30 ++
1 file changed, 10 insertions(+), 20 deletions(-)
Good idea!
Reviewed-by: Thomas Huth
-by: Markus Armbruster
---
tests/unit/test-qmp-event.c | 5 -
1 file changed, 5 deletions(-)
Reviewed-by: Thomas Huth
files changed, 46 insertions(+), 11 deletions(-)
Thanks, I can confirm that this series fixes the problem for me - both
locally when compiling with CFI, and on Travis (where I observed the problem
first):
https://app.travis-ci.com/github/huth/qemu/jobs/613659935#L6580
Tested-by: Thomas Huth
fingerprint: BE07 D9FD 5480 9AB2 C4B0 FF5F 6376 2CDA 67E2 F359
* tag 'pull-xenfv-20231121' of git://git.infradead.org/users/dwmw2/qemu:
hw/xen: clean up xen_block_find_free_vdev() to avoid Coverity false positive
vl: disable default serial when xen-console is enabled
Signed-off-by: Stefan Hajnoczi
fingerprint: BE07 D9FD 5480 9AB2 C4B0 FF5F 6376 2CDA 67E2 F359
* tag 'pull-xenfv-20231121' of git://git.infradead.org/users/dwmw2/qemu:
hw/xen: clean up xen_block_find_free_vdev() to avoid Coverity false positive
vl: disable default serial when xen-console is enabled
Signed-off-by: Stefan Hajnoczi
This reverts commit c4d74ab24a02c90b7a3240510b3dd4e1bec536dd.
The reverse debugging test is sometimes still failing. See:
https://gitlab.com/qemu-project/qemu/-/issues/1992
Signed-off-by: Thomas Huth
---
tests/avocado/reverse_debugging.py | 8
1 file changed, 8 insertions(+)
diff
On 21/11/2023 10.39, Marc-André Lureau wrote:
Hi
On Mon, Nov 20, 2023 at 5:36 PM Nicholas Piggin wrote:
On Mon Nov 20, 2023 at 10:06 PM AEST, Marc-André Lureau wrote:
Hi
On Thu, Nov 16, 2023 at 3:54 PM Nicholas Piggin wrote:
If a chardev socket is not read, it will eventually fill and
On 21/11/2023 10.14, Daniel P. Berrangé wrote:
On Tue, Nov 21, 2023 at 09:56:24AM +0100, Thomas Huth wrote:
On 17/11/2023 08.35, Nicholas Piggin wrote:
On Fri Nov 17, 2023 at 4:11 AM AEST, Thomas Huth wrote:
On 16/11/2023 12.53, Nicholas Piggin wrote:
Let's try enable reverse_debugging.py
ing remote
display and it used the TTY instead.
Fixes: commit 1bec1cc0d ("ui/console: allow to override the default VC")
Reported-by: German Maglione
Signed-off-by: Marc-André Lureau
---
ui/console.c | 18 --
1 file changed, 8 insertions(+), 10 deletions(-)
Acked-by: Thomas Huth
c | 1 +
ui/spice-app.c | 1 +
3 files changed, 3 insertions(+)
FWIW,
Acked-by: Thomas Huth
On 17/11/2023 08.35, Nicholas Piggin wrote:
On Fri Nov 17, 2023 at 4:11 AM AEST, Thomas Huth wrote:
On 16/11/2023 12.53, Nicholas Piggin wrote:
Let's try enable reverse_debugging.py in gitlab CI.
Signed-off-by: Nicholas Piggin
---
Maybe we could try this again at some point? The bug might
On 21/11/2023 09.08, Zhu Jun wrote:
These variables "ret" are never referenced in the code, thus
add check logic for the "ret"
Signed-off-by: Zhu Jun
Thanks!
Reviewed-by: Thomas Huth
I'll queue it for my next pull request.
Thanks, patch looks better this way, but I'd still like to ask you to fix
two small issues:
On 21/11/2023 07.16, zhujun2 wrote:
These variables "ret" are never referenced in the code, that
s/, that/, thus/
add check logic for the "ret"
Signed-off-by: zhujun2
Could you *please* fix
On 19/11/2023 19.19, Michael Tokarev wrote:
The following patches are queued for QEMU stable v8.1.3:
https://gitlab.com/qemu-project/qemu/-/commits/staging-8.1
Patch freeze is 2023-11-19 (frozen), and the release is planned for 2023-11-21:
https://wiki.qemu.org/Planning/8.1
Please
The migration stream on s390x contains data for the storage_attributes
which the analyze-migration.py cannot handle yet. Add the basic code
for handling this, so we can re-enable the check in the migration-test.
Signed-off-by: Thomas Huth
---
tests/qtest/migration-test.c | 4 +---
scripts
, 80 insertions(+), 9 deletions(-)
For s390x part:
Acked-by: Thomas Huth
On 17/11/2023 13.25, Philippe Mathieu-Daudé wrote:
On 17/11/23 12:44, Thomas Huth wrote:
QOM names currently don't have any enforced naming rules. This
can be problematic, e.g. when they are used on the command line
for the "-device" option (where the comma is used to separate
From: Markus Armbruster
Fixes: b65b4b7ae3c8 (xlnx-bbram: hw/nvram: Use dot in device type name)
Signed-off-by: Markus Armbruster
[thuth: Use longhand syntax to avoid problems with the "." in the name]
Reviewed-by: Peter Maydell
Signed-off-by: Thomas Huth
---
docs/system/arm/x
migration.
Reviewed-by: David Hildenbrand
Reviewed-by: Philippe Mathieu-Daudé
Signed-off-by: Thomas Huth
---
include/exec/memory.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/exec/memory.h b/include/exec/memory.h
index 831f7c996d..f172e82ac9 100644
--- a/include/exe
ry.
"xlnx,efuse", "xlnx,versal-efuse", "xlnx,pmc-efuse-cache",
"xlnx-zynqmp-efuse" are from v6.2.0, "xlnx,versal-crl" is from v7.1.0,
and the remainder are new.
Rename them all to "xlnx-FOO", like commit e178113ff64 did.
Reported-by: T
Philippe Mathieu-Daudé
Signed-off-by: Thomas Huth
---
tests/unit/test-io-task.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/unit/test-io-task.c b/tests/unit/test-io-task.c
index 953a50ae66..115dba8970 100644
--- a/tests/unit/test-io-task.c
+++ b/tests/unit/test-io-task
" and "Sun-UltraSparc" CPUs).
We also cannot enforce the rule that names must start with a letter
yet, since there are lot of types that start with a digit. Still,
at least limiting the first characters to the alphanumerical range
should be way better than
xlnx.bbram-ctrl"
hw: Replace anti-social QOM type names (again)
Thomas Huth (3):
memory: Remove "qemu:" prefix from the "qemu:ram-discard-manager" type
name
tests/unit/test-io-task: Rename "qemu:dummy" to avoid colon in the
name
qom/object: Limit
or so long in QEMU, I guess it makes sense to mark it as
deprecated here now. We can still reconsider if a new maintainer shows up...
Reviewed-by: Thomas Huth
or so long in QEMU, I guess it makes sense to mark it as
deprecated here now. We can still reconsider if a new maintainer shows up...
Reviewed-by: Thomas Huth
___
Devel mailing list -- devel@lists.libvirt.org
To unsubscribe send an email t
://wiki.qemu.org/Planning/8.2
That means, only bug fixes now, no new features will be merged until 8.2 has
been released.
Thomas
On 2023/11/16 下午9:19, Thomas Huth wrote:
On 15/11/2023 11.07, gaosong wrote:
Hi,
Can this patch be merged in during the 8.2 cycle?
Hi!
We normally do linux
On 16/11/2023 12.53, Nicholas Piggin wrote:
Let's try enable reverse_debugging.py in gitlab CI.
Signed-off-by: Nicholas Piggin
---
Maybe we could try this again at some point? The bug might have been
noticed sooner.
They only take a couple of seconds to run so should not take too much
It's failing very often, so don't run this by default anymore
until it gets fixed.
Message-ID: <20231114153019.295131-1-th...@redhat.com>
Reviewed-by: Peter Maydell
Signed-off-by: Thomas Huth
---
tests/avocado/replay_kernel.py | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff
:
$ avocado run tests/avocado/multiprocess.py
(1/2) tests/avocado/multiprocess.py:Multiprocess.test_multiprocess_x86_64:
ERROR: QemuBaseTest.fetch_asset() missing 1 required positional argument:
'asset_hash' (0.19 s)
Inspired-by: Thomas Huth
Signed-off-by: Philippe Mathieu-Daudé
Reviewed
From: Nicholas Piggin
Let's try enable reverse_debugging.py in gitlab CI.
Signed-off-by: Nicholas Piggin
Message-ID: <20231116115354.228678-3-npig...@gmail.com>
Signed-off-by: Thomas Huth
---
tests/avocado/reverse_debugging.py | 7 ---
1 file changed, 7 deletions(-)
diff --git a
assertEquals() has been removed in Python 3.12 and should be replaced by
assertEqual(). See: https://docs.python.org/3.12/whatsnew/3.12.html#id3
Message-ID: <20231114134326.287242-1-th...@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé
Signed-off-by: Thomas Huth
---
docs/devel/testi
now!
While we're at it, move the expected string to the preceeding
exec_command_and_wait_for_pattern() instead (since waiting for
empty string here does not make too much sense).
Message-ID: <20231114203456.319093-1-th...@redhat.com>
Reviewed-by: Antonio Caggiano
Signed-off-by: Thomas Huth
---
t
The intel_iommu test is currently succeeding with annoying warnings.
Add the proper asset hashes to avoid those.
Message-ID: <20231114143531.291820-1-th...@redhat.com>
Reviewed-by: Eric Auger
Signed-off-by: Thomas Huth
---
tests/avocado/intel_iommu.py | 6 --
1 file changed, 4 inse
("python/machine: use socketpair() for console connections")
Fixes: 761a13b23946 ("tests/avocado: ppc64 reverse debugging tests for pseries
and powernv")
Fixes: be52eca30978 ("tests/acceptance: add reverse debugging test")
Tested-by: Thomas Huth
Signed-off-b
assertEquals() has been removed in Python 3.12 and should be replaced by
assertEqual(). See: https://docs.python.org/3.12/whatsnew/3.12.html#id3
Message-ID: <20231116061956.14676-1-th...@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé
Acked-by: Ani Sinha
Signed-off-by: Thomas Huth
---
| No hash provided. Cannot check the asset file integrity.
Add the proper asset hashes to avoid those warnings.
Message-ID: <20231115145852.494052-1-th...@redhat.com>
Signed-off-by: Thomas Huth
---
tests/avocado/multiprocess.py | 18 --
1 file changed, 12 insertions
From: Philippe Mathieu-Daudé
assertRegexpMatches() has been removed in Python 3.12 and should be replaced by
assertRegex(). See: https://docs.python.org/3.12/whatsnew/3.12.html#id3
Inspired-by: Thomas Huth
Signed-off-by: Philippe Mathieu-Daudé
Message-ID: <20231114144832.71612-1-
Mathieu-Daudé (2):
tests/avocado: Replace assertRegexpMatches() for Python 3.12 compatibility
tests/avocado: Make fetch_asset() unconditionally require a crypto hash
Thomas Huth (6):
tests/avocado: Replace assertEquals() for Python 3.12 compatibility
tests/avocado/virtio
On 16/11/2023 14.24, Daniel P. Berrangé wrote:
On Thu, Nov 16, 2023 at 02:14:54PM +0100, Thomas Huth wrote:
QOM names currently don't have any enforced naming rules. This
can be problematic, e.g. when they are used on the command line
for the "-device" option (where the com
On 15/11/2023 11.07, gaosong wrote:
Hi,
Can this patch be merged in during the 8.2 cycle?
Hi!
We normally do linux-headers updates along with the patches that need the
new definitions ... so is there an urgent reason to push this stand-alone
patch into 8.2 without any patches that require
From: Markus Armbruster
Fixes: b65b4b7ae3c8 (xlnx-bbram: hw/nvram: Use dot in device type name)
Signed-off-by: Markus Armbruster
[thuth: Use longhand syntax to avoid problems with the "." in the name]
Signed-off-by: Thomas Huth
---
docs/system/arm/xlnx-versal-virt.rst | 2 +-
1 fi
" and "Sun-UltraSparc" CPUs).
We also cannot enforce the rule that names must start with a letter
yet, since there are lot of types that start with a digit. Still,
at least limiting the first characters to the alphanumerical range
should be way better than
Type names should not contain special characters like ":" (so that
they are easier to use with QAPI and other parts). We are going to
forbid such names in an upcoming patch. Thus let's replace the ":"
here with a "-".
Signed-off-by: Thomas Huth
---
tests/unit
ry.
"xlnx,efuse", "xlnx,versal-efuse", "xlnx,pmc-efuse-cache",
"xlnx-zynqmp-efuse" are from v6.2.0, "xlnx,versal-crl" is from v7.1.0,
and the remainder are new.
Rename them all to "xlnx-FOO", like commit e178113ff64 did.
Reported-by: T
the check to type_register_internal() instead of type_new()
so that we can disallow colons, too
- Only allow '+' in legacy names
Markus Armbruster (2):
docs/system/arm: Fix for rename of type "xlnx.bbram-ctrl"
hw: Replace anti-social QOM type names (again)
Thomas Huth (3):
memory: Remove
n.
Signed-off-by: Thomas Huth
---
include/exec/memory.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/exec/memory.h b/include/exec/memory.h
index 831f7c996d..f172e82ac9 100644
--- a/include/exec/memory.h
+++ b/include/exec/memory.h
@@ -43,7 +43,7 @@ type
On 16/11/2023 12.53, Nicholas Piggin wrote:
Let's try enable reverse_debugging.py in gitlab CI.
Signed-off-by: Nicholas Piggin
---
Maybe we could try this again at some point? The bug might have been
noticed sooner.
They only take a couple of seconds to run so should not take too much
On 14/11/2023 14.21, Peter Maydell wrote:
On Tue, 14 Nov 2023 at 13:05, Thomas Huth wrote:
QOM names currently don't have any enforced naming rules. This
can be problematic, e.g. when they are used on the command line
for the "-device" option (where the comma is used to separate
d('console'),
+stop_check=(lambda : not vm.is_running()))
+console_drainer.start()
return vm
@staticmethod
Tested-by: Thomas Huth
Could you please send this as a proper patch, with a S-o-b line, and a short
comment in front of the newly added code explaining it?
Thanks,
Thomas
assertEquals() has been removed in Python 3.12 and should be replaced by
assertEqual(). See: https://docs.python.org/3.12/whatsnew/3.12.html#id3
Signed-off-by: Thomas Huth
---
Note: The test has just been merged, that's why I missed this file in
my earlier patch that replaces the assertEquals
artifacts produce:
$ avocado run tests/avocado/multiprocess.py
(1/2) tests/avocado/multiprocess.py:Multiprocess.test_multiprocess_x86_64:
ERROR: QemuBaseTest.fetch_asset() missing 1 required positional argument:
'asset_hash' (0.19 s)
Inspired-by: Thomas Huth
Signed-off-by: Philippe
artifacts produce:
$ avocado run tests/avocado/multiprocess.py
(1/2) tests/avocado/multiprocess.py:Multiprocess.test_multiprocess_x86_64:
ERROR: QemuBaseTest.fetch_asset() missing 1 required positional argument:
'asset_hash' (0.19 s)
Inspired-by: Thomas Huth
Signed-off-by: Philippe
| No hash provided. Cannot check the asset file integrity.
Add the proper asset hashes to avoid those warnings.
Signed-off-by: Thomas Huth
---
tests/avocado/multiprocess.py | 18 --
1 file changed, 12 insertions(+), 6 deletions(-)
diff --git a/tests/avocado/multiprocess.py b
On 15/11/2023 09.00, zhujun2 wrote:
These variables are never referenced in the code, just remove them
Signed-off-by: zhujun2
---
tests/qtest/test-filter-mirror.c | 2 +-
tests/qtest/test-filter-redirector.c | 4 ++--
tests/qtest/virtio-net-test.c| 2 +-
3 files changed, 4
[i] (mie[i]) and needs to be maintained separately.
*/
uint64_t vsie;
Reviewed-by: Thomas Huth
led while the device and all its vrings are stopped
* (`VHOST_TRANSFER_STATE_PHASE_STOPPED`).
*
- * @dev: The vhost device to which to send the sate
+ * @dev: The vhost device to which to send the state
* @f: Migration stream from which to load the state
* @errp: Potential error message
*
ter buffer lifetime has a limit set by vendor.
* If it is over the limit, WriteBooster feature will be disabled.
*/
#define UFS_WB_EXCEED_LIFETIME 0x0B
Reviewed-by: Thomas Huth
Reviewed-by: Thomas Huth
}
Reviewed-by: Thomas Huth
at all. I will asume that RDMA is magicaly and don't
+ * overhead at all. I will assume that RDMA is magicaly and don't
* need to transfer (at least) the addresses where it wants to
* write the pages. Here it looks like it should be something
* like:
Reviewed-by: Thomas Huth
ny CPUs we are going
to
* step or resume, and a pointer to the state structure of one of them,
- * respectivelly
+ * respectively
*/
int target_count = 0;
CPUState *last_target = NULL;
Reviewed-by: Thomas Huth
o mailbox for switch
cci."
Cc: Jonathan Cameron
Cc: Fan Ni
Signed-off-by: Michael Tokarev
---
hw/cxl/cxl-component-utils.c | 4 ++--
hw/cxl/cxl-mailbox-utils.c | 2 +-
include/hw/cxl/cxl_device.h | 2 +-
3 files changed, 4 insertions(+), 4 deletions(-)
Reviewed-by: Thomas Huth
| 2 +-
linux-user/syscall.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
Reviewed-by: Thomas Huth
On 15/11/2023 07.41, Michael Tokarev wrote:
14.11.2023 22:05, Thomas Huth wrote:
On 14/11/2023 17.58, Michael Tokarev wrote:
Fixes: 1aefe2ca1423 "migration/doc: Add documentation for backwards
compatiblity"
Cc: Juan Quintela
Signed-off-by: Michael Tokarev
---
docs/devel/migrati
On 15/11/2023 02.15, Nicholas Piggin wrote:
On Wed Nov 15, 2023 at 4:29 AM AEST, Thomas Huth wrote:
On 14/11/2023 17.37, Philippe Mathieu-Daudé wrote:
On 14/11/23 17:31, Thomas Huth wrote:
The tests seem currently to be broken. Disable them by default
until someone fixes them.
Signed-off
On 14/11/2023 21.42, Eric Auger wrote:
Hi Thomas,
On 11/14/23 15:35, Thomas Huth wrote:
The intel_iommu test is currently succeeding with annoying warnings.
nit: you may have precised the nature of the warning or quotes
The annoying warnings look like this (in the summary):
(031/174
now!
While we're at it, move the expected string to the preceeding
exec_command_and_wait_for_pattern() instead (since waiting for
empty string here does not make too much sense).
Signed-off-by: Thomas Huth
---
tests/avocado/virtio-gpu.py | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff -
imit_value = 50;
Reviewed-by: Thomas Huth
nt request */
+/* Enable transfer request and task management request */
cap = ufs_rreg(ufs, A_CAP);
nutrs = FIELD_EX32(cap, CAP, NUTRS) + 1;
nutmrs = FIELD_EX32(cap, CAP, NUTMRS) + 1;
Reviewed-by: Thomas Huth
Reviewed-by: Thomas Huth
On 14/11/2023 17.58, Michael Tokarev wrote:
Fixes: 1aefe2ca1423 "migration/doc: Add documentation for backwards
compatiblity"
Cc: Juan Quintela
Signed-off-by: Michael Tokarev
---
docs/devel/migration.rst | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git
r starters, migration-test
-compression tests are disabled becase they fail randomly. If you need
+compression tests are disabled because they fail randomly. If you need
compression, use multifd compression methods.
Reviewed-by: Thomas Huth
On 14/11/2023 17.22, Peter Maydell wrote:
On Tue, 14 Nov 2023 at 15:30, Thomas Huth wrote:
It's failing very often, so don't run this by default anymore
until it gets fixed.
Signed-off-by: Thomas Huth
---
tests/avocado/replay_kernel.py | 3 ++-
1 file changed, 2 insertions(+), 1 deletion
On 14/11/2023 17.36, Philippe Mathieu-Daudé wrote:
On 14/11/23 16:30, Thomas Huth wrote:
It's failing very often, so don't run this by default anymore
until it gets fixed.
Signed-off-by: Thomas Huth
---
tests/avocado/replay_kernel.py | 3 ++-
1 file changed, 2 insertions(+), 1 deletion
On 14/11/2023 17.37, Philippe Mathieu-Daudé wrote:
On 14/11/23 17:31, Thomas Huth wrote:
The tests seem currently to be broken. Disable them by default
until someone fixes them.
Signed-off-by: Thomas Huth
---
tests/avocado/reverse_debugging.py | 7 ---
1 file changed, 4 insertions
The tests seem currently to be broken. Disable them by default
until someone fixes them.
Signed-off-by: Thomas Huth
---
tests/avocado/reverse_debugging.py | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/tests/avocado/reverse_debugging.py
b/tests/avocado
On 14/11/2023 15.48, Philippe Mathieu-Daudé wrote:
assertRegexpMatches() has been removed in Python 3.12 and should be replaced by
assertRegex(). See: https://docs.python.org/3.12/whatsnew/3.12.html#id3
Inspired-by: Thomas Huth
Signed-off-by: Philippe Mathieu-Daudé
---
docs/devel
It's failing very often, so don't run this by default anymore
until it gets fixed.
Signed-off-by: Thomas Huth
---
tests/avocado/replay_kernel.py | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/tests/avocado/replay_kernel.py b/tests/avocado/replay_kernel.py
index a18610542e
The intel_iommu test is currently succeeding with annoying warnings.
Add the proper asset hashes to avoid those.
Signed-off-by: Thomas Huth
---
tests/avocado/intel_iommu.py | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/tests/avocado/intel_iommu.py b/tests/avocado
On 14/11/2023 15.10, Philippe Mathieu-Daudé wrote:
Hi Thomas,
On 14/11/23 14:43, Thomas Huth wrote:
assertEquals() has been removed in Python 3.12 and should be replaced by
assertEqual(). See: https://docs.python.org/3.12/whatsnew/3.12.html#id3
From this list I see assertEquals() has been
assertEquals() has been removed in Python 3.12 and should be replaced by
assertEqual(). See: https://docs.python.org/3.12/whatsnew/3.12.html#id3
Signed-off-by: Thomas Huth
---
docs/devel/testing.rst | 2 +-
tests/avocado/cpu_queries.py| 2 +-
tests/avocado
t start
with a letter yet. Still, having at least some rules enforced here
should be way better than nothing.
Signed-off-by: Thomas Huth
---
qom/object.c | 19 +++
1 file changed, 19 insertions(+)
diff --git a/qom/object.c b/qom/object.c
index 95c0dc8285..8ff85e0239 100644
--- a/q
From: Janosch Frank
Some architectures (s390x) need to cleanup after a failed dump to be
able to continue to run the vm. Add a cleanup function pointer and
call it if it's set.
Signed-off-by: Janosch Frank
Reviewed-by: Thomas Huth
Reviewed-by: Marc-André Lureau
Message-ID
From: Janosch Frank
dump_state_prepare() now sets the function pointers to NULL so we only
need to touch them if we're going to use them.
Signed-off-by: Janosch Frank
Reviewed-by: Marc-André Lureau
Reviewed-by: Thomas Huth
Message-ID: <20231109120443.185979-2-fran...@linux.ibm.com>
The following changes since commit f78ea7ddb0e18766ece9fdfe02061744a7afc41b:
Merge tag 'pull-request-2023-11-13' of https://gitlab.com/thuth/qemu into
staging (2023-11-13 07:15:43 -0500)
are available in the Git repository at:
https://gitlab.com/thuth/qemu.git tags/pull-request-2023-11-14
a dump end. The dump
completion data is discarded but there's no point in writing it to a
file anyway if there's a possibility that other PV dump data is
missing.
Signed-off-by: Janosch Frank
Reviewed-by: Thomas Huth
Reviewed-by: Claudio Imbrenda
Reviewed-by: Marc-André Lureau
Message-ID
Forgot to CC: qemu-devel (sorry) - thanks to Markus for the hint.
So let's repeat it here:
On 14/11/2023 10.31, Daniel P. Berrangé wrote:
On Tue, Nov 14, 2023 at 10:05:37AM +0100, Thomas Huth wrote:
QOM names currently don't have any enforced naming rules. This can
be problematic, e.g. when
On 14/11/2023 08.41, Markus Armbruster wrote:
Cc: the other QOM maintainers
Daniel P. Berrangé writes:
On Mon, Nov 13, 2023 at 02:43:42PM +0100, Markus Armbruster wrote:
We got rid of QOM type names containing ',' in 6.0, but some have
crept back in. Replace them just like we did in 6.0.
generated (indicating if the IRQ line was toggled).
Signed-off-by: Gustavo Romero
---
tests/qtest/libqtest.c | 24
tests/qtest/libqtest.h | 28
2 files changed, 52 insertions(+)
Acked-by: Thomas Huth
1301 - 1400 of 20565 matches
Mail list logo