Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1330750 > unrolled thread

[PATCH V3 0/5] perf/amd/iommu: Enable multi-IOMMU support

Back to article view | Back to linux.kernel

Contents

  [PATCH V3 0/5] perf/amd/iommu: Enable multi-IOMMU support Suravee Suthikulpanit <Suravee.Suthikulpanit@amd.com> - 2016-02-10 00:00 +0100
    Re: [PATCH V3 0/5] perf/amd/iommu: Enable multi-IOMMU support Borislav Petkov <bp@alien8.de> - 2016-02-10 01:10 +0100
      Re: [PATCH V3 0/5] perf/amd/iommu: Enable multi-IOMMU support Suravee Suthikulpanit <Suravee.Suthikulpanit@amd.com> - 2016-02-10 14:50 +0100

#1330750 — [PATCH V3 0/5] perf/amd/iommu: Enable multi-IOMMU support

From: Suravee Suthikulpanit <suravee.suthikulpanit@amd.com>

This patch series modifies the existing perf_event_amd_iommu driver
to support systems with multiple IOMMUs. It introduces new AMD IOMMU APIs,
which will are used by the AMD IOMMU Perf driver to access performance
counters in multiple IOMMUs.

In addition, this series should also fix current AMD IOMMU PMU driver
initialization issue in some existing KV and CZ platform.

Note that this patch also fixes the issue where IOMMU driver fails
to write to IOMMU perf counter as reported by Andreas Hartmann here
(http://comments.gmane.org/gmane.linux.kernel.pci/49147).

Git branch containing this patch series is available here:

    https://github.com/ssuthiku/linux.git  perf-iommu-v3

Changes from V2 ( https://lkml.org/lkml/2016/1/1/141)
  * Ported to 4.5.0-rc2
  * Add reviewed by Joerg for patch 1 and 2
  * Remove EXPORT_SYMBOL from patch 3 (per Joerg suggestion)
  * Merge patch 4/6 and 6/6 from V2 into 5/5 in V3 and add
    more description in the commit message and in code comment.
  * Patch 5: modify the logic to update counts to get rid off
    un-necessary local64_cmpxchg().

Changes from V1 (https://lkml.org/lkml/2015/12/22/535):
  * Update patch3 and 6 to use amd_iommus_present instead of introducing
    amd_iommu_cnt static variable since they are the same thing

Suravee Suthikulpanit (5):
  perf/amd/iommu: Consolidate and move perf_event_amd_iommu header
  perf/amd/iommu: Modify functions to query max banks and counters
  iommu/amd: Introduce amd_iommu_get_num_iommus()
  perf/amd/iommu: Introduce get_iommu_bnk_cnt_evt_idx
  perf/amd/iommu: Enable support for multiple IOMMUs

 arch/x86/kernel/cpu/perf_event_amd_iommu.c | 164 ++++++++++++++++++++---------
 arch/x86/kernel/cpu/perf_event_amd_iommu.h |  40 -------
 drivers/iommu/amd_iommu_init.c             | 116 ++++++++++++++++----
 drivers/iommu/amd_iommu_proto.h            |   7 --
 include/linux/perf/perf_event_amd_iommu.h  |  43 ++++++++
 5 files changed, 254 insertions(+), 116 deletions(-)
 delete mode 100644 arch/x86/kernel/cpu/perf_event_amd_iommu.h
 create mode 100644 include/linux/perf/perf_event_amd_iommu.h

-- 
2.5.0

[toc] | [next] | [standalone]

#1330782

On Tue, Feb 09, 2016 at 04:53:50PM -0600, Suravee Suthikulpanit wrote:
> From: Suravee Suthikulpanit <suravee.suthikulpanit@amd.com>
> 
> This patch series modifies the existing perf_event_amd_iommu driver
> to support systems with multiple IOMMUs. It introduces new AMD IOMMU APIs,
> which will are used by the AMD IOMMU Perf driver to access performance
> counters in multiple IOMMUs.
> 
> In addition, this series should also fix current AMD IOMMU PMU driver
> initialization issue in some existing KV and CZ platform.
> 
> Note that this patch also fixes the issue where IOMMU driver fails
> to write to IOMMU perf counter as reported by Andreas Hartmann here
> (http://comments.gmane.org/gmane.linux.kernel.pci/49147).
> 
> Git branch containing this patch series is available here:
> 
>     https://github.com/ssuthiku/linux.git  perf-iommu-v3
> 
> Changes from V2 ( https://lkml.org/lkml/2016/1/1/141)
>   * Ported to 4.5.0-rc2
>   * Add reviewed by Joerg for patch 1 and 2
>   * Remove EXPORT_SYMBOL from patch 3 (per Joerg suggestion)
>   * Merge patch 4/6 and 6/6 from V2 into 5/5 in V3 and add
>     more description in the commit message and in code comment.
>   * Patch 5: modify the logic to update counts to get rid off
>     un-necessary local64_cmpxchg().
> 
> Changes from V1 (https://lkml.org/lkml/2015/12/22/535):
>   * Update patch3 and 6 to use amd_iommus_present instead of introducing
>     amd_iommu_cnt static variable since they are the same thing
> 
> Suravee Suthikulpanit (5):
>   perf/amd/iommu: Consolidate and move perf_event_amd_iommu header
>   perf/amd/iommu: Modify functions to query max banks and counters
>   iommu/amd: Introduce amd_iommu_get_num_iommus()
>   perf/amd/iommu: Introduce get_iommu_bnk_cnt_evt_idx
>   perf/amd/iommu: Enable support for multiple IOMMUs
> 
>  arch/x86/kernel/cpu/perf_event_amd_iommu.c | 164 ++++++++++++++++++++---------
>  arch/x86/kernel/cpu/perf_event_amd_iommu.h |  40 -------

You probably want to redo those against latest tip/master:

http://git.kernel.org/tip/5b26547dd7faa84e1293baa144a0f3e74ed7d4c7

-- 
Regards/Gruss,
    Boris.

ECO tip #101: Trim your mails when you reply.

[toc] | [prev] | [next] | [standalone]

#1331176

Hi Boris,

On 02/09/2016 06:08 PM, Borislav Petkov wrote:
> You probably want to redo those against latest tip/master:
>
> http://git.kernel.org/tip/5b26547dd7faa84e1293baa144a0f3e74ed7d4c7
>
> -- Regards/Gruss, Boris.

Sure, I'll re-base this. By the way, do you have any other comments 
regarding the changes?

Thanks,
Suravee

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web