mirror of
				https://github.com/torvalds/linux.git
				synced 2025-11-04 02:30:34 +02:00 
			
		
		
		
	powerpc/perf: Fix branch event code for power9
Correct "branch" event code of Power9 is "r4d05e". Replace the current
"branch" event code with "r4d05e" and add a hack to use "r10012" as
event code for Power9 DD1.
Fixes: d89f473ff6 ("powerpc/perf: Fix PM_BRU_CMPL event code for power9")
Reported-by: Anton Blanchard <anton@samba.org>
Signed-off-by: Madhavan Srinivasan <maddy@linux.vnet.ibm.com>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
			
			
This commit is contained in:
		
							parent
							
								
									89d8bb1638
								
							
						
					
					
						commit
						24bedcb7c8
					
				
					 2 changed files with 10 additions and 2 deletions
				
			
		| 
						 | 
					@ -16,7 +16,7 @@ EVENT(PM_CYC,					0x0001e)
 | 
				
			||||||
EVENT(PM_ICT_NOSLOT_CYC,			0x100f8)
 | 
					EVENT(PM_ICT_NOSLOT_CYC,			0x100f8)
 | 
				
			||||||
EVENT(PM_CMPLU_STALL,				0x1e054)
 | 
					EVENT(PM_CMPLU_STALL,				0x1e054)
 | 
				
			||||||
EVENT(PM_INST_CMPL,				0x00002)
 | 
					EVENT(PM_INST_CMPL,				0x00002)
 | 
				
			||||||
EVENT(PM_BRU_CMPL,				0x10012)
 | 
					EVENT(PM_BRU_CMPL,				0x4d05e)
 | 
				
			||||||
EVENT(PM_BR_MPRED_CMPL,				0x400f6)
 | 
					EVENT(PM_BR_MPRED_CMPL,				0x400f6)
 | 
				
			||||||
 | 
					
 | 
				
			||||||
/* All L1 D cache load references counted at finish, gated by reject */
 | 
					/* All L1 D cache load references counted at finish, gated by reject */
 | 
				
			||||||
| 
						 | 
					@ -56,3 +56,5 @@ EVENT(PM_RUN_CYC,				0x600f4)
 | 
				
			||||||
/* Instruction Dispatched */
 | 
					/* Instruction Dispatched */
 | 
				
			||||||
EVENT(PM_INST_DISP,				0x200f2)
 | 
					EVENT(PM_INST_DISP,				0x200f2)
 | 
				
			||||||
EVENT(PM_INST_DISP_ALT,				0x300f2)
 | 
					EVENT(PM_INST_DISP_ALT,				0x300f2)
 | 
				
			||||||
 | 
					/* Alternate Branch event code */
 | 
				
			||||||
 | 
					EVENT(PM_BR_CMPL_ALT,				0x10012)
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -231,7 +231,7 @@ static int power9_generic_events_dd1[] = {
 | 
				
			||||||
	[PERF_COUNT_HW_STALLED_CYCLES_FRONTEND] =	PM_ICT_NOSLOT_CYC,
 | 
						[PERF_COUNT_HW_STALLED_CYCLES_FRONTEND] =	PM_ICT_NOSLOT_CYC,
 | 
				
			||||||
	[PERF_COUNT_HW_STALLED_CYCLES_BACKEND] =	PM_CMPLU_STALL,
 | 
						[PERF_COUNT_HW_STALLED_CYCLES_BACKEND] =	PM_CMPLU_STALL,
 | 
				
			||||||
	[PERF_COUNT_HW_INSTRUCTIONS] =			PM_INST_DISP,
 | 
						[PERF_COUNT_HW_INSTRUCTIONS] =			PM_INST_DISP,
 | 
				
			||||||
	[PERF_COUNT_HW_BRANCH_INSTRUCTIONS] =		PM_BRU_CMPL,
 | 
						[PERF_COUNT_HW_BRANCH_INSTRUCTIONS] =		PM_BR_CMPL_ALT,
 | 
				
			||||||
	[PERF_COUNT_HW_BRANCH_MISSES] =			PM_BR_MPRED_CMPL,
 | 
						[PERF_COUNT_HW_BRANCH_MISSES] =			PM_BR_MPRED_CMPL,
 | 
				
			||||||
	[PERF_COUNT_HW_CACHE_REFERENCES] =		PM_LD_REF_L1,
 | 
						[PERF_COUNT_HW_CACHE_REFERENCES] =		PM_LD_REF_L1,
 | 
				
			||||||
	[PERF_COUNT_HW_CACHE_MISSES] =			PM_LD_MISS_L1_FIN,
 | 
						[PERF_COUNT_HW_CACHE_MISSES] =			PM_LD_MISS_L1_FIN,
 | 
				
			||||||
| 
						 | 
					@ -453,6 +453,12 @@ static int __init init_power9_pmu(void)
 | 
				
			||||||
		 * sampling scenarios in power9 DD1, instead use PM_INST_DISP.
 | 
							 * sampling scenarios in power9 DD1, instead use PM_INST_DISP.
 | 
				
			||||||
		 */
 | 
							 */
 | 
				
			||||||
		EVENT_VAR(PM_INST_CMPL, _g).id = PM_INST_DISP;
 | 
							EVENT_VAR(PM_INST_CMPL, _g).id = PM_INST_DISP;
 | 
				
			||||||
 | 
							/*
 | 
				
			||||||
 | 
							 * Power9 DD1 should use PM_BR_CMPL_ALT event code for
 | 
				
			||||||
 | 
							 * "branches" to provide correct counter value.
 | 
				
			||||||
 | 
							 */
 | 
				
			||||||
 | 
							EVENT_VAR(PM_BRU_CMPL, _g).id = PM_BR_CMPL_ALT;
 | 
				
			||||||
 | 
							EVENT_VAR(PM_BRU_CMPL, _c).id = PM_BR_CMPL_ALT;
 | 
				
			||||||
		rc = register_power_pmu(&power9_isa207_pmu);
 | 
							rc = register_power_pmu(&power9_isa207_pmu);
 | 
				
			||||||
	} else {
 | 
						} else {
 | 
				
			||||||
		rc = register_power_pmu(&power9_pmu);
 | 
							rc = register_power_pmu(&power9_pmu);
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
		Reference in a new issue