[8u60] RFR of backport for 8066875: VirtualSpace does not use large pages
Thomas Schatzl
thomas.schatzl at oracle.com
Thu Mar 26 12:55:16 UTC 2015
Hi all,
can I have reviews for the backport of "8066875: VirtualSpace does not
use large pages" for 8u60? I also would like to have one review from the
compiler team (cc'ed) since the change touches some compiler files.
It did only apply with minor changes, so I need re-reviews. The problem
is that in jdk9 the code cache sizing has been changed. In particular:
- dropped the hunk in code/codeCache.cpp because the code to determine
memory sizes in 8u60 is much simpler i.e. .
E.g. this change:
--- a/src/share/vm/code/codeCache.cpp Thu Jan 15 16:05:20 2015 +0100
+++ b/src/share/vm/code/codeCache.cpp Fri Jan 16 10:29:12 2015 +0100
@@ -233,8 +233,8 @@
ReservedCodeSpace CodeCache::reserve_heap_memory(size_t size) {
// Determine alignment
const size_t page_size = os::can_execute_large_page_memory() ?
- MIN2(os::page_size_for_region(InitialCodeCacheSize, 8),
- os::page_size_for_region(size, 8)) :
+ MIN2(os::page_size_for_region_aligned(InitialCodeCacheSize, 8),
+ os::page_size_for_region_aligned(size, 8)) :
os::vm_page_size();
const size_t granularity = os::vm_allocation_granularity();
const size_t r_align = MAX2(page_size, granularity);
- fixed the code in heap.cpp because of the same change (JDK-8015774:
Add support for multiple code heaps) is not in 8u60.
Note that this change is based on "8049864: TestParallelHeapSizeFlags
fails with unexpected heap size"
which is also out for review (on hotspot-gc-dev), and "8053995: Add
method to WhiteBox to get vm_pagesize" which applies cleanly.
Full 8u60 changeset:
http://cr.openjdk.java.net/~tschatzl/8066875-8u60/webrev.8u60/
Fix changeset:
http://cr.openjdk.java.net/~tschatzl/8066875-8u60/webrev.8u60-fix/
CR:
https://bugs.openjdk.java.net/browse/JDK-8066875
Original change:
http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/4321214d5dbc
Testing: jprt
With that changeset in place, JDK-8058354 can be merged relatively
easily, which is the goal of most of the recent backports.
Thanks,
Thomas
More information about the hotspot-compiler-dev
mailing list