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

2016-04-29 Thread Radim Krčmář
2016-04-28 17:08-0500, Suravee Suthikulanit: > On 4/12/2016 11:22 AM, Radim Krčmář wrote: >> 2016-04-07 03:20-0500, Suravee Suthikulpanit: >> > From: Suravee Suthikulpanit >> > >> > This patch introduces VMEXIT handlers, avic_incomplete_ipi_interception() >> > and

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

2016-04-29 Thread Radim Krčmář
2016-04-28 17:08-0500, Suravee Suthikulanit: > On 4/12/2016 11:22 AM, Radim Krčmář wrote: >> 2016-04-07 03:20-0500, Suravee Suthikulpanit: >> > From: Suravee Suthikulpanit >> > >> > This patch introduces VMEXIT handlers, avic_incomplete_ipi_interception() >> > and

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

2016-04-28 Thread Suravee Suthikulanit
Hi Radim / Paolo, Sorry for delay in response. On 4/12/2016 11:22 AM, Radim Krčmář wrote: 2016-04-07 03:20-0500, Suravee Suthikulpanit: From: Suravee Suthikulpanit This patch introduces VMEXIT handlers, avic_incomplete_ipi_interception() and

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

2016-04-28 Thread Suravee Suthikulanit
Hi Radim / Paolo, Sorry for delay in response. On 4/12/2016 11:22 AM, Radim Krčmář wrote: 2016-04-07 03:20-0500, Suravee Suthikulpanit: From: Suravee Suthikulpanit This patch introduces VMEXIT handlers, avic_incomplete_ipi_interception() and avic_unaccelerated_access_interception() along

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

2016-04-13 Thread Radim Krčmář
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. > >

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

2016-04-13 Thread Radim Krčmář
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. > >

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

2016-04-12 Thread 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. Thanks

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

2016-04-12 Thread 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. Thanks

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

2016-04-12 Thread Radim Krčmář
2016-04-07 03:20-0500, Suravee Suthikulpanit: > From: Suravee Suthikulpanit > > This patch introduces VMEXIT handlers, avic_incomplete_ipi_interception() > and avic_unaccelerated_access_interception() along with two trace points > (trace_kvm_avic_incomplete_ipi and

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

2016-04-12 Thread Radim Krčmář
2016-04-07 03:20-0500, Suravee Suthikulpanit: > From: Suravee Suthikulpanit > > This patch introduces VMEXIT handlers, avic_incomplete_ipi_interception() > and avic_unaccelerated_access_interception() along with two trace points > (trace_kvm_avic_incomplete_ipi and

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

2016-04-07 Thread Suravee Suthikulpanit
From: Suravee Suthikulpanit This patch introduces VMEXIT handlers, avic_incomplete_ipi_interception() and avic_unaccelerated_access_interception() along with two trace points (trace_kvm_avic_incomplete_ipi and trace_kvm_avic_unaccelerated_access). Signed-off-by:

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

2016-04-07 Thread Suravee Suthikulpanit
From: Suravee Suthikulpanit This patch introduces VMEXIT handlers, avic_incomplete_ipi_interception() and avic_unaccelerated_access_interception() along with two trace points (trace_kvm_avic_incomplete_ipi and trace_kvm_avic_unaccelerated_access). Signed-off-by: Suravee Suthikulpanit ---