Re: [PATCH 1/3] KVM: APIC: remove unnecessary double checks on APIC existence

From: Radim KrÄmÃÅ
Date: Tue Feb 09 2016 - 10:19:08 EST


2016-02-09 16:06+0100, Paolo Bonzini:
> On 09/02/2016 14:55, Radim KrÄmÃÅ wrote:
>> This is most likely going to bug on the following path:
>> handle_apic_access -> kvm_lapic_set_eoi
>>
>> Before the change, handle_apic_access would just drop EOIs that should
>> have gone to user space ... I'm not sure if we tested it, or the path is
>> really never taken.
>
> cpu_need_virtualize_apic_accesses is needed to set
> SECONDARY_EXEC_VIRTUALIZE_APIC_ACCESSES (vmx_secondary_exec_control) and
> SECONDARY_EXEC_VIRTUALIZE_APIC_ACCESSES is required to get to
> handle_apic_access.

Thanks,

Reviewed-by: Radim KrÄmÃÅ <rkrcmar@xxxxxxxxxx>