Re: [PATCH] drivers/perf: Enable PID_IN_CONTEXTIDR with SPE

2020-12-14 Thread James Clark
On 02/12/2020 01:09, Will Deacon wrote: > On Tue, Dec 01, 2020 at 12:10:40PM +0800, Leo Yan wrote: >> On Mon, Nov 30, 2020 at 04:46:51PM +, Will Deacon wrote: >>> On Mon, Nov 30, 2020 at 06:24:54PM +0200, James Clark wrote: Enable PID_IN_CONTEXTIDR by default when Arm SPE is enabled. >>

Re: [PATCH] drivers/perf: Enable PID_IN_CONTEXTIDR with SPE

2020-12-03 Thread Mathieu Poirier
On Thu, Dec 03, 2020 at 02:39:41PM +0800, Leo Yan wrote: > Hi Will, > > [ + Mathieu ] > > On Tue, Dec 01, 2020 at 11:09:36PM +, Will Deacon wrote: > > On Tue, Dec 01, 2020 at 12:10:40PM +0800, Leo Yan wrote: > > > On Mon, Nov 30, 2020 at 04:46:51PM +, Will Deacon wrote: > > > > On Mon, No

Re: [PATCH] drivers/perf: Enable PID_IN_CONTEXTIDR with SPE

2020-12-02 Thread Leo Yan
Looping in Mathieu at this time. On Thu, Dec 03, 2020 at 02:39:41PM +0800, Leo Yan wrote: > Hi Will, > > [ + Mathieu ] > > On Tue, Dec 01, 2020 at 11:09:36PM +, Will Deacon wrote: > > On Tue, Dec 01, 2020 at 12:10:40PM +0800, Leo Yan wrote: > > > On Mon, Nov 30, 2020 at 04:46:51PM +, Wil

Re: [PATCH] drivers/perf: Enable PID_IN_CONTEXTIDR with SPE

2020-12-02 Thread Leo Yan
Hi Will, [ + Mathieu ] On Tue, Dec 01, 2020 at 11:09:36PM +, Will Deacon wrote: > On Tue, Dec 01, 2020 at 12:10:40PM +0800, Leo Yan wrote: > > On Mon, Nov 30, 2020 at 04:46:51PM +, Will Deacon wrote: > > > On Mon, Nov 30, 2020 at 06:24:54PM +0200, James Clark wrote: > > > > Enable PID_IN_

Re: [PATCH] drivers/perf: Enable PID_IN_CONTEXTIDR with SPE

2020-12-01 Thread Will Deacon
On Tue, Dec 01, 2020 at 12:10:40PM +0800, Leo Yan wrote: > On Mon, Nov 30, 2020 at 04:46:51PM +, Will Deacon wrote: > > On Mon, Nov 30, 2020 at 06:24:54PM +0200, James Clark wrote: > > > Enable PID_IN_CONTEXTIDR by default when Arm SPE is enabled. > > > This flag is required to get PID data in

Re: [PATCH] drivers/perf: Enable PID_IN_CONTEXTIDR with SPE

2020-11-30 Thread Leo Yan
Hi Will, On Mon, Nov 30, 2020 at 04:46:51PM +, Will Deacon wrote: > On Mon, Nov 30, 2020 at 06:24:54PM +0200, James Clark wrote: > > Enable PID_IN_CONTEXTIDR by default when Arm SPE is enabled. > > This flag is required to get PID data in the SPE trace. Without > > it the perf tool will report

Re: [PATCH] drivers/perf: Enable PID_IN_CONTEXTIDR with SPE

2020-11-30 Thread Will Deacon
On Mon, Nov 30, 2020 at 06:24:54PM +0200, James Clark wrote: > Enable PID_IN_CONTEXTIDR by default when Arm SPE is enabled. > This flag is required to get PID data in the SPE trace. Without > it the perf tool will report 0 for PID which isn't very useful, > especially when doing system wide profili

[PATCH] drivers/perf: Enable PID_IN_CONTEXTIDR with SPE

2020-11-30 Thread James Clark
Enable PID_IN_CONTEXTIDR by default when Arm SPE is enabled. This flag is required to get PID data in the SPE trace. Without it the perf tool will report 0 for PID which isn't very useful, especially when doing system wide profiling or profiling applications that fork. There is a small performance