diff mbox series

[v2] arm64: perf: remove unsupported events for Cortex-A73

Message ID 1510731566-22016-1-git-send-email-xuyiping@hisilicon.com
State Accepted
Commit f8ada189550984ee21f27be736042b74a7da1d68
Headers show
Series [v2] arm64: perf: remove unsupported events for Cortex-A73 | expand

Commit Message

Xu YiPing Nov. 15, 2017, 7:39 a.m. UTC
bus access read/write events are not supported in A73, based on the
Cortex-A73 TRM r0p2, section 11.9 Events (pages 11-457 to 11-460).

Fixes: 5561b6c5e981 "arm64: perf: add support for Cortex-A73"
Signed-off-by: Xu YiPing <xuyiping@hisilicon.com>

---
 arch/arm64/kernel/perf_event.c | 6 ------
 1 file changed, 6 deletions(-)

-- 
2.7.4

Comments

Will Deacon Nov. 15, 2017, 5:52 p.m. UTC | #1
On Wed, Nov 15, 2017 at 03:39:26PM +0800, Xu YiPing wrote:
> bus access read/write events are not supported in A73, based on the

> Cortex-A73 TRM r0p2, section 11.9 Events (pages 11-457 to 11-460).

> 

> Fixes: 5561b6c5e981 "arm64: perf: add support for Cortex-A73"

> Signed-off-by: Xu YiPing <xuyiping@hisilicon.com>

> ---

>  arch/arm64/kernel/perf_event.c | 6 ------

>  1 file changed, 6 deletions(-)


Julien -- can I have your ack on this, please?

Will
Julien Thierry Nov. 16, 2017, 9:28 a.m. UTC | #2
On 15/11/17 17:52, Will Deacon wrote:
> On Wed, Nov 15, 2017 at 03:39:26PM +0800, Xu YiPing wrote:

>> bus access read/write events are not supported in A73, based on the

>> Cortex-A73 TRM r0p2, section 11.9 Events (pages 11-457 to 11-460).

>>

>> Fixes: 5561b6c5e981 "arm64: perf: add support for Cortex-A73"

>> Signed-off-by: Xu YiPing <xuyiping@hisilicon.com>

>> ---

>>   arch/arm64/kernel/perf_event.c | 6 ------

>>   1 file changed, 6 deletions(-)

> 

> Julien -- can I have your ack on this, please?


Yes, of course.

Acked-by: Julien Thierry <julien.thierry@arm.com>


> 

> Will

> 


-- 
Julien Thierry
diff mbox series

Patch

diff --git a/arch/arm64/kernel/perf_event.c b/arch/arm64/kernel/perf_event.c
index 9eaef51..3affca3 100644
--- a/arch/arm64/kernel/perf_event.c
+++ b/arch/arm64/kernel/perf_event.c
@@ -262,12 +262,6 @@  static const unsigned armv8_a73_perf_cache_map[PERF_COUNT_HW_CACHE_MAX]
 
 	[C(L1D)][C(OP_READ)][C(RESULT_ACCESS)]	= ARMV8_IMPDEF_PERFCTR_L1D_CACHE_RD,
 	[C(L1D)][C(OP_WRITE)][C(RESULT_ACCESS)]	= ARMV8_IMPDEF_PERFCTR_L1D_CACHE_WR,
-
-	[C(NODE)][C(OP_READ)][C(RESULT_ACCESS)]	= ARMV8_IMPDEF_PERFCTR_BUS_ACCESS_RD,
-	[C(NODE)][C(OP_WRITE)][C(RESULT_ACCESS)] = ARMV8_IMPDEF_PERFCTR_BUS_ACCESS_WR,
-
-	[C(NODE)][C(OP_READ)][C(RESULT_ACCESS)]	= ARMV8_IMPDEF_PERFCTR_BUS_ACCESS_RD,
-	[C(NODE)][C(OP_WRITE)][C(RESULT_ACCESS)] = ARMV8_IMPDEF_PERFCTR_BUS_ACCESS_WR,
 };
 
 static const unsigned armv8_thunder_perf_cache_map[PERF_COUNT_HW_CACHE_MAX]