Markus Rechberger wrote:
This patch synchronises the APIC reset with the inkernel implementation which fixes the reboot issues which can be seen with Linux and Windows.

Signed-off-by: Joerg Roedel <joerg.roedel-5C7GfCeVMHo@public.gmane.org>
Signed-off-by: Markus Rechberger <markus.rechberger-5C7GfCeVMHo@public.gmane.org>

diff --git a/qemu/hw/apic.c b/qemu/hw/apic.c index 2011247..60d31fa 100644 --- a/qemu/hw/apic.c +++ b/qemu/hw/apic.c @@ -930,6 +930,13 @@ static void apic_reset(void *opaque) * processor when local APIC is enabled. */ s->lvt[APIC_LVT_LINT0] = 0x700; +#ifdef USE_KVM +#ifdef KVM_CAP_IRQCHIP + if (kvm_allowed && kvm_irqchip_in_kernel(kvm_context)) { + kvm_kernel_lapic_load_from_user(s); + } +#endif +#endif }
In general your approach might work but there are some inconsistencies - there is kvm_apic_reset
in the kernel too. We can either drop it and use your load method or add an apic-reset ioctl.
Eddie/Avi which approach do you think is better?
There is also the ioapic who should get reset too.
Thanks,
Dor.
 static CPUReadMemoryFunc *apic_mem_read[3] = {
  
-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/

_______________________________________________ kvm-devel mailing list kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org https://lists.sourceforge.net/lists/listinfo/kvm-devel