Fri, 20 Sep 2013 10:53:28 +0200
8024974: Incorrect use of GC_locker::is_active()
Summary: SymbolTable and StringTable can make calls to GC_locker::is_active() outside a safepoint. This isn't safe because the GC_locker active state (lock count) is only updated at a safepoint and only remains valid as long as _needs_gc is true. However, outside a safepoint_needs_gc can change to false at any time, which makes it impossible to do a correct call to is_active() in that context. In this case these calls can just be removed since the input argument to basic_add() should never be on the heap and so there's no need to check the GC_locker state. This change also adjusts the assert() in is_active() to makes sure all calls to this function are always done under a safepoint.
Reviewed-by: brutisso, dcubed
Contributed-by: per.liden@oracle.com
duke@435 | 1 | # |
dholmes@5417 | 2 | # Copyright (c) 1997, 2013, Oracle and/or its affiliates. All rights reserved. |
duke@435 | 3 | # DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
duke@435 | 4 | # |
duke@435 | 5 | # This code is free software; you can redistribute it and/or modify it |
duke@435 | 6 | # under the terms of the GNU General Public License version 2 only, as |
duke@435 | 7 | # published by the Free Software Foundation. |
duke@435 | 8 | # |
duke@435 | 9 | # This code is distributed in the hope that it will be useful, but WITHOUT |
duke@435 | 10 | # ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
duke@435 | 11 | # FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
duke@435 | 12 | # version 2 for more details (a copy is included in the LICENSE file that |
duke@435 | 13 | # accompanied this code). |
duke@435 | 14 | # |
duke@435 | 15 | # You should have received a copy of the GNU General Public License version |
duke@435 | 16 | # 2 along with this work; if not, write to the Free Software Foundation, |
duke@435 | 17 | # Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
duke@435 | 18 | # |
trims@1907 | 19 | # Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
trims@1907 | 20 | # or visit www.oracle.com if you need additional information or have any |
trims@1907 | 21 | # questions. |
duke@435 | 22 | # |
duke@435 | 23 | # |
duke@435 | 24 | |
duke@435 | 25 | # Generic compiler settings |
erikj@3600 | 26 | !if "x$(CXX)" == "x" |
erikj@3518 | 27 | CXX=cl.exe |
erikj@3600 | 28 | !endif |
duke@435 | 29 | |
erikj@3518 | 30 | # CXX Flags: (these vary slightly from VC6->VS2003->VS2005 compilers) |
duke@435 | 31 | # /nologo Supress copyright message at every cl.exe startup |
duke@435 | 32 | # /W3 Warning level 3 |
duke@435 | 33 | # /Zi Include debugging information |
duke@435 | 34 | # /WX Treat any warning error as a fatal error |
tbell@862 | 35 | # /MD Use dynamic multi-threaded runtime (msvcrt.dll or msvc*NN.dll) |
duke@435 | 36 | # /MTd Use static multi-threaded runtime debug versions |
duke@435 | 37 | # /O1 Optimize for size (/Os), skips /Oi |
duke@435 | 38 | # /O2 Optimize for speed (/Ot), adds /Oi to /O1 |
duke@435 | 39 | # /Ox Old "all optimizations flag" for VC6 (in /O1) |
duke@435 | 40 | # /Oy Use frame pointer register as GP reg (in /Ox and /O1) |
duke@435 | 41 | # /GF Merge string constants and put in read-only memory (in /O1) |
duke@435 | 42 | # /Gy Func level link (in /O1, allows for link-time func ordering) |
duke@435 | 43 | # /Gs Inserts stack probes (in /O1) |
duke@435 | 44 | # /GS Inserts security stack checks in some functions (VS2005 default) |
duke@435 | 45 | # /Oi Use intrinsics (in /O2) |
duke@435 | 46 | # /Od Disable all optimizations |
duke@435 | 47 | # |
duke@435 | 48 | # NOTE: Normally following any of the above with a '-' will turn off that flag |
sbohne@492 | 49 | # |
sbohne@492 | 50 | # 6655385: For VS2003/2005 we now specify /Oy- (disable frame pointer |
sbohne@492 | 51 | # omission.) This has little to no effect on performance while vastly |
sbohne@492 | 52 | # improving the quality of crash log stack traces involving jvm.dll. |
duke@435 | 53 | |
duke@435 | 54 | # These are always used in all compiles |
sla@5046 | 55 | CXX_FLAGS=$(EXTRA_CFLAGS) /nologo /W3 /WX |
duke@435 | 56 | |
dcubed@3724 | 57 | # Let's add debug information when Full Debug Symbols is enabled |
dcubed@3724 | 58 | !if "$(ENABLE_FULL_DEBUG_SYMBOLS)" == "1" |
erikj@3518 | 59 | CXX_FLAGS=$(CXX_FLAGS) /Zi |
dcubed@3724 | 60 | !endif |
duke@435 | 61 | |
duke@435 | 62 | # Based on BUILDARCH we add some flags and select the default compiler name |
duke@435 | 63 | !if "$(BUILDARCH)" == "ia64" |
duke@435 | 64 | MACHINE=IA64 |
duke@435 | 65 | DEFAULT_COMPILER_NAME=VS2003 |
erikj@3518 | 66 | CXX_FLAGS=$(CXX_FLAGS) /D "CC_INTERP" /D "_LP64" /D "IA64" |
duke@435 | 67 | !endif |
duke@435 | 68 | |
duke@435 | 69 | !if "$(BUILDARCH)" == "amd64" |
duke@435 | 70 | MACHINE=AMD64 |
duke@435 | 71 | DEFAULT_COMPILER_NAME=VS2005 |
erikj@3518 | 72 | CXX_FLAGS=$(CXX_FLAGS) /D "_LP64" /D "AMD64" |
duke@435 | 73 | LP64=1 |
duke@435 | 74 | !endif |
duke@435 | 75 | |
duke@435 | 76 | !if "$(BUILDARCH)" == "i486" |
duke@435 | 77 | MACHINE=I386 |
duke@435 | 78 | DEFAULT_COMPILER_NAME=VS2003 |
erikj@3518 | 79 | CXX_FLAGS=$(CXX_FLAGS) /D "IA32" |
duke@435 | 80 | !endif |
duke@435 | 81 | |
duke@435 | 82 | # Sanity check, this is the default if not amd64, ia64, or i486 |
duke@435 | 83 | !ifndef DEFAULT_COMPILER_NAME |
erikj@3518 | 84 | CXX=ARCH_ERROR |
duke@435 | 85 | !endif |
duke@435 | 86 | |
erikj@3518 | 87 | CXX_FLAGS=$(CXX_FLAGS) /D "WIN32" /D "_WINDOWS" |
sla@2369 | 88 | # Must specify this for sharedRuntimeTrig.cpp |
erikj@3518 | 89 | CXX_FLAGS=$(CXX_FLAGS) /D "VM_LITTLE_ENDIAN" |
sla@2369 | 90 | |
sla@2369 | 91 | # Used for platform dispatching |
erikj@3518 | 92 | CXX_FLAGS=$(CXX_FLAGS) /D TARGET_OS_FAMILY_windows |
erikj@3518 | 93 | CXX_FLAGS=$(CXX_FLAGS) /D TARGET_ARCH_$(Platform_arch) |
erikj@3518 | 94 | CXX_FLAGS=$(CXX_FLAGS) /D TARGET_ARCH_MODEL_$(Platform_arch_model) |
erikj@3518 | 95 | CXX_FLAGS=$(CXX_FLAGS) /D TARGET_OS_ARCH_windows_$(Platform_arch) |
erikj@3518 | 96 | CXX_FLAGS=$(CXX_FLAGS) /D TARGET_OS_ARCH_MODEL_windows_$(Platform_arch_model) |
erikj@3518 | 97 | CXX_FLAGS=$(CXX_FLAGS) /D TARGET_COMPILER_visCPP |
sla@2369 | 98 | |
sla@2369 | 99 | |
tbell@862 | 100 | # MSC_VER is a 4 digit number that tells us what compiler is being used |
tbell@862 | 101 | # and is generated when the local.make file is created by build.make |
tbell@862 | 102 | # via the script get_msc_ver.sh |
tbell@862 | 103 | # |
duke@435 | 104 | # If MSC_VER is set, it overrides the above default setting. |
duke@435 | 105 | # But it should be set. |
duke@435 | 106 | # Possible values: |
duke@435 | 107 | # 1200 is for VC6 |
duke@435 | 108 | # 1300 and 1310 is VS2003 or VC7 |
duke@435 | 109 | # 1399 is our fake number for the VS2005 compiler that really isn't 1400 |
duke@435 | 110 | # 1400 is for VS2005 |
tbell@862 | 111 | # 1500 is for VS2008 |
prr@1840 | 112 | # 1600 is for VS2010 |
dholmes@5417 | 113 | # 1700 is for VS2012 |
duke@435 | 114 | # Do not confuse this MSC_VER with the predefined macro _MSC_VER that the |
duke@435 | 115 | # compiler provides, when MSC_VER==1399, _MSC_VER will be 1400. |
duke@435 | 116 | # Normally they are the same, but a pre-release of the VS2005 compilers |
duke@435 | 117 | # in the Windows 64bit Platform SDK said it was 1400 when it was really |
duke@435 | 118 | # closer to VS2003 in terms of option spellings, so we use 1399 for that |
duke@435 | 119 | # 1400 version that really isn't 1400. |
tbell@862 | 120 | # See the file get_msc_ver.sh for more info. |
duke@435 | 121 | !if "x$(MSC_VER)" == "x" |
duke@435 | 122 | COMPILER_NAME=$(DEFAULT_COMPILER_NAME) |
duke@435 | 123 | !else |
duke@435 | 124 | !if "$(MSC_VER)" == "1200" |
duke@435 | 125 | COMPILER_NAME=VC6 |
duke@435 | 126 | !endif |
duke@435 | 127 | !if "$(MSC_VER)" == "1300" |
duke@435 | 128 | COMPILER_NAME=VS2003 |
duke@435 | 129 | !endif |
duke@435 | 130 | !if "$(MSC_VER)" == "1310" |
duke@435 | 131 | COMPILER_NAME=VS2003 |
duke@435 | 132 | !endif |
duke@435 | 133 | !if "$(MSC_VER)" == "1399" |
duke@435 | 134 | # Compiler might say 1400, but if it's 14.00.30701, it isn't really VS2005 |
duke@435 | 135 | COMPILER_NAME=VS2003 |
duke@435 | 136 | !endif |
duke@435 | 137 | !if "$(MSC_VER)" == "1400" |
duke@435 | 138 | COMPILER_NAME=VS2005 |
duke@435 | 139 | !endif |
tbell@862 | 140 | !if "$(MSC_VER)" == "1500" |
tbell@862 | 141 | COMPILER_NAME=VS2008 |
tbell@862 | 142 | !endif |
prr@1840 | 143 | !if "$(MSC_VER)" == "1600" |
prr@1840 | 144 | COMPILER_NAME=VS2010 |
prr@1840 | 145 | !endif |
dholmes@5417 | 146 | !if "$(MSC_VER)" == "1700" |
dholmes@5417 | 147 | COMPILER_NAME=VS2012 |
dholmes@5417 | 148 | !endif |
duke@435 | 149 | !endif |
duke@435 | 150 | |
duke@435 | 151 | # By default, we do not want to use the debug version of the msvcrt.dll file |
duke@435 | 152 | # but if MFC_DEBUG is defined in the environment it will be used. |
duke@435 | 153 | MS_RUNTIME_OPTION = /MD |
duke@435 | 154 | !if "$(MFC_DEBUG)" == "true" |
duke@435 | 155 | MS_RUNTIME_OPTION = /MTd /D "_DEBUG" |
duke@435 | 156 | !endif |
duke@435 | 157 | |
dholmes@5417 | 158 | # VS2012 and later won't work with: |
dholmes@5417 | 159 | # /D _STATIC_CPPLIB /D _DISABLE_DEPRECATE_STATIC_CPPLIB |
dholmes@5417 | 160 | !if "$(MSC_VER)" < "1700" |
duke@435 | 161 | # Always add the _STATIC_CPPLIB flag |
zgu@2396 | 162 | STATIC_CPPLIB_OPTION = /D _STATIC_CPPLIB /D _DISABLE_DEPRECATE_STATIC_CPPLIB |
duke@435 | 163 | MS_RUNTIME_OPTION = $(MS_RUNTIME_OPTION) $(STATIC_CPPLIB_OPTION) |
dholmes@5417 | 164 | !endif |
erikj@3518 | 165 | CXX_FLAGS=$(CXX_FLAGS) $(MS_RUNTIME_OPTION) |
duke@435 | 166 | |
duke@435 | 167 | # How /GX option is spelled |
duke@435 | 168 | GX_OPTION = /GX |
duke@435 | 169 | |
duke@435 | 170 | # Optimization settings for various versions of the compilers and types of |
duke@435 | 171 | # builds. Three basic sets of settings: product, fastdebug, and debug. |
erikj@3518 | 172 | # These get added into CXX_FLAGS as needed by other makefiles. |
duke@435 | 173 | !if "$(COMPILER_NAME)" == "VC6" |
duke@435 | 174 | PRODUCT_OPT_OPTION = /Ox /Os /Gy /GF |
duke@435 | 175 | FASTDEBUG_OPT_OPTION = /Ox /Os /Gy /GF |
duke@435 | 176 | DEBUG_OPT_OPTION = /Od |
duke@435 | 177 | !endif |
duke@435 | 178 | |
duke@435 | 179 | !if "$(COMPILER_NAME)" == "VS2003" |
sbohne@492 | 180 | PRODUCT_OPT_OPTION = /O2 /Oy- |
sbohne@492 | 181 | FASTDEBUG_OPT_OPTION = /O2 /Oy- |
duke@435 | 182 | DEBUG_OPT_OPTION = /Od |
duke@435 | 183 | !endif |
duke@435 | 184 | |
duke@435 | 185 | !if "$(COMPILER_NAME)" == "VS2005" |
sbohne@492 | 186 | PRODUCT_OPT_OPTION = /O2 /Oy- |
sbohne@492 | 187 | FASTDEBUG_OPT_OPTION = /O2 /Oy- |
duke@435 | 188 | DEBUG_OPT_OPTION = /Od |
duke@435 | 189 | GX_OPTION = /EHsc |
duke@435 | 190 | # This VS2005 compiler has /GS as a default and requires bufferoverflowU.lib |
duke@435 | 191 | # on the link command line, otherwise we get missing __security_check_cookie |
duke@435 | 192 | # externals at link time. Even with /GS-, you need bufferoverflowU.lib. |
duke@435 | 193 | # NOTE: Currently we decided to not use /GS- |
duke@435 | 194 | BUFFEROVERFLOWLIB = bufferoverflowU.lib |
erikj@3518 | 195 | LD_FLAGS = /manifest $(LD_FLAGS) $(BUFFEROVERFLOWLIB) |
tbell@862 | 196 | # Manifest Tool - used in VS2005 and later to adjust manifests stored |
tbell@862 | 197 | # as resources inside build artifacts. |
erikj@3600 | 198 | !if "x$(MT)" == "x" |
tbell@862 | 199 | MT=mt.exe |
tbell@862 | 200 | !endif |
erikj@3600 | 201 | !endif |
tbell@862 | 202 | |
tbell@862 | 203 | !if "$(COMPILER_NAME)" == "VS2008" |
tbell@862 | 204 | PRODUCT_OPT_OPTION = /O2 /Oy- |
tbell@862 | 205 | FASTDEBUG_OPT_OPTION = /O2 /Oy- |
tbell@862 | 206 | DEBUG_OPT_OPTION = /Od |
tbell@862 | 207 | GX_OPTION = /EHsc |
erikj@3518 | 208 | LD_FLAGS = /manifest $(LD_FLAGS) |
tbell@862 | 209 | # Manifest Tool - used in VS2005 and later to adjust manifests stored |
tbell@862 | 210 | # as resources inside build artifacts. |
erikj@3600 | 211 | !if "x$(MT)" == "x" |
tbell@862 | 212 | MT=mt.exe |
duke@435 | 213 | !endif |
erikj@3600 | 214 | !endif |
duke@435 | 215 | |
prr@1840 | 216 | !if "$(COMPILER_NAME)" == "VS2010" |
prr@1840 | 217 | PRODUCT_OPT_OPTION = /O2 /Oy- |
prr@1840 | 218 | FASTDEBUG_OPT_OPTION = /O2 /Oy- |
prr@1840 | 219 | DEBUG_OPT_OPTION = /Od |
prr@1840 | 220 | GX_OPTION = /EHsc |
erikj@3518 | 221 | LD_FLAGS = /manifest $(LD_FLAGS) |
prr@1840 | 222 | # Manifest Tool - used in VS2005 and later to adjust manifests stored |
prr@1840 | 223 | # as resources inside build artifacts. |
erikj@3600 | 224 | !if "x$(MT)" == "x" |
prr@1840 | 225 | MT=mt.exe |
erikj@3600 | 226 | !endif |
zgu@2590 | 227 | !if "$(BUILDARCH)" == "i486" |
erikj@3518 | 228 | LD_FLAGS = /SAFESEH $(LD_FLAGS) |
zgu@2590 | 229 | !endif |
prr@1840 | 230 | !endif |
prr@1840 | 231 | |
dholmes@5417 | 232 | !if "$(COMPILER_NAME)" == "VS2012" |
dholmes@5417 | 233 | PRODUCT_OPT_OPTION = /O2 /Oy- |
dholmes@5417 | 234 | FASTDEBUG_OPT_OPTION = /O2 /Oy- |
dholmes@5417 | 235 | DEBUG_OPT_OPTION = /Od |
dholmes@5417 | 236 | GX_OPTION = /EHsc |
dholmes@5417 | 237 | LD_FLAGS = /manifest $(LD_FLAGS) |
dholmes@5417 | 238 | # Manifest Tool - used in VS2005 and later to adjust manifests stored |
dholmes@5417 | 239 | # as resources inside build artifacts. |
dholmes@5417 | 240 | !if "x$(MT)" == "x" |
dholmes@5417 | 241 | MT=mt.exe |
dholmes@5417 | 242 | !endif |
dholmes@5417 | 243 | !if "$(BUILDARCH)" == "i486" |
dholmes@5417 | 244 | LD_FLAGS = /SAFESEH $(LD_FLAGS) |
dholmes@5417 | 245 | !endif |
dholmes@5417 | 246 | !endif |
dholmes@5417 | 247 | |
duke@435 | 248 | # If NO_OPTIMIZATIONS is defined in the environment, turn everything off |
duke@435 | 249 | !ifdef NO_OPTIMIZATIONS |
duke@435 | 250 | PRODUCT_OPT_OPTION = $(DEBUG_OPT_OPTION) |
duke@435 | 251 | FASTDEBUG_OPT_OPTION = $(DEBUG_OPT_OPTION) |
duke@435 | 252 | !endif |
duke@435 | 253 | |
duke@435 | 254 | # Generic linker settings |
erikj@3600 | 255 | !if "x$(LD)" == "x" |
erikj@3518 | 256 | LD=link.exe |
erikj@3600 | 257 | !endif |
erikj@3518 | 258 | LD_FLAGS= $(LD_FLAGS) kernel32.lib user32.lib gdi32.lib winspool.lib \ |
duke@435 | 259 | comdlg32.lib advapi32.lib shell32.lib ole32.lib oleaut32.lib \ |
duke@435 | 260 | uuid.lib Wsock32.lib winmm.lib /nologo /machine:$(MACHINE) /opt:REF \ |
dcubed@3724 | 261 | /opt:ICF,8 |
dcubed@3724 | 262 | !if "$(ENABLE_FULL_DEBUG_SYMBOLS)" == "1" |
dcubed@3724 | 263 | LD_FLAGS= $(LD_FLAGS) /map /debug |
dcubed@3724 | 264 | !endif |
duke@435 | 265 | |
zgu@3031 | 266 | |
zgu@3031 | 267 | !if $(MSC_VER) >= 1600 |
erikj@3518 | 268 | LD_FLAGS= $(LD_FLAGS) psapi.lib |
zgu@3031 | 269 | !endif |
zgu@3031 | 270 | |
duke@435 | 271 | # Resource compiler settings |
erikj@3600 | 272 | !if "x$(RC)" == "x" |
duke@435 | 273 | RC=rc.exe |
erikj@3600 | 274 | !endif |
duke@435 | 275 | RC_FLAGS=/D "HS_VER=$(HS_VER)" \ |
duke@435 | 276 | /D "HS_DOTVER=$(HS_DOTVER)" \ |
duke@435 | 277 | /D "HS_BUILD_ID=$(HS_BUILD_ID)" \ |
duke@435 | 278 | /D "JDK_VER=$(JDK_VER)" \ |
duke@435 | 279 | /D "JDK_DOTVER=$(JDK_DOTVER)" \ |
duke@435 | 280 | /D "HS_COMPANY=$(HS_COMPANY)" \ |
duke@435 | 281 | /D "HS_FILEDESC=$(HS_FILEDESC)" \ |
duke@435 | 282 | /D "HS_COPYRIGHT=$(HS_COPYRIGHT)" \ |
duke@435 | 283 | /D "HS_FNAME=$(HS_FNAME)" \ |
duke@435 | 284 | /D "HS_INTERNAL_NAME=$(HS_INTERNAL_NAME)" \ |
duke@435 | 285 | /D "HS_NAME=$(HS_NAME)" |
duke@435 | 286 | |
erikj@3518 | 287 | # Need this to match the CXX_FLAGS settings |
duke@435 | 288 | !if "$(MFC_DEBUG)" == "true" |
duke@435 | 289 | RC_FLAGS = $(RC_FLAGS) /D "_DEBUG" |
duke@435 | 290 | !endif |
duke@435 | 291 |