On 16/06/2020 15.31, Cathy Avery wrote: > On 6/16/20 9:02 AM, Thomas Huth wrote: >> On 08/06/2020 18.00, Paolo Bonzini wrote: >>> Currently setup_vm cannot assume that it can invoke IPIs, and therefore >>> only initializes CR0/CR3/CR4 on the CPU it runs on. In order to keep >>> the >>> initialization code clean, let's just call smp_init (and therefore >>> setup_idt) unconditionally. >>> >>> Signed-off-by: Paolo Bonzini <pbonzini@xxxxxxxxxx> >>> --- >>> x86/access.c | 2 -- >>> x86/apic.c | 1 - >>> x86/asyncpf.c | 1 - >>> x86/cmpxchg8b.c | 1 - >>> x86/cstart.S | 6 +++--- >>> x86/cstart64.S | 6 +++--- >>> x86/debug.c | 1 - >>> x86/emulator.c | 1 - >>> x86/eventinj.c | 1 - >> Hi Paolo, >> >> this patch broke the eventinj test on i386 on gitlab: >> >> https://gitlab.com/huth/kvm-unit-tests/-/jobs/597447047#L1933 >> >> if I revert the patch, the test works again: >> >> https://gitlab.com/huth/kvm-unit-tests/-/jobs/597455720#L1934 >> >> Any ideas how to fix that? >> >> Thanks, >> Thomas > > I am not seeing a failure running eventinj standalone on AMD. > > SUMMARY: 13 tests > PASS eventinj (13 tests) It's only the 32-bit build that is failing, 64-bit is fine. And gitlab-CI runs with tcg instead of kvm, so maybe that's related, too? Thomas
