RFR(XS) [8u] 8153267: nmethod's exception cache not multi-thread safe

David Buck david.buck at oracle.com
Wed Apr 5 13:37:30 UTC 2017


Hi!
(explicitly CCing everyone involved with the original JDK 9 review)

Please have a look at this very straightforward backport to 8u-dev of 
Martin's fix for 8153267.

8153267 bug report:   https://bugs.openjdk.java.net/browse/JDK-8153267

8153267 JDK 9 push: 
http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/95b8ce0f1228

8u backport webrev:   http://cr.openjdk.java.net/~dbuck/8153267.0_jdk8u/

The only non-trivial difference between Martin's JDK 9 push and my 
backport is that I also included what remained of the change for 
8143897. 8143897 was pushed before 8153267 but then completely 
superseded by 8153267. Specifically, I included the addition of the 
"index" local (as opposed to the multiple calls to count()) from 8143897 
as it will keep both code lines (8 and 9) better in-sync and also makes 
the code more readable in my opinion. So you can think of the above 
webrev as a combined delta of both 8153267 and 8153267.

8143897 bug report:  https://bugs.openjdk.java.net/browse/JDK-8143897

8143897 jdk 9 push: 
http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/f918c20107d9

All standard JPRT tests (default and "hotspot" testsets) run and passed.

Cheers,
-Buck


More information about the hotspot-compiler-dev mailing list