Re: [PATCH kvm-unit-tests] x86: always set up SMP

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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)




[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux