Avi Kivity wrote:
> Zhang, Xiantao wrote:
>> From 720dab659e581adba3a7846db6d2f612ffcd83a0 Mon Sep 17 00:00:00
>> 2001 From: Xiantao Zhang <[EMAIL PROTECTED]>
>> Date: Mon, 6 Oct 2008 13:48:45 +0800
>> Subject: [PATCH] kvm: Split arch/x86/kvm/irq.c to two parts.
>>
>> Moving irq ack notification logic as common, and make
>> it shared with ia64 side.
>>
>
>
>> +/* This should be called with the kvm->lock mutex held */
>> +void kvm_set_irq(struct kvm *kvm, int irq, int level) +{
>> + /* Not possible to detect if the guest uses the PIC or the
>> + * IOAPIC. So set the bit in both. The guest will ignore
>> + * writes to the unused one.
>> + */
>> + kvm_ioapic_set_irq(kvm->arch.vioapic, irq, level); +#ifdef X86
>> + kvm_pic_set_irq(pic_irqchip(kvm), irq, level);
>> +#endif
>> +}
>> +
>>
>
> This needs to be CONFIG_X86, not X86. As is, Windows doesn't even
> boot. Fixing it causes compile errors for missing declarations and
> pic_irqchip().
Hi, Avi
I have fixed it on my ia32 machine but seems forget to commit
it.Okay, I will retest it before sending out the new one.
Thanks
Xiantao
--
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
