]> git.proxmox.com Git - mirror_ubuntu-bionic-kernel.git/commitdiff
powerpc/8xx: Remove macro that checks kernel address
authorChristophe Leroy <christophe.leroy@c-s.fr>
Wed, 12 Jul 2017 10:08:47 +0000 (12:08 +0200)
committerMichael Ellerman <mpe@ellerman.id.au>
Tue, 15 Aug 2017 12:55:52 +0000 (22:55 +1000)
The macro to check if an address is a kernel address or not is
not used anymore in DTLBmiss handler. It is used in ITLB miss handler
and in DTLB error handler. DTLB error handler is not a hot path, it
doesn't need such optimisation.

In order to simplify a following patch which will rework ITLB miss
handler, we remove the macros and reintroduce them inside the handler.

Signed-off-by: Christophe Leroy <christophe.leroy@c-s.fr>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
arch/powerpc/kernel/head_8xx.S

index 471936704261cd784236c960c3757b66ccf462f8..35058661288f0352e971a8eb928cf1e3b2d4e824 100644 (file)
        mtspr   spr, reg
 #endif
 
-/* Macro to test if an address is a kernel address */
 #if CONFIG_TASK_SIZE <= 0x80000000 && CONFIG_PAGE_OFFSET >= 0x80000000
-#define IS_KERNEL(tmp, addr)           \
-       andis.  tmp, addr, 0x8000       /* Address >= 0x80000000 */
-#define BRANCH_UNLESS_KERNEL(label)    beq     label
-#else
-#define IS_KERNEL(tmp, addr)           \
-       rlwinm  tmp, addr, 16, 16, 31;  \
-       cmpli   cr0, tmp, PAGE_OFFSET >> 16
-#define BRANCH_UNLESS_KERNEL(label)    blt     label
+/* By simply checking Address >= 0x80000000, we know if its a kernel address */
+#define SIMPLE_KERNEL_ADDRESS          1
 #endif
 
 
@@ -346,11 +339,20 @@ InstructionTLBMiss:
        mfcr    r3
 #endif
 #if defined(CONFIG_MODULES) || defined (CONFIG_DEBUG_PAGEALLOC)
-       IS_KERNEL(r11, r10)
+#ifdef SIMPLE_KERNEL_ADDRESS
+       andis.  r11, r10, 0x8000        /* Address >= 0x80000000 */
+#else
+       rlwinm  r11, r10, 16, 0xfff8
+       cmpli   cr0, r11, PAGE_OFFSET@h
+#endif
 #endif
        mfspr   r11, SPRN_M_TW  /* Get level 1 table */
 #if defined(CONFIG_MODULES) || defined (CONFIG_DEBUG_PAGEALLOC)
-       BRANCH_UNLESS_KERNEL(3f)
+#ifdef SIMPLE_KERNEL_ADDRESS
+       beq+    3f
+#else
+       blt+    3f
+#endif
        lis     r11, (swapper_pg_dir-PAGE_OFFSET)@ha
 3:
 #endif
@@ -704,9 +706,10 @@ FixupDAR:/* Entry point for dcbx workaround. */
        mtspr   SPRN_SPRG_SCRATCH2, r10
        /* fetch instruction from memory. */
        mfspr   r10, SPRN_SRR0
-       IS_KERNEL(r11, r10)
+       rlwinm  r11, r10, 16, 0xfff8
+       cmpli   cr0, r11, PAGE_OFFSET@h
        mfspr   r11, SPRN_M_TW  /* Get level 1 table */
-       BRANCH_UNLESS_KERNEL(3f)
+       blt+    3f
        rlwinm  r11, r10, 16, 0xfff8
 _ENTRY(FixupDAR_cmp)
        cmpli   cr7, r11, (PAGE_OFFSET + 0x1800000)@h