[src/trunk]: src/sys/arch g/c vm_page_zero_enable

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

details:   https://anonhg.NetBSD.org/src/rev/4a14cc3a75cc
branches:  trunk
changeset: 934546:4a14cc3a75cc
user:      ad <ad%NetBSD.org@localhost>
date:      Sat Jun 13 20:01:27 2020 +0000

description:
g/c vm_page_zero_enable

diffstat:

 sys/arch/arc/arc/p_acer_pica_61.c   |  13 ++-----------
 sys/arch/arc/arc/p_dti_arcstation.c |  12 ++----------
 sys/arch/arc/arc/p_dti_tyne.c       |  12 ++----------
 sys/arch/arc/arc/p_sni_rm200pci.c   |  12 ++----------
 sys/arch/sparc/sparc/cpu.c          |  10 ++--------
 sys/arch/x86/x86/cpu.c              |   7 ++-----
 6 files changed, 12 insertions(+), 54 deletions(-)

diffs (192 lines):

diff -r 02557a84f62f -r 4a14cc3a75cc sys/arch/arc/arc/p_acer_pica_61.c
--- a/sys/arch/arc/arc/p_acer_pica_61.c Sat Jun 13 19:55:39 2020 +0000
+++ b/sys/arch/arc/arc/p_acer_pica_61.c Sat Jun 13 20:01:27 2020 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: p_acer_pica_61.c,v 1.12 2011/03/06 14:58:42 tsutsui Exp $      */
+/*     $NetBSD: p_acer_pica_61.c,v 1.13 2020/06/13 20:01:27 ad Exp $   */
 /*     $OpenBSD: picabus.c,v 1.11 1999/01/11 05:11:10 millert Exp $    */
 
 /*
@@ -30,7 +30,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: p_acer_pica_61.c,v 1.12 2011/03/06 14:58:42 tsutsui Exp $");
+__KERNEL_RCSID(0, "$NetBSD: p_acer_pica_61.c,v 1.13 2020/06/13 20:01:27 ad Exp $");
 
 #include <sys/param.h>
 #include <uvm/uvm.h>
@@ -87,15 +87,6 @@
 p_acer_pica_61_init(void)
 {
 
-       /*
-        * PICA-61 has PC-style coherent(?) 128KB L2 cache,
-        * and mips_L2CachePresent == 0 on this machine.
-        *
-        * if page zero in the idle loop is enabled,
-        * commands dump core due to incoherent cache.
-        */
-       vm_page_zero_enable = false; /* XXX - should be enabled */
-
        c_magnum_init();
 
        /* chipset-dependent jazzio bus configuration */
diff -r 02557a84f62f -r 4a14cc3a75cc sys/arch/arc/arc/p_dti_arcstation.c
--- a/sys/arch/arc/arc/p_dti_arcstation.c       Sat Jun 13 19:55:39 2020 +0000
+++ b/sys/arch/arc/arc/p_dti_arcstation.c       Sat Jun 13 20:01:27 2020 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: p_dti_arcstation.c,v 1.18 2019/11/10 21:16:22 chs Exp $        */
+/*     $NetBSD: p_dti_arcstation.c,v 1.19 2020/06/13 20:01:27 ad Exp $ */
 /*     $OpenBSD: machdep.c,v 1.36 1999/05/22 21:22:19 weingart Exp $   */
 
 /*
@@ -39,7 +39,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: p_dti_arcstation.c,v 1.18 2019/11/10 21:16:22 chs Exp $");
+__KERNEL_RCSID(0, "$NetBSD: p_dti_arcstation.c,v 1.19 2020/06/13 20:01:27 ad Exp $");
 
 #define __INTR_PRIVATE
 #include <sys/param.h>
@@ -205,14 +205,6 @@
        ipl_sr_map = dti_arcstation_ipl_sr_map;
 
        /*
-        * XXX - should be enabled, if tested.
-        *
-        * We use safe default for now, because this platform is untested.
-        * In other words, the following may not be needed at all.
-        */
-       vm_page_zero_enable = false;
-
-       /*
         * Initialize I/O address offset
         */
        arc_bus_space_init(&arc_bus_io, "rpc44isaio",
diff -r 02557a84f62f -r 4a14cc3a75cc sys/arch/arc/arc/p_dti_tyne.c
--- a/sys/arch/arc/arc/p_dti_tyne.c     Sat Jun 13 19:55:39 2020 +0000
+++ b/sys/arch/arc/arc/p_dti_tyne.c     Sat Jun 13 20:01:27 2020 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: p_dti_tyne.c,v 1.19 2011/02/20 07:52:42 matt Exp $     */
+/*     $NetBSD: p_dti_tyne.c,v 1.20 2020/06/13 20:01:27 ad Exp $       */
 /*     $OpenBSD: machdep.c,v 1.36 1999/05/22 21:22:19 weingart Exp $   */
 
 /*
@@ -40,7 +40,7 @@
 
 #define __INTR_PRIVATE
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: p_dti_tyne.c,v 1.19 2011/02/20 07:52:42 matt Exp $");
+__KERNEL_RCSID(0, "$NetBSD: p_dti_tyne.c,v 1.20 2020/06/13 20:01:27 ad Exp $");
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -200,14 +200,6 @@
        ipl_sr_map = dti_tyne_ipl_sr_map;
 
        /*
-        * XXX - should be enabled, if tested.
-        *
-        * We use safe default for now, because this platform is untested.
-        * In other words, the following may not be needed at all.
-        */
-       vm_page_zero_enable = false;
-
-       /*
         * Initialize I/O address offset
         */
        arc_bus_space_init(&arc_bus_io, "tyneisaio",
diff -r 02557a84f62f -r 4a14cc3a75cc sys/arch/arc/arc/p_sni_rm200pci.c
--- a/sys/arch/arc/arc/p_sni_rm200pci.c Sat Jun 13 19:55:39 2020 +0000
+++ b/sys/arch/arc/arc/p_sni_rm200pci.c Sat Jun 13 20:01:27 2020 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: p_sni_rm200pci.c,v 1.15 2011/02/20 07:52:42 matt Exp $ */
+/*     $NetBSD: p_sni_rm200pci.c,v 1.16 2020/06/13 20:01:27 ad Exp $   */
 /*     $OpenBSD: machdep.c,v 1.36 1999/05/22 21:22:19 weingart Exp $   */
 
 /*
@@ -40,7 +40,7 @@
 
 #define __INTR_PRIVATE
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: p_sni_rm200pci.c,v 1.15 2011/02/20 07:52:42 matt Exp $");
+__KERNEL_RCSID(0, "$NetBSD: p_sni_rm200pci.c,v 1.16 2020/06/13 20:01:27 ad Exp $");
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -116,14 +116,6 @@
        ipl_sr_map = sni_rm200pci_ipl_sr_map;
 
        /*
-        * XXX - should be enabled, if tested.
-        *
-        * We use safe default for now, because this platform is untested.
-        * In other words, the following may not be needed at all.
-        */
-       vm_page_zero_enable = false;
-
-       /*
         * Initialize I/O address offset
         */
        arc_init_wired_map();
diff -r 02557a84f62f -r 4a14cc3a75cc sys/arch/sparc/sparc/cpu.c
--- a/sys/arch/sparc/sparc/cpu.c        Sat Jun 13 19:55:39 2020 +0000
+++ b/sys/arch/sparc/sparc/cpu.c        Sat Jun 13 20:01:27 2020 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: cpu.c,v 1.255 2019/10/01 18:00:07 chs Exp $ */
+/*     $NetBSD: cpu.c,v 1.256 2020/06/13 20:01:27 ad Exp $ */
 
 /*
  * Copyright (c) 1996
@@ -52,7 +52,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: cpu.c,v 1.255 2019/10/01 18:00:07 chs Exp $");
+__KERNEL_RCSID(0, "$NetBSD: cpu.c,v 1.256 2020/06/13 20:01:27 ad Exp $");
 
 #include "opt_multiprocessor.h"
 #include "opt_lockdebug.h"
@@ -1500,12 +1500,6 @@
 void
 cpumatch_ms1(struct cpu_info *sc, struct module_info *mp, int node)
 {
-
-       /*
-        * Turn off page zeroing in the idle loop; an unidentified
-        * bug causes (very sporadic) user process corruption.
-        */
-       vm_page_zero_enable = 0;
 }
 
 void
diff -r 02557a84f62f -r 4a14cc3a75cc sys/arch/x86/x86/cpu.c
--- a/sys/arch/x86/x86/cpu.c    Sat Jun 13 19:55:39 2020 +0000
+++ b/sys/arch/x86/x86/cpu.c    Sat Jun 13 20:01:27 2020 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: cpu.c,v 1.192 2020/05/21 21:12:30 ad Exp $     */
+/*     $NetBSD: cpu.c,v 1.193 2020/06/13 20:01:27 ad Exp $     */
 
 /*
  * Copyright (c) 2000-2020 NetBSD Foundation, Inc.
@@ -62,7 +62,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: cpu.c,v 1.192 2020/05/21 21:12:30 ad Exp $");
+__KERNEL_RCSID(0, "$NetBSD: cpu.c,v 1.193 2020/06/13 20:01:27 ad Exp $");
 
 #include "opt_ddb.h"
 #include "opt_mpbios.h"                /* for MPDEBUG */
@@ -768,9 +768,6 @@
 
        /* Now that we know about the TSC, attach the timecounter. */
        tsc_tc_init();
-
-       /* Enable zeroing of pages in the idle loop if we have SSE2. */
-       vm_page_zero_enable = false; /* ((cpu_feature[0] & CPUID_SSE2) != 0); */
 }
 #endif