Re: [PATCH] accel/tcg: test CPUJumpCache in tb_jmp_cache_clear_page()

2023-02-03 Thread Richard Henderson
On 2/3/23 07:15, Eric Auger wrote: After commit 4e4fa6c12d ("accel/tcg: Complete cpu initialization before registration"), it looks the CPUJumpCache handle can be NULL. This causes a SIGSEV when running debug-wp-migration kvm unit test. At the first place it should be clarified why this TCG

Re: [PATCH] accel/tcg: test CPUJumpCache in tb_jmp_cache_clear_page()

2023-02-03 Thread Eric Auger
Hi Philippe, On 2/3/23 18:21, Philippe Mathieu-Daudé wrote: > On 3/2/23 18:15, Eric Auger wrote: >> After commit 4e4fa6c12d ("accel/tcg: Complete cpu initialization >> before registration"), it looks the CPUJumpCache handle can be NULL. >> This causes a SIGSEV when running debug-wp-migration kvm

Re: [PATCH] accel/tcg: test CPUJumpCache in tb_jmp_cache_clear_page()

2023-02-03 Thread Philippe Mathieu-Daudé
On 3/2/23 18:15, Eric Auger wrote: After commit 4e4fa6c12d ("accel/tcg: Complete cpu initialization before registration"), it looks the CPUJumpCache handle can be NULL. This causes a SIGSEV when running debug-wp-migration kvm unit test. Do you mean commit a976a99a29 ("include/hw/core: Create

[PATCH] accel/tcg: test CPUJumpCache in tb_jmp_cache_clear_page()

2023-02-03 Thread Eric Auger
After commit 4e4fa6c12d ("accel/tcg: Complete cpu initialization before registration"), it looks the CPUJumpCache handle can be NULL. This causes a SIGSEV when running debug-wp-migration kvm unit test. At the first place it should be clarified why this TCG code is called with KVM acceleration.