[PATCH][PPC32] Performance Monitor/Oprofile support for e500

Andrew Morton akpm at osdl.org
Wed Dec 1 10:30:39 EST 2004

Kumar Gala <galak at linen.sps.mot.com> wrote:
> Andrew,
> Adds oprofile support for the e500 PowerPC core.

- arch/ppc/kernel/perfmon_fsl_booke.c has prototypes for init_pmc_stop()
  and friends, but those prototypes are already in

- please don't put prototypes and extern declarations in .c files.  Ever.
  It defeats typechecking.  Put them in a header file which is visible to
  all callers/users as well as to the implementation.

- Do these need to be exported to modules?


  and if so, does an EXPORT_SYMBOL_GPL() not suffice?

- This:

	+extern void (*perf_irq)(struct pt_regs *);

  should be in a header file.

I'll queue the patch up.  Fixups relative to this patch would be
appreciated, thanks.

More information about the Linuxppc-embedded mailing list