Hi,
On Arndale board, perf works well on llct as well as Samsung topic tree[1], but fails on linux-linaro[2]. Git bisect points to commit 6fea652c638e "ARM: perf: [WIP] Add register emulation for offline ARMv7 PMUs".
Is there anything missing w.r.t. Arndale/Exynos to get perf to work on linux-linaro?
[1] https://pastebin.linaro.org/view/bb559643 [2] https://pastebin.linaro.org/view/824f8629
On Fri, 2013-06-14 at 15:50 +0530, Tushar Behera wrote:
Hi,
On Arndale board, perf works well on llct as well as Samsung topic tree[1], but fails on linux-linaro[2]. Git bisect points to commit 6fea652c638e "ARM: perf: [WIP] Add register emulation for offline ARMv7 PMUs".
Is there anything missing w.r.t. Arndale/Exynos to get perf to work on linux-linaro?
Probably not, and I now understand the patches you identified may not work for their intended purpose anyway, so it is likely that they will just get dropped when I next put together a branch for linux-linaro.
On Fri, Jun 14, 2013 at 03:50:38PM +0530, Tushar Behera wrote:
Hi,
On Arndale board, perf works well on llct as well as Samsung topic tree[1], but fails on linux-linaro[2]. Git bisect points to commit 6fea652c638e "ARM: perf: [WIP] Add register emulation for offline ARMv7 PMUs".
Is there anything missing w.r.t. Arndale/Exynos to get perf to work on linux-linaro?
[1] https://pastebin.linaro.org/view/bb559643 [2] https://pastebin.linaro.org/view/824f8629
Since the series containing that patch was a temporary hack, I'm working on a replacement for it. So I'm not sure if it's worth trying to fix it at this stage. Nico already dropped them from his master.
Can you try reverting that whole set of patches and report on whether the resulting kernel works for you?
These are the patches:
ARM: perf: [WIP] Skip PMU register save/restore when no active counters ARM: perf: [WIP] Map PMU IRQ affinities correctly ARM: perf: [WIP] Initial bL switcher support ARM: perf: [WIP] Manipulate the right shadow register for PM*CLR ARM: perf: [WIP] Check PMU is valid for the CPU in armpmu_disable() ARM: perf: [WIP] Add a separate cpu_init() method for ARM PMUs ARM: perf: [WIP] Add register emulation for offline ARMv7 PMUs ARM: perf: Allow multiple CPU PMUs per CPU
Cheers ---Dave
On Thu, 2013-06-20 at 10:29 +0100, Dave Martin wrote:
On Fri, Jun 14, 2013 at 03:50:38PM +0530, Tushar Behera wrote:
Hi,
On Arndale board, perf works well on llct as well as Samsung topic tree[1], but fails on linux-linaro[2]. Git bisect points to commit 6fea652c638e "ARM: perf: [WIP] Add register emulation for offline ARMv7 PMUs".
Is there anything missing w.r.t. Arndale/Exynos to get perf to work on linux-linaro?
[1] https://pastebin.linaro.org/view/bb559643 [2] https://pastebin.linaro.org/view/824f8629
Since the series containing that patch was a temporary hack, I'm working on a replacement for it. So I'm not sure if it's worth trying to fix it at this stage. Nico already dropped them from his master.
Can you try reverting that whole set of patches and report on whether the resulting kernel works for you?
These are the patches:
ARM: perf: [WIP] Skip PMU register save/restore when no active counters ARM: perf: [WIP] Map PMU IRQ affinities correctly ARM: perf: [WIP] Initial bL switcher support ARM: perf: [WIP] Manipulate the right shadow register for PM*CLR ARM: perf: [WIP] Check PMU is valid for the CPU in armpmu_disable() ARM: perf: [WIP] Add a separate cpu_init() method for ARM PMUs ARM: perf: [WIP] Add register emulation for offline ARMv7 PMUs ARM: perf: Allow multiple CPU PMUs per CPU
I dropped these in the IKS branch I feed into linux-linaro, and as of today they are gone, i.e. in ll_20130620.0.
On Thu, Jun 20, 2013 at 10:46:26AM +0100, Jon Medhurst (Tixy) wrote:
On Thu, 2013-06-20 at 10:29 +0100, Dave Martin wrote:
On Fri, Jun 14, 2013 at 03:50:38PM +0530, Tushar Behera wrote:
Hi,
On Arndale board, perf works well on llct as well as Samsung topic tree[1], but fails on linux-linaro[2]. Git bisect points to commit 6fea652c638e "ARM: perf: [WIP] Add register emulation for offline ARMv7 PMUs".
Is there anything missing w.r.t. Arndale/Exynos to get perf to work on linux-linaro?
[1] https://pastebin.linaro.org/view/bb559643 [2] https://pastebin.linaro.org/view/824f8629
Since the series containing that patch was a temporary hack, I'm working on a replacement for it. So I'm not sure if it's worth trying to fix it at this stage. Nico already dropped them from his master.
Can you try reverting that whole set of patches and report on whether the resulting kernel works for you?
These are the patches:
ARM: perf: [WIP] Skip PMU register save/restore when no active counters ARM: perf: [WIP] Map PMU IRQ affinities correctly ARM: perf: [WIP] Initial bL switcher support ARM: perf: [WIP] Manipulate the right shadow register for PM*CLR ARM: perf: [WIP] Check PMU is valid for the CPU in armpmu_disable() ARM: perf: [WIP] Add a separate cpu_init() method for ARM PMUs ARM: perf: [WIP] Add register emulation for offline ARMv7 PMUs ARM: perf: Allow multiple CPU PMUs per CPU
I dropped these in the IKS branch I feed into linux-linaro, and as of today they are gone, i.e. in ll_20130620.0.
OK, that's good.
Thanks ---Dave
-
Dave Martin -
Jon Medhurst (Tixy) -
Tushar Behera