From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joerg Roedel Subject: Re: [PATCH 3/6] kvm/x86/mmu: don't unnecessarily recalculate table_gfn in *fetch Date: Thu, 5 Mar 2009 15:36:08 +0100 Message-ID: <20090305143608.GF8751@amd.com> References: <1236255153-4432-1-git-send-email-joerg.roedel@amd.com> <1236255153-4432-4-git-send-email-joerg.roedel@amd.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: kvm@vger.kernel.org, linux-kernel@vger.kernel.org To: Avi Kivity , Marcelo Tosatti Return-path: Received: from outbound-sin.frontbridge.com ([207.46.51.80]:19363 "EHLO SG2EHSOBE001.bigfish.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752108AbZCEOgi convert rfc822-to-8bit (ORCPT ); Thu, 5 Mar 2009 09:36:38 -0500 Content-Disposition: inline In-Reply-To: <1236255153-4432-4-git-send-email-joerg.roedel@amd.com> Sender: kvm-owner@vger.kernel.org List-ID: On Thu, Mar 05, 2009 at 01:12:30PM +0100, Joerg Roedel wrote: > Signed-off-by: Joerg Roedel > --- > arch/x86/kvm/paging_tmpl.h | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) >=20 > diff --git a/arch/x86/kvm/paging_tmpl.h b/arch/x86/kvm/paging_tmpl.h > index a0c11ea..79668ba 100644 > --- a/arch/x86/kvm/paging_tmpl.h > +++ b/arch/x86/kvm/paging_tmpl.h > @@ -315,7 +315,7 @@ static u64 *FNAME(fetch)(struct kvm_vcpu *vcpu, g= va_t addr, > direct =3D 1; > if (!is_dirty_pte(gw->ptes[level - 1])) > access &=3D ~ACC_WRITE_MASK; > - table_gfn =3D gpte_to_gfn(gw->ptes[level - 1]); > + table_gfn =3D gw->table_gfn[level - delta]; > } else { > direct =3D 0; > table_gfn =3D gw->table_gfn[level - 2]; Made a mistake here when rebasing these patches. Updated patch is below= : =46rom de38a141a0ba44b3d46e333620d518cd831326c0 Mon Sep 17 00:00:00 200= 1 =46rom: Joerg Roedel Date: Wed, 4 Mar 2009 20:49:15 +0100 Subject: [PATCH 3/6] kvm/x86/mmu: don't unnecessarily recalculate table= _gfn in *fetch Signed-off-by: Joerg Roedel --- arch/x86/kvm/paging_tmpl.h | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/arch/x86/kvm/paging_tmpl.h b/arch/x86/kvm/paging_tmpl.h index a0c11ea..79668ba 100644 --- a/arch/x86/kvm/paging_tmpl.h +++ b/arch/x86/kvm/paging_tmpl.h @@ -315,7 +315,7 @@ static u64 *FNAME(fetch)(struct kvm_vcpu *vcpu, gva= _t addr, direct =3D 1; if (!is_dirty_pte(gw->ptes[level - 1])) access &=3D ~ACC_WRITE_MASK; - table_gfn =3D gpte_to_gfn(gw->ptes[level - 1]); + table_gfn =3D gw->table_gfn[level - 1]; } else { direct =3D 0; table_gfn =3D gw->table_gfn[level - 2]; --=20 1.5.6.4 --=20 | Advanced Micro Devices GmbH Operating | Karl-Hammerschmidt-Str. 34, 85609 Dornach bei M=C3=BCnchen System |=20 Research | Gesch=C3=A4ftsf=C3=BChrer: Jochen Polster, Thomas M. McCoy= , Giuliano Meroni Center | Sitz: Dornach, Gemeinde Aschheim, Landkreis M=C3=BCnchen | Registergericht M=C3=BCnchen, HRB Nr. 43632