Re: [PATCH 1/1] perf/x86/intel: Add extended event constraints for Knights Landing

Hi, On Fri, Jun 24, 2016 at 7:54 AM, Odzioba, Lukasz wrote: > On Tuesday, June 21, 2016 11:38 AM, Peter Zijlstra wrote: >> Yes, that is the intent, but how is this achieved? I'm not sure I see >> how the patch ensure this. > > If you are confused, then it is likely that

Re: [PATCH 1/1] perf/x86/intel: Add extended event constraints for Knights Landing

Hi, On Fri, Jun 24, 2016 at 7:54 AM, Odzioba, Lukasz wrote: > On Tuesday, June 21, 2016 11:38 AM, Peter Zijlstra wrote: >> Yes, that is the intent, but how is this achieved? I'm not sure I see >> how the patch ensure this. > > If you are confused, then it is likely that I did something wrong

RE: [PATCH 1/1] perf/x86/intel: Add extended event constraints for Knights Landing

On Tuesday, June 21, 2016 11:38 AM, Peter Zijlstra wrote: > Yes, that is the intent, but how is this achieved? I'm not sure I see > how the patch ensure this. If you are confused, then it is likely that I did something wrong here. Let me explain myself. We already have a mechanism to create

RE: [PATCH 1/1] perf/x86/intel: Add extended event constraints for Knights Landing

On Tuesday, June 21, 2016 11:38 AM, Peter Zijlstra wrote: > Yes, that is the intent, but how is this achieved? I'm not sure I see > how the patch ensure this. If you are confused, then it is likely that I did something wrong here. Let me explain myself. We already have a mechanism to create

Re: [PATCH 1/1] perf/x86/intel: Add extended event constraints for Knights Landing

On Mon, Jun 20, 2016 at 10:26:30AM +, Odzioba, Lukasz wrote: > On 08.06.2016 Peter Zijlstra wrote: > > How does this work in the light of intel_alt_er() ? > > Hi Peter, > > If the constrained bit is valid on only one of the OCR MSRs (like in case of > KNL), > then OCR valid mask will

Re: [PATCH 1/1] perf/x86/intel: Add extended event constraints for Knights Landing

On Mon, Jun 20, 2016 at 10:26:30AM +, Odzioba, Lukasz wrote: > On 08.06.2016 Peter Zijlstra wrote: > > How does this work in the light of intel_alt_er() ? > > Hi Peter, > > If the constrained bit is valid on only one of the OCR MSRs (like in case of > KNL), > then OCR valid mask will

RE: [PATCH 1/1] perf/x86/intel: Add extended event constraints for Knights Landing

On 08.06.2016 Peter Zijlstra wrote: > How does this work in the light of intel_alt_er() ? Hi Peter, If the constrained bit is valid on only one of the OCR MSRs (like in case of KNL), then OCR valid mask will forbid altering it by the other MSR in intel_alt_er. If constrained bit is valid on

RE: [PATCH 1/1] perf/x86/intel: Add extended event constraints for Knights Landing

On 08.06.2016 Peter Zijlstra wrote: > How does this work in the light of intel_alt_er() ? Hi Peter, If the constrained bit is valid on only one of the OCR MSRs (like in case of KNL), then OCR valid mask will forbid altering it by the other MSR in intel_alt_er. If constrained bit is valid on

Re: [PATCH 1/1] perf/x86/intel: Add extended event constraints for Knights Landing

On Wed, Jun 08, 2016 at 06:02:16AM +0200, Lukasz Odzioba wrote: > For Knights Landing processor we need to filter OFFCORE_RESPONSE > events by config1 parameter to make sure that it will end up in > an appropriate PMC to meet specification. > > On Knights Landing: > MSR_OFFCORE_RSP_1 bits 8, 11,

Re: [PATCH 1/1] perf/x86/intel: Add extended event constraints for Knights Landing

On Wed, Jun 08, 2016 at 06:02:16AM +0200, Lukasz Odzioba wrote: > For Knights Landing processor we need to filter OFFCORE_RESPONSE > events by config1 parameter to make sure that it will end up in > an appropriate PMC to meet specification. > > On Knights Landing: > MSR_OFFCORE_RSP_1 bits 8, 11,

[PATCH 1/1] perf/x86/intel: Add extended event constraints for Knights Landing

For Knights Landing processor we need to filter OFFCORE_RESPONSE events by config1 parameter to make sure that it will end up in an appropriate PMC to meet specification. On Knights Landing: MSR_OFFCORE_RSP_1 bits 8, 11, 14 can be used only on PMC1 MSR_OFFCORE_RSP_0 bit 38 can be used only on

[PATCH 1/1] perf/x86/intel: Add extended event constraints for Knights Landing

For Knights Landing processor we need to filter OFFCORE_RESPONSE events by config1 parameter to make sure that it will end up in an appropriate PMC to meet specification. On Knights Landing: MSR_OFFCORE_RSP_1 bits 8, 11, 14 can be used only on PMC1 MSR_OFFCORE_RSP_0 bit 38 can be used only on