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 _______________________________________________ linaro-dev mailing list linaro-dev@lists.linaro.org http://lists.linaro.org/mailman/listinfo/linaro-dev