All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] tests/functional: Require TCG to run reverse debugging tests
@ 2026-01-15 16:10 Philippe Mathieu-Daudé
  2026-01-15 16:15 ` Daniel P. Berrangé
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Philippe Mathieu-Daudé @ 2026-01-15 16:10 UTC (permalink / raw)
  To: qemu-devel
  Cc: Gustavo Romero, Alex Bennée, Paolo Bonzini, Pavel Dovgalyuk,
	Philippe Mathieu-Daudé

Record/replay is specific to TCG. Require it to avoid failure
when using a HVF-only build on Darwin:

  qemu-system-aarch64: -icount shift=7,rr=record,rrfile=/scratch/replay.bin,rrsnapshot=init: cannot configure icount, TCG support not available

Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
 tests/functional/reverse_debugging.py | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/tests/functional/reverse_debugging.py b/tests/functional/reverse_debugging.py
index 8b9507674a0..8e6f0b0e923 100644
--- a/tests/functional/reverse_debugging.py
+++ b/tests/functional/reverse_debugging.py
@@ -67,6 +67,8 @@ def vm_get_icount(vm):
     def reverse_debugging(self, gdb_arch, shift=7, args=None, big_endian=False):
         from qemu_test import GDB
 
+        self.require_accelerator("tcg")
+
         # create qcow2 for snapshots
         self.log.info('creating qcow2 image for VM snapshots')
         image_path = os.path.join(self.workdir, 'disk.qcow2')
-- 
2.52.0



^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] tests/functional: Require TCG to run reverse debugging tests
  2026-01-15 16:10 [PATCH] tests/functional: Require TCG to run reverse debugging tests Philippe Mathieu-Daudé
@ 2026-01-15 16:15 ` Daniel P. Berrangé
  2026-01-15 16:39 ` Alex Bennée
  2026-01-15 19:53 ` Philippe Mathieu-Daudé
  2 siblings, 0 replies; 4+ messages in thread
From: Daniel P. Berrangé @ 2026-01-15 16:15 UTC (permalink / raw)
  To: Philippe Mathieu-Daudé
  Cc: qemu-devel, Gustavo Romero, Alex Bennée, Paolo Bonzini,
	Pavel Dovgalyuk

On Thu, Jan 15, 2026 at 05:10:29PM +0100, Philippe Mathieu-Daudé wrote:
> Record/replay is specific to TCG. Require it to avoid failure
> when using a HVF-only build on Darwin:
> 
>   qemu-system-aarch64: -icount shift=7,rr=record,rrfile=/scratch/replay.bin,rrsnapshot=init: cannot configure icount, TCG support not available
> 
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---
>  tests/functional/reverse_debugging.py | 2 ++
>  1 file changed, 2 insertions(+)

Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>


With regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] tests/functional: Require TCG to run reverse debugging tests
  2026-01-15 16:10 [PATCH] tests/functional: Require TCG to run reverse debugging tests Philippe Mathieu-Daudé
  2026-01-15 16:15 ` Daniel P. Berrangé
@ 2026-01-15 16:39 ` Alex Bennée
  2026-01-15 19:53 ` Philippe Mathieu-Daudé
  2 siblings, 0 replies; 4+ messages in thread
From: Alex Bennée @ 2026-01-15 16:39 UTC (permalink / raw)
  To: Philippe Mathieu-Daudé
  Cc: qemu-devel, Gustavo Romero, Paolo Bonzini, Pavel Dovgalyuk

Philippe Mathieu-Daudé <philmd@linaro.org> writes:

> Record/replay is specific to TCG. Require it to avoid failure
> when using a HVF-only build on Darwin:
>
>   qemu-system-aarch64: -icount shift=7,rr=record,rrfile=/scratch/replay.bin,rrsnapshot=init: cannot configure icount, TCG support not available
>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>

Reviewed-by: Alex Bennée <alex.bennee@linaro.org>

-- 
Alex Bennée
Virtualisation Tech Lead @ Linaro


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] tests/functional: Require TCG to run reverse debugging tests
  2026-01-15 16:10 [PATCH] tests/functional: Require TCG to run reverse debugging tests Philippe Mathieu-Daudé
  2026-01-15 16:15 ` Daniel P. Berrangé
  2026-01-15 16:39 ` Alex Bennée
@ 2026-01-15 19:53 ` Philippe Mathieu-Daudé
  2 siblings, 0 replies; 4+ messages in thread
From: Philippe Mathieu-Daudé @ 2026-01-15 19:53 UTC (permalink / raw)
  To: qemu-devel
  Cc: Gustavo Romero, Alex Bennée, Paolo Bonzini, Pavel Dovgalyuk

On 15/1/26 17:10, Philippe Mathieu-Daudé wrote:
> Record/replay is specific to TCG. Require it to avoid failure
> when using a HVF-only build on Darwin:
> 
>    qemu-system-aarch64: -icount shift=7,rr=record,rrfile=/scratch/replay.bin,rrsnapshot=init: cannot configure icount, TCG support not available
> 
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---
>   tests/functional/reverse_debugging.py | 2 ++
>   1 file changed, 2 insertions(+)

Queued via accel-next tree, thanks.


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2026-01-15 19:54 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-01-15 16:10 [PATCH] tests/functional: Require TCG to run reverse debugging tests Philippe Mathieu-Daudé
2026-01-15 16:15 ` Daniel P. Berrangé
2026-01-15 16:39 ` Alex Bennée
2026-01-15 19:53 ` Philippe Mathieu-Daudé

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.