Re: [PATCH] arm64: perf: remove duplicated and unsupported events for Cortex-A73

[Julien Thierry]

Hi,

On 14/11/17 12:49, Alexander Shishkin wrote:
> On Tue, Nov 14, 2017 at 08:27:59PM +0800, Xu YiPing wrote:
> > bus access read/write events are not supported in A73, and the definition
> > is duplicated, so delete it.
>
> In the changelog, it would be nice to mention the commit that added these:
>
> 5561b6c5e9813 ("arm64: perf: add support for Cortex-A73")
>
> Also it's not clear what is this new statement based on. The original
> commit at least references a TRM document.

I re-checked the TRM for this core and Xu is right, the BUS_ACCESS 
events are not available on Cortex-A73. Plus those entries are 
duplicated, I think something went wrong when I rebased the patch.

So I think this new patch is doing the right thing.

Thanks for catching that.

Cheers,

--
Julien Thierry