Re: [Xen-devel] [PATCH v2 05/10] x86: Implement Intel Processor Trace context switch

2018-07-04 Thread Kang, Luwei
> >> >> > @@ -40,3 +42,102 @@ static int __init parse_ipt_params(const > >> >> > char > >> >> > +static inline void ipt_save_msr(struct ipt_ctx *ctx, unsigned > >> >> > +int > >> >> > +addr_range) { > >> >> > +unsigned int i; > >> >> > + > >> >> > +rdmsrl(MSR_IA32_RTIT_STATUS,

Re: [Xen-devel] [PATCH v2 05/10] x86: Implement Intel Processor Trace context switch

2018-07-04 Thread Jan Beulich
>>> On 04.07.18 at 10:48, wrote: >> >> > @@ -40,3 +42,102 @@ static int __init parse_ipt_params(const char >> >> > +static inline void ipt_save_msr(struct ipt_ctx *ctx, unsigned int >> >> > +addr_range) { >> >> > +unsigned int i; >> >> > + >> >> > +rdmsrl(MSR_IA32_RTIT_STATUS,

Re: [Xen-devel] [PATCH v2 05/10] x86: Implement Intel Processor Trace context switch

2018-07-04 Thread Kang, Luwei
> >> > @@ -40,3 +42,102 @@ static int __init parse_ipt_params(const char > >> > +static inline void ipt_save_msr(struct ipt_ctx *ctx, unsigned int > >> > +addr_range) { > >> > +unsigned int i; > >> > + > >> > +rdmsrl(MSR_IA32_RTIT_STATUS, ctx->status); > >> > +

Re: [Xen-devel] [PATCH v2 05/10] x86: Implement Intel Processor Trace context switch

2018-07-03 Thread Jan Beulich
>>> On 03.07.18 at 12:18, wrote: >> > @@ -40,3 +42,102 @@ static int __init parse_ipt_params(const char >> > +static inline void ipt_save_msr(struct ipt_ctx *ctx, unsigned int >> > +addr_range) { >> > +unsigned int i; >> > + >> > +rdmsrl(MSR_IA32_RTIT_STATUS, ctx->status); >> > +

Re: [Xen-devel] [PATCH v2 05/10] x86: Implement Intel Processor Trace context switch

2018-07-03 Thread Kang, Luwei
> > @@ -40,3 +42,102 @@ static int __init parse_ipt_params(const char > > *str) > > > > return 0; > > } > > + > > +static inline void ipt_load_msr(const struct ipt_ctx *ctx, > > + unsigned int addr_range) > > Please let the compiler decide whether to inline such

Re: [Xen-devel] [PATCH v2 05/10] x86: Implement Intel Processor Trace context switch

2018-06-29 Thread Jan Beulich
>>> On 30.05.18 at 15:27, wrote: > @@ -40,3 +42,102 @@ static int __init parse_ipt_params(const char *str) > > return 0; > } > + > +static inline void ipt_load_msr(const struct ipt_ctx *ctx, > + unsigned int addr_range) Please let the compiler decide whether to

[Xen-devel] [PATCH v2 05/10] x86: Implement Intel Processor Trace context switch

2018-05-30 Thread Luwei Kang
Load/Restore Intel Processor Trace Register in context switch. MSR IA32_RTIT_CTL is loaded/stored automatically from VMCS. When Intel Processor Trace is supported in guest, we need to load/restore MSRs only when this feature is enabled in guest. Signed-off-by: Luwei Kang ---