From mboxrd@z Thu Jan 1 00:00:00 1970 From: Liliang Subject: [PATCH] x86/HVM: Fixed the bug which cause the nested hvm guest fail to boot Date: Tue, 19 Aug 2014 10:32:45 +0800 Message-ID: <1408415565-28913-1-git-send-email-liangx.z.li@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: xen-devel@lists.xen.org Cc: Li Liang , keir@xen.org, jbeulich@suse.com List-Id: xen-devel@lists.xenproject.org From: Li Liang The commit aa9114edd added the needless function call memory_type_changed, it is unnessary because the PAT content does not influence the result of epte_get_entry_emt(). If it is called, the cache will be flushed excessively and make the nested guest very slowly, just like blocked. Signed-off-by: Li Liang --- xen/arch/x86/hvm/hvm.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/xen/arch/x86/hvm/hvm.c b/xen/arch/x86/hvm/hvm.c index d40c48e..e8f41c1 100644 --- a/xen/arch/x86/hvm/hvm.c +++ b/xen/arch/x86/hvm/hvm.c @@ -257,8 +257,6 @@ int hvm_set_guest_pat(struct vcpu *v, u64 guest_pat) if ( !hvm_funcs.set_guest_pat(v, guest_pat) ) v->arch.hvm_vcpu.pat_cr = guest_pat; - memory_type_changed(v->domain); - return 1; } -- 1.9.1