From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marcelo Tosatti Subject: Re: [PATCH 2/6] KVM: SVM: Add manipulation functions for CRx intercepts Date: Thu, 2 Dec 2010 14:43:50 -0200 Message-ID: <20101202164350.GB23017@amt.cnet> References: <1291136641-4874-1-git-send-email-joerg.roedel@amd.com> <1291136641-4874-3-git-send-email-joerg.roedel@amd.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Avi Kivity , kvm@vger.kernel.org, linux-kernel@vger.kernel.org To: Joerg Roedel Return-path: Content-Disposition: inline In-Reply-To: <1291136641-4874-3-git-send-email-joerg.roedel@amd.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: kvm.vger.kernel.org On Tue, Nov 30, 2010 at 06:03:57PM +0100, Joerg Roedel wrote: > This patch wraps changes to the CRx intercepts of SVM into > seperate functions to abstract nested-svm better and prepare > the implementation of the vmcb-clean-bits feature. > > Signed-off-by: Joerg Roedel > --- > arch/x86/include/asm/svm.h | 15 +++-- > arch/x86/kvm/svm.c | 119 +++++++++++++++++++++++--------------------- > 2 files changed, 72 insertions(+), 62 deletions(-) > > - control->intercept_cr_read = INTERCEPT_CR0_MASK | > - INTERCEPT_CR3_MASK | > - INTERCEPT_CR4_MASK; > - > - control->intercept_cr_write = INTERCEPT_CR0_MASK | > - INTERCEPT_CR3_MASK | > - INTERCEPT_CR4_MASK | > - INTERCEPT_CR8_MASK; > + set_cr_intercept(svm, INTERCEPT_CR0_READ); > + set_cr_intercept(svm, INTERCEPT_CR3_READ); > + set_cr_intercept(svm, INTERCEPT_CR4_READ); > + set_cr_intercept(svm, INTERCEPT_CR0_WRITE); > + set_cr_intercept(svm, INTERCEPT_CR3_WRITE); > + set_cr_intercept(svm, INTERCEPT_CR4_WRITE); > + set_cr_intercept(svm, INTERCEPT_CR8_WRITE); Should clear hflags before using is_guest_mode().