On Friday, October 17, 2008 1:11 PM Sheng Yang wrote:
> On Wednesday 15 October 2008 22:27:49 Jan Kiszka wrote:
>> Aligning in-kernel kvm_apic_accept_pic_intr with its user space mate,
>> this patch relaxes the conditions under which PIC IRQs are accepted
>> by LVT0. This reflects reality and allows to reuse the service for
>> the NMI watchdog use case.
>>
>> Signed-off-by: Jan Kiszka <[EMAIL PROTECTED]>
>> ---
>> arch/x86/kvm/lapic.c | 13 ++++---------
>> 1 file changed, 4 insertions(+), 9 deletions(-)
>>
>> Index: b/arch/x86/kvm/lapic.c
>> ===================================================================
>> --- a/arch/x86/kvm/lapic.c +++ b/arch/x86/kvm/lapic.c
>> @@ -1072,16 +1072,11 @@ int kvm_apic_has_interrupt(struct kvm_vc
>> int kvm_apic_accept_pic_intr(struct kvm_vcpu *vcpu) {
>> u32 lvt0 = apic_get_reg(vcpu->arch.apic, APIC_LVT0); - int
>> r = 0;
>>
>> - if (vcpu->vcpu_id == 0) {
>> - if (!apic_hw_enabled(vcpu->arch.apic))
>> - r = 1;
>> - if ((lvt0 & APIC_LVT_MASKED) == 0 &&
>> - GET_APIC_DELIVERY_MODE(lvt0) == APIC_MODE_EXTINT)
>> - r = 1;
>> - }
>> - return r;
>> + if (!apic_hw_enabled(vcpu->arch.apic) ||
>> + (lvt0 & APIC_LVT_MASKED) == 0)
>> + return 1;
>> + return 0;
>> }
>>
>> void kvm_inject_apic_timer_irqs(struct kvm_vcpu *vcpu)
>>
> (sorry for late review...)
>
> Thanks to find out the root cause of BSOD!
With the patch, we can boot up 64bit windows (XP, Win2k3, Vista, Win2k8)
without BSOD now.
Best Regards
Jiajun
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
