From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dietmar Hahn Subject: [PATCH] msr: Use defines for bits of MSR_IA32_DEBUGCTLMSR instead of numbers Date: Tue, 31 Jan 2012 09:42:23 +0100 Message-ID: <2717113.UZgetSD5rT@amur> 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.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: "xen-devel@lists.xensource.com" List-Id: xen-devel@lists.xenproject.org # HG changeset patch # Parent e2722b24dc0962de37215320b05d1bb7c4c42864 Use defines for bits of MSR_IA32_DEBUGCTLMSR instead of numbers. Signed-off-by: Dietmar Hahn diff -r e2722b24dc09 xen/arch/x86/hvm/vmx/vmx.c --- a/xen/arch/x86/hvm/vmx/vmx.c Thu Jan 26 17:43:31 2012 +0000 +++ b/xen/arch/x86/hvm/vmx/vmx.c Tue Jan 31 09:40:31 2012 +0100 @@ -1944,11 +1944,12 @@ static int vmx_msr_write_intercept(unsig break; case MSR_IA32_DEBUGCTLMSR: { int i, rc = 0; - - if ( !msr_content || (msr_content & ~3) ) + uint64_t allowed = MSR_IA32_DEBUGCTLMSR_LBR | MSR_IA32_DEBUGCTLMSR_BTF; + + if ( !msr_content || (msr_content & ~allowed) ) break; - if ( msr_content & 1 ) + if ( msr_content & MSR_IA32_DEBUGCTLMSR_LBR ) { const struct lbr_info *lbr = last_branch_msr_get(); if ( lbr == NULL ) diff -r e2722b24dc09 xen/include/asm-x86/msr-index.h --- a/xen/include/asm-x86/msr-index.h Thu Jan 26 17:43:31 2012 +0000 +++ b/xen/include/asm-x86/msr-index.h Tue Jan 31 09:40:31 2012 +0100 @@ -64,12 +64,18 @@ #define MSR_MTRRfix4K_F8000 0x0000026f #define MSR_MTRRdefType 0x000002ff +/* MSR_IA32_DEBUGCTLMSR bits: */ +#define _IA32_DEBUGCTLMSR_LBR 0 /* Last Branch Record */ +#define _IA32_DEBUGCTLMSR_BTF 1 /* Single Step on Branches */ +#define MSR_IA32_DEBUGCTLMSR_LBR (1<<_IA32_DEBUGCTLMSR_LBR) +#define MSR_IA32_DEBUGCTLMSR_BTF (1<<_IA32_DEBUGCTLMSR_BTF) + #define MSR_IA32_DEBUGCTLMSR 0x000001d9 #define MSR_IA32_LASTBRANCHFROMIP 0x000001db #define MSR_IA32_LASTBRANCHTOIP 0x000001dc #define MSR_IA32_LASTINTFROMIP 0x000001dd #define MSR_IA32_LASTINTTOIP 0x000001de - + #define MSR_IA32_MTRR_PHYSBASE0 0x00000200 #define MSR_IA32_MTRR_PHYSMASK0 0x00000201 #define MSR_IA32_MTRR_PHYSBASE1 0x00000202 -- Company details: http://ts.fujitsu.com/imprint.html