From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marcelo Tosatti Subject: Re: [PATCH v12 0/3] x86, apicv: Add APIC virtualization support Date: Wed, 23 Jan 2013 22:05:57 -0200 Message-ID: <20130124000557.GA22800@amt.cnet> References: <1358952446-17416-1-git-send-email-yang.z.zhang@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: kvm@vger.kernel.org, gleb@redhat.com, haitao.shan@intel.com, xiantao.zhang@intel.com To: Yang Zhang Return-path: Received: from mx1.redhat.com ([209.132.183.28]:20279 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752299Ab3AXAIu (ORCPT ); Wed, 23 Jan 2013 19:08:50 -0500 Content-Disposition: inline In-Reply-To: <1358952446-17416-1-git-send-email-yang.z.zhang@intel.com> Sender: kvm-owner@vger.kernel.org List-ID: On Wed, Jan 23, 2013 at 10:47:23PM +0800, Yang Zhang wrote: > From: Yang Zhang > > APIC virtualization is a new feature which can eliminate most of VM exit > when vcpu handle a interrupt: > > APIC register virtualization: > APIC read access doesn't cause APIC-access VM exits. > APIC write becomes trap-like. > > Virtual interrupt delivery: > Virtual interrupt delivery avoids KVM to inject vAPIC interrupts > manually, which is fully taken care of by the hardware. > > Please refer to Intel SDM volume 3, chapter 29 for more details. > Changes v11 to v12: > * Check irqchip in kernel when enabling apicv, if using userspace irq chip, > apicv cannot be used and must be disabled. > * Rename some fucntion to more descriptive name. > * Move ioapic entry pase logic to lapic.c > * Rebased on top of KVM upstream > > Changes v10 to v11: > * Use two new msr bitmaps for guest that enabling x2apic mode: > Since msr bitmap is shared by all guests, it will break guest that > not using x2apic when updating the global msr bitmap. To solve this, > we use two new msr bitmap for guest which using x2apic. > > Changes v9 to v10: > * Enable virtualize x2apic mode when guest is using x2apic and apicv: > There is no point to enable x2apic mode when apicv is disabled. > * Grep ioapic_lock when traversing ioapic entry to set eoi exit bitmap > * Rebased on top of KVM upstream > > Changes v8 to v9: > * Update eoi exit bitmap by vcpu itself. > * Enable virtualize x2apic mode when guest is using x2apic. > * Rebase on top of KVM upstream > > Changes v7 to v8: > * According Marcelo's suggestion, add comments for irr_pending and isr_count, > since the two valiables have different meaning when using apicv. > * Set highest bit in vISR to SVI after migation. > * Use spinlock to access eoi exit bitmap synchronously. > * Enable virtualize x2apic mode when guest is using x2apic > * Rebased on top of KVM upstream. > > Yang Zhang (3): > x86, apicv: add APICv register virtualization support > x86, apicv: add virtual x2apic support > x86, apicv: add virtual interrupt delivery support > > arch/ia64/kvm/lapic.h | 6 + > arch/x86/include/asm/kvm_host.h | 8 + > arch/x86/include/asm/vmx.h | 14 ++ > arch/x86/kvm/irq.c | 56 ++++++- > arch/x86/kvm/lapic.c | 135 ++++++++++++--- > arch/x86/kvm/lapic.h | 29 ++++ > arch/x86/kvm/svm.c | 37 ++++ > arch/x86/kvm/vmx.c | 349 ++++++++++++++++++++++++++++++++++++--- > arch/x86/kvm/x86.c | 11 +- > include/linux/kvm_host.h | 3 + > virt/kvm/ioapic.c | 39 +++++ > virt/kvm/ioapic.h | 4 + > virt/kvm/irq_comm.c | 25 +++ > virt/kvm/kvm_main.c | 5 + > 14 files changed, 667 insertions(+), 54 deletions(-) Reviewed-by: Marcelo Tosatti