Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

arch/powerpc/perf: Update get_mem_data_src function to use saved values of sier and mmcra regs

During performance monitor interrupt handling, the regs are setup using
perf_read_regs function. Here some of the pt_regs fields is overloaded.
Samples Instruction Event Register (SIER) is loaded into pt_regs,
overloading regs->dar. And regs->dsisr to store MMCRA (Monitor Mode
Control Register A) so that we only need to read these once on each
interrupt.

Update the isa207_get_mem_data_src function to use regs->dar instead of
reading from SPRN_SIER again. Also use regs->dsisr to read the mmcra
value

Signed-off-by: Athira Rajeev <atrajeev@linux.vnet.ibm.com>
Signed-off-by: Madhavan Srinivasan <maddy@linux.ibm.com>
Link: https://patch.msgid.link/20250121131621.39054-2-atrajeev@linux.vnet.ibm.com

authored by

Athira Rajeev and committed by
Madhavan Srinivasan
f848e7ee 2ffb26af

+12 -2
+12 -2
arch/powerpc/perf/isa207-common.c
··· 319 319 return; 320 320 } 321 321 322 - sier = mfspr(SPRN_SIER); 322 + /* 323 + * Use regs-dar for SPRN_SIER which is saved 324 + * during perf_read_regs at the beginning 325 + * of the PMU interrupt handler to avoid multiple 326 + * reads of SPRN_SIER 327 + */ 328 + sier = regs->dar; 323 329 val = (sier & ISA207_SIER_TYPE_MASK) >> ISA207_SIER_TYPE_SHIFT; 324 330 if (val != 1 && val != 2 && !(val == 7 && cpu_has_feature(CPU_FTR_ARCH_31))) { 325 331 dsrc->val = 0; ··· 346 340 * to determine the exact instruction type. If the sampling 347 341 * criteria is neither load or store, set the type as default 348 342 * to NA. 343 + * 344 + * Use regs->dsisr for MMCRA which is saved during perf_read_regs 345 + * at the beginning of the PMU interrupt handler to avoid 346 + * multiple reads of SPRN_MMCRA 349 347 */ 350 - mmcra = mfspr(SPRN_MMCRA); 348 + mmcra = regs->dsisr; 351 349 352 350 op_type = (mmcra >> MMCRA_SAMP_ELIG_SHIFT) & MMCRA_SAMP_ELIG_MASK; 353 351 switch (op_type) {