[PATCH] powerpc: reduce oprofile/common.c differences
authorStephen Rothwell <sfr@canb.auug.org.au>
Mon, 19 Sep 2005 13:28:21 +0000 (23:28 +1000)
committerPaul Mackerras <paulus@samba.org>
Wed, 21 Sep 2005 09:21:08 +0000 (19:21 +1000)
Rename and slightly modify {request,free}_perfmon_irq in the ppc code.

Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au>
Signed-off-by: Paul Mackerras <paulus@samba.org>
arch/powerpc/oprofile/common.c
arch/ppc/kernel/perfmon.c
include/asm-ppc/perfmon.h

index 486314a0defd0a250057354c90e43ca55dabf711..88b4118fd0c509c31a6f3f7f7ebb6390657b9624 100644 (file)
@@ -46,19 +46,12 @@ static void op_handle_interrupt(struct pt_regs *regs)
 
 static int op_powerpc_setup(void)
 {
 
 static int op_powerpc_setup(void)
 {
-#ifdef __powerpc64__
        int err;
 
        /* Grab the hardware */
        err = reserve_pmc_hardware(op_handle_interrupt);
        if (err)
                return err;
        int err;
 
        /* Grab the hardware */
        err = reserve_pmc_hardware(op_handle_interrupt);
        if (err)
                return err;
-#else /* __powerpc64__ */
-       /* Install our interrupt handler into the existing hook.  */
-       if (request_perfmon_irq(&op_handle_interrupt))
-               return -EBUSY;
-       mb();
-#endif /* __powerpc64__ */
 
        /* Pre-compute the values to stuff in the hardware registers.  */
        model->reg_setup(ctr, &sys, model->num_counters);
 
        /* Pre-compute the values to stuff in the hardware registers.  */
        model->reg_setup(ctr, &sys, model->num_counters);
@@ -78,13 +71,7 @@ static int op_powerpc_setup(void)
 
 static void op_powerpc_shutdown(void)
 {
 
 static void op_powerpc_shutdown(void)
 {
-#ifdef __powerpc64__
        release_pmc_hardware();
        release_pmc_hardware();
-#else /* __powerpc64__ */
-       mb();
-       /* Remove our interrupt handler. We may be removing this module. */
-       free_perfmon_irq();
-#endif /* __powerpc64__ */
 }
 
 static void op_powerpc_cpu_start(void *dummy)
 }
 
 static void op_powerpc_cpu_start(void *dummy)
index fa1dad96b8309f1b44a4a40aef2748a36b4cf3e0..91e2786ea097536455a25eddefcfba1d898dc4ca 100644 (file)
@@ -61,7 +61,7 @@ void (*perf_irq)(struct pt_regs *) = dummy_perf;
 
 /* Grab the interrupt, if it's free.
  * Returns 0 on success, -1 if the interrupt is taken already */
 
 /* Grab the interrupt, if it's free.
  * Returns 0 on success, -1 if the interrupt is taken already */
-int request_perfmon_irq(void (*handler)(struct pt_regs *))
+int reserve_pmc_hardware(void (*handler)(struct pt_regs *))
 {
        int err = 0;
 
 {
        int err = 0;
 
@@ -71,7 +71,7 @@ int request_perfmon_irq(void (*handler)(struct pt_regs *))
                perf_irq = handler;
        else {
                pr_info("perfmon irq already handled by %p\n", perf_irq);
                perf_irq = handler;
        else {
                pr_info("perfmon irq already handled by %p\n", perf_irq);
-               err = -1;
+               err = -EBUSY;
        }
 
        spin_unlock(&perfmon_lock);
        }
 
        spin_unlock(&perfmon_lock);
@@ -79,7 +79,7 @@ int request_perfmon_irq(void (*handler)(struct pt_regs *))
        return err;
 }
 
        return err;
 }
 
-void free_perfmon_irq(void)
+void release_pmc_hardware(void)
 {
        spin_lock(&perfmon_lock);
 
 {
        spin_lock(&perfmon_lock);
 
@@ -89,5 +89,5 @@ void free_perfmon_irq(void)
 }
 
 EXPORT_SYMBOL(perf_irq);
 }
 
 EXPORT_SYMBOL(perf_irq);
-EXPORT_SYMBOL(request_perfmon_irq);
-EXPORT_SYMBOL(free_perfmon_irq);
+EXPORT_SYMBOL(reserve_pmc_hardware);
+EXPORT_SYMBOL(release_pmc_hardware);
index e9692a603cff170c78fa1d7ce52c1f1b523f7f33..2ae031594a4e67c06f8873b5735d72357b7909b9 100644 (file)
@@ -3,8 +3,8 @@
 
 extern void (*perf_irq)(struct pt_regs *);
 
 
 extern void (*perf_irq)(struct pt_regs *);
 
-int request_perfmon_irq(void (*handler)(struct pt_regs *));
-void free_perfmon_irq(void);
+int reserve_pmc_hardware(void (*handler)(struct pt_regs *));
+void release_pmc_hardware(void);
 
 #ifdef CONFIG_FSL_BOOKE
 void init_pmc_stop(int ctr);
 
 #ifdef CONFIG_FSL_BOOKE
 void init_pmc_stop(int ctr);