ARC: entry.S: common'ize scrtach reg freeup in intr + exceptions
authorVineet Gupta <vgupta@synopsys.com>
Mon, 13 Oct 2014 09:42:25 +0000 (15:12 +0530)
committerVineet Gupta <vgupta@synopsys.com>
Fri, 19 Jun 2015 12:39:34 +0000 (18:09 +0530)
Signed-off-by: Vineet Gupta <vgupta@synopsys.com>
arch/arc/include/asm/entry.h
arch/arc/kernel/entry.S

index 884081099f800fd6b4ba133bc1f1746084845c49..9e844ab5eede7aa13a959a4e9f3b93c40fcc0fc2 100644 (file)
 .endm
 
 /*--------------------------------------------------------------
- * For early Exception Prologue, a core reg is temporarily needed to
+ * For early Exception/ISR Prologue, a core reg is temporarily needed to
  * code the rest of prolog (stack switching). This is done by stashing
  * it to memory (non-SMP case) or SCRATCH0 Aux Reg (SMP).
  *
  * Before saving the full regfile - this reg is restored back, only
  * to be saved again on kernel mode stack, as part of pt_regs.
  *-------------------------------------------------------------*/
-.macro EXCPN_PROLOG_FREEUP_REG reg
+.macro PROLOG_FREEUP_REG       reg, mem
 #ifdef CONFIG_SMP
        sr  \reg, [ARC_REG_SCRATCH_DATA0]
 #else
-       st  \reg, [@ex_saved_reg1]
+       st  \reg, [\mem]
 #endif
 .endm
 
-.macro EXCPN_PROLOG_RESTORE_REG        reg
+.macro PROLOG_RESTORE_REG      reg, mem
 #ifdef CONFIG_SMP
        lr  \reg, [ARC_REG_SCRATCH_DATA0]
 #else
-       ld  \reg, [@ex_saved_reg1]
+       ld  \reg, [\mem]
 #endif
 .endm
 
 .macro EXCEPTION_PROLOGUE
 
        /* Need at least 1 reg to code the early exception prologue */
-       EXCPN_PROLOG_FREEUP_REG r9
+       PROLOG_FREEUP_REG r9, @ex_saved_reg1
 
        /* U/K mode at time of exception (stack not switched if already K) */
        lr  r9, [erstatus]
        st      r0, [sp, 4]    /* orig_r0, needed only for sys calls */
 
        /* Restore r9 used to code the early prologue */
-       EXCPN_PROLOG_RESTORE_REG  r9
+       PROLOG_RESTORE_REG  r9, @ex_saved_reg1
 
        SAVE_R0_TO_R12
        PUSH    gp
  *-------------------------------------------------------------*/
 .macro SAVE_ALL_INT1
 
-       /* restore original r9 to be saved as part of reg-file */
-#ifdef CONFIG_SMP
-       lr  r9, [ARC_REG_SCRATCH_DATA0]
-#else
-       ld  r9, [@int1_saved_reg]
-#endif
+       /* restore original r9 */
+       PROLOG_RESTORE_REG  r9, @int1_saved_reg
 
        /* now we are ready to save the remaining context :) */
        st      event_IRQ1, [sp, 8]    /* Dummy ECR */
 
 .macro SAVE_ALL_INT2
 
-       /* TODO-vineetg: SMP we can't use global nor can we use
-       *   SCRATCH0 as we do for int1 because while int1 is using
-       *   it, int2 can come
-       */
-       /* retsore original r9 , saved in sys_saved_r9 */
-       ld  r9, [@int2_saved_reg]
+       /*
+        * In SMP we can't use mem nor can we use SCRARCH_DATA0
+        * as we do for int1 because int2 can clobber it
+        * Hence 2 levels of intr are NOT allowed in SMP (by Kconfig)
+        */
+       /* restore original r9 */
+       PROLOG_RESTORE_REG r9, @int2_saved_reg
 
        /* now we are ready to save the remaining context :) */
        st      event_IRQ2, [sp, 8]    /* Dummy ECR */
index d868289c5a26f74e780ee2e59ff6174987d68d68..13b14b8dcd8dfd32fb568f6ac2bbcaa5214a42ae 100644 (file)
@@ -186,9 +186,8 @@ reserved:           ; processor restart
 ; ---------------------------------------------
 ENTRY(handle_interrupt_level2)
 
-       ; TODO-vineetg for SMP this wont work
        ; free up r9 as scratchpad
-       st  r9, [@int2_saved_reg]
+       PROLOG_FREEUP_REG r9, @int2_saved_reg
 
        ;Which mode (user/kernel) was the system in when intr occured
        lr  r9, [status32_l2]
@@ -234,12 +233,7 @@ END(handle_interrupt_level2)
 ; ---------------------------------------------
 ENTRY(handle_interrupt_level1)
 
-       /* free up r9 as scratchpad */
-#ifdef CONFIG_SMP
-       sr  r9, [ARC_REG_SCRATCH_DATA0]
-#else
-       st   r9, [@int1_saved_reg]
-#endif
+       PROLOG_FREEUP_REG r9, @int1_saved_reg
 
        ;Which mode (user/kernel) was the system in when intr occured
        lr  r9, [status32_l1]