[PATCH v2] ARM: perf: split up perf_event.c by architecture

Jean Pihet jean.pihet at newoldbits.com
Tue Nov 23 09:26:59 EST 2010


Hi Will,

On Mon, Nov 22, 2010 at 5:38 PM, Will Deacon <will.deacon at arm.com> wrote:
> Hello,
>
> This is version 2 of the patches originally posted here:
>
> http://lists.infradead.org/pipermail/linux-arm-kernel/2010-November/031503.html
>
> The only patch that has changed is 5/5 which addresses some style comments
> raised by Jean. Since the patch is over 100k, I've put it in my git repository
> rather than posting it to the list (where it would be held for moderation).
>
> Feedback welcome.
>
>
> The following changes since commit 3561d43fd289f590fdae672e5eb831b8d5cf0bf6:
>  Linus Torvalds (1):
>        Linux 2.6.37-rc3
>
> are available in the git repository at:
>
>  git://linux-arm.org/linux-2.6-wd.git perf-split
>
> Will Deacon (5):
>      ARM: perf: consolidate common PMU behaviour
>      ARM: perf: avoid exposing internal stop function for v6 PMU
>      ARM: perf: add _init() functions to PMUs
>      ARM: perf: encode PMU name in arm_pmu structure
>      ARM: perf: separate PMU backends into multiple files

About the patch 5/5:

This probably is nitpicking from my side, but there still are warnings
generated by checkpatch:

WARNING: please, no space before tabs
#3999: FILE: arch/arm/kernel/perf_event_xscale.c:9:
+ * ^I- xscale1pmu: 2 event counters and a cycle counter$

WARNING: please, no space before tabs
#4000: FILE: arch/arm/kernel/perf_event_xscale.c:10:
+ * ^I- xscale2pmu: 4 event counters and a cycle counter$

total: 0 errors, 2 warnings, 4754 lines checked

Other than that, nice work!

Acked-by: Jean Pihet <j-pihet at ti.com>

Thanks,
Jean


>  arch/arm/kernel/perf_event.c        | 2448 +----------------------------------
>  arch/arm/kernel/perf_event_v6.c     |  672 ++++++++++
>  arch/arm/kernel/perf_event_v7.c     |  906 +++++++++++++
>  arch/arm/kernel/perf_event_xscale.c |  807 ++++++++++++
>  4 files changed, 2419 insertions(+), 2414 deletions(-)
>  create mode 100644 arch/arm/kernel/perf_event_v6.c
>  create mode 100644 arch/arm/kernel/perf_event_v7.c
>  create mode 100644 arch/arm/kernel/perf_event_xscale.c
>
>
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>
>



More information about the linux-arm-kernel mailing list