8187809: UseMembar should be set true and deprecate the flag
authorrehn
Fri, 10 Nov 2017 09:16:42 +0100
changeset 47821 0cd18aa4f7b6
parent 47820 1bc021ddeae0
child 47822 7437dc810834
8187809: UseMembar should be set true and deprecate the flag Reviewed-by: dcubed, sspitsyn, dholmes
src/hotspot/cpu/ppc/globals_ppc.hpp
src/hotspot/cpu/s390/globals_s390.hpp
src/hotspot/cpu/sparc/globals_sparc.hpp
src/hotspot/cpu/x86/globals_x86.hpp
src/hotspot/share/runtime/arguments.cpp
--- a/src/hotspot/cpu/ppc/globals_ppc.hpp	Thu Nov 09 12:43:13 2017 -0800
+++ b/src/hotspot/cpu/ppc/globals_ppc.hpp	Fri Nov 10 09:16:42 2017 +0100
@@ -69,7 +69,7 @@
 define_pd_global(bool, RewriteBytecodes,      true);
 define_pd_global(bool, RewriteFrequentPairs,  true);
 
-define_pd_global(bool, UseMembar,             false);
+define_pd_global(bool, UseMembar,             true);
 
 define_pd_global(bool, PreserveFramePointer,  false);
 
--- a/src/hotspot/cpu/s390/globals_s390.hpp	Thu Nov 09 12:43:13 2017 -0800
+++ b/src/hotspot/cpu/s390/globals_s390.hpp	Fri Nov 10 09:16:42 2017 +0100
@@ -71,7 +71,7 @@
 define_pd_global(bool, RewriteBytecodes,     true);
 define_pd_global(bool, RewriteFrequentPairs, true);
 
-define_pd_global(bool, UseMembar,            false);
+define_pd_global(bool, UseMembar,            true);
 
 define_pd_global(bool, PreserveFramePointer, false);
 
--- a/src/hotspot/cpu/sparc/globals_sparc.hpp	Thu Nov 09 12:43:13 2017 -0800
+++ b/src/hotspot/cpu/sparc/globals_sparc.hpp	Fri Nov 10 09:16:42 2017 +0100
@@ -74,7 +74,7 @@
 define_pd_global(bool, RewriteBytecodes,     true);
 define_pd_global(bool, RewriteFrequentPairs, true);
 
-define_pd_global(bool, UseMembar,            false);
+define_pd_global(bool, UseMembar,            true);
 
 define_pd_global(bool, PreserveFramePointer, false);
 
--- a/src/hotspot/cpu/x86/globals_x86.hpp	Thu Nov 09 12:43:13 2017 -0800
+++ b/src/hotspot/cpu/x86/globals_x86.hpp	Fri Nov 10 09:16:42 2017 +0100
@@ -84,11 +84,7 @@
 define_pd_global(bool, RewriteBytecodes,     true);
 define_pd_global(bool, RewriteFrequentPairs, true);
 
-#ifdef _ALLBSD_SOURCE
 define_pd_global(bool, UseMembar,            true);
-#else
-define_pd_global(bool, UseMembar,            false);
-#endif
 
 // GC Ergo Flags
 define_pd_global(size_t, CMSYoungGenPerWorker, 64*M);  // default max size of CMS young gen, per GC worker thread
--- a/src/hotspot/share/runtime/arguments.cpp	Thu Nov 09 12:43:13 2017 -0800
+++ b/src/hotspot/share/runtime/arguments.cpp	Fri Nov 10 09:16:42 2017 +0100
@@ -382,6 +382,7 @@
   { "MaxRAMFraction",               JDK_Version::jdk(10),  JDK_Version::undefined(), JDK_Version::undefined() },
   { "MinRAMFraction",               JDK_Version::jdk(10),  JDK_Version::undefined(), JDK_Version::undefined() },
   { "InitialRAMFraction",           JDK_Version::jdk(10),  JDK_Version::undefined(), JDK_Version::undefined() },
+  { "UseMembar",                    JDK_Version::jdk(10), JDK_Version::jdk(11), JDK_Version::jdk(12) },
   { "IgnoreUnverifiableClassesDuringDump", JDK_Version::jdk(10),  JDK_Version::undefined(), JDK_Version::undefined() },
 
   // --- Deprecated alias flags (see also aliased_jvm_flags) - sorted by obsolete_in then expired_in: