> -----Original Message-----
> From: Kang, Luwei
> Sent: Tuesday, May 22, 2018 12:52 PM
> To: k...@vger.kernel.org
> Cc: t...@linutronix.de; mi...@redhat.com; h...@zytor.com; x...@kernel.org; 
> chao.p.p...@linux.intel.com;
> thomas.lenda...@amd.com; b...@suse.de; Liang, Kan <kan.li...@intel.com>; 
> janakarajan.natara...@amd.com;
> d...@amazon.co.uk; linux-kernel@vger.kernel.org; 
> alexander.shish...@linux.intel.com; pet...@infradead.org;
> mathieu.poir...@linaro.org; kstew...@linuxfoundation.org; 
> gre...@linuxfoundation.org; pbonz...@redhat.com;
> rkrc...@redhat.com; da...@redhat.com; b...@redhat.com; 
> yu.c.zh...@linux.intel.com; j...@8bytes.org; Kang, Luwei
> <luwei.k...@intel.com>
> Subject: [PATCH v9 01/12] perf/x86/intel/pt: Move Intel-PT MSRs bit 
> definitions to a public header
> 
> From: Chao Peng <chao.p.p...@linux.intel.com>
> 
> Intel Processor Trace virtualization enabling in KVM guest need to access 
> these MSRs bit definitions, so move them to public header
> file msr-index.h.
> 
> Signed-off-by: Chao Peng <chao.p.p...@linux.intel.com>
> Signed-off-by: Luwei Kang <luwei.k...@intel.com>
> ---
>  arch/x86/events/intel/pt.h       | 37 -------------------------------------
>  arch/x86/include/asm/msr-index.h | 33 +++++++++++++++++++++++++++++++++
>  2 files changed, 33 insertions(+), 37 deletions(-)
> 
> diff --git a/arch/x86/events/intel/pt.h b/arch/x86/events/intel/pt.h index 
> 0eb41d0..0050ca1 100644
> --- a/arch/x86/events/intel/pt.h
> +++ b/arch/x86/events/intel/pt.h
> @@ -20,43 +20,6 @@
>  #define __INTEL_PT_H__
> 
>  /*
> - * PT MSR bit definitions
> - */
> -#define RTIT_CTL_TRACEEN             BIT(0)
> -#define RTIT_CTL_CYCLEACC            BIT(1)
> -#define RTIT_CTL_OS                  BIT(2)
> -#define RTIT_CTL_USR                 BIT(3)
> -#define RTIT_CTL_PWR_EVT_EN          BIT(4)
> -#define RTIT_CTL_FUP_ON_PTW          BIT(5)
> -#define RTIT_CTL_CR3EN                       BIT(7)
> -#define RTIT_CTL_TOPA                        BIT(8)
> -#define RTIT_CTL_MTC_EN                      BIT(9)
> -#define RTIT_CTL_TSC_EN                      BIT(10)
> -#define RTIT_CTL_DISRETC             BIT(11)
> -#define RTIT_CTL_PTW_EN                      BIT(12)
> -#define RTIT_CTL_BRANCH_EN           BIT(13)
> -#define RTIT_CTL_MTC_RANGE_OFFSET    14
> -#define RTIT_CTL_MTC_RANGE           (0x0full << RTIT_CTL_MTC_RANGE_OFFSET)
> -#define RTIT_CTL_CYC_THRESH_OFFSET   19
> -#define RTIT_CTL_CYC_THRESH          (0x0full << RTIT_CTL_CYC_THRESH_OFFSET)
> -#define RTIT_CTL_PSB_FREQ_OFFSET     24
> -#define RTIT_CTL_PSB_FREQ                    (0x0full << 
> RTIT_CTL_PSB_FREQ_OFFSET)
> -#define RTIT_CTL_ADDR0_OFFSET                32
> -#define RTIT_CTL_ADDR0               (0x0full << RTIT_CTL_ADDR0_OFFSET)
> -#define RTIT_CTL_ADDR1_OFFSET                36
> -#define RTIT_CTL_ADDR1               (0x0full << RTIT_CTL_ADDR1_OFFSET)
> -#define RTIT_CTL_ADDR2_OFFSET                40
> -#define RTIT_CTL_ADDR2               (0x0full << RTIT_CTL_ADDR2_OFFSET)
> -#define RTIT_CTL_ADDR3_OFFSET                44
> -#define RTIT_CTL_ADDR3               (0x0full << RTIT_CTL_ADDR3_OFFSET)
> -#define RTIT_STATUS_FILTEREN         BIT(0)
> -#define RTIT_STATUS_CONTEXTEN                BIT(1)
> -#define RTIT_STATUS_TRIGGEREN                BIT(2)
> -#define RTIT_STATUS_BUFFOVF          BIT(3)
> -#define RTIT_STATUS_ERROR            BIT(4)
> -#define RTIT_STATUS_STOPPED          BIT(5)
> -
> -/*
>   * Single-entry ToPA: when this close to region boundary, switch
>   * buffers to avoid losing data.
>   */
> diff --git a/arch/x86/include/asm/msr-index.h 
> b/arch/x86/include/asm/msr-index.h
> index 53d5b1b..afe4e13 100644
> --- a/arch/x86/include/asm/msr-index.h
> +++ b/arch/x86/include/asm/msr-index.h
> @@ -106,7 +106,40 @@
>  #define MSR_PEBS_LD_LAT_THRESHOLD    0x000003f6
> 
>  #define MSR_IA32_RTIT_CTL            0x00000570
> +#define RTIT_CTL_TRACEEN             BIT(0)
> +#define RTIT_CTL_CYCLEACC            BIT(1)
> +#define RTIT_CTL_OS                  BIT(2)
> +#define RTIT_CTL_USR                 BIT(3)
> +#define RTIT_CTL_PWR_EVT_EN          BIT(4)
> +#define RTIT_CTL_FUP_ON_PTW          BIT(5)
> +#define RTIT_CTL_CR3EN                       BIT(7)
> +#define RTIT_CTL_TOPA                        BIT(8)
> +#define RTIT_CTL_MTC_EN                      BIT(9)
> +#define RTIT_CTL_TSC_EN                      BIT(10)
> +#define RTIT_CTL_DISRETC             BIT(11)
> +#define RTIT_CTL_PTW_EN                      BIT(12)
> +#define RTIT_CTL_BRANCH_EN           BIT(13)
> +#define RTIT_CTL_MTC_RANGE_OFFSET    14
> +#define RTIT_CTL_MTC_RANGE           (0x0full << RTIT_CTL_MTC_RANGE_OFFSET)
> +#define RTIT_CTL_CYC_THRESH_OFFSET   19
> +#define RTIT_CTL_CYC_THRESH          (0x0full << RTIT_CTL_CYC_THRESH_OFFSET)
> +#define RTIT_CTL_PSB_FREQ_OFFSET     24
> +#define RTIT_CTL_PSB_FREQ            (0x0full << RTIT_CTL_PSB_FREQ_OFFSET)
> +#define RTIT_CTL_ADDR0_OFFSET                32
> +#define RTIT_CTL_ADDR0                       (0x0full << 
> RTIT_CTL_ADDR0_OFFSET)
> +#define RTIT_CTL_ADDR1_OFFSET                36
> +#define RTIT_CTL_ADDR1                       (0x0full << 
> RTIT_CTL_ADDR1_OFFSET)
> +#define RTIT_CTL_ADDR2_OFFSET                40
> +#define RTIT_CTL_ADDR2                       (0x0full << 
> RTIT_CTL_ADDR2_OFFSET)
> +#define RTIT_CTL_ADDR3_OFFSET                44
> +#define RTIT_CTL_ADDR3                       (0x0full << 
> RTIT_CTL_ADDR3_OFFSET)
>  #define MSR_IA32_RTIT_STATUS         0x00000571
> +#define RTIT_STATUS_FILTEREN         BIT(0)
> +#define RTIT_STATUS_CONTEXTEN                BIT(1)
> +#define RTIT_STATUS_TRIGGEREN                BIT(2)
> +#define RTIT_STATUS_BUFFOVF          BIT(3)
> +#define RTIT_STATUS_ERROR            BIT(4)
> +#define RTIT_STATUS_STOPPED          BIT(5)
>  #define MSR_IA32_RTIT_ADDR0_A                0x00000580
>  #define MSR_IA32_RTIT_ADDR0_B                0x00000581
>  #define MSR_IA32_RTIT_ADDR1_A                0x00000582

Hi,
    Patch 1~5 have some code changes in x86 native for Intel Processor Trace 
virtualization enabling in KVM guest.
    I have sent patch set v9 which include some minor changes from old version.
    Do you have any comments?

Thanks,
Luwei Kang

Reply via email to