Re: [PATCH V3 5/5] perf/amd/iommu: Enable support for multiple IOMMUs

2016-02-11 Thread Suravee Suthikulpanit
Hi Boris, Thank you very much for catching several styling issues. I should have taken care of that earlier. I'll clean them up. Please see my other comments below. Thanks, Suravee On 02/11/2016 12:14 AM, Borislav Petkov wrote: On Tue, Feb 09, 2016 at 04:53:55PM -0600, Suravee

Re: [PATCH V3 5/5] perf/amd/iommu: Enable support for multiple IOMMUs

2016-02-11 Thread Suravee Suthikulpanit
Hi Boris, Thank you very much for catching several styling issues. I should have taken care of that earlier. I'll clean them up. Please see my other comments below. Thanks, Suravee On 02/11/2016 12:14 AM, Borislav Petkov wrote: On Tue, Feb 09, 2016 at 04:53:55PM -0600, Suravee

Re: [PATCH V3 5/5] perf/amd/iommu: Enable support for multiple IOMMUs

2016-02-10 Thread Borislav Petkov
On Tue, Feb 09, 2016 at 04:53:55PM -0600, Suravee Suthikulpanit wrote: > The current amd_iommu_pc_get_set_reg_val() does not support muli-IOMMU > system. This patch replace amd_iommu_pc_get_set_reg_val() with > amd_iommu_pc_set_reg_val() and amd_iommu_pc_[set|get]_cnt_vals(). > > Also, the

Re: [PATCH V3 5/5] perf/amd/iommu: Enable support for multiple IOMMUs

2016-02-10 Thread Borislav Petkov
On Tue, Feb 09, 2016 at 04:53:55PM -0600, Suravee Suthikulpanit wrote: > The current amd_iommu_pc_get_set_reg_val() does not support muli-IOMMU > system. This patch replace amd_iommu_pc_get_set_reg_val() with > amd_iommu_pc_set_reg_val() and amd_iommu_pc_[set|get]_cnt_vals(). > > Also, the

[PATCH V3 5/5] perf/amd/iommu: Enable support for multiple IOMMUs

2016-02-09 Thread Suravee Suthikulpanit
The current amd_iommu_pc_get_set_reg_val() does not support muli-IOMMU system. This patch replace amd_iommu_pc_get_set_reg_val() with amd_iommu_pc_set_reg_val() and amd_iommu_pc_[set|get]_cnt_vals(). Also, the current struct hw_perf_event.prev_count can only store the previous counter value only

[PATCH V3 5/5] perf/amd/iommu: Enable support for multiple IOMMUs

2016-02-09 Thread Suravee Suthikulpanit
The current amd_iommu_pc_get_set_reg_val() does not support muli-IOMMU system. This patch replace amd_iommu_pc_get_set_reg_val() with amd_iommu_pc_set_reg_val() and amd_iommu_pc_[set|get]_cnt_vals(). Also, the current struct hw_perf_event.prev_count can only store the previous counter value only