Re: [PART1 RFC v4 08/11] svm: Add VMEXIT handlers for AVIC

From: Radim KrÄmÃÅ
Date: Wed Apr 13 2016 - 08:37:44 EST


2016-04-13 00:29+0200, Paolo Bonzini:
> On 12/04/2016 18:22, Radim KrÄmÃÅ wrote:
>>> > + if (apic_id > 4 || cluster_id >= 0xf)
>>> > + return NULL;
>>> > + index = (cluster_id << 2) + apic_id;
>> ffs(apic_id), because 'apic_id' must be compacted into 2 bits.
>
> ffs(apic_id)-1 actually.

Yes, thanks.

(And I missed that the confusion begins by passing "ffs(dlid) - 1" as
mda to avic_handle_ldr_write, because the rest cannot work after that.)