[lkml]   [2017]   [Nov]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH] arm64: perf: remove duplicated and unsupported events for Cortex-A73

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.


Julien Thierry

 \ /
  Last update: 2017-11-14 14:28    [W:0.023 / U:2.076 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site