On 18 October 2012 18:41, Jan Kiszka <jan.kiszka@xxxxxxxxxxx> wrote: > Minor: If you do > > function() > { > #ifdef CONFIG_KVM > ... > #endif > } > >> + >> qemu_irq *arm_pic_init_cpu(ARMCPU *cpu) >> { >> +#ifdef CONFIG_KVM >> + if (kvm_enabled()) { >> + return qemu_allocate_irqs(kvm_arm_pic_cpu_handler, cpu, 2); >> + } >> +#endif > > ...you can avoid the ifdef'ery here. Good idea -- will do that in next version. -- PMM -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html