[src/trunk]: src/sys/arch/sparc64/sparc64 smp_dcache_flush_page_cpuset(): Av...

[palle <palle%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/55f705579c8e
branches:  trunk
changeset: 335358:55f705579c8e
user:      palle <palle%NetBSD.org@localhost>
date:      Sun Jan 04 15:24:18 2015 +0000

description:
smp_dcache_flush_page_cpuset():  Avoid run-time check for cpu type/implementation by installing correct function in sparc64_ipi_init()

diffstat:

 sys/arch/sparc64/sparc64/ipifuncs.c |  33 +++++++++++++++++----------------
 1 files changed, 17 insertions(+), 16 deletions(-)

diffs (72 lines):

diff -r 13267e40b4a5 -r 55f705579c8e sys/arch/sparc64/sparc64/ipifuncs.c
--- a/sys/arch/sparc64/sparc64/ipifuncs.c       Sun Jan 04 14:23:37 2015 +0000
+++ b/sys/arch/sparc64/sparc64/ipifuncs.c       Sun Jan 04 15:24:18 2015 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: ipifuncs.c,v 1.54 2015/01/03 11:22:14 palle Exp $ */
+/*     $NetBSD: ipifuncs.c,v 1.55 2015/01/04 15:24:18 palle Exp $ */
 
 /*-
  * Copyright (c) 2004 The NetBSD Foundation, Inc.
@@ -27,7 +27,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: ipifuncs.c,v 1.54 2015/01/03 11:22:14 palle Exp $");
+__KERNEL_RCSID(0, "$NetBSD: ipifuncs.c,v 1.55 2015/01/04 15:24:18 palle Exp $");
 
 #include "opt_ddb.h"
 
@@ -84,8 +84,9 @@
 void   sparc64_ipi_blast_dcache(void *, void *);
 void   sparc64_ipi_ccall(void *, void *);
 
-/* Function pointer for use in smp_tlb_flush() - setup in sparc64_ipi_init() */
+/* Function pointers to be setup in sparc64_ipi_init() */
 static ipifunc_t smp_tlb_flush_pte_func = NULL;
+static ipifunc_t sparc64_ipi_dcache_flush_page_func = NULL;
 
 /*
  * Process cpu stop-self event.
@@ -180,12 +181,21 @@
         * Prepare cpu type dependent function pointers
         */
 
-       if (CPU_ISSUN4V)
+       if (CPU_ISSUN4V) {
                smp_tlb_flush_pte_func = sparc64_ipi_flush_pte_sun4v;
-       else if (CPU_IS_USIII_UP())
+               sparc64_ipi_dcache_flush_page_func =
+                   sparc64_ipi_dcache_flush_page_sun4v;
+       }
+       else if (CPU_IS_USIII_UP()) {
                smp_tlb_flush_pte_func = sparc64_ipi_flush_pte_usiii;
-       else
+               sparc64_ipi_dcache_flush_page_func =
+                   sparc64_ipi_dcache_flush_page_usiii;
+       }
+       else {
                smp_tlb_flush_pte_func = sparc64_ipi_flush_pte_us;
+               sparc64_ipi_dcache_flush_page_func =
+                   sparc64_ipi_dcache_flush_page_us;
+       }
 
        if (CPU_ISSUN4V)
                sparc64_send_ipi = sparc64_send_ipi_sun4v;
@@ -476,16 +486,7 @@
 void
 smp_dcache_flush_page_cpuset(paddr_t pa, sparc64_cpuset_t activecpus)
 {
-       ipifunc_t func;
-
-       if (CPU_ISSUN4V)
-               func = sparc64_ipi_dcache_flush_page_sun4v;
-       else if (CPU_IS_USIII_UP())
-               func = sparc64_ipi_dcache_flush_page_usiii;
-       else
-               func = sparc64_ipi_dcache_flush_page_us;
-
-       sparc64_multicast_ipi(activecpus, func, pa, dcache_line_size);
+       sparc64_multicast_ipi(activecpus, sparc64_ipi_dcache_flush_page_func, pa, dcache_line_size);
        sp_dcache_flush_page(pa);
 }