Messages in this thread | | | From | Robin Getz <> | Subject | Re: Fix Blackfin HARDWARE_PM support | Date | Tue, 15 Jan 2008 15:22:38 -0500 |
| |
On Tue 15 Jan 2008 12:42, Mathieu Desnoyers pondered: > This patch restores the blackfin Hardware Performance Monitor Profiling > support that was killed by > commit 09cadedbdc01f1a4bea1f427d4fb4642eaa19da9. > > Since there seems to be no good reason to behave differently from other > architectures, it now automatically selects the hardware performance > counters whenever the profiling is activated. > > mach-common/irqpanic.c: pm_overflow > calls pm_overflow_handler which is in oprofile/op_model_bf533.c. I doubt > that setting HARDWARE_PM as "m" will work at all, since the > pm_overflow_handler should be in the core kernel image because it is > called by irqpanic.c. > > Therefore, I change HARDWARE_PM from a tristate to a bool. > > The whole arch/$(ARCH)/oprofile/ is built depending on CONFIG_OPROFILE. > Since part of the HARDWARE_PM support files sits in this directory, it makes > sense to also depend on OPROFILE, not only PROFILING. Since OPROFILE already > depends on PROFILING, it is correct to only depend on OPROFILE only. > > Thanks to Adrian Bunk for finding this bug and providing an initial > patch. > > Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@polymtl.ca> > CC: Linus Torvalds <torvalds@linux-foundation.org> > CC: Adrian Bunk <adrian.bunk@movial.fi> > CC: Randy Dunlap <randy.dunlap@oracle.com> > CC: bryan.wu@analog.com > CC: Robin Getz <rgetz@blackfin.uclinux.org>
Acked-by: Robin Getz <rgetz@blackfin.uclinux.org>
> --- > arch/blackfin/Kconfig | 4 ++++ > 1 file changed, 4 insertions(+) > > Index: linux-2.6-lttng/arch/blackfin/Kconfig > =================================================================== > --- linux-2.6-lttng.orig/arch/blackfin/Kconfig 2007-12-29 > 11:00:05.000000000 -0500 > +++ linux-2.6-lttng/arch/blackfin/Kconfig 2007-12-29 > 11:25:39.000000000 -0500 > @@ -65,6 +65,10 @@ config GENERIC_CALIBRATE_DELAY > bool > default y > > +config HARDWARE_PM > + def_bool y > + depends on OPROFILE > + > source "init/Kconfig" > source "kernel/Kconfig.preempt" > > >
| |