lkml.org 
[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
From
Date
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

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