Messages in this thread | | | Subject | Re: [PATCH v2 2/6] perf/amd/iommu: Modify functions to query max banks and counters | From | Suravee Suthikulpanit <> | Date | Thu, 7 Jan 2016 09:28:33 -0600 |
| |
On 01/07/2016 06:34 AM, Joerg Roedel wrote: > On Fri, Jan 01, 2016 at 12:13:36PM -0600, Suravee Suthikulpanit wrote: >> Currently, amd_iommu_pc_get_max_[banks|counters]() require devid, >> which should not be the case. Also, these don't properly support >> multi-IOMMU system. >> >> Current and future AMD systems with IOMMU that support perf counter >> would likely contain homogeneous IOMMUs where multiple IOMMUs are >> availalbe. So, this patch modifies these function to iterate all IOMMU >> to check the max banks and counters reported by the hardware. >> >> Signed-off-by: Suravee Suthikulpanit <Suravee.Suthikulpanit@amd.com> >> --- >> arch/x86/kernel/cpu/perf_event_amd_iommu.c | 17 +++++++---------- >> drivers/iommu/amd_iommu_init.c | 20 ++++++++++++-------- >> include/linux/perf/perf_event_amd_iommu.h | 7 ++----- >> 3 files changed, 21 insertions(+), 23 deletions(-) > > Reviewed-by: Joerg Roedel <jroedel@suse.de> > > This looks like the fix for the CZ issue, right? If true, it is > probably v4.5 material.
Yes in deed :)
Suravee
| |