[PATCH] KVM: change PT_FIRST_AVAIL_BITS_SHIFT to avoid conflict with EPT Dirty bit

[PATCH] KVM: change PT_FIRST_AVAIL_BITS_SHIFT to avoid conflict with EPT Dirty bit

[Xudong Hao]

EPT Dirty bit use bit 9 as Intel SDM definition, to avoid conflict, change
PT_FIRST_AVAIL_BITS_SHIFT to 10.

Signed-off-by: Xudong Hao <xudong.hao@intel.com>
Signed-off-by: Xiantao Zhang <xiantao.zhang@intel.com>

---
 arch/x86/kvm/mmu.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/x86/kvm/mmu.c b/arch/x86/kvm/mmu.c
index 24dd43d..f5b996a 100644
--- a/arch/x86/kvm/mmu.c
+++ b/arch/x86/kvm/mmu.c
@@ -90,7 +90,7 @@ module_param(dbg, bool, 0644);
 
 #define PTE_PREFETCH_NUM		8
 
-#define PT_FIRST_AVAIL_BITS_SHIFT 9
+#define PT_FIRST_AVAIL_BITS_SHIFT 10
 #define PT64_SECOND_AVAIL_BITS_SHIFT 52
 
 #define PT64_LEVEL_BITS 9
-- 
1.5.5

RE: [PATCH] KVM: change PT_FIRST_AVAIL_BITS_SHIFT to avoid conflict with EPT Dirty bit

[Hao, Xudong]

Hi, Avi

Do you have comments for this patch?

Thanks,
-Xudong

> -----Original Message-----
> From: Hao, Xudong
> Sent: Thursday, June 07, 2012 6:26 PM
> To: avi@redhat.com
> Cc: kvm@vger.kernel.org; Hao, Xudong; Zhang, Xiantao
> Subject: [PATCH] KVM: change PT_FIRST_AVAIL_BITS_SHIFT to avoid conflict
> with EPT Dirty bit
> 
> EPT Dirty bit use bit 9 as Intel SDM definition, to avoid conflict, change
> PT_FIRST_AVAIL_BITS_SHIFT to 10.
> 
> Signed-off-by: Xudong Hao <xudong.hao@intel.com>
> Signed-off-by: Xiantao Zhang <xiantao.zhang@intel.com>
> 
> ---
>  arch/x86/kvm/mmu.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/arch/x86/kvm/mmu.c b/arch/x86/kvm/mmu.c
> index 24dd43d..f5b996a 100644
> --- a/arch/x86/kvm/mmu.c
> +++ b/arch/x86/kvm/mmu.c
> @@ -90,7 +90,7 @@ module_param(dbg, bool, 0644);
> 
>  #define PTE_PREFETCH_NUM		8
> 
> -#define PT_FIRST_AVAIL_BITS_SHIFT 9
> +#define PT_FIRST_AVAIL_BITS_SHIFT 10
>  #define PT64_SECOND_AVAIL_BITS_SHIFT 52
> 
>  #define PT64_LEVEL_BITS 9
> --
> 1.5.5

RE: [PATCH] KVM: change PT_FIRST_AVAIL_BITS_SHIFT to avoid conflict with EPT Dirty bit

[Hao, Xudong]

Hi, Avi

What's your opinion of this patch?

Thanks,
-Xudong

> -----Original Message-----
> From: kvm-owner@vger.kernel.org [mailto:kvm-owner@vger.kernel.org] On
> Behalf Of Hao, Xudong
> Sent: Monday, June 11, 2012 9:10 AM
> To: avi@redhat.com
> Cc: kvm@vger.kernel.org; Zhang, Xiantao
> Subject: RE: [PATCH] KVM: change PT_FIRST_AVAIL_BITS_SHIFT to avoid
> conflict with EPT Dirty bit
> 
> Hi, Avi
> 
> Do you have comments for this patch?
> 
> Thanks,
> -Xudong
> 
> > -----Original Message-----
> > From: Hao, Xudong
> > Sent: Thursday, June 07, 2012 6:26 PM
> > To: avi@redhat.com
> > Cc: kvm@vger.kernel.org; Hao, Xudong; Zhang, Xiantao
> > Subject: [PATCH] KVM: change PT_FIRST_AVAIL_BITS_SHIFT to avoid conflict
> > with EPT Dirty bit
> >
> > EPT Dirty bit use bit 9 as Intel SDM definition, to avoid conflict, change
> > PT_FIRST_AVAIL_BITS_SHIFT to 10.
> >
> > Signed-off-by: Xudong Hao <xudong.hao@intel.com>
> > Signed-off-by: Xiantao Zhang <xiantao.zhang@intel.com>
> >
> > ---
> >  arch/x86/kvm/mmu.c |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> >
> > diff --git a/arch/x86/kvm/mmu.c b/arch/x86/kvm/mmu.c
> > index 24dd43d..f5b996a 100644
> > --- a/arch/x86/kvm/mmu.c
> > +++ b/arch/x86/kvm/mmu.c
> > @@ -90,7 +90,7 @@ module_param(dbg, bool, 0644);
> >
> >  #define PTE_PREFETCH_NUM		8
> >
> > -#define PT_FIRST_AVAIL_BITS_SHIFT 9
> > +#define PT_FIRST_AVAIL_BITS_SHIFT 10
> >  #define PT64_SECOND_AVAIL_BITS_SHIFT 52
> >
> >  #define PT64_LEVEL_BITS 9
> > --
> > 1.5.5
> 
> --
> To unsubscribe from this list: send the line "unsubscribe kvm" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] KVM: change PT_FIRST_AVAIL_BITS_SHIFT to avoid conflict with EPT Dirty bit

[Marcelo Tosatti]

On Thu, Jun 07, 2012 at 06:26:07PM +0800, Xudong Hao wrote:
> EPT Dirty bit use bit 9 as Intel SDM definition, to avoid conflict, change
> PT_FIRST_AVAIL_BITS_SHIFT to 10.
> 
> Signed-off-by: Xudong Hao <xudong.hao@intel.com>
> Signed-off-by: Xiantao Zhang <xiantao.zhang@intel.com>

Applied, thanks.