From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paolo Bonzini Subject: Re: [PATCH v5 6/7] kvm, mem-hotplug: Unpin and remove kvm_arch->apic_access_page. Date: Thu, 11 Sep 2014 11:34:51 +0200 Message-ID: <54116CBB.7050505@redhat.com> References: <1410413886-32213-1-git-send-email-tangchen@cn.fujitsu.com> <1410413886-32213-7-git-send-email-tangchen@cn.fujitsu.com> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-15 Content-Transfer-Encoding: 7bit Cc: kvm@vger.kernel.org, laijs@cn.fujitsu.com, isimatu.yasuaki@jp.fujitsu.com, guz.fnst@cn.fujitsu.com, linux-kernel@vger.kernel.org To: Tang Chen , gleb@kernel.org, mtosatti@redhat.com, nadav.amit@gmail.com, jan.kiszka@web.de Return-path: In-Reply-To: <1410413886-32213-7-git-send-email-tangchen@cn.fujitsu.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: kvm.vger.kernel.org Il 11/09/2014 07:38, Tang Chen ha scritto: > + if (vm_need_virtualize_apic_accesses(vmx->vcpu.kvm)) { > + struct page *page = gfn_to_page(vmx->vcpu.kvm, > + APIC_DEFAULT_PHYS_BASE >> PAGE_SHIFT); > + vmcs_write64(APIC_ACCESS_ADDR, page_to_phys(page)); > + /* > + * Do not pin apic access page in memory so that memory > + * hotplug process is able to migrate it. > + */ > + put_page(page); > + } Please reuse vcpu_reload_apic_access_page here, too. Paolo