sparc32: use flushi when run-time patching in per_cpu_patch
authorSam Ravnborg <sam@ravnborg.org>
Sun, 20 May 2012 15:27:53 +0000 (17:27 +0200)
committerDavid S. Miller <davem@davemloft.net>
Sun, 20 May 2012 20:33:36 +0000 (13:33 -0700)
Davis S. Miller wrote:
"
The way we do that now is overkill.  We only needed to use the MMU
cache ops when we had sun4c around because sun4c lacked support for
the "flush" instruction.

But all sun4m and later chips have it so we can use it
unconditionally.

So in the per_cpu_patch() code, get rid of the cache ops invocation,
and instead execute a "flush %reg" after each of the instruction patch
assignments, where %reg is set to the address of the instruction that
was stored into.

Perhaps take the flushi() definition from asm/cacheflush_64.h and
place it into asm/cacheflush.h, then you can simply use that.
"

Implemented as per suggestion.
Moved run-time patching before we call paging_init(),
so helper methods in paging_init() may utilise run-time patching too.

Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
arch/sparc/include/asm/cacheflush.h
arch/sparc/include/asm/cacheflush_64.h
arch/sparc/kernel/setup_32.c

index 049168087b198fd37e5a144bc55e7682ef1ffc6f..f6c4839b83888220e4b621248824e334ed2d9afe 100644 (file)
@@ -1,5 +1,9 @@
 #ifndef ___ASM_SPARC_CACHEFLUSH_H
 #define ___ASM_SPARC_CACHEFLUSH_H
+
+/* flush addr - to allow use of self-modifying code */
+#define flushi(addr)   __asm__ __volatile__ ("flush %0" : : "r" (addr) : "memory")
+
 #if defined(__sparc__) && defined(__arch64__)
 #include <asm/cacheflush_64.h>
 #else
index 2efea2ff88b7d4571920b0dca517dfb4edb81470..301736d9e7a1c4e2392f851209d35b86b08c76bf 100644 (file)
@@ -8,9 +8,6 @@
 #include <linux/mm.h>
 
 /* Cache flush operations. */
-
-
-#define flushi(addr)   __asm__ __volatile__ ("flush %0" : : "r" (addr) : "memory")
 #define flushw_all()   __asm__ __volatile__("flushw")
 
 extern void __flushw_user(void);
index d65b5a1c220917fb50d039cf5d264507dd60f29b..c052313f4dc578700193abaf5fc6e9b127e66e0a 100644 (file)
@@ -227,16 +227,14 @@ static void __init per_cpu_patch(void)
                        prom_halt();
                }
                *(unsigned int *) (addr + 0) = insns[0];
+               flushi(addr + 0);
                *(unsigned int *) (addr + 4) = insns[1];
+               flushi(addr + 4);
                *(unsigned int *) (addr + 8) = insns[2];
+               flushi(addr + 8);
 
                p++;
        }
-#ifdef CONFIG_SMP
-       local_ops->cache_all();
-#else
-       sparc32_cachetlb_ops->cache_all();
-#endif
 }
 
 enum sparc_cpu sparc_cpu_model;
@@ -340,13 +338,11 @@ void __init setup_arch(char **cmdline_p)
        init_mm.context = (unsigned long) NO_CONTEXT;
        init_task.thread.kregs = &fake_swapper_regs;
 
-       paging_init();
-
-       /* Now that we have the cache ops hooked up, we can patch
-        * instructions.
-        */
+       /* Run-time patch instructions to match the cpu model */
        per_cpu_patch();
 
+       paging_init();
+
        smp_setup_cpu_possible_map();
 }