On Wed, Feb 15, 2023 at 05:53:23PM +0100, Eric Auger wrote: > > + SMMUTLBEntry *tlbe, SMMUPTWEventInfo *info) > > { > > dma_addr_t baseaddr, indexmask; > > int stage = cfg->stage; > > @@ -384,7 +384,7 @@ int smmu_ptw(SMMUTransCfg *cfg, dma_addr_t iova, > > IOMMUAccessFlags perm, > > g_assert_not_reached(); > > } > > > > - return smmu_ptw_64(cfg, iova, perm, tlbe, info); > > + return smmu_ptw_64_s1(cfg, iova, perm, tlbe, info); > May need to rename the trace points as well > All PTW trace points have a stage argument which is set correctly from stage-1/stage-2 ptw functions: trace_smmu_ptw_page_pte trace_smmu_ptw_block_pte trace_smmu_ptw_invalid_pte
trace_smmu_ptw_level is the only one that had no stage argument, I can change it to be consistent with the others. Thanks, Mostafa